ActionListener中HashMap存储正确答案重复/丢失问题求助
问题分析与解决方案
我仔细看了你的代码,问题根源出在**全局静态的truesList**上。你每次收集完当前题目的正确答案后,把这个列表存入truesMap,紧接着调用truesList.clear()——但HashMap里存储的是列表的引用,不是副本!这就导致所有题目对应的答案列表其实都是同一个对象:你清空它之后,之前存的条目也跟着变空;而每次新增题目时,又往同一个列表里加内容,最终所有条目都会指向最后一次添加的内容,自然就出现了重复的情况。
你之前处理选项时,最后用循环拆分全局列表的方式避开了问题,但正确答案的逻辑没这么做,这就是两者的差异所在。
具体修复步骤
- 移除全局的
truesList静态变量,不要再共享同一个列表实例 - 每次处理新题目时,创建一个新的
ArrayList<String>来存储当前题目的正确答案 - 存入
truesMap后不需要清空列表(因为是局部变量,用完就会被回收,不会影响下一次的题目处理)
修改后的关键代码片段
先删掉全局的truesList变量,然后在suivant的事件处理逻辑里做如下修改:
// 移除这行全局静态变量:static ArrayList<String> truesList = new ArrayList<>(); // 在actionPerformed方法的suivant分支中修改: if (e.getSource() == suivant) { String prompt = questionField.getText(); prompts.add(prompt); ExamPanel.this.setVisible(false); ExamFrame.this.setContentPane(new ExamPanel()); // 保留原有的选项收集逻辑(也可以后续优化) for (int i=0;i<2;i++) { choicesList.add(respnsesFields.get(i).getText()); } // 创建局部列表存储当前题目的正确答案 ArrayList<String> currentTrues = new ArrayList<>(); for (int i=0;i<2;i++) { if (jCheckBoxes.get(i).isSelected()) { currentTrues.add(respnsesFields.get(i).getText()); } } truesMap.put(count, currentTrues); // 无需clear,局部变量会自动完成生命周期管理 count++; }
额外优化建议
- 尽量避免使用静态全局集合(比如
choicesMap、choicesList这些),这类共享实例很容易引发引用相关的bug,建议把这些集合封装在ExamFrame的实例中,而非静态变量。 - 选项收集逻辑也可以和正确答案统一,每次创建局部列表存入
choicesMap,不用等到最后再拆分全局列表,代码逻辑会更清晰:// 移除全局choicesList,在suivant事件中替换为: ArrayList<String> currentChoices = new ArrayList<>(); for (int i=0;i<2;i++) { currentChoices.add(respnsesFields.get(i).getText()); } choicesMap.put(count, currentChoices);
内容的提问来源于stack exchange,提问作者匿名用户
相关产品推荐
相关产品推荐

