IDE-suggested drive-by: extract comparator creation

Author: greenc-FNAL

modules/dcache/src/main/java/org/dcache/pool/classic/IoQueueManager.java

@@ -32,6 +32,7 @@
 import org.dcache.pool.FaultEvent;
 import org.dcache.pool.FaultListener;
 import org.dcache.pool.classic.MoverRequestScheduler.Order;
+import org.dcache.pool.classic.MoverRequestScheduler.PrioritizedRequest;
 import org.dcache.util.IoPriority;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -443,24 +444,7 @@ public Serializable call() throws CommandException {
                       .toArray(IoJobInfo[]::new);
             } else {
 
-                Comparator<MoverRequestScheduler.PrioritizedRequest> comparator;
-                if (sortBySize) {
-                    comparator = (b, a) -> Long.compare(
-                          a.getMover().getBytesTransferred(), b.getMover().getBytesTransferred()
-                    );
-                } else if (sortByTime) {
-                    comparator = (b, a) -> Long.compare(
-                          a.getMover().getLastTransferred(), b.getMover().getLastTransferred()
-                    );
-                } else {
-                    comparator = (b, a) -> Integer.compare(
-                          a.getId(), b.getId()
-                    );
-                }
-
-                if (reverseSort) {
-                    comparator = comparator.reversed();
-                }
+                Comparator<PrioritizedRequest> comparator = getPrioritizedRequestComparator();
 
                 StringBuilder sb = new StringBuilder();
                 if (groupByQueue) {
@@ -478,6 +462,28 @@ public Serializable call() throws CommandException {
                 return sb.toString();
             }
         }
+
+        private Comparator<PrioritizedRequest> getPrioritizedRequestComparator() {
+            Comparator<PrioritizedRequest> comparator;
+            if (sortBySize) {
+                comparator = (b, a) -> Long.compare(
+                      a.getMover().getBytesTransferred(), b.getMover().getBytesTransferred()
+                );
+            } else if (sortByTime) {
+                comparator = (b, a) -> Long.compare(
+                      a.getMover().getLastTransferred(), b.getMover().getLastTransferred()
+                );
+            } else {
+                comparator = (b, a) -> Integer.compare(
+                      a.getId(), b.getId()
+                );
+            }
+
+            if (reverseSort) {
+                comparator = comparator.reversed();
+            }
+            return comparator;
+        }
     }
 
     @Command(name = "p2p ls", hint = "list pool to pool source movers",