fix: Version number pattern update overwritten #2968 (#2980)

* Fix bug #2968

* Fix bug #2968

version-number/src/main/java/com/iluwatar/versionnumber/BookRepository.java

@@ -24,8 +24,7 @@
  */
 package com.iluwatar.versionnumber;
 
-import java.util.HashMap;
+import java.util.concurrent.ConcurrentHashMap;
-import java.util.Map;
 
 /**
  * This repository represents simplified database.
@@ -34,7 +33,8 @@ import java.util.Map;
  * as much as in real databases.
  */
 public class BookRepository {
-  private final Map<Long, Book> collection = new HashMap<>();
+  private final ConcurrentHashMap<Long, Book> collection = new ConcurrentHashMap<>();
+  private final Object lock = new Object();
 
   /**
    * Adds book to collection.
@@ -57,19 +57,22 @@ public class BookRepository {
       throw new BookNotFoundException("Not found book with id: " + book.getId());
     }
 
-    var latestBook = collection.get(book.getId());
+    // used synchronized block to ensure only one thread compares and update the version
-    if (book.getVersion() != latestBook.getVersion()) {
+    synchronized (lock) {
-      throw new VersionMismatchException(
+      var latestBook = collection.get(book.getId());
-        "Tried to update stale version " + book.getVersion()
+      if (book.getVersion() != latestBook.getVersion()) {
-          + " while actual version is " + latestBook.getVersion()
+        throw new VersionMismatchException(
-      );
+            "Tried to update stale version " + book.getVersion()
+                + " while actual version is " + latestBook.getVersion()
+        );
+      }
+
+      // update version, including client representation - modify by reference here
+      book.setVersion(book.getVersion() + 1);
+
+      // save book copy to repository
+      collection.put(book.getId(), new Book(book));
     }
-
-    // update version, including client representation - modify by reference here
-    book.setVersion(book.getVersion() + 1);
-
-    // save book copy to repository
-    collection.put(book.getId(), new Book(book));
   }
 
   /**