refactor 458

fishercoder1534 · fishercoder1534 · commit f92a373364f5 · 2019-04-19T12:52:05.000-07:00
diff --git a/src/main/java/com/fishercoder/solutions/_458.java b/src/main/java/com/fishercoder/solutions/_458.java
@@ -17,17 +17,19 @@ how many pigs (x) you need to figure out the "poison" bucket within p minutes? T
  */
 public class _458 {
 
-    public int poorPigs(int buckets, int minutesToDie, int minutesToTest) {
-        if (buckets-- == 1) {
-            return 0;
+    public static class Solution1 {
+        public int poorPigs(int buckets, int minutesToDie, int minutesToTest) {
+            if (buckets-- == 1) {
+                return 0;
+            }
+            int base = minutesToTest / minutesToDie + 1;
+            int count = 0;
+            while (buckets > 0) {
+                buckets /= base;
+                count++;
+            }
+            return count;
         }
-        int base = minutesToTest / minutesToDie + 1;
-        int count = 0;
-        while (buckets > 0) {
-            buckets /= base;
-            count++;
-        }
-        return count;
     }
 
 }
diff --git a/src/test/java/com/fishercoder/_458Test.java b/src/test/java/com/fishercoder/_458Test.java
@@ -8,7 +8,7 @@
 import static junit.framework.Assert.assertEquals;
 
 public class _458Test {
-    private static _458 test;
+    private static _458.Solution1 solution1;
     private static int expected;
     private static int actual;
     private static int buckets;
@@ -17,7 +17,7 @@ public class _458Test {
 
     @BeforeClass
     public static void setup() {
-        test = new _458();
+        solution1 = new _458.Solution1();
     }
 
     @Before
@@ -32,7 +32,7 @@ public void test1() {
         minutesToDie = 15;
         minutesToTest = 60;
         expected = 5;
-        actual = test.poorPigs(buckets, minutesToDie, minutesToTest);
+        actual = solution1.poorPigs(buckets, minutesToDie, minutesToTest);
         assertEquals(expected, actual);
     }
 
@@ -42,7 +42,7 @@ public void test2() {
         minutesToDie = 1;
         minutesToTest = 1;
         expected = 0;
-        actual = test.poorPigs(buckets, minutesToDie, minutesToTest);
+        actual = solution1.poorPigs(buckets, minutesToDie, minutesToTest);
         assertEquals(expected, actual);
     }
 
@@ -52,7 +52,7 @@ public void test3() {
         minutesToDie = 12;
         minutesToTest = 60;
         expected = 4;
-        actual = test.poorPigs(buckets, minutesToDie, minutesToTest);
+        actual = solution1.poorPigs(buckets, minutesToDie, minutesToTest);
         assertEquals(expected, actual);
     }
 }
