Made SailsInCommand and SailsOutCommand one class. #story[1098]
This commit is contained in:
@@ -33,7 +33,7 @@ public class MockBoat extends Boat {
|
||||
/**
|
||||
* Stores whether the boat is on autoVMG or not
|
||||
*/
|
||||
private boolean autoVMG = true;
|
||||
private boolean autoVMG = false;
|
||||
|
||||
|
||||
|
||||
|
||||
@@ -321,10 +321,10 @@ public class MockRace extends Race {
|
||||
|
||||
if (!finish && totalElapsedMilliseconds >= updatePeriodMilliseconds && !boat.isSailsOut()) {
|
||||
|
||||
if (boat.getCurrentSpeed() == 0) {
|
||||
newOptimalVMG(boat);
|
||||
boat.setBearing(boat.calculateBearingToNextMarker());
|
||||
}
|
||||
// if (boat.getCurrentSpeed() == 0) {
|
||||
// newOptimalVMG(boat);
|
||||
// boat.setBearing(boat.calculateBearingToNextMarker());
|
||||
// }
|
||||
|
||||
setBoatSpeed(boat);
|
||||
|
||||
|
||||
@@ -19,8 +19,10 @@ public class CommandFactory {
|
||||
switch(action) {
|
||||
case AUTO_PILOT: return new VMGCommand(race, boat);
|
||||
case TACK_GYBE: return new TackGybeCommand(race, boat);
|
||||
case SAILS_IN: return new SailsInCommand(race, boat);
|
||||
case SAILS_OUT: return new SailsOutCommand(race, boat);
|
||||
case UPWIND: return new WindCommand(race, boat, true);
|
||||
case DOWNWIND: return new WindCommand(race, boat, false);
|
||||
case SAILS_OUT: return new SailsCommand(race, boat, true);
|
||||
case SAILS_IN: return new SailsCommand(race, boat, false);
|
||||
default: return null; // TODO - please please have discussion over what to default to
|
||||
}
|
||||
}
|
||||
|
||||
+5
-4
@@ -3,18 +3,19 @@ package mock.model.commandFactory;
|
||||
import mock.model.MockBoat;
|
||||
import mock.model.MockRace;
|
||||
|
||||
public class SailsInCommand implements Command {
|
||||
public class SailsCommand implements Command {
|
||||
private MockRace race;
|
||||
private MockBoat boat;
|
||||
private boolean sailsOut;
|
||||
|
||||
public SailsInCommand(MockRace race, MockBoat boat) {
|
||||
public SailsCommand(MockRace race, MockBoat boat, Boolean sailsOut) {
|
||||
this.race = race;
|
||||
this.boat = boat;
|
||||
this.sailsOut = sailsOut;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void execute() {
|
||||
this.boat.setSailsOut(false);
|
||||
System.out.println("hi");
|
||||
this.boat.setSailsOut(this.sailsOut);
|
||||
}
|
||||
}
|
||||
@@ -1,20 +0,0 @@
|
||||
package mock.model.commandFactory;
|
||||
|
||||
import mock.model.MockBoat;
|
||||
import mock.model.MockRace;
|
||||
|
||||
public class SailsOutCommand implements Command {
|
||||
private MockRace race;
|
||||
private MockBoat boat;
|
||||
|
||||
public SailsOutCommand(MockRace race, MockBoat boat) {
|
||||
this.race = race;
|
||||
this.boat = boat;
|
||||
}
|
||||
|
||||
@Override
|
||||
public void execute() {
|
||||
this.boat.setSailsOut(true);
|
||||
System.out.println("hi2");
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user