java - 我需要同步方法和对象吗

Question

所以我有以下列表，可以从多个线程访问：

ArrayList<String> listOfString = Collections.synchronizedList(new ArrayList<String>());

我知道当我遍历列表时，我必须像这样同步：

synchronized(listOfString)
{
    for(String s : listOfString) System.out.println(s);

    listOfString.clear();
}

如果我想删除一些东西怎么办，我这样做：

public void removeString(String s)
{
    listOfString.remove(s);
}

或这个：

public synchronized void removeString(String s)
{
    listOfString.remove(s);
}

Answer 1

正如你所说，列表已经同步，所以你的removeString方法不需要synchronized太。

但是请注意，如果您的某个方法包含非原子操作（例如，您想检查列表是否包含某些内容，然后相应地修改列表），您可能需要添加另一层同步。

最后，你似乎没有注意到这个方法：

public synchronized void removeString(String s)

在不同的锁上同步（它在 上同步this）。所以回到我上面给出的例子，你可以这样写：

public void someMethod() {
    synchronized(listOfString) { //use the same lock!
        if(listOfString.size() == 123) {
            listOfString.add("lucky day");
        }
    }
}