java - 如果在单个方法中递增，static int 线程安全吗？

Question

我想使用下面的代码跟踪 getVariableAndLogAccess(RequestInfo requestInfo)。如果只有这两种方法访问变量，是否线程安全？ 使其线程安全的标准方法是什么？

public class MyAccessLog(){

    private int recordIndex = 0;
    private int variableWithAccessTracking = 42; 
    private final Map<Integer, RequestInfo> requestsLog = new HashMap<>();
    public int getVariableAndLogAccess(RequestInfo requestInfo){
        Integer myID = recordIndex++;
        int variableValue = variableWithAccessTracking;
        requestInfo.saveValue(variableValue);
        requestLog.put(myID, requestInfo);
        return variableValue;
     }
     public void setValueAndLog(RequestInfo requestInfo, int newValue){
        Integer myID = recordIndex++;
        variableWithAccessTracking = variableValue;
        requestInfo.saveValue(variableValue);
        requestLog.put(myID, requestInfo);
     }

/*other methods*/
}

Answer 1

Will it be thread safe if only these two methods access variable?

没有。

例如，如果两个线程调用 setValueAndLog，它们可能会以相同的 myID 值结束。

What is the standard way to make it thread safe?

你应该用 AtomicInteger 替换你的 int , 使用 lock , 或 syncrhonized block 以防止并发修改。

根据经验，使用原子变量(例如前面提到的 AtomicInteger)比使用锁要好，因为锁涉及操作系统。调用操作系统就像请律师 - 对于您可以自己解决的事情，最好避免两者。

注意如果使用锁或者同步块(synchronized block)，setter和getter都需要使用同一个锁。否则，当 setter 仍在更新变量时，getter 可能会被访问​​，从而导致并发错误。