refactor 328

fishercoder1534 · fishercoder1534 · commit fe2c7df5207b · 2019-01-02T07:22:39.000-08:00
diff --git a/src/main/java/com/fishercoder/solutions/_328.java b/src/main/java/com/fishercoder/solutions/_328.java
@@ -19,22 +19,23 @@ You should try to do it in place. The program should run in O(1) space complexit
  */
 public class _328 {
 
-    public ListNode oddEvenList(ListNode head) {
-        if (head != null) {
-            ListNode odd = head;
-            ListNode even = head.next;
-            ListNode evenHead = even;
-
-            while (even != null && even.next != null) {
-                odd.next = odd.next.next;
-                even.next = even.next.next;
-                odd = odd.next;
-                even = even.next;
+    public static class Solution1 {
+        public ListNode oddEvenList(ListNode head) {
+            if (head != null) {
+                ListNode odd = head;
+                ListNode even = head.next;
+                ListNode evenHead = even;
+
+                while (even != null && even.next != null) {
+                    odd.next = odd.next.next;
+                    even.next = even.next.next;
+                    odd = odd.next;
+                    even = even.next;
+                }
+
+                odd.next = evenHead;
             }
-
-            odd.next = evenHead;
+            return head;
         }
-        return head;
     }
-
 }
diff --git a/src/test/java/com/fishercoder/_328Test.java b/src/test/java/com/fishercoder/_328Test.java
@@ -7,32 +7,29 @@
 
 import static org.junit.Assert.assertEquals;
 
-/**
- * Created by stevesun on 5/29/17.
- */
 public class _328Test {
-    private static _328 test;
-    private static ListNode expected;
-    private static ListNode node;
+  private static _328.Solution1 solution1;
+  private static ListNode expected;
+  private static ListNode node;
 
-    @BeforeClass
-    public static void setup() {
-        test = new _328();
-    }
+  @BeforeClass
+  public static void setup() {
+    solution1 = new _328.Solution1();
+  }
 
-    @Test
-    public void test1() {
-        node = new ListNode(1);
-        node.next = new ListNode(2);
-        node.next.next = new ListNode(3);
-        node.next.next.next = new ListNode(4);
-        node.next.next.next.next = new ListNode(5);
+  @Test
+  public void test1() {
+    node = new ListNode(1);
+    node.next = new ListNode(2);
+    node.next.next = new ListNode(3);
+    node.next.next.next = new ListNode(4);
+    node.next.next.next.next = new ListNode(5);
 
-        expected = new ListNode(1);
-        expected.next = new ListNode(3);
-        expected.next.next = new ListNode(5);
-        expected.next.next.next = new ListNode(2);
-        expected.next.next.next.next = new ListNode(4);
-        assertEquals(expected, test.oddEvenList(node));
-    }
+    expected = new ListNode(1);
+    expected.next = new ListNode(3);
+    expected.next.next = new ListNode(5);
+    expected.next.next.next = new ListNode(2);
+    expected.next.next.next.next = new ListNode(4);
+    assertEquals(expected, solution1.oddEvenList(node));
+  }
 }
