Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.java.programmer > #7073

Re: Synchronization of the constructor

Show all headers | View raw

On 13.08.2011 11:58, MaciekL wrote:

> I have a doubt because Java disables synchronization of the constructor
> by default.

Btw, it's "question" and not "doubt".  Side note: I see this quite 
often.  Does anybody have an idea why non native speakers often use 
"doubt" instead of "question"?  I'm not a native speaker myself but I 
can't remember having mixed up the two.  So I guess it has to do with 
the way English is taught or the mother tongue.

> Following example shows that lack of synchronization makes
> NullPointerException.
> ................
> [Thread[main,5,main]]  {TestApp(0)} Begin
> [Thread[runner,5,main]] Begin
> [Thread[runner,5,main]] Test
> Exception in thread "runner" java.lang.NullPointerException
>          at TestApp.test(TestApp.java:61)
>          at TestApp.run(TestApp.java:55)
>          at java.lang.Thread.run(Thread.java:662)
> [Thread[main,5,main]] Test
> [Thread[main,5,main]] OBJ = java.lang.Object@9304b1
> [Thread[main,5,main]]  {TestApp(0)} End
> ................

Where is line 61?  From what I can see your example only has about 30 lines.

> Synchronized 'test' method may be called by another thread in case
> the Object is not created.

What do you mean by that?

> This undefined behaviour is very tricky, note that following example
> may works correctly, it depends on the Constructor duration time.

You see an error.  That doesn't necessarily mean that behavior is undefined.

> I have found a soultion, adding whole definition of the Constructor into
> synchronized block.
> ...............
> synchronized (this)
> {
>    ... // ConstructorBody
> }
> ...............
>
> I don't understand why Java forbids simple declaration:
> ...............
> public synchronized TestApp(int timeout)
> ...............

Probably because usually "this" is not leaked (to other objects or 
threads) during construction.  This is generally considered dangerous 
because you leak a reference to a potentially incomplete instance (the 
constructor whose task it is to initialize the object to a consistent 
state did not yet finish).

I also remember that they changed the language spec at one point with 
regard to thread visibility of "this" during construction but I don't 
recall the details right now.

I have to say that your design is questionable: you inherit Runnable 
which is intended to decouple thread handling from what is executed in 
the thread.  Yet inside the constructor you create a thread and start it.

Rather you should first let the construction of the object finish 
properly and then create a Thread with that instance and start the 
thread.  If you want to do that in the same class a static method could 
do, e.g.

public static void runTest(final int timeout)
   throws InterruptedException {
   final TestApp ta = new TestApp();
   final Thread th = new Thread(ta);
   th.start();
   Thread.sleep(timeout);
   ta.setObj(new Object());
   ta.test();
   th.join();
}

> Because of that all calls to "add*Listener(this)" from constructor
> should be carrefully implemented.
>
> /*--:BEG:--[TestApp.java]----------------------------------------------------*/
> public class TestApp implements Runnable
> {
>    Object obj = null;
>    public static void info(String s)
>    {
>      System.err.println("[" + Thread.currentThread().toString() + "] " + s);
>    }
>    public TestApp(int timeout)
>    {
>      info(" {TestApp(" + timeout + ")} Begin");
>      Thread runner = (new Thread(this));
>      runner.setName("runner");
>      runner.start();
>      try { Thread.sleep(timeout); }
>      catch (InterruptedException ie) { }

Empty catch block without any handling or at least a comment explaining 
why you consciously ignore the exception is really bad.

>      obj = new Object();
>      test();
>      info(" {TestApp(" + timeout + ")} End");
>    }
>    public void run()
>    {
>      info("Begin");
>      test();
>      info("End");
>    }
>    public synchronized void test()
>    {
>      info("Test");
>      info("OBJ = " + obj.toString());

You don't need toString() - it's done automatically.

>    }
>    public static void main(String [] args)
>    {
>      new TestApp(1000);
>    }
> }
> /*--:EOF:--[TestApp.java]----------------------------------------------------*/

Now why do you want to start a thread in a constructor and have it 
executed code of "this"?

Kind regards

	robert

-- 
remember.guy do |as, often| as.you_can - without end
http://blog.rubybestpractices.com/

Back to comp.lang.java.programmer | Previous | Next — Previous in thread | Next in thread | Find similar

Thread

Synchronization of the constructor MaciekL <__nospam__maclab@o2.pl> - 2011-08-13 11:58 +0200
  Re: Synchronization of the constructor Robert Klemme <shortcutter@googlemail.com> - 2011-08-13 12:17 +0200
    Re: Synchronization of the constructor "Qu0ll" <Qu0llSixFour@gmail.com> - 2011-08-13 21:36 +1000
      Re: Synchronization of the constructor Arved Sandstrom <asandstrom3minus1@eastlink.ca> - 2011-08-13 09:30 -0300
    [OT] Natural language. Was: Re: Synchronization of the constructor Patricia Shanahan <pats@acm.org> - 2011-08-13 07:10 -0700
  Re: Synchronization of the constructor Eric Sosman <esosman@ieee-dot-org.invalid> - 2011-08-13 09:36 -0400
    Re: Synchronization of the constructor markspace <-@.> - 2011-08-13 08:13 -0700
      Re: Synchronization of the constructor Lew <lewbloch@gmail.com> - 2011-08-13 09:23 -0700
        Re: Synchronization of the constructor Patricia Shanahan <pats@acm.org> - 2011-08-13 10:04 -0700
          Re: Synchronization of the constructor Lew <lewbloch@gmail.com> - 2011-08-13 21:29 -0700
            Re: Synchronization of the constructor kedar mhaswade <kedar.mhaswade@gmail.com> - 2011-08-13 23:16 -0700
            Re: Synchronization of the constructor "Qu0ll" <Qu0llSixFour@gmail.com> - 2011-08-15 14:24 +1000
              Re: Synchronization of the constructor Lew <lewbloch@gmail.com> - 2011-08-14 23:03 -0700
      Re: Synchronization of the constructor kedar mhaswade <kedar.mhaswade@gmail.com> - 2011-08-13 23:12 -0700
        Re: Synchronization of the constructor markspace <-@.> - 2011-08-14 07:42 -0700
          Re: Synchronization of the constructor markspace <-@.> - 2011-08-14 07:51 -0700
            Re: Synchronization of the constructor Lew <lewbloch@gmail.com> - 2011-08-14 09:10 -0700
              Re: Synchronization of the constructor markspace <-@.> - 2011-08-14 11:28 -0700

csiph-web