java - 在对象上同步似乎没有同步

Question

我运行一个程序，其中包含以下类(不仅如此，而且这些是与问题相关的类)

在Results类下，我有一个同步的LinkedHashMap，例如:

private static Map<Integer,Result>    resultsHashMap=Collections.synchronizedMap(new LinkedHashMap<Integer, Result>());

和一个 getter 方法:

public static Map<Integer,Result> getResultsHashMap() {
        return resultsHashMap;
}

我的 Result 类中也有一个带有以下同步代码的构造函数:

public Result(){
    synchronized (Lock.lock) {
        uniqueIdResult++;
    }
}

以及一个同步的 getter 方法:

public static int getUniqueIdResult() {
    synchronized (Lock.lock) {
        return uniqueIdResult;
    }

}

uniqueIdResult 定义如下:

private static int uniqueIdResult=0;

我还有一个 Lock 类，其中包含此对象:

public static final Lock lock=new Lock();

现在，这是我所追求的重要问题。在我的程序中，我有接下来的两行，它们创建一个 Result 并将其放入 HashMap

Result result = new Result();
Results.getResultsHashMap().put(Result.getUniqueIdResult(), result);

我尝试使用不同数量的线程运行我的程序。当它使用 1 个线程运行时，输出正如我所期望的那样(具体来说，但不一定重要，Results.resultsHashMap 包含 433 个键，这是应该的，并且键从 1 开始)。

但是当我使用不同数量的线程运行它时，它会给出不同的输出。例如，使用 6 个线程运行每次都会给出不同数量的键，有时 430，有时 428，有时 427 等。并且起始键并不总是与键总数相关(例如，total_number_of_keys-starting_key_number+1，其中一开始在我看来是某种模式，但后来意识到事实并非如此)

迭代是这样的:

int counterOfResults=0;
    for (Integer key : Results.getResultsHashMap().keySet()) {
        System.out.println(key + " " + Results.getResultsHashMap().get(key));
        counterOfResults++;
    }
    System.out.println(counterOfResults);

此外，当同步获取hashMap的getter方法时，如果不同步Result的创建和向hashMap的插入，多线程的输出会给出错误的输出。
另外，当仅同步其中一行时(创建 Result 并放入 hashMap)，多线程下的输出不一致。

但是，当我同步两行这些行(创建结果并放入 map 中)时，如下所示:

Result result;
    synchronized (Lock.lock) {
         result = new Result(currentLineTimeNationalityNameYearofbirth.getName(),currentLineTimeNationalityNameYearofbirth.getTime(),citycompetionwas,date,distance,stroke,gender,kindofpool);
        Results.getResultsHashMap().put(Result.getUniqueIdResult(), result);
    }

无论我使用多少个线程，输出都是完美的。

另外，我会注意到，只有在所有线程完成后，通过对创建的所有线程使用 join 方法，才会打印输出。

所以我的问题是:
据我所知，在同步这两行(创建 Result 并放入 hashMap)之前，我的所有关键部分，例如，更改和获取 uniqueIdResult，获取 < em>resultsHashMap (正如我提到的，我也尝试同步这个 getter 方法)在同一个对象上同步，另外，在将 hashMap 与 Collections.synchronizedMap 放在一起时，我采取了一种更安全的方法，据我所知知道，应该使 hashMap 线程安全。

为什么输出不符合我的预期？哪里存在安全问题？

Answer 1

这些行没有排除:

Result result = new Result();
Results.getResultsHashMap().put(Result.getUniqueIdResult(), result);

如果您有 4 个线程，它们可能都会执行第一行(这会将 uniqueIdResult 变量增加四次)，然后都执行第二行(此时它们都会看到getUniqueIdResult() 的返回值相同)。这解释了当您有 4 个(或更多)线程时，您的键如何从 4 开始。

因为您有多个线程可能(并且不可预测地)存储到同一个键，所以您最终也会在映射中得到可变数量的条目。

您可能应该从 Result 类构造函数中删除增量，而是在 getUniqueIdResult 方法中执行此操作:

public static int getUniqueIdResult() {
    synchronized (Lock.lock) {
        return ++uniqueIdResult;
    }
}

(完成此操作后，根本不再需要创建 Result 实例)。