# HG changeset patch # User tatsuki # Date 1438702617 -32400 # Node ID a0c9710e05666c3ce6ae4234247e7850159610ee # Parent 7da9056e9357b5fd380ae1445cde9126a2b1781b List iterator bug fix diff -r 7da9056e9357 -r a0c9710e0566 src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/data/list/List.java --- a/src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/data/list/List.java Tue Aug 04 23:07:43 2015 +0900 +++ b/src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/data/list/List.java Wed Aug 05 00:36:57 2015 +0900 @@ -61,7 +61,7 @@ @Override public boolean hasNext() { - return currentNode.getNext().getAttribute() != null; + return currentNode.getAttribute() != null; } @Override @@ -76,7 +76,7 @@ public Iterator reverseIterator() { Node currentNode = head.getNext(); Stack stack = new Stack<>(); - while (currentNode.getNext().getAttribute() != null) { + while (currentNode.getAttribute() != null) { stack.push(currentNode.getAttribute()); currentNode = currentNode.getNext(); } diff -r 7da9056e9357 -r a0c9710e0566 src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/transaction/DefaultTreeNode.java --- a/src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/transaction/DefaultTreeNode.java Tue Aug 04 23:07:43 2015 +0900 +++ b/src/main/java/jp/ac/u_ryukyu/ie/cr/jungle/transaction/DefaultTreeNode.java Wed Aug 05 00:36:57 2015 +0900 @@ -54,11 +54,6 @@ } @Override - public String getNodeId() { - return nodeId; - } - - @Override public int compareTo(TreeNode o) { return this.hashCode() - o.hashCode(); } diff -r 7da9056e9357 -r a0c9710e0566 src/test/java/jp/ac/u_ryukyu/ie/cr/tatsuki/list/listIterator.java --- a/src/test/java/jp/ac/u_ryukyu/ie/cr/tatsuki/list/listIterator.java Tue Aug 04 23:07:43 2015 +0900 +++ b/src/test/java/jp/ac/u_ryukyu/ie/cr/tatsuki/list/listIterator.java Wed Aug 05 00:36:57 2015 +0900 @@ -13,18 +13,18 @@ @Test public void listIteratorTest() { List list = new List(); - for (int count = 1; count <= 100; count++) { + for (int count = 1; count < 10; count++) { list = list.addLast(count); } Iterator iterator = list.iterator(); - for (int count = 1; count < 100; count++) { + for (int count = 1; count < 10; count++) { Assert.assertTrue(iterator.hasNext()); int attribute = iterator.next(); Assert.assertEquals(attribute, count); } iterator = list.reverseIterator(); - for (int count = 100; count > 0; count--) { + for (int count = 9; count > 0; count--) { Assert.assertTrue(iterator.hasNext()); int attribute = iterator.next(); Assert.assertEquals(attribute, count);