有人能告诉我为什么我的 actionListener for-loop 不起作用吗？

Question

我有一个程序，它接受一个输入文件，从中提取一个颜色字 + 十六进制值（例如红色 0xFF0000）。我的代码工作得很好，除了我试图用 HashMap 替换我的 2 个数组列表......那是事情发生了错误的地方。我的代码恢复到我认为之前的状态，但现在按下单选按钮时它不会改变颜色。有人想看看吗？

public HashMapTests() {
        JPanel p1 = new JPanel();
        this.getContentPane().setLayout(new GridLayout(5,4));
        ButtonGroup group = new ButtonGroup();
        for (int i = 0; i < colorCollection.size(); i++) {
            jrbColor[i] = new JRadioButton(colorCollection.get(i));
            jrbColor[i].setText(colorCollection.get(i));
            group.add(jrbColor[i]); 
            p1.add(jrbColor[i]);
            }
        for(int i = 0; i < colorCollection.size(); i++){
            jrbColor[i].addActionListener(new ActionListener() {
                public void actionPerformed(ActionEvent e){
                    for (int j = 0; j < colorCollection.size(); j++){
                        String hexColor = hexCollection.get(j);
                        if(hexCollection.get(j).equals(((JRadioButton)e.getSource()).getText())){
                            getContentPane().setBackground(Color.decode(hexColor));

                            repaint();
                        }
                    }                   
            }
        }); 
        }
        add(p1);
            }

Answer 1

第一次调查：

while (colorCollection.size() < 10)

应替换为

if (colorCollection.size() < 10)

第二次观察：

jrbColor[i] = new JRadioButton(colorCollection.get(i));
jrbColor[i].setText(colorCollection.get(i));

第二行没用，见构造函数的javadoc。

第三：

附加监听器的第二个循环没用，你可以把这段代码放到你创建按钮的第一个循环中。

最后：

if (hexCollection.get(j).equals(((JRadioButton) e.getSource()).getText())) {

您将此处的 hexCollection 内容与单选按钮文本进行比较，但该按钮具有来自 colorCollection 的标签。我无法查看您的文件，但我认为这将是问题所在。

地图解决方案：

初始化

String name = fileInput.next();
String hexValue = fileInput.next();
colors.put(name, hexValue);

按钮

    int i = 0;
    for (String s : colors.keySet()) {
        jrbColor[i] = new JRadioButton(s);
        group.add(jrbColor[i]);
        p1.add(jrbColor[i]);
        jrbColor[i].addActionListener(new ActionListener() {
        public void actionPerformed(ActionEvent e) {
            String hexColor = colors.get(((JRadioButton) e.getSource()).getText());
            getContentPane().setBackground(Color.decode(hexColor));
        }
        });
    }