From 148108a6581b16d346c891f4a1165e20ed832a96 Mon Sep 17 00:00:00 2001 From: fjc40 Date: Thu, 3 Aug 2017 21:11:25 +1200 Subject: [PATCH] CommandFactory uses BoatActionEnum instead of raw ints. #story[1094] --- .../java/mock/model/commandFactory/CommandFactory.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/racevisionGame/src/main/java/mock/model/commandFactory/CommandFactory.java b/racevisionGame/src/main/java/mock/model/commandFactory/CommandFactory.java index 17834963..6e87f11b 100644 --- a/racevisionGame/src/main/java/mock/model/commandFactory/CommandFactory.java +++ b/racevisionGame/src/main/java/mock/model/commandFactory/CommandFactory.java @@ -2,6 +2,7 @@ package mock.model.commandFactory; import mock.model.MockBoat; import mock.model.MockRace; +import network.Messages.Enums.BoatActionEnum; /** * Factory class for Command objects @@ -14,10 +15,10 @@ public class CommandFactory { * @param action number to select command * @return */ - public static Command createCommand(MockRace race, MockBoat boat, int action) { + public static Command createCommand(MockRace race, MockBoat boat, BoatActionEnum action) { switch(action) { - case 1: return new VMGCommand(race, boat); - case 4: return new TackGybeCommand(race, boat); + case AUTO_PILOT: return new VMGCommand(race, boat); + case TACK_GYBE: return new TackGybeCommand(race, boat); default: return null; // TODO - please please have discussion over what to default to } }