From 1ee3962c6326f6176e5b241cb9c564285b6d5336 Mon Sep 17 00:00:00 2001 From: rake6682 <97271338+rake6682@users.noreply.github.com> Date: Sun, 1 Jun 2025 10:46:22 -0700 Subject: [PATCH] Peer Review --- src/main/java/edu/sdccd/cisc191/game/Game.java | 6 ++++-- src/main/java/edu/sdccd/cisc191/game/Shipyard.java | 2 ++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/main/java/edu/sdccd/cisc191/game/Game.java b/src/main/java/edu/sdccd/cisc191/game/Game.java index b79a3cd..5f8830c 100644 --- a/src/main/java/edu/sdccd/cisc191/game/Game.java +++ b/src/main/java/edu/sdccd/cisc191/game/Game.java @@ -23,8 +23,10 @@ import java.util.concurrent.ScheduledExecutorService; // 12 import java.util.concurrent.TimeUnit; // 13 - - +// There is a lot going on here. Perhaps split this into +// A game Initializer/controller +// and a JavaFX UI controller? +// Game logic also appears to be non-functional // Main Game Class (Integrates JavaFX, Shipyard System, and Exploration System) public class Game extends Application { private Shipyard shipyard; diff --git a/src/main/java/edu/sdccd/cisc191/game/Shipyard.java b/src/main/java/edu/sdccd/cisc191/game/Shipyard.java index 108b9bc..48c5ae9 100644 --- a/src/main/java/edu/sdccd/cisc191/game/Shipyard.java +++ b/src/main/java/edu/sdccd/cisc191/game/Shipyard.java @@ -19,6 +19,8 @@ * Interactive Testing: main method allows quick testing of shipyard features */ +// If this is a test class, shouldn't this be in the tests folder? + public class Shipyard { private final Map availableShips; private final List playerFleet;