Review

.gitignore

@@ -0,0 +1,9 @@
+/target/
+/.settings/
+/.classpath
+/.project
+
+target/
+.idea/
+*.iml
+*.iws

src/main/java/edu/kis/vh/nursery/DefaultCountingOutRhymer.java

@@ -0,0 +1,86 @@
+package edu.kis.vh.nursery;
+
+/**
+ * Klasa DefaultCountingOutRhymer implementuje licznik, który przechowuje
+ * elementy w tablicy o stałej wielkości. Ma takie opercje jak dodawanie elementu
+ * do tablicy, sprawdzenie czy licznik jest pusty lub pełny oraz podgląd
+ * ostatniego elementu i jego usuwanie z tablicy.
+ */
+public class DefaultCountingOutRhymer {
+    /**
+     * Wartość, która oznacza, że tablica jest pusta.
+     */
+    private static final int EMPTY = -1;
+    /**
+     * Wartość minimalna zwracana przez metodę peekaboo, gdy licznik jest pusty.
+     */
+    private static final int PEEKABOO_MIN = -1;
+    /**
+     * Rozmiar tablicy przechowującej elementy licznika.
+     */
+    private static final int MAX_SIZE = 12;
+    /**
+     * Tablica przechowująca elementy licznika.
+     */
+    private final int[] numbers = new int[MAX_SIZE];
+    /**
+     * Indeks wskazujący na ostatni dodany element w tablicy
+     * z przypisaną wartością domyślną EMPTY.
+     */
+    private int total = EMPTY;
+
+    /**
+     * Zwraca obecny indeks ostatniego elementu w liczniku.
+     * @return indeks ostatniego elementu
+     */
+    public int getTotal() {
+        return total;
+    }
+
+    /**
+     * Dodaje nowy element do tablicy.
+     * Przed dododwaniem sprawdza, czy tablica nie jest pełna
+     * @param in wartość dodawanego elementu
+     */
+    public void countIn(int in) {
+        if (!isFull())
+            numbers[++total] = in;
+    }
+
+    /**
+     * Sprawdza czy licznik jest pusty.
+     * @return true, jak licznik jest pusty, false w przeciwnym przypadku
+     */
+    public boolean callCheck() {
+        return getTotal() == EMPTY;
+    }
+
+    /**
+     * Sprawdza czy licznik jest pełny.
+     * @return true, jak licznik jest pełny, false w przeciwnym przypadku
+     */
+    public boolean isFull() {
+        return getTotal() == MAX_SIZE - 1;
+    }
+
+    /**
+     * Zwraca ostatni dodany element bez jego usuwania z tablicy.
+     * @return ostatni element w tablicy lub -1 (PEEKABOO_MIN) w przypadku pustej tablicy licznika
+     */
+    protected int peekaboo() {
+        if (callCheck())
+            return PEEKABOO_MIN;
+        return numbers[getTotal()];
+    }
+
+    /**
+     * Zwraca ostatni element z tablicy oraz usuwa go.
+     * @return osatni element w tablicy lub -1 (MIN) jeżeli tablica jest pusta
+     */
+    public int countOut() {
+        if (callCheck())
+            return EMPTY;
+        return numbers[total--];
+    }
+
+}

src/main/java/edu/kis/vh/nursery/FIFORhymer.java

@@ -1,21 +1,23 @@
 package edu.kis.vh.nursery;
 
