From 7b3df173ad8d34592d75c8b62c88eee244abe6a6 Mon Sep 17 00:00:00 2001
From: miermontoto <mier@mier.info>
Date: Thu, 20 Oct 2022 16:49:03 +0200
Subject: [PATCH] small fixes

---
 src/game/LauncherRF.java |  9 ++++++---
 src/handler/Loader.java  | 26 ++++++++++----------------
 src/obj/Settings.java    | 20 ++++++++++----------
 3 files changed, 26 insertions(+), 29 deletions(-)

diff --git a/src/game/LauncherRF.java b/src/game/LauncherRF.java
index 3c19e62..5db9114 100755
--- a/src/game/LauncherRF.java
+++ b/src/game/LauncherRF.java
@@ -876,12 +876,15 @@ private void jmiNuevaPartidaActionPerformed(java.awt.event.ActionEvent evt) {//G
     }//GEN-LAST:event_jmiNuevaPartidaActionPerformed
 
     private void jmiTrucosLoopSlowdownActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_jmiTrucosLoopSlowdownActionPerformed
-        var result = Integer.parseInt(Dialog.input("Introduzca el valor del nuevo slowdown (ms):", (x) -> (0 <= Integer.parseInt(x))));
-        loopComms.setSlowdown(result);
+        try {
+            loopComms.setSlowdown(Integer.parseInt(Dialog.input("Introduzca el valor del nuevo slowdown (ms):", (x) -> (0 <= Integer.parseInt(x)))));
+        } catch (NumberFormatException nfe) {Dialog.showError("El valor introducido no es un número.");}
     }//GEN-LAST:event_jmiTrucosLoopSlowdownActionPerformed
 
     private void jmiTrucosLoopLimitActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_jmiTrucosLoopLimitActionPerformed
-        loopComms.setLimit(Integer.parseInt(Dialog.input("Introduzca el límite de iteraciones:", (x) -> (Integer.parseInt(x) >= 0))));
+        try {
+            loopComms.setLimit(Integer.parseInt(Dialog.input("Introduzca el límite de iteraciones:", (x) -> (Integer.parseInt(x) >= 0))));
+        } catch (NumberFormatException nfe) {Dialog.showError("El valor introducido no es un número.");}
     }//GEN-LAST:event_jmiTrucosLoopLimitActionPerformed
 
     private void pneInfoComponentShown(java.awt.event.ComponentEvent evt) {//GEN-FIRST:event_pneInfoComponentShown
diff --git a/src/handler/Loader.java b/src/handler/Loader.java
index 8382014..5d74ed1 100644
--- a/src/handler/Loader.java
+++ b/src/handler/Loader.java
@@ -15,27 +15,21 @@
  * @author Juan Mier
  */
 public class Loader {
-    
+
     private final String title;
     private final FileNameExtensionFilter fnef;
-    
-    /**
-     * Constrctor generado para cumplir con SonarLint:S1118.
-     * 
-     * @see <a href="https://sonarcloud.io/organizations/default/rules?languages=java&open=java%3AS1118&q=S1118">
-     * 		Regla SonarLint:S1118 </a>
-     */
-    public Loader(String s, FileNameExtensionFilter f) {
-        title = s;
-        fnef = f;
+
+    public Loader(String windowTitle, FileNameExtensionFilter fnef) {
+        this.title = windowTitle;
+        this.fnef = fnef;
     }
-    
+
     public File load() {
         JFileChooser jfc = jfcGenerator();
         int res = jfc.showOpenDialog(null);
         return res == JFileChooser.APPROVE_OPTION ? jfc.getSelectedFile() : null;
     }
-    
+
     private JFileChooser jfcGenerator() {
         JFileChooser jfc = new JFileChooser("./");
         jfc.setMultiSelectionEnabled(false);
@@ -44,7 +38,7 @@ private JFileChooser jfcGenerator() {
         jfc.setFileFilter(fnef);
         return jfc;
     }
-    
+
     public String loadString() {
         File f = this.load();
         if(f == null) return null;
@@ -52,11 +46,11 @@ public String loadString() {
             return Files.readString(f.toPath());
         } catch (IOException ex) {return null;}
     }
-    
+
     public File save() {
         JFileChooser jfc = jfcGenerator();
         int res = jfc.showSaveDialog(null);
         return res == JFileChooser.APPROVE_OPTION ? jfc.getSelectedFile() : null;
     }
-    
+
 }
diff --git a/src/obj/Settings.java b/src/obj/Settings.java
index e62a2d1..f81d947 100755
--- a/src/obj/Settings.java
+++ b/src/obj/Settings.java
@@ -68,27 +68,27 @@ public boolean getStatus(String settingName) {
 
     /**
      * Método que establece el valor de una opción.
-     * @param s Opción del que se quiere cambiar el valor.
-     * @param b Valor a establecer.
+     * @param settingName Opción del que se quiere cambiar el valor.
+     * @param bool Valor a establecer.
      * @return Devuelve el valor booleano que devuelve {@code put} o {@code false} si la opción no existe.
      */
-    public boolean setStatus(String s, boolean b) {
-        if(!settingsMap.containsKey(s)) {
-            System.err.println("Opción no existente: " + s);
+    public boolean setStatus(String settingName, boolean bool) {
+        if(!settingsMap.containsKey(settingName)) {
+            System.err.println("Opción no existente: " + settingName);
             return false;
         }
-        return settingsMap.put(s, b);
+        return settingsMap.put(settingName, bool);
     }
 
     /**
      * Método que conmuta el valor de una opción.
-     * @param s Nombre de la opción a conmutar.
+     * @param settingName Nombre de la opción a conmutar.
      * @return Devuelve el nuevo valor booleano de la opción.
      */
-    public boolean toggleStatus(String s) {
+    public boolean toggleStatus(String settingName) {
         //if(!settingsMap.containsKey(s)) throw new OperationNotSupportedException("Opción no existente.");
-        boolean newValue = !settingsMap.get(s);
-        settingsMap.put(s, newValue);
+        boolean newValue = !settingsMap.get(settingName);
+        settingsMap.put(settingName, newValue);
         return newValue;
     }