updates

vogella · vogella · commit a626e6c8a7f5 · 2012-10-10T17:22:24.000+02:00
diff --git a/de.vogella.datastructures.list/src/de/vogella/datastructures/list/MyList.java b/de.vogella.datastructures.list/src/de/vogella/datastructures/list/MyList.java
@@ -26,7 +26,7 @@ private void ensureCapa() {
 
 	@SuppressWarnings("unchecked")
 	public E get(int i) {
-		if (i>= elements.length) {
+		if (i>= size || i <0) {
 			throw new IndexOutOfBoundsException("Index: " + i + ", Size " + i );
 		}
 		return (E) elements[i];
diff --git a/de.vogella.datastructures.list/test/de/vogella/datastructures/list/MyListTest.java b/de.vogella.datastructures.list/test/de/vogella/datastructures/list/MyListTest.java
@@ -21,7 +21,19 @@ public void testMyList() {
 		assertTrue(2 == list.get(1));
 		assertTrue(3 == list.get(2));
 		
-		list.get(100);
+		list.get(6);
+	}
+	
+
+	@Test(expected=IndexOutOfBoundsException.class)
+	public void testNegative() {
+		MyList<Integer> list = new MyList<Integer>();
+		list.add(1);
+		list.add(2);
+		list.add(3);
+		list.add(3);
+		list.add(4);
+		list.get(-1);
 	}
 
 	@Test(expected=IndexOutOfBoundsException.class)
@@ -36,7 +48,7 @@ public void testList() {
 		assertTrue(2 == list.get(1));
 		assertTrue(3 == list.get(2));
 		
-		list.get(100);
+		list.get(6);
 	}
 	
 	

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ private void ensureCapa() {`
`26`	`26`
`27`	`27`	`@SuppressWarnings("unchecked")`
`28`	`28`	`public E get(int i) {`
`29`		`- if (i>= elements.length) {`
	`29`	`+ if (i>= size \|\| i <0) {`
`30`	`30`	`throw new IndexOutOfBoundsException("Index: " + i + ", Size " + i );`
`31`	`31`	`}`
`32`	`32`	`return (E) elements[i];`