minor refactoring

raydac · Mar 25, 2021 · 3db7098 · 3db7098
1 parent d53d4dd
commit 3db7098
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/...ships-client-pc/src/main/java/com/igormaznitsa/battleships/gui/BattleshipsCommDaemon.java b/...ships-client-pc/src/main/java/com/igormaznitsa/battleships/gui/BattleshipsCommDaemon.java
@@ -20,6 +20,7 @@
 import com.igormaznitsa.battleships.opponent.FirstMoveOrderProvider;
 import com.igormaznitsa.battleships.opponent.GameEventType;
 import com.igormaznitsa.battleships.utils.Utils;
+
 import java.time.Duration;
 import java.util.HashMap;
 import java.util.Map;
@@ -139,13 +140,7 @@ private void checkPlayersReady() {
         firstMoveOrderProvider = (FirstMoveOrderProvider) this.playerB;
       } else {
         LOGGER.info("Using internal dice to choose first turn player");
-        firstMoveOrderProvider = new FirstMoveOrderProvider() {
-          @Override
-          public BattleshipsPlayer findFirstTurnPlayer(final BattleshipsPlayer playerA,
-                                                       final BattleshipsPlayer playerB) {
-            return Utils.RND.nextBoolean() ? playerA : playerB;
-          }
-        };
+        firstMoveOrderProvider = (playerA, playerB) -> Utils.RND.nextBoolean() ? playerA : playerB;
       }
 
       final BattleshipsPlayer firstTurnPlayer = Objects

diff --git a/...n/battleships-client-pc/src/main/java/com/igormaznitsa/battleships/gui/OpeningDialog.java b/...n/battleships-client-pc/src/main/java/com/igormaznitsa/battleships/gui/OpeningDialog.java
@@ -101,7 +101,7 @@ public OpeningDialog(final StartOptions startOptions) {
     this.buttonGo.addActionListener(e -> {
       int hostPort = -1;
       try {
-        hostPort = Integer.parseInt(this.textFieldPort.getText().toString().trim());
+        hostPort = Integer.parseInt(this.textFieldPort.getText().trim());
       } catch (Exception ex) {
         // ignoring
       }