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.notify()当你没有锁定时你正在做p2(你必须握住显示器来调用通知).更改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();
   }
}

在volatile此处创建cont字段,并根据需要镜像if语句的其他部分.

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