Small refactoring

lukas-krecan-lt · lukas-krecan-lt · commit 493ea44924bb · 2017-01-09T21:09:40.000+01:00
diff --git a/shedlock-core/src/main/java/net/javacrumbs/shedlock/core/DefaultLockManager.java b/shedlock-core/src/main/java/net/javacrumbs/shedlock/core/DefaultLockManager.java
@@ -44,19 +44,22 @@ public void executeIfNotLocked(Runnable task) {
             logger.debug("No lock configuration for {}. Executing without lock.", task);
             task.run();
         } else {
-            LockConfiguration lockConfig = lockConfigOptional.get();
-            Optional<SimpleLock> lock = lockProvider.lock(lockConfig);
-            if (lock.isPresent()) {
-                try {
-                    logger.debug("Locked {}.", lockConfig.getName());
-                    task.run();
-                } finally {
-                    lock.get().unlock();
-                    logger.debug("Unlocked {}.", lockConfig.getName());
-                }
-            } else {
-                logger.info("Not executing {}. It's locked.", lockConfig.getName());
+            executeIfNotLocked(task, lockConfigOptional.get());
+        }
+    }
+
+    private void executeIfNotLocked(Runnable task, LockConfiguration lockConfig) {
+        Optional<SimpleLock> lock = lockProvider.lock(lockConfig);
+        if (lock.isPresent()) {
+            try {
+                logger.debug("Locked {}.", lockConfig.getName());
+                task.run();
+            } finally {
+                lock.get().unlock();
+                logger.debug("Unlocked {}.", lockConfig.getName());
             }
+        } else {
+            logger.info("Not executing {}. It's locked.", lockConfig.getName());
         }
     }
 }
