ActionListener - Радио кнопки

Question

ActionListener - Радио кнопки

1

Я пытаюсь заставить программу работать, когда вы нажимаете конкретный переключатель, который говорит о цвете на нем, и он делает всю страницу такой. Окно открывается как квадрат с 4 различными цветовыми опциями. Я просто не могу, чтобы жизнь меня заставила работать actionPerformed. Вот мой код. Любая помощь приветствуется.

public class ColorPanel3 extends JPanel implements ActionListener {

Color darkBlue = new Color(5,41,186);
Color lightBlue = new Color(35,253,253);
Color darkRed = new Color(158,19,47);
Color lightRed = new Color(255,105,105);


JRadioButton lightRedButton = new JRadioButton("Light Red");
JRadioButton darkBlueButton = new JRadioButton("Dark Blue");
JRadioButton lightBlueButton = new JRadioButton("Light Blue");
JRadioButton darkRedButton = new JRadioButton("Dark Red");

public  ColorPanel3() {
    setName("Color Panel");
    setLayout(new GridLayout(1, 0, 0, 0));
    JPanel panel = new JPanel();
    add(panel);
    panel.setLayout(null);
    panel.setOpaque(true);

    lightRedButton.setHorizontalAlignment(SwingConstants.CENTER);
    lightRedButton.setBounds(0, 150, 150, 150);
    lightRedButton.setBackground(lightRed);
    panel.add(lightRedButton);


    darkBlueButton.setHorizontalAlignment(SwingConstants.CENTER);
    darkBlueButton.setBounds(0, 0, 150, 150);
    darkBlueButton.setBackground(darkBlue);
    panel.add(darkBlueButton);


    lightBlueButton.setHorizontalAlignment(SwingConstants.CENTER);
    lightBlueButton.setBounds(150, 0, 150, 150);
    lightBlueButton.setBackground(lightBlue);
    panel.add(lightBlueButton);

    darkRedButton.setHorizontalAlignment(SwingConstants.CENTER);
    darkRedButton.setBounds(150, 150, 150, 150);
    darkRedButton.setBackground(darkRed);

    panel.add(darkRedButton);

    ButtonGroup group = new ButtonGroup();   //creates a button group so that only one radio button may be pressed at a time.
    group.add(darkBlueButton);
    group.add(lightBlueButton);
    group.add(darkRedButton);
    group.add(lightRedButton);

    lightRedButton.addActionListener(actionListener);
    darkRedButton.addActionListener(actionListener);
    lightBlueButton.addActionListener(actionListener);
    darkBlueButton.addActionListener(actionListener);

}

ActionListener actionListener = new ActionListener(){
    public void actionPerformed (ActionEvent e){


        if(darkBlueButton.isSelected()){
            darkBlueButton.setBackground(darkBlue);
            lightBlueButton.setBackground(darkBlue);
            lightRedButton.setBackground(darkBlue);
            darkRedButton.setBackground(darkBlue);
        }

        if(lightBlueButton.isSelected()){
            darkBlueButton.setBackground(lightBlue);
            lightBlueButton.setBackground(lightBlue);
            lightRedButton.setBackground(lightBlue);
            darkRedButton.setBackground(lightBlue);
        }

        if(darkRedButton.isSelected()){
            darkBlueButton.setBackground(darkRed);
            lightBlueButton.setBackground(darkRed);
            lightRedButton.setBackground(darkRed);
            darkRedButton.setBackground(darkRed);
        }

        if(lightRedButton.isSelected()){
            darkBlueButton.setBackground(lightRed);
            lightBlueButton.setBackground(lightRed);
            lightRedButton.setBackground(lightRed);
            darkRedButton.setBackground(lightRed);
        }
    }

И я создал новый файл класса, чтобы включить панель. Это сработало.

public class ColorFrame{
public static void main(String[] args) {
    JFrame frame = new JFrame();
    frame.add(new ColorPanel3());
    frame.setVisible(true);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setTitle("Color Frame");

}

}
    };

}

Окончательное редактирование: приведенный выше код - это то, с чем я закончил.

Jacob Wiebe 23 март 2015, в 04:04

Источник

0

Попробуйте добавить System.out.println(1); В действии выполните часть, чтобы проверить, где она работает нормально, затем добавьте эту информацию к вашему вопросу, и я смогу вам помочь. Еще одна вещь, вы пытаетесь изменить фон окна или переключателя?
nom 23 март 2015, в 02:44
0

Что вы ожидаете, и что на самом деле происходит?
TNT 23 март 2015, в 02:49
0

Извините, Набиэль, я не уверен, что вы подразумеваете под System.out.println (1); часть. Я попытался добавить его в actionPerformed, но ничего не изменилось. Я пытаюсь изменить цвет фона радиоприемников. Когда я нажимаю на любую из радиобоксов, цвета не меняются. Конечно, это что-то в моем методе actionPerformed?
Jacob Wiebe 23 март 2015, в 02:52
0

Я хочу, чтобы все цвета фона блоков менялись одновременно. Если я нажму на темно-красный блок радио, я хочу, чтобы фон всех блоков (всего окна) изменился на этот цвет.
Jacob Wiebe 23 март 2015, в 02:54
0

Итак, вы хотите, чтобы весь JPanel изменил цвет?
TNT 23 март 2015, в 02:56
0

Да, но переключатели все большие (каждая занимает 1/4 от JPanel), поэтому я думаю, что мне нужно изменить фон каждой кнопки по отдельности, чтобы все изменило цвет.
Jacob Wiebe 23 март 2015, в 02:57
0

