fix lab 10

src/ca/bcit/comp1510/lab10/Collisions.java

@@ -37,15 +37,23 @@ public class Collisions {
         boolean collided = false;
         int stepsTaken = 0;
         
-        for (int i = 0; i < steps && !collided; i++) {
+        for (int i = 0; i < steps; i++) {
             w1.takeStep();
             w2.takeStep();
-            collided = samePosition(w1, w2);
+
+            if (samePosition(w1, w2)) {                
+                collided = true;
+                System.out.println("Collision at (" 
+                        + w1.getX() + ", "
+                        + w1.getY() + ") after " 
+                        + stepsTaken + " steps");
+            }
+            
             stepsTaken++;
         }
 
-        String message = collided ? "Collision!" : "No Collision.";
+        if (!collided) {
-        System.out.println(message);
+            System.out.println("No collisions.");
-        System.out.println("After " + stepsTaken + " steps");
+        }
     }
 }

src/ca/bcit/comp1510/lab10/Name.java

@@ -101,12 +101,10 @@ public class Name implements Comparable<Name> {
     public int compareTo(Name n) {
         if (!this.last.equals(n.last)) {
             return this.last.compareTo(n.last);
+        } else if (!this.first.equals(n.first)) {
+            return this.first.compareTo(n.first);
         } else {
-            if (!this.first.equals(n.first)) {
+            return this.middle.compareTo(n.middle);
-                return this.first.compareTo(n.first);
-            } else {
-                return this.middle.compareTo(n.middle);
-            }
         }
     }