Double check locking using Volatile.

Question

Hi gurus,
Please suggest if the following lazy loading would work.
public CommandFactory{
       private static volatile ICommand command= null;
       private static ICommand instanceCommand = null;
       public static ICommand getCommand(){
                  if(instanceCommand == null){
                         synchronized(CommandFactory.class){
                              if(instanceCommand == null){
                                  command = new Command();
                                  instanceCommand = command;
           return instanceCommand;
}

Answer 1

For danny like "Boss of Talented people..lol...lmao....." understand
The Double checking with Volatile which would work with the new improved volatile contract.
public CommandFactory{
       private static volatile ICommand command= null;
              public static ICommand getCommand(){
                  if(command== null){
                         synchronized(CommandFactory.class){
                              if(command== null){
                                  command = new Command();
           return command;
}And the code I wrote which isn't any good as per JTahlborn is here
public class CommandFactory {
     private static volatile ICommand command = null;
     private static ICommand instanceCommand = null;
     public static ICommand getCommand() {
          if (instanceCommand == null) {
               synchronized (CommandFactory.class) {
                    if (instanceCommand == null) {
                         command = new Command();
                         instanceCommand = command;
          return instanceCommand;
}