Нет, просто использовать Component класс, setBackground метод.
TNT 23 март 2015, в 03:02
0

@JacobWiebe добавление System.out.println() в ваш код несколько раз - очень распространенный метод отладки, который сообщает вам, где ваш код работает нормально
nom 23 март 2015, в 03:14
0

@JacobWiebe я также заметил, что ваш код пытается изменить фон переключателя, а не JFrame.
nom 23 март 2015, в 03:16
0

@JacobWiebe use ColorPanel3.this.setBackground(\*your color*\); вместо lightRedButton.setBackground(darkBlue);
nom 23 март 2015, в 03:19

Показать ещё 8 комментариев

Теги:

java

swing

actionlistener

2 ответа

Ещё вопросы

Попробуйте добавить System.out.println(1); В действии выполните часть, чтобы проверить, где она работает нормально, затем добавьте эту информацию к вашему вопросу, и я смогу вам помочь. Еще одна вещь, вы пытаетесь изменить фон окна или переключателя?
Что вы ожидаете, и что на самом деле происходит?
Извините, Набиэль, я не уверен, что вы подразумеваете под System.out.println (1); часть. Я попытался добавить его в actionPerformed, но ничего не изменилось. Я пытаюсь изменить цвет фона радиоприемников. Когда я нажимаю на любую из радиобоксов, цвета не меняются. Конечно, это что-то в моем методе actionPerformed?
Я хочу, чтобы все цвета фона блоков менялись одновременно. Если я нажму на темно-красный блок радио, я хочу, чтобы фон всех блоков (всего окна) изменился на этот цвет.
Итак, вы хотите, чтобы весь JPanel изменил цвет?
Да, но переключатели все большие (каждая занимает 1/4 от JPanel), поэтому я думаю, что мне нужно изменить фон каждой кнопки по отдельности, чтобы все изменило цвет.
Нет, просто использовать Component класс, setBackground метод.
@JacobWiebe добавление System.out.println() в ваш код несколько раз - очень распространенный метод отладки, который сообщает вам, где ваш код работает нормально
@JacobWiebe я также заметил, что ваш код пытается изменить фон переключателя, а не JFrame.
@JacobWiebe use ColorPanel3.this.setBackground(\*your color*\); вместо lightRedButton.setBackground(darkBlue);

nom · Answer 1 · 2015-03-23T01-59-00.000Z

1

Ваш слушатель действия должен выглядеть следующим образом:

ColorPanel3.setOpaque(true);
ColourPanel3.this.setBackground(yourcolor);

И более того, если вы не пишете этот код для обфускации кода, пожалуйста, используйте разные исполнители действий для всех переключателей.

Приветствия.

Изменить: вам нужно будет добавить панель в JFrame

nom 23 март 2015, в 01:59

0

ColorPanel3.setOpaque(false); На непрозрачной панели цвет фона не будет виден. please use different actionlisteners Это действительно вопрос предпочтений, который также зависит от приложения. Можно уставать от написания addActionListener снова и снова, особенно если действия нескольких кнопок похожи.
TNT 23 март 2015, в 03:44
0

@TNT Первое, что вы сказали, было правильно, что ColorPanel3.setOpaque(false); неправильно. Во второй части использование разных слушателей действий является очень хорошей идеей, поскольку позволяет избежать запутывания кода, но, как вы сказали, это вопрос предпочтений.
nom 23 март 2015, в 04:17
0

Я отредактировал свой основной пост, чтобы показать мой исправленный код. У меня все еще есть та же самая проблема, где нажатие любой из кнопок ничего не делает. :(
Jacob Wiebe 23 март 2015, в 04:39
0

@JacobWiebe подожди немного, дай мне использовать IDE.
nom 23 март 2015, в 06:02
0

Хорошо, в итоге я заставил его работать, поместив ColorPanel3 в рамку. Новая концепция для меня. Я опубликую свой обновленный код выше - но теперь он работает как задумано. В итоге я изменил цвет фона каждой кнопки, используя операторы if.
Jacob Wiebe 23 март 2015, в 06:24
0

@JacobWiebe Я предполагал, что вы добавили панель в jframe в других частях вашего кода, который, как я снова предполагал, существует, потому что в вашем классе нет main.
nom 23 март 2015, в 06:32

Показать ещё 4 комментария

TNT · Answer 2 · 2015-03-23T00-18-00.000Z

Прежде всего, избавьтесь от этих строк:

JRadioButton darkBlueButton = (JRadioButton) e.getSource();
JRadioButton lightBlueButton = (JRadioButton) e.getSource();
JRadioButton darkRedButton = (JRadioButton) e.getSource();
JRadioButton lightRedButton = (JRadioButton) e.getSource();

JRadioButton что вы создаете четыре экземпляра JRadioButton со ссылкой на JRadioButton, заданные e.getSource(). Если вы хотите, чтобы фон менял цвет, это необязательно. Я чувствую, что эффект от этого не то, что вы намеревались сделать, так или иначе.

При создании объектов JRadioButton с конструктором, который принимает String, вы даете ему ярлык (т. JRadioButton Темно-синий) вместе с командой действий, которая совпадает с текстом. Вы можете воспользоваться этим, используя getActionCommand метод ActionEvent класса, чтобы определить, какой цвет должен быть JPanel:

if (e.getActionCommand().equals("Dark Blue")) {
    ColorPanel3.this.setBackground(darkBlue);
}

Но для того, чтобы любое изменение цвета стало очевидным, вы должны установить прозрачность верхней панели:

panel.setOpaque(false);