0

我正在使用ActionListener以在选择项目时更新JLIST。

jComboBox.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent e) {
        JComboBox cb = (JComboBox) e.getSource();
        updateLocalFileList( cb.getSelectedItem().toString() );
    }
});

它正在为 UI 调用此方法。

public void updateLocalFileList( String path ){
    DefaultListModel model = new DefaultListModel();
    for (String str : LocalFileSystem.getFileListFromDirectory( path )) {
        model.addElement( str );
    }
    getJList().setModel(model);
}

如果 getFileListFromDirectory() 给出 NullPointerException，比如选择了空 dvd 驱动器的盘符，它似乎会阻止 ActionListener 按预期工作。

我不确定到底发生了什么，但我怀疑将空值传递给模型会导致此问题。

有任何想法吗？

编辑

这是所要求的堆栈跟踪。如您所见，该方法显然会在无法访问的驱动器上触发 NullPointerException。我不知道为什么它会阻止 JList 更新，因为应用程序的其余部分工作正常。

java.lang.NullPointerException
    at mine.View.updateLocalFileList(View.java:274)
    at mine.View$1.actionPerformed(View.java:262)
    at javax.swing.JComboBox.fireActionEvent(Unknown Source)
    at javax.swing.JComboBox.setSelectedItem(Unknown Source)
    at javax.swing.JComboBox.setSelectedIndex(Unknown Source)
    at javax.swing.plaf.basic.BasicComboPopup$Handler.mouseReleased(Unknown Source)
    at java.awt.AWTEventMulticaster.mouseReleased(Unknown Source)
    at java.awt.Component.processMouseEvent(Unknown Source)
    at javax.swing.JComponent.processMouseEvent(Unknown Source)
    at javax.swing.plaf.basic.BasicComboPopup$1.processMouseEvent(Unknown Source)
    at java.awt.Component.processEvent(Unknown Source)
    at java.awt.Container.processEvent(Unknown Source)
    at java.awt.Component.dispatchEventImpl(Unknown Source)
    at java.awt.Container.dispatchEventImpl(Unknown Source)
    at java.awt.Component.dispatchEvent(Unknown Source)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
    at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
    at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
    at java.awt.Container.dispatchEventImpl(Unknown Source)
    at java.awt.Window.dispatchEventImpl(Unknown Source)
    at java.awt.Component.dispatchEvent(Unknown Source)
    at java.awt.EventQueue.dispatchEvent(Unknown Source)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
    at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
    at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
    at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
    at java.awt.EventDispatchThread.run(Unknown Source)
4

4 回答 4

2

我会做两件事来使它更健壮。

  1. 确保 getFileListFromDirectory 永远不会返回 null。如果没有项目，则返回Collections.emptyList而不是 null。如果这不可能，请在使用“for each”迭代之前专门检查返回值。实际上，空指针将阻止模型更新。（因此单击空驱动器不会清除文件列表。）
  2. 将 cb.getSelectedItem() 分配给局部变量并在调用 updateLocalFileList 之前检查 null。如果 cb.getSelectedItem() 为空，您可以选择清除文件列表。
于 2010-06-25T03:14:15.477 回答
1

NullPointerException 发生在 EventDispatchThread（这是所有 GUI 更新代码通常发生的地方），因此它会中断事件侦听器本身。因此，您的 GUI 无法正确更新（不再）。为防止这种情况发生，您必须显式处理异常，或拦截可能触发它们的原因。

于 2010-06-25T03:12:12.357 回答
1

您的侦听器（引发异常的那个）不是唯一侦听特定事件的侦听器。核心 Swing 类（尤其是 UI 委托）注册自己的侦听器，以便它们可以正确更新 UI。如果您的侦听器失败，则无法保证所有（或任何）其他侦听器都会收到此特定事件的通知。

于 2010-06-25T07:13:45.697 回答
0

几点：

究竟是什么LocalFileSystem类？那是自定义类吗？

如果在某些情况下预期它会返回 null，则对其使用局部变量并在调用任何方法之前检查它是否为 null。

exists您可以使用on 方法检查给定文件或目录是否存在File。还有一种isDirectory方法。我建议你考虑使用它。

于 2010-06-25T17:25:51.693 回答