How to correctly make a thread safe Singleton Factory in Java? [duplicate]

Answer 1

In fact your factory isn't thread safe, because in race condition you can have more than one ClientFactory living in application. Lets assume two threads:

1. ThreadA is evaluating condition 'if (instance == null)' and instance is null, so it enters statement
2. ThreadB is evaluating condition 'if (instance == null)' and instance is null (because A didn't make to instantiate it), so it enters statement
3. ThreadA creates new ClientFactory() and returns it
4. ThreadB creates new ClientFactory() and returns it
5. Now we have more than one ClientFactory in application. Of course other threads trying to retrieve instance some time later will always return single instance.

In my opinion the easiest way to write singleton in Java is to use enum. In your case it will looks:

public enum ClientFactory {
  INSTANCE;

  public Company getClient() {
    return new Company();
  }
}

And usage:

ClientFactory.INSTANCE.getClient()