Android. Тот же экземпляр выдал другой результат.

Question

Я вывел один и тот же объект одновременно, но получил разные результаты ... Что может быть причиной РАЗЛИЧНОГО результата?

Функция в UserHelper.class:

public void login(String phone, String password) { 
    UserModel.logInInBackground(phone, password, new LogInCallback<UserModel>() { 
     @Override 
     public void done(UserModel userModel, AVException e) { 
      if (null != userModel) { 
       if (userModel.getPosition() == UserModel.USER_BUYER) { 
        refresh(); 
        DebugLog.e("fuck" + mUserStatus + UserInstance.getInstance().getUserStatus()); 
        for (UserListener listener : listeners) 
         listener.OnUserLogin(); 
       } else if (userModel.getPosition() == UserModel.USER_SELLER) 
        logout(); 
      } else for (UserListener listener : listeners) 
       listener.HandleError(e.getCode()); 
     } 
    }, UserModel.class); 

public USER_STATUS getUserStatus() { 
    return mUserStatus; 
} 

И UserInstance.class.

public class UserInstance { 
    public static UserHelper mInstance; 

public static UserHelper getInstance() { 
    if (null == mInstance) mInstance = new UserHelper(); 
    DebugLog.e(mInstance.toString()); 
    return mInstance; 
    } 
} 

Answer 1

Прежде всего, если вы имели в виду в UserHelper класс быть синглтон, почему вы получаете доступ к USER_STATUS экземпляра с помощью UserInstance.getInstance().getUserStatus() вместо того, чтобы просто getUserStatus()?

Во-вторых, вы, возможно, получаете разные экземпляры UserHelper, если однотонный доступ доступен из разных потоков, потому что ваша реализация не является потокобезопасной.

Правильная реализация будет с помощью double locking pattern:

public class UserInstance { 
    public static UserHelper mInstance; 
    private static final ReentrantLock lock = new ReentrantLock(); 

    public static UserHelper getInstance() { 
     if (null == mInstance){ 
      lock.lock(); 
      try{ 
       if (null == mInstance){ 
        mInstance = new UserHelper(); 
       } 
      } 
      finally{ 
       lock.unlock(); 
      } 
     } 
     DebugLog.e(mInstance.toString()); 
     return mInstance; 
    } 

}

Answer 2

В конце концов, я получаю тот же экземпляр ..

Благодаря Шломи Uziei. Я забыл использовать шаблон двойной блокировки. И я не должен делать mInstance static ...