Java同步块-锁定整个方法

Question

我不太确定如何正确使用同步块。 我知道使用synced关键字会锁定整个类。 所以那不是我想要的。 我想锁定类中的单个方法。 这是我班上的一个例子。

//class does only contain static methods
public class MessageFactory implements IValues {

  public static Message createRegisterRequest(String name, String password) {
      Payload payload = createPayload(name, password);
      return new Message(VALUE_REGISTRATION, payload);
  }
}

如何正确使用同步块来锁定整个方法？ 我可以选择任何参数（或名称或密码）吗？ 之间有什么区别：

  public static Message createRegisterRequest(String name, String password) {
     synchronized(name){
         Payload payload = createPayload(name, password);
         return new Message(VALUE_REGISTRATION, payload);
      }
  }

和：

  public static Message createRegisterRequest(String name, String password) {
     synchronized(password){
         Payload payload = createPayload(name, password);
         return new Message(VALUE_REGISTRATION, payload);
      }
  }

和：

  public static Message createRegisterRequest(String name, String password) {
     Object lock = new Object();
     synchronized(lock){
         Payload payload = createPayload(name, password);
         return new Message(VALUE_REGISTRATION, payload);
     }
  }

createPayload没有访问共享的任何内容。 看起来像：

 private static Payload createPayload(int playerID, String roomName){
    Payload payload = new Payload();
    payload.setPlayerID(playerID);
    payload.setRoomName(roomName);
    return payload;
}

我以为我必须同步方法，因为有多个实例同时访问MessageFactory中的方法。 如果我错了，请告诉我。

Answer 1

Java中的同步是使用共享对象进行的。 因此，您应该在方法之外进行同步。 任何局部变量或参数都将不起作用，因为该数据对于每个方法调用而言都是局部的，并且不会共享。

您应该这样做：

public class MessageFactory implements IValues {
    private static final Object LOCK = new Object();

    public static Message createRegisterRequest(String name, String password) {
        synchronized(LOCK){
            Payload payload = createPayload(name, password);
            return new Message(VALUE_REGISTRATION, payload);
        }
    }
}