Java线程问题

Question

我在我的应用程序中使用多个线程。 基本上我有一个组合框，当选择收件箱时，p1恢复，p2暂停，选择发送后，p2开始，p1停止。 下面是代码（我确定它不完美）

public void modifyText(ModifyEvent e) {
                if (combo.getText().equals("Inbox"))
                {
                    synchronized(p2) 
                    {
                        p2.cont = false;
                    }
                    table.removeAll();
                    synchronized(p1)
                    {
                        p1.cont = true;
                        p1.notify();
                    }
                }


                else if (combo.getText().equals("Sent"))
                {
                    synchronized(p2) 
                    {
                        p1.cont = false;
                    }
                    table.removeAll();
                    synchronized(p1)
                    {
                        p2.cont = true;
                        p2.notify();
                    }
                }
            }
        });

对于P1和P2，我在他们的while循环中有这个：

synchronized (this) {
            while (cont == false)
                try {
                    wait();
                } catch (Exception e) {
                }
        } 

...因为它现在正在工作（我是线程的初学者）。 在组合框中按下Sent后，我收到IllegalStateMonitorException。 谁能帮我解决问题PLZ？

谢谢和问候，Krt_Malta

Answer 1

问题出在这里：

synchronized(p1)
{
    p2.cont = true;
    p2.notify();
}

当你没有锁定p2时你正在做p2.notify() （你必须让监视器调用它上面的通知）。 将synchronized(p1)更改为synchronized(p2) 。 此外，您还需要反转其他同步子句，这也是错误的。 所以，作为一个例子：

synchronized(p1) 
{
    p1.cont = false;
    // p1.notify(); <- do you need this here?
}
table.removeAll();
synchronized(p2)
{
    p2.cont = true;
    p2.notify();
}

另外，你的其他代码也有点不对，锁定整个循环是非常糟糕的做法，使它更具原子性。

while (!cont) {
    synchronized (this) {
       try {
           wait();
       } catch (Exception e) {
       }
    }
}

其他优化，尽可能避免synchronised ：

if (p1.cont) {
   synchronized(p1) 
   {
       p1.cont = false;
       // p1.notify(); <- do you need this here?
   }
}
table.removeAll();

if (!p2.cont) {
   synchronized(p2)
   {
       p2.cont = true;
       p2.notify();
   }
}

在这里使cont字段变为volatile ，并在适当的情况下镜像if语句的其他部分。

编辑：回顾这一点，并与我最近面临的并发错误作斗争，任何实现此模式的人都可能面临无限等待的问题，如果被锁定和半锁定的对象正在被有条件的同时看待循环（这是因为在评估条件和强制等待语句之间状态可以改变的间隙）。 在这种情况下，将同步块放在循环的外部 。

Answer 2

在这段代码中

                synchronized(p1)
                {
                    p2.cont = true;
                    p2.notify();
                }

您正在p1同步，但在p2上调用notify() ，这会导致异常。

Answer 3

您不能在awt事件调度线程中等待，否则会占用整个应用程序。 阅读http://en.wikipedia.org/wiki/Event_dispatching_thread

除非你真的知道自己在做什么，否则你不应该使用原始线程。 查看http://java.sun.com/javase/6/docs/api/java/util/concurrent/package-summary.html并阅读Executors