Использование одной кнопки для изменения цвета нескольких JPanels - PullRequest
Новая
       7

Использование одной кнопки для изменения цвета нескольких JPanels

2 голосов
/ 13 апреля 2020

Итак, у меня есть три панели, для которых у меня есть три разные кнопки, чтобы изменить их на свои соответствующие цвета. Мне нужно добавить четвертую кнопку, которая вернет все три панели к их исходному светло-серому цвету по умолчанию. Я добавляю эту кнопку «сброс», и она только изменяет первую панель назад. Что я делаю неправильно? 

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.FlowLayout;
import java.awt.Color;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

public class PanelDemo extends JFrame implements ActionListener
{
public static final int WIDTH = 300;
public static final int HEIGHT = 200;
private JPanel redPanel;
private JPanel whitePanel;
private JPanel bluePanel;

public static void main(String[] args)
{
    PanelDemo gui = new PanelDemo();
    gui.setVisible(true);
}
public PanelDemo()
{
    super("Panel Demonstration");
    setSize(WIDTH, HEIGHT);
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setLayout(new BorderLayout());
    JPanel biggerPanel = new JPanel();
    biggerPanel.setLayout(new GridLayout(1, 3));

    redPanel = new JPanel();
    redPanel.setBackground(Color.LIGHT_GRAY);
    biggerPanel.add(redPanel);

    whitePanel = new JPanel();
    whitePanel.setBackground(Color.LIGHT_GRAY);
    biggerPanel.add(whitePanel);

    bluePanel = new JPanel();
    bluePanel.setBackground(Color.LIGHT_GRAY);
    biggerPanel.add(bluePanel);

    add(biggerPanel, BorderLayout.CENTER);

    JPanel buttonPanel = new JPanel();
    buttonPanel.setBackground(Color.LIGHT_GRAY);
    buttonPanel.setLayout(new FlowLayout());

    JButton redButton = new JButton("Red");
    redButton.setBackground(Color.RED);
    redButton.addActionListener(this);
    buttonPanel.add(redButton);

    JButton whiteButton = new JButton("White");
    whiteButton.setBackground(Color.WHITE);
    whiteButton.addActionListener(this);
    buttonPanel.add(whiteButton);

    JButton blueButton = new JButton("Blue");
    blueButton.setBackground(Color.BLUE);
    blueButton.addActionListener(this);
    buttonPanel.add(blueButton);

    JButton resetButton = new JButton("Reset");
    resetButton.setBackground(Color.LIGHT_GRAY);
    resetButton.addActionListener(this);
    buttonPanel.add(resetButton);

    add(buttonPanel, BorderLayout.SOUTH);
}
@Override
public void actionPerformed(ActionEvent e)
{
    String buttonString = e.getActionCommand();

    if (buttonString.equals("Red"))
        redPanel.setBackground(Color.RED);
    else if (buttonString.equals("White"))
        whitePanel.setBackground(Color.WHITE);
    else if (buttonString.equals("Blue"))
        bluePanel.setBackground(Color.BLUE);
    else if (buttonString.equals("Reset"))
        redPanel.setBackground(Color.LIGHT_GRAY);
    else if (buttonString.equals("Reset"))
        bluePanel.setBackground(Color.LIGHT_GRAY);
    else if (buttonString.equals("Reset"))
        whitePanel.setBackground(Color.LIGHT_GRAY);
    else
        System.out.println("Unexpected error.");


}
}

1 Ответ

1 голос
/ 13 апреля 2020

Здесь была ваша проблема. Вы имели, если еще на каждой панели для сброса. Сравните код ниже с тем, что у вас есть. Это была просто логическая проблема c. 


    public void actionPerformed(ActionEvent e) {
        String buttonString = e.getActionCommand();

        if (buttonString.equals("Red"))
            redPanel.setBackground(Color.RED);
        else if (buttonString.equals("White"))
            whitePanel.setBackground(Color.WHITE);
        else if (buttonString.equals("Blue"))
            bluePanel.setBackground(Color.BLUE);
        else if (buttonString.equals("Reset")) {
            redPanel.setBackground(Color.LIGHT_GRAY);
            bluePanel.setBackground(Color.LIGHT_GRAY);
            whitePanel.setBackground(Color.LIGHT_GRAY);
        }
        else
            System.out.println("Unexpected error.");

И пара предложений.

  • Не расширяйте JFrame. Просто используйте экземпляр этого. Это лучшая техника.
  • Поместите следующее как последний оператор в вашем конструкторе. Она будет центрировать панель на вашем экране. 
setLocationRelativeTo(null);
// or when using a frame instance.
frame.setLocationRelativeTo(null);
...