JLabel не меняет цвет дважды

Question

У меня есть следующий код:

   public class Test extends JFrame implements ActionListener{
 private static final Color TRANSP_WHITE = new Color(new Float(1), new Float(1), new Float(1), new Float(0.5)); 
 private static final Color TRANSP_RED = new Color(new Float(1), new Float(0), new Float(0), new Float(0.1));
 private static final Color[] COLORS = new Color[]{ TRANSP_RED, TRANSP_WHITE};
 private int index = 0;

 private JLabel label;
 private JButton button; 
 public Test(){
  super();

  setLayout(new BoxLayout(getContentPane(), BoxLayout.Y_AXIS));
  label = new JLabel("hello world");
  label.setOpaque(true);
  label.setBackground(TRANSP_WHITE);

  getContentPane().add(label);

  button = new JButton("Click Me");
  button.addActionListener(this);

  getContentPane().add(button);

  pack();
  setVisible(true);
 }

 @Override
 public void actionPerformed(ActionEvent e) {
  if(e.getSource().equals(button)){
   label.setBackground(COLORS[index % (COLORS.length - 1)]);
index++;
      }
     }

 public static void main(String[] args) {
  new Test();
 }
    }

Когда я запускаю его, я получаю метку с TRANSP_WHITE фон, а затем, когда я нажимаю кнопку, этот цвет меняется на TRANSP_RED но когда я нажимаю еще раз, я не вижу изменений в цвете.Кто-нибудь знает, почему?

Спасибо

Answer 1

Вы делаете это не правильно.Это надо сделать так

label = new JLabel("hello world"){
     public void paintComponent(Graphics g)
     {
         //draw background
         Color old=g.getColor();
         g.setColor(getBackground());
         g.fillRect(0,0,getWidth(),getHeight());
         g.setColor(old);
         super.paintComponent(g);
     }
};
label.setOpaque(false); // your component is not opaque!

Answer 2

Ну, а чего ты ожидал?

label.setBackground(COLORS[index % (COLORS.length - 1)]);

Индексная переменная жестко запрограммирована на 0.и COLORS.length -1 по сути является константой.Поэтому каждый раз, когда вы нажимаете кнопку установки фона на COLORS[0];

Если вы измените метод действия на следующий, вы получите желаемые результаты:

 @Override
 public void actionPerformed(ActionEvent e) {
  if(e.getSource().equals(button)){
   label.setBackground(COLORS[index % COLORS.length]);
   index++;
  }
 }

Первый:Оператор по модулю всегда возвращает значение от 0 до одного меньше переданного ему значения.Так

index % COLORS.length

Всегда будет возвращать значение от 0 до COLORS.length -1.

Второй:Вы забыли увеличивать индекс после каждого вызова.

Answer 3

Привет!Вы забыли увеличить индекс.В этом выражении:

label.setBackground(COLORS[index % (COLORS.length - 1)]);

index % (COLORS.length - 1) всегда 0.

КСТАТИ.вам не обязательно использовать new Float(1) при создании Color. 1F тоже должно работать.

Answer 4

Вот код, который вы должны использовать

label.setBackground(COLORS[index % (COLORS.length)]);
index++;