2

我为我的班级制作了一个简单的摇摆程序，它根据 ComboBox 中的选定索引更改时区和其他一些东西。当方法 run() 如下所示时工作正常：

public void run() {
        while(true){
            Calendar c = Calendar.getInstance();
            int h = c.get(Calendar.HOUR);
            int m = c.get(Calendar.MINUTE);
            int s = c.get(Calendar.SECOND);
            l.setText(""+h+":"+m+":"+(s<10?"0"+s:s));
            try {
                t.sleep(1000);
            } catch (InterruptedException ex) {}
        }
    }

但是，当我尝试重新定义它并使时间更改起作用时，我在这一行中得到一个空指针异常：

p.cb.addItemListener(new ItemListener() {

方法 run() 看起来像这样，它不会工作。有任何想法吗？

public void run() {
    while(true){

        p.cb.addItemListener(new ItemListener() {

            @Override
            public void itemStateChanged(ItemEvent e) {
                Calendar c = Calendar.getInstance();
                int h = c.get(Calendar.HOUR);
                int m = c.get(Calendar.MINUTE);
                int s = c.get(Calendar.SECOND);
                int index = p.cb.getSelectedIndex();

                if(index == 0){
                    l.setText(""+h+":"+m+":"+(s<10?"0"+s:s));
                }
                else if(index == 1){
                    l.setText(""+(h-6)+":"+m+":"+(s<10?"0"+s:s));
                }
                else if(index == 2){
                    l.setText(""+(h-1)+":"+m+":"+(s<10?"0"+s:s));
                }
                else if(index == 3){
                    l.setText(""+(h-6)+":"+m+":"+(s<10?"0"+s:s));
                }
                else if(index == 4){
                    l.setText(""+(h+8)+":"+m+":"+(s<10?"0"+s:s));
                }
            }

        });

        try {
            t.sleep(1000);
        } catch (InterruptedException ex) {}
    }
}

如果有人感到困惑，p 是 JFrame 类的一个实例，而 cb 是对 JFrame 类中 ComboBox 的引用。

4

3 回答 3

4

ANullPointerException被抛出是因为要么p或cb有一个null值。

于 2013-03-05T16:02:29.740 回答
1

你的线程现在没有意义了。它每秒钟向组合框添加一个 ItemListener，但每个 ItemListener 都在做同样的事情。此外，您的线程正在修改 EDT 之外的 gui 元素。您需要在 EDT 上添加 ItemListener。

我假设您尝试做的是每隔几秒或更改组合框时更新标签。

您应该在没有线程的情况下执行此操作，而是使用 Swing Timer 和 ItemListener 的组合。

// called on the EDT
void setup() {
     // create gui elements
     p.cb.addItemListener(new ItemListener() {
        updateTimeLabel();
     }
     new Timer(1000, new ActionListener() {
        public void actionPerformed(ActionEvent evt) {
           updateTimeLable()
        }
     });
 }

 private void updateTimeLabel() {
     Calendar c = Calendar.getInstance();
     int h = c.get(Calendar.HOUR);
     int m = c.get(Calendar.MINUTE);
     int s = c.get(Calendar.SECOND);
     int index = p.cb.getSelectedIndex();

     if(index == 0){
        l.setText(""+h+":"+m+":"+(s<10?"0"+s:s));
     }
     else if(index == 1){
         l.setText(""+(h-6)+":"+m+":"+(s<10?"0"+s:s));
     }
     else if(index == 2){
         l.setText(""+(h-1)+":"+m+":"+(s<10?"0"+s:s));
     }
     else if(index == 3){
          l.setText(""+(h-6)+":"+m+":"+(s<10?"0"+s:s));
     }
     else if(index == 4){
          l.setText(""+(h+8)+":"+m+":"+(s<10?"0"+s:s));
     }
 }

至于为什么现在出现空指针异常，这可能是由于访问尚未初始化的不同变量、内存可见性问题、线程竞争条件或修改 EDT 之外的 GUI 元素引起的。如果你删除你的线程，除了第一个之外，所有这些条件都会消失。然后很容易确保在调用 addItemListener 方法之前初始化 p.cb。

此外，您确实应该使用比 p、cb、l 更具描述性的名称。

于 2013-03-05T17:35:11.307 回答
0

如果有人感到困惑，p 是 JFrame 类的一个实例，而 cb 是对 JFrame 类中 ComboBox 的引用。

JFrame 的实例？你为什么要遵循这种方法？只需调用ComboBox变量并添加监听器！p似乎是NULL。

于 2013-03-05T18:36:16.963 回答