refactor 555

fishercoder1534 · fishercoder1534 · commit 179e05403382 · 2019-06-29T07:21:51.000-07:00
diff --git a/src/main/java/com/fishercoder/solutions/_555.java b/src/main/java/com/fishercoder/solutions/_555.java
@@ -33,41 +33,43 @@
  */
 public class _555 {
 
-    /**
-     * credit: https://discuss.leetcode.com/topic/86477/neat-java-solution
-     * and article: https://leetcode.com/articles/split-assembled-strings/#approach-3-optimized-solution-accepted
-     */
-    public String splitLoopedString(String[] strs) {
-        StringBuilder sb = new StringBuilder();
-        for (int i = 0; i < strs.length; i++) {
-            sb.setLength(0);
-            String reverse = sb.append(strs[i]).reverse().toString();
-            if (strs[i].compareTo(reverse) < 0) {
-                strs[i] = reverse;
+    public static class Solution1 {
+
+        /**
+         * credit: https://discuss.leetcode.com/topic/86477/neat-java-solution
+         * and article: https://leetcode.com/articles/split-assembled-strings/#approach-3-optimized-solution-accepted
+         */
+        public String splitLoopedString(String[] strs) {
+            StringBuilder sb = new StringBuilder();
+            for (int i = 0; i < strs.length; i++) {
+                sb.setLength(0);
+                String reverse = sb.append(strs[i]).reverse().toString();
+                if (strs[i].compareTo(reverse) < 0) {
+                    strs[i] = reverse;
+                }
             }
-        }
-        String result = "";
-        for (int i = 0; i < strs.length; i++) {
-            sb.setLength(0);
-            String reverse = sb.append(strs[i]).reverse().toString();
-            for (String str : new String[]{strs[i], reverse}) {
-                for (int k = 0; k < str.length(); k++) {
-                    sb.setLength(0);
-                    sb.append(str.substring(k));
-                    for (int j = i + 1; j < strs.length; j++) {
-                        sb.append(strs[j]);
-                    }
-                    for (int j = 0; j < i; j++) {
-                        sb.append(strs[j]);
-                    }
-                    sb.append(str.substring(0, k));
-                    if (sb.toString().compareTo(result) > 0) {
-                        result = sb.toString();
+            String result = "";
+            for (int i = 0; i < strs.length; i++) {
+                sb.setLength(0);
+                String reverse = sb.append(strs[i]).reverse().toString();
+                for (String str : new String[]{strs[i], reverse}) {
+                    for (int k = 0; k < str.length(); k++) {
+                        sb.setLength(0);
+                        sb.append(str.substring(k));
+                        for (int j = i + 1; j < strs.length; j++) {
+                            sb.append(strs[j]);
+                        }
+                        for (int j = 0; j < i; j++) {
+                            sb.append(strs[j]);
+                        }
+                        sb.append(str.substring(0, k));
+                        if (sb.toString().compareTo(result) > 0) {
+                            result = sb.toString();
+                        }
                     }
                 }
             }
+            return result;
         }
-        return result;
     }
-
 }
diff --git a/src/test/java/com/fishercoder/_555Test.java b/src/test/java/com/fishercoder/_555Test.java
@@ -10,29 +10,29 @@
  * Created by fishercoder on 4/29/17.
  */
 public class _555Test {
-    private static _555 test;
+    private static _555.Solution1 solution1;
     private static String expected;
     private static String actual;
     private static String[] strs;
 
     @BeforeClass
     public static void setup() {
-        test = new _555();
+        solution1 = new _555.Solution1();
     }
 
     @Test
     public void test1() {
         strs = new String[]{"abc", "xyz"};
         expected = "zyxcba";
-        actual = test.splitLoopedString(strs);
+        actual = solution1.splitLoopedString(strs);
         assertEquals(expected, actual);
     }
 
     @Test
     public void test2() {
         strs = new String[]{"lc", "evol", "cdy"};
         expected = "ylclovecd";
-        actual = test.splitLoopedString(strs);
+        actual = solution1.splitLoopedString(strs);
         assertEquals(expected, actual);
     }
 }
