refactor 151

fishercoder1534 · fishercoder1534 · commit c44bf48aea45 · 2021-01-14T11:27:08.000-08:00
diff --git a/src/main/java/com/fishercoder/solutions/_151.java b/src/main/java/com/fishercoder/solutions/_151.java
@@ -27,6 +27,7 @@ public String reverseWords(String s) {
             return stringBuilder.substring(0, stringBuilder.length() - 1).toString();
         }
     }
+
     public static class Solution2 {
         public String reverseWords(String s) {
             int len = s.length();
@@ -50,11 +51,10 @@ public String reverseWords(String s) {
                 }
                 // word found
                 String word = s.substring(i, j);
-                if(result.length() == 0) {
+                if (result.length() == 0) {
                     result = word;
-                }
-                else {
-                    result = word + " "+ result;
+                } else {
+                    result = word + " " + result;
                 }
                 i = j + 1;
             }
diff --git a/src/test/java/com/fishercoder/_151Test.java b/src/test/java/com/fishercoder/_151Test.java
@@ -7,44 +7,43 @@
 import static org.junit.Assert.assertEquals;
 
 public class _151Test {
-  private static _151.Solution1 solution1;
-  private static _151.Solution2 solution2;
-  private static String s;
-
-  @BeforeClass
-  public static void setup()
-  {
-    solution1 = new _151.Solution1();
-    solution2 = new _151.Solution2();
-  }
-
-  @Test
-  public void test1() {
-    s = " ";
-    assertEquals("", solution1.reverseWords(s));
-  }
-
-  @Test
-  public void test2() {
-    s = " 1";
-    assertEquals("1", solution1.reverseWords(s));
-  }
-
-  @Test
-  public void test3() {
-    s = "   a   b ";
-    assertEquals("b a", solution1.reverseWords(s));
-  }
-
-  @Test
-  public void test4() {
-    s = "a b  c";
-    assertEquals("c b a", solution1.reverseWords(s));
-  }
-
-  @Test
-  public void test5() {
-    s = "  hello world  ";
-    assertEquals("world hello", solution2.reverseWords(s));
-  }
+    private static _151.Solution1 solution1;
+    private static _151.Solution2 solution2;
+    private static String s;
+
+    @BeforeClass
+    public static void setup() {
+        solution1 = new _151.Solution1();
+        solution2 = new _151.Solution2();
+    }
+
+    @Test
+    public void test1() {
+        s = " ";
+        assertEquals("", solution1.reverseWords(s));
+    }
+
+    @Test
+    public void test2() {
+        s = " 1";
+        assertEquals("1", solution1.reverseWords(s));
+    }
+
+    @Test
+    public void test3() {
+        s = "   a   b ";
+        assertEquals("b a", solution1.reverseWords(s));
+    }
+
+    @Test
+    public void test4() {
+        s = "a b  c";
+        assertEquals("c b a", solution1.reverseWords(s));
+    }
+
+    @Test
+    public void test5() {
+        s = "  hello world  ";
+        assertEquals("world hello", solution2.reverseWords(s));
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@ public String reverseWords(String s) {`
`27`	`27`	`return stringBuilder.substring(0, stringBuilder.length() - 1).toString();`
`28`	`28`	`}`
`29`	`29`	`}`
	`30`	`+`
`30`	`31`	`public static class Solution2 {`
`31`	`32`	`public String reverseWords(String s) {`
`32`	`33`	`int len = s.length();`
`@@ -50,11 +51,10 @@ public String reverseWords(String s) {`
`50`	`51`	`}`
`51`	`52`	`// word found`
`52`	`53`	`String word = s.substring(i, j);`
`53`		`- if(result.length() == 0) {`
	`54`	`+ if (result.length() == 0) {`
`54`	`55`	`result = word;`
`55`		`- }`
`56`		`- else {`
`57`		`- result = word + " "+ result;`
	`56`	`+ } else {`
	`57`	`+ result = word + " " + result;`
`58`	`58`	`}`
`59`	`59`	`i = j + 1;`
`60`	`60`	`}`