BAEL-3855

core-java-modules/core-java-concurrency-collections-2/src/main/java/com/baeldung/concurrent/lock/SingleLock.java

@@ -14,30 +14,30 @@ public class SingleLock extends ConcurrentAccessExperiment {
 
     protected synchronized Supplier<?> putSupplier(Map<String,String> map, int key) {
         return (()-> {
-            boolean done = false;
             try {
-                while(!done) {
+                lock.lock();
-                    done = lock.tryLock();
-                }
                 map.put("key" + key, "value" + key);
+            } catch (Exception e) {
+                this.putSupplier(map, key);
             } finally {
-                lock.unlock();
+                try {
-            }
+                    lock.unlock();
+                } catch (Exception e) {}            }
             return null;
         });
     }
 
     protected synchronized Supplier<?> getSupplier(Map<String,String> map, int key) {
         return (()-> {
-            boolean done = false;
             try {
-                while(!done) {
+                lock.lock();
-                	done = lock.tryLock();
-                }
                 map.get("key" + key);
+            } catch (Exception e) {
+                this.getSupplier(map, key);
             } finally {
-                lock.unlock();
+                try {
-            }
+                    lock.unlock();
+                } catch (Exception e) {}            }
             return null;
         });
     }

core-java-modules/core-java-concurrency-collections-2/src/main/java/com/baeldung/concurrent/lock/StripedLock.java

@@ -7,23 +7,25 @@ import com.google.common.base.Supplier;
 import com.google.common.util.concurrent.Striped;
 
 public class StripedLock extends ConcurrentAccessExperiment {
-    Striped<Lock> lock;
+    Striped<Lock> stripedLock;
 
     public StripedLock(int buckets) {
-        lock = Striped.lock(buckets);
+        stripedLock = Striped.lock(buckets);
     }
 
     protected synchronized Supplier<?> putSupplier(Map<String,String> map, int key) {
         return (()-> {
-            Lock currentLock = lock.get("key" + key);
+        	int bucket = key % stripedLock.size();
-            boolean done = false;
+            Lock lock = stripedLock.get(bucket);
             try {
-                while(!done) {
+                lock.tryLock();
-                    done = currentLock.tryLock();
-                }
                 map.put("key" + key, "value" + key);
+            } catch (Exception e) {
+                this.putSupplier(map, key);
             } finally {
-                currentLock.unlock();
+                try {
+                    lock.unlock();
+                } catch (Exception e) {}
             }
             return null;
         });
@@ -31,15 +33,17 @@ public class StripedLock extends ConcurrentAccessExperiment {
 
     protected synchronized Supplier<?> getSupplier(Map<String,String> map, int key) {
         return (()-> {
-            Lock currentLock = lock.get("key" + key);
+        	int bucket = key % stripedLock.size();
-            boolean done = false;
+            Lock lock = stripedLock.get(bucket);
             try {
-                while(!done) {
+                lock.tryLock();
-                    done = currentLock.tryLock();
-                }
                 map.get("key" + key);
+            } catch (Exception e) {
+                this.getSupplier(map, key);
             } finally {
-                currentLock.unlock();
+                try {
+                    lock.unlock();
+                } catch (Exception e) {}
             }
             return null;
         });