Skip to content

Commit 0a7b9a2

Browse files
committed
refactor(server): more Configuration option sorting
1 parent a42c7e5 commit 0a7b9a2

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

server/src/main/java/sc/server/Application.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -75,17 +75,19 @@ public static void parseArguments(String[] params)
7575
throws IllegalOptionValueException, UnknownOptionException {
7676
CmdLineParser parser = new CmdLineParser();
7777
Option pluginDirOption = parser.addStringOption(Configuration.PLUGINS_OPTION);
78+
Option portOption = parser.addIntegerOption('p', Configuration.PORT_KEY);
79+
7880
Option loadGameFileOption = parser.addStringOption(Configuration.GAMELOADFILE);
7981
Option turnToLoadOption = parser.addIntegerOption(Configuration.TURN_TO_LOAD);
8082
Option saveReplayOption = parser.addBooleanOption(Configuration.SAVE_REPLAY);
81-
Option portOption = parser.addIntegerOption('p', Configuration.PORT_KEY);
8283
parser.parse(params);
8384

84-
String path = (String) parser.getOptionValue(pluginDirOption, null);
85+
String pluginPath = (String) parser.getOptionValue(pluginDirOption, null);
86+
String port = parser.getOptionValue(portOption, SharedConfiguration.DEFAULT_PORT).toString();
87+
8588
String loadGameFile = (String) parser.getOptionValue(loadGameFileOption, null);
8689
Integer turnToLoad = (Integer) parser.getOptionValue(turnToLoadOption, 0);
8790
Boolean saveReplay = (Boolean) parser.getOptionValue(saveReplayOption, false);
88-
String port = parser.getOptionValue(portOption, SharedConfiguration.DEFAULT_PORT).toString();
8991

9092
Configuration.set(Configuration.PORT_KEY, port);
9193
if (loadGameFile != null) {
@@ -97,10 +99,10 @@ public static void parseArguments(String[] params)
9799
if (saveReplay)
98100
Configuration.set(Configuration.SAVE_REPLAY, saveReplay.toString());
99101

100-
if (path != null) {
101-
File pluginDir = new File(path).getAbsoluteFile();
102+
if (pluginPath != null) {
103+
File pluginDir = new File(pluginPath).getAbsoluteFile();
102104
if (pluginDir.exists() && pluginDir.isDirectory()) {
103-
Configuration.set(Configuration.PLUGINS_OPTION, path);
105+
Configuration.set(Configuration.PLUGINS_OPTION, pluginPath);
104106
logger.info("Loading plugins from {}", pluginDir);
105107
} else {
106108
logger.warn("Could not find {} to load plugins from", pluginDir);

server/src/main/java/sc/server/gaming/GameRoom.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,16 @@
2323
import sc.protocol.room.*;
2424
import sc.server.Configuration;
2525
import sc.server.network.Client;
26-
import sc.shared.*;
26+
import sc.shared.GameResult;
27+
import sc.shared.InvalidMoveException;
28+
import sc.shared.SlotDescriptor;
29+
import sc.shared.Violation;
2730

2831
import java.io.*;
29-
import java.util.*;
32+
import java.util.ArrayList;
33+
import java.util.Collection;
34+
import java.util.Collections;
35+
import java.util.List;
3036
import java.util.stream.Collectors;
3137

3238
/**

0 commit comments

Comments
 (0)