CountryRunnerJPanel.java is over 500 lines long. Without reading the code, one can safely assume this class is responsible for more than one aspect of the application. Consider refactoring this class and the project as a whole into separate packages (i.e. model, view, controller) where each class is only responsible for one aspect.
For example, CountryRunnerJPanel.java should only be responsible for updating the actual GUI, and it should ask (or be told by) the controller to update with provided information that the controller gets from the model (Information about the game's state).
Sierrasch
commented
6 years ago
CountryRunnerJPanel.javais over 500 lines long. Without reading the code, one can safely assume this class is responsible for more than one aspect of the application. Consider refactoring this class and the project as a whole into separate packages (i.e. model, view, controller) where each class is only responsible for one aspect.For example,
CountryRunnerJPanel.javashould only be responsible for updating the actual GUI, and it should ask (or be told by) the controller to update with provided information that the controller gets from the model (Information about the game's state).F17 OK 250pts