Refactored BoatinRace to observable list

- Changed starting Boats into ObservableList from array
- stopped checkPosition in Race abstract class from updating the table as everytime as it is checked by a listener
- Changed array methods .length to .size() and [i] to .get() to reflect the type change.
#refactor #story[9]

.idea/copyright/profiles_settings.xml

@@ -0,0 +1,3 @@
+<component name="CopyrightManager">
+  <settings default="" />
+</component>

.idea/modules.xml

@@ -2,6 +2,7 @@
 <project version="4">
   <component name="ProjectModuleManager">
     <modules>
+      <module fileurl="file://$PROJECT_DIR$/team-7.iml" filepath="$PROJECT_DIR$/team-7.iml" />
       <module fileurl="file://$PROJECT_DIR$/.idea/team-7.iml" filepath="$PROJECT_DIR$/.idea/team-7.iml" />
     </modules>
   </component>

src/main/java/seng302/Controllers/RaceController.java

@@ -4,6 +4,7 @@ package seng302.Controllers;
 import javafx.beans.property.ReadOnlyObjectWrapper;
 import javafx.beans.value.ObservableValue;
 import javafx.collections.FXCollections;
+import javafx.collections.ObservableList;
 import javafx.fxml.FXML;
 import javafx.scene.canvas.GraphicsContext;
 import javafx.scene.control.TableColumn;
@@ -47,15 +48,16 @@ public class RaceController extends Controller{
     private RaceMap map;
 
 
-    public void updateMap(BoatInRace[] boats) {
-        raceMap.setBoats(boats);
+    public void updateMap(ObservableList<BoatInRace> boats) {
+        BoatInRace[] boatInRaces = new BoatInRace[boats.size()];
+        raceMap.setBoats(boats.toArray(boatInRaces));
         raceMap.drawRaceMap();
 
     }
 
     public void updateInfoTable(Race race) {
         boatInfoTable.getItems().clear();
-        boatInfoTable.setItems(FXCollections.observableArrayList(race.getStartingBoats()));
+        boatInfoTable.setItems(race.getStartingBoats());
 
         boatTeamColumn.setCellValueFactory(new PropertyValueFactory<BoatInRace,String>("Name"));
         boatMarkColumn.setCellValueFactory(new PropertyValueFactory<BoatInRace, String>("CurrentLeg"));

src/main/java/seng302/Model/Race.java

@@ -1,6 +1,9 @@
 package seng302.Model;
 
 
+import javafx.collections.FXCollections;
+import javafx.collections.ObservableArray;
+import javafx.collections.ObservableList;
 import seng302.Controllers.RaceController;
 import seng302.GPSCoordinate;
 
@@ -11,7 +14,8 @@ import java.util.*;
  * Created by fwy13 on 3/03/17.
  */
 public abstract class Race implements Runnable {
-    protected BoatInRace[] startingBoats;
+    //protected BoatInRace[] startingBoats;
+    protected ObservableList<BoatInRace> startingBoats;
     protected ArrayList<Leg> legs;
     protected RaceController controller;
     protected int boatsFinished = 0;
@@ -25,7 +29,7 @@ public abstract class Race implements Runnable {
      * @param legs Number of marks in order that the boats pass in order to complete the race.
      */
     public Race(BoatInRace[] boats, ArrayList<Leg> legs, RaceController controller) {
-        this.startingBoats = boats;
+        this.startingBoats = FXCollections.observableArrayList(boats);
         this.legs = legs;
         this.legs.add(new Leg("Finish"));
         this.controller = controller;
@@ -45,9 +49,9 @@ public abstract class Race implements Runnable {
         //show the boats participating.
         System.out.println("Boats Participating:");
         System.out.println("====================");
-        for (int i = 0; i < startingBoats.length; i++) {
-            System.out.println(i + 1 + ". " + startingBoats[i].getName() + ", Speed: " + Math.round(startingBoats[i].getVelocity() * 1.94384) + "kn");
-            startingBoats[i].setCurrentLeg(legs.get(0));
+        for (int i = 0; i < startingBoats.size(); i++) {
+            System.out.println(i + 1 + ". " + startingBoats.get(i).getName() + ", Speed: " + Math.round(startingBoats.get(i).getVelocity() * 1.94384) + "kn");
+            startingBoats.get(i).setCurrentLeg(legs.get(0));
         }
     }
 
@@ -68,7 +72,7 @@ public abstract class Race implements Runnable {
         long remainingSeconds;
 
 
-        while (boatsFinished < startingBoats.length) {
+        while (boatsFinished < startingBoats.size()) {
             timeLoopStarted = System.currentTimeMillis();
             totalTimeElapsed = System.currentTimeMillis() - timeRaceStarted;
             long currentTime = System.currentTimeMillis() - timeRaceStarted;
@@ -95,7 +99,7 @@ public abstract class Race implements Runnable {
 
     protected void checkPosition(BoatInRace boat, long timeElapsed) {
         if (boat.getDistanceTravelledInLeg() > boat.getCurrentLeg().getDistance()){
-            updateController();
+            //updateController();
             //boat has passed onto new leg
             if (boat.getCurrentLeg().getName().equals("Finish")) {
                 //boat has finished
@@ -115,7 +119,7 @@ public abstract class Race implements Runnable {
     }
 
 
-    public BoatInRace[] getStartingBoats() {
+    public ObservableList<BoatInRace> getStartingBoats() {
         return startingBoats;
     }