Merge branch 'CommandFactory' into 'master'

Tacking and gybing merge request

Note: this allows the player to perform a common maneuver, changing tack, quickly. The goal is to change the boat's heading from one tack to another with a single keypress.
If the boat is heading upwind, the keypress will cause the boat to tack through head-to-wind, so that the wind shifts from one side of the boat to the other. The TWA (true wind angle) after tacking should be 360-TWA(before_tacking).
Similarly for gybing if the boat is initially heading downwind, except that the boat's heading always remains away from the wind.

Acceptance criteria:

- The wind shifts from one side of the boat to the other on keypress
- When heading upwind the boat should tack on keypress
- When heading downwind the boat should gybe on keypress

[Manual test](https://eng-git.canterbury.ac.nz/seng302-2017/team-7/wikis/manual-test:-tacking-and-gybing)

See merge request !24

.idea/codeStyleSettings.xml

@@ -1,6 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project version="4">
-  <component name="ProjectCodeStyleSettingsManager">
-    <option name="PREFERRED_PROJECT_CODE_STYLE" value="Default" />
-  </component>
-</project>

racevisionGame/src/main/java/mock/model/MockRace.java

@@ -346,7 +346,7 @@ public class MockRace extends Race {
 
             this.updateEstimatedTime(boat);
         }
-
+        checkPosition(boat, totalElapsedMilliseconds);
     }
 
     private void newOptimalVMG(MockBoat boat) {
@@ -688,6 +688,7 @@ public class MockRace extends Race {
 
     }
 
+
     public List<CompoundMark> getCompoundMarks() {
         return compoundMarks;
     }

racevisionGame/src/main/java/mock/model/commandFactory/CommandFactory.java

@@ -13,7 +13,7 @@ public class CommandFactory {
      * @param race to receive command
      * @param boat to receive command in race
      * @param action number to select command
-     * @return
+     * @return command object corresponding to action
      */
     public static Command createCommand(MockRace race, MockBoat boat, BoatActionEnum action) {
         switch(action) {

racevisionGame/src/main/java/mock/model/commandFactory/TackGybeCommand.java

@@ -2,34 +2,49 @@ package mock.model.commandFactory;
 
 import mock.model.MockBoat;
 import mock.model.MockRace;
+import shared.model.Bearing;
 
 /**
- * Created by David on 2/08/2017.
+ * Command class for tacking and gybing
  */
 public class TackGybeCommand implements Command {
     private MockRace race;
     private MockBoat boat;
 
+    /**
+     * Constructor for class
+     * @param race mock race
+     * @param boat mock boat to update
+     */
     public TackGybeCommand(MockRace race, MockBoat boat) {
         this.race = race;
         this.boat = boat;
     }
 
-    //The refactoring of MockRace will require changes to be made
     @Override
     public void execute() {
+        double boatAngle = boat.getBearing().degrees();
+        double windAngle =race.getWindDirection().degrees();
+        double differenceAngle = calcDistance(boatAngle, windAngle);
+        double angleA = windAngle + differenceAngle;
+        double angleB = windAngle - differenceAngle;
+        if(angleA % 360 == boatAngle){
+            boat.setBearing(Bearing.fromDegrees(angleB));
+        } else {
+            boat.setBearing(Bearing.fromDegrees(angleA));
+        }
+    }
 
-        boat.setAutoVMG(false);
+    /**
-
+     * Method to calculate smallest angle between 2 angles
-        /*VMG newVMG = boat.getPolars().calculateVMG(
+     * @param degreeA first angle degree
-                race.getWindDirection(),
+     * @param degreeB second angle degree
-                race.getWindSpeed(),
+     * @return the calculated smallest angle
-                boat.calculateBearingToNextMarker(),
+     */
-                Bearing.fromDegrees(0d),
+    public double calcDistance(double degreeA, double degreeB){
-                Bearing.fromDegrees(359.99999d));
+        double phi = Math.abs(degreeB - degreeA) % 360;
-        VMG boatVMG = new VMG(boat.getCurrentSpeed(), boat.getBearing());
+        return phi > 180 ? 360 - phi : phi;
-        if(race.improvesVelocity(boatVMG, newVMG, boat.calculateBearingToNextMarker())){
-            boat.setVMG(newVMG);
-        }*/
     }
+
 }
+

racevisionGame/src/main/java/mock/model/commandFactory/VMGCommand.java

@@ -4,27 +4,30 @@ import mock.model.MockBoat;
 import mock.model.MockRace;
 
 /**
- * Created by David on 2/08/2017.
+ * Command class for autoVMG
  */
 public class VMGCommand implements Command {
     private MockRace race;
     private MockBoat boat;
 
+    /**
+     * Constructor for class
+     * @param race mock race
+     * @param boat mock boat to update
+     */
     public VMGCommand(MockRace race, MockBoat boat) {
         this.race = race;
         this.boat = boat;
     }
 
-    //The refactoring of MockRace will require changes to be made
     @Override
     public void execute() {
+        if (boat.getAutoVMG()){
+            boat.setAutoVMG(false);
+            System.out.println("Auto VMG off!");
+        } else {
             boat.setAutoVMG(true);
-        /*VMG newVMG = boat.getPolars().calculateVMG(
+            System.out.println("Auto VMG on!");
-                race.getWindDirection(),
+        }
-                race.getWindSpeed(),
-                boat.calculateBearingToNextMarker(),
-                Bearing.fromDegrees(0d),
-                Bearing.fromDegrees(359.99999d));
-        boat.setVMG(newVMG);*/
     }
 }

racevisionGame/src/main/java/visualiser/gameController/ControllerServer.java

@@ -30,19 +30,15 @@ public class ControllerServer extends Observable implements Runnable {
      * Last received boat action
      */
     private BoatActionEnum action;
-    /**
-     *
-     */
-    private RaceLogic rc;
 
     /**
      * Initialise server-side controller with live client socket
      * @param socket to client
+     * @param race logic loop observing controls
      */
-    public ControllerServer(Socket socket, RaceLogic rc) {
+    public ControllerServer(Socket socket, RaceLogic race) {
         this.socket = socket;
-        this.rc = rc;
+        this.addObserver(race);
-        this.addObserver(rc);
         try {
             this.inputStream = new DataInputStream(this.socket.getInputStream());
         } catch (IOException e) {