-public class FIFORhymer extends defaultCountingOutRhymer {
+public class FifoRhymer extends DefaultCountingOutRhymer {
+
+    private final DefaultCountingOutRhymer temp = new DefaultCountingOutRhymer();
 
-    public defaultCountingOutRhymer temp = new defaultCountingOutRhymer();
-
     @Override
     public int countOut() {
         while (!callCheck())
-
-        temp.countIn(super.countOut());
-
-        int ret = temp.countOut();
-
-        while (!temp.callCheck())
-
-        countIn(temp.countOut());
-
+            getTemp().countIn(super.countOut());
+
+        int ret = getTemp().countOut();
+
+        while (!getTemp().callCheck())
+            countIn(getTemp().countOut());
+
         return ret;
     }
+
+    public DefaultCountingOutRhymer getTemp() {
+        return temp;
+    }
 }

src/main/java/edu/kis/vh/nursery/HanoiRhymer.java

@@ -1,17 +1,26 @@
 package edu.kis.vh.nursery;
 
-public class HanoiRhymer extends defaultCountingOutRhymer {
+public class HanoiRhymer extends DefaultCountingOutRhymer {
 
-int totalRejected = 0;
+    private int totalRejected = 0;
 
     public int reportRejected() {
-        return totalRejected;
+        return getTotalRejected();
     }
 
+    @Override
     public void countIn(int in) {
-    if (!callCheck() && in > peekaboo())
-            totalRejected++;
-            else
-                super.countIn(in);
+        if (!callCheck() && in > peekaboo())
+            setTotalRejected(getTotalRejected() + 1);
+        else
+            super.countIn(in);
+    }
+
+    public int getTotalRejected() {
+        return totalRejected;
+    }
+
+    public void setTotalRejected(int totalRejected) {
+        this.totalRejected = totalRejected;
     }
 }

src/main/java/edu/kis/vh/nursery/factory/DefaultRhymersFactory.java

@@ -1,29 +1,28 @@
 package edu.kis.vh.nursery.factory;
 
-import edu.kis.vh.nursery.defaultCountingOutRhymer;
-import edu.kis.vh.nursery.FIFORhymer;
+import edu.kis.vh.nursery.DefaultCountingOutRhymer;
+import edu.kis.vh.nursery.FifoRhymer;
 import edu.kis.vh.nursery.HanoiRhymer;
-import edu.kis.vh.nursery.factory.Rhymersfactory;
 
-public class DefaultRhymersFactory implements Rhymersfactory {
+public class DefaultRhymersFactory implements RhymersFactory {
 
     @Override
-    public defaultCountingOutRhymer GetStandardRhymer() {
-        return new defaultCountingOutRhymer();
+    public DefaultCountingOutRhymer getStandardRhymer() {
+        return new DefaultCountingOutRhymer();
     }
 
     @Override
-    public defaultCountingOutRhymer GetFalseRhymer() {
-        return new defaultCountingOutRhymer();
+    public DefaultCountingOutRhymer getFalseRhymer() {
+        return new DefaultCountingOutRhymer();
     }
 
     @Override
-    public defaultCountingOutRhymer GetFIFORhymer() {
-        return new FIFORhymer();
+    public DefaultCountingOutRhymer getFIFORhymer() {
+        return new FifoRhymer();
     }
 
     @Override
-    public defaultCountingOutRhymer GetHanoiRhymer() {
+    public DefaultCountingOutRhymer getHanoiRhymer() {
         return new HanoiRhymer();
     }
 

src/main/java/edu/kis/vh/nursery/factory/RhymersFactory.java

@@ -0,0 +1,15 @@
+package edu.kis.vh.nursery.factory;
+
+import edu.kis.vh.nursery.DefaultCountingOutRhymer;
+
+public interface RhymersFactory {
+
+    public DefaultCountingOutRhymer getStandardRhymer();
+
+    public DefaultCountingOutRhymer getFalseRhymer();
+
+    public DefaultCountingOutRhymer getFIFORhymer();
+
+    public DefaultCountingOutRhymer getHanoiRhymer();
+
+}

src/main/java/edu/kis/vh/nursery/list/IntLinkedList.java

@@ -2,21 +2,22 @@
 
 public class IntLinkedList {
 
-    Node last;
-    int i;
+    private static final int MIN = -1;
+    private Node last;
+    private int i;
 
     public void push(int i) {
-        if (last == null)
-            last = new Node(i);
+        if (getLast() == null)
+            setLast(new Node(i));
         else {
-            last.next = new Node(i);
-            last.next.prev = last;
-            last = last.next;
+            getLast().setNext(new Node(i));
+            getLast().getNext().setPrev(getLast());
+            setLast(getLast().getNext());
         }
     }
 
     public boolean isEmpty() {
-        return last == null;
+        return getLast() == null;
     }
 
     public boolean isFull() {
@@ -25,16 +26,24 @@ public boolean isFull() {
 
     public int top() {
         if (isEmpty())
-            return -1;
-        return last.value;
+            return MIN;
+        return getLast().getValue();
     }
 
     public int pop() {
         if (isEmpty())
-            return -1;
-        int ret = last.value;
-        last = last.prev;
+            return MIN;
+        int ret = getLast().getValue();
+        setLast(getLast().getPrev());
         return ret;
     }
 
+    public Node getLast() {
+        return last;
+    }
+
+    public void setLast(Node last) {
+        this.last = last;
+    }
+
 }

src/main/java/edu/kis/vh/nursery/list/Node.java

@@ -2,11 +2,31 @@
 
 public class Node {
 
-    public int value;
-    public Node prev, next;
+    private final int value;
+    private Node prev;
+    private Node next;
 
     public Node(int i) {
         value = i;
     }
-
+
+    public int getValue() {
+        return value;
+    }
+
+    public Node getPrev() {
+        return prev;
+    }
+
+    public void setPrev(Node prev) {
+        this.prev = prev;
+    }
+
+    public Node getNext() {
+        return next;
+    }
+
+    public void setNext(Node next) {
+        this.next = next;
+    }
 }