0

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

//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 中的方法。如果我错了，请告诉我。

4

1 回答 1

0

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);
        }
    }
}
于 2016-04-02T12:34:14.820 回答