Re: Using "synchronized" but still getting IllegalMonitorStateException

From:

"Peter Duniho" <NpOeStPeAdM@nnowslpianmk.com>

Newsgroups:

comp.lang.java.programmer

Date:

Mon, 06 Apr 2009 10:01:55 -0700

Message-ID:

<op.ury5lhjs8jd0ej@macbook-pro.local>

On Mon, 06 Apr 2009 09:54:47 -0700, laredotornado
<laredotornado@zipmail.com> wrote:

Hi,

I'm trying to execute this bit of JUnit test code (Java 1.5):

    public void testEventLoggerFailure() {
        try {
            synchronized(this) {
                EventLogger el = new EventLogger(null,
                       null,
                       null,
                       null);
                el.start();
                el.wait();
            } // synchronized
        } catch (Exception e) {
            e.printStackTrace();
            fail(e.getMessage());
        } // catch
    } // testEventLoggerFailure

but yet, at the "el.wait()" line the below exceptioin is thrown ...

What's "EventLogger"? Where's your concise-but-complete code example that
reliably demonstrates the problem? Or as they say around here, your SSCCE?

That said, if you're going to wait on a synchronization object, it's not
sufficient to have synchronized on any random object. You have to own the
monitor for the object you're trying to wait on. Obviously in this case,
"this" isn't the appropriate object required to hold the monitor for, for
calling "el.wait()".

Pete