慕碼人8056858
2023-06-21 13:53:20
這是我當(dāng)前的代碼。我想知道如果已經(jīng)同步,我是否需要包裝isObjectActive()在同步塊中?getState()private final Object lock = new Object();@GuardedBy("lock")private EventTypes state;@GuardedBy("lock")public void setState(final EventTypes state) { synchronized (lock) { this.state = state; }}@GuardedBy("lock")@VisibleForTestingpublic EventTypes getState() { synchronized (lock) { return state; }}@GuardedBy("lock")public boolean isObjectActive() { synchronized (lock) { return getState() == EventTypes.ACTIVE; }}
1 回答

慕工程0101907
TA貢獻(xiàn)1887條經(jīng)驗 獲得超5個贊
[Do] 我需要將 isObjectActive() 包裝在同步塊中......?
不。
同步不是針對方法的,而是針對數(shù)據(jù)的。在上面顯示的示例中,您可以synchronized從 中刪除塊isObjectActive(),因為該函數(shù)不直接訪問state變量,并且它調(diào)用的唯一其他函數(shù)在訪問 時確實使用同步state。
你可能應(yīng)該考慮一下調(diào)用的結(jié)果意味著什么,isObjectActive()如果其他線程的操作可以在調(diào)用者有機(jī)會之前改變對象的狀態(tài)根據(jù)結(jié)果采取行動:
MyType mt = ...;
if (mt.isObjectActive()) {
? ? // This is broken because there's no guarantee that mt still will be "active"
? ? // by the time doSomething...() gets called.
? ? //
? ? doSomethingThatOnlyMakesSenseWhen_mt_IsActive(...);
}
添加回答
舉報
0/150
提交
取消