Gwendal in main : add test on futur constraints and enhance input symbol method (#1)

Co-authored-by: ROGER <gwendal.roger@etu.univ-lyon1.fr>
Reviewed-on: #1

app/src/main/java/sudoku/App.java

@@ -44,5 +44,14 @@ public class App {
 
         System.out.println("Symboles possibles :");
         sudoku.getGrille().printSymbolesPossibles();
+
+        //Création d'un second sudoku
+        System.out.println("Création d'un second SUDOKU");
+
+        Sudoku sudoku2 = new Sudoku(9);
+        sudoku2.getGrille().askSetSymbolesPossibles();
+
+        System.out.println("Symboles possibles :");
+        sudoku2.getGrille().printSymbolesPossibles();
     }
 }

app/src/main/java/sudoku/Grille.java

@@ -44,18 +44,19 @@ public class Grille {
 
     /**
      * Crée un bloc à partir des positions spécifiées
-            
+     * 
-            Exemple :
+     * Exemple :
-            sudoku.getGrille().createBloc(Arrays.asList(
+     * sudoku.getGrille().createBloc(Arrays.asList(
-            new int[] { 0, 0 },
+     * new int[] { 0, 0 },
-            new int[] { 0, 1 },
+     * new int[] { 0, 1 },
-            new int[] { 0, 2 },
+     * new int[] { 0, 2 },
-            new int[] { 1, 0 },
+     * new int[] { 1, 0 },
-            new int[] { 1, 1 },
+     * new int[] { 1, 1 },
-            new int[] { 1, 2 },
+     * new int[] { 1, 2 },
-            new int[] { 2, 0 },
+     * new int[] { 2, 0 },
-            new int[] { 2, 1 },
+     * new int[] { 2, 1 },
-            new int[] { 2, 2 }));
+     * new int[] { 2, 2 }));
+     * 
      * @param positions
      * @return
      */
@@ -107,15 +108,19 @@ public class Grille {
     }
 
     public void askSetSymbolesPossibles() {
-            try (Scanner scanner = new Scanner(System.in)) {
+        Scanner scanner = new Scanner(System.in);
-                System.out.println(("Choisissez le type de symboles :"));
+        try {
-                System.out.println(("1. Nombres"));
+            Console.infoln("Choisissez le type de symboles :");
-                System.out.println(("2. Caractères"));
+            Console.infoln("1. Nombres");
-                System.out.println(("3. Texte/Emoji"));
+            Console.infoln("2. Lettres");
+            Console.infoln("3. Texte/Emoji");
 
             int choix = 0;
             try {
                 choix = Integer.parseInt(scanner.nextLine());
+                if (choix < 1 || choix > 3) {
+                    throw new NumberFormatException("Choix invalide");
+                }
             } catch (NumberFormatException e) {
                 Console.errorln("Choix invalide");
                 return;
@@ -127,29 +132,42 @@ public class Grille {
 
                 switch (choix) {
                     case 1: // Nombres
-                            try {
+                        Symbole intTemp = Symbole.of(input);
-                                symbolesPossibles.add(Symbole.of(Integer.parseInt(input)));
+                        if (intTemp.isInt()) {
-                            } catch (NumberFormatException e) {
+                            if (symbolesPossibles.contains(intTemp)) {
+                                Console.errorln("Ce symbole existe déjà, veuillez entrer un autre symbole");
+                                i--;
+                            } else {
+                                symbolesPossibles.add(intTemp);
+                            }
+                        } else {
                             Console.errorln("Veuillez entrer un nombre valide");
                             i--;
                         }
                         break;
 
-                        case 2: // Caractères
+                    case 2: // Lettres
-                            if (input.length() == 1) {
+                        if (input.length() == 1 && Symbole.of(input.charAt(0)).isLetter()) {
-                                symbolesPossibles.add(Symbole.of(input.charAt(0)));
+                            Symbole charTemp = Symbole.of(input.charAt(0));
+                            if (symbolesPossibles.contains(charTemp)) {
+                                Console.errorln("Ce symbole existe déjà, veuillez entrer un autre symbole");
+                                i--;
                             } else {
-                                Console.errorln("Veuillez entrer un seul caractère");
+                                symbolesPossibles.add(charTemp);
+                            }
+                        } else {
+                            Console.errorln("Veuillez entrer une lettre valide");
                             i--;
                         }
                         break;
 
                     case 3: // Texte/Emoji
-                            if (!input.isEmpty()) {
+                        Symbole stringTemp = Symbole.of(input);
-                                symbolesPossibles.add(Symbole.of(input));
+                        if (symbolesPossibles.contains(stringTemp)) {
-                            } else {
+                            Console.errorln("Ce symbole existe déjà, veuillez entrer un autre symbole");
-                                Console.errorln("Le symbole ne peut pas être vide");
                             i--;
+                        } else {
+                            symbolesPossibles.add(stringTemp);
                         }
                         break;
 
@@ -158,6 +176,10 @@ public class Grille {
                         return;
                 }
             }
+        } catch (Exception e) {
+            System.out.println("Une erreur est survenue : " + e.getMessage());
+        } finally {
+            scanner.close();
         }
     }
 

app/src/main/java/sudoku/Symbole.java

@@ -20,6 +20,19 @@ public class Symbole {
         return new Symbole(String.valueOf(c));
     }
 
+    public boolean isInt() {
+        try {
+            Integer.parseInt(valeur);
+            return true;
+        } catch (NumberFormatException e) {
+            return false;
+        }
+    }
+
+    public boolean isLetter() {
+        return valeur.length() == 1 && Character.isLetter(valeur.charAt(0));
+    }
+
     @Override
     public String toString() {
         return valeur;
@@ -27,8 +40,11 @@ public class Symbole {
 
     @Override
     public boolean equals(Object obj) {
-        if (this == obj) return true;
+        if (this == obj)
-        if (obj == null || getClass() != obj.getClass()) return false;
+            return true;
+
+        if (obj == null || getClass() != obj.getClass())
+            return false;
         Symbole symbole = (Symbole) obj;
         return valeur.equals(symbole.valeur);
     }

app/src/main/java/sudoku/core/Console.java

@@ -11,14 +11,18 @@ public class Console {
     static final String ANSI_BOLD = "\u001B[1m";
 
     public static void errorln(String message) {
-        System.err.println(ANSI_RED + ANSI_BOLD + message + ANSI_RESET);
+        System.err.println(ANSI_RED + ANSI_BOLD + message + ANSI_RESET + "\n");
     }
 
     public static void successln(String message) {
-        System.out.println(ANSI_GREEN + ANSI_BOLD + message + ANSI_RESET);
+        System.out.println(ANSI_GREEN + ANSI_BOLD + message + ANSI_RESET + "\n");
     }
 
     public static void warnln(String message) {
-        System.out.println(ANSI_YELLOW + ANSI_BOLD+ message + ANSI_RESET);
+        System.out.println(ANSI_YELLOW + ANSI_BOLD + message + ANSI_RESET + "\n");
+    }
+
+    public static void infoln(String message) {
+        System.out.println(message + "\n");
     }
 }

app/src/test/java/sudoku/AppTest.java

@@ -1,17 +1,35 @@
-/*
- * This Java source file was generated by the Gradle 'init' task.
- */
 package sudoku;
 
 import org.junit.jupiter.api.Test;
 
+import java.io.ByteArrayInputStream;
+import java.util.ArrayList;
+import java.util.Arrays;
+
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class AppTest {
     @Test
     public void testExample() {
-        System.out.println("DOING TEST");
+        // Simulate user input
-        assertEquals(2 + 2, 4);
+        String simulatedInput = "1\n1\n2\n3\n4\n5\n6\n7\n8\n9\n";
-        System.out.println("DONE TEST");   
+        System.setIn(new ByteArrayInputStream(simulatedInput.getBytes()));
+
+        // Create a new Sudoku
+        Sudoku sudoku = new Sudoku(9);
+        sudoku.getGrille().askSetSymbolesPossibles();
+
+        // Verify the symbols
+        ArrayList<Symbole> expectedSymbols = new ArrayList<>(Arrays.asList(
+                Symbole.of(1),
+                Symbole.of(2),
+                Symbole.of(3),
+                Symbole.of(4),
+                Symbole.of(5),
+                Symbole.of(6),
+                Symbole.of(7),
+                Symbole.of(8),
+                Symbole.of(9)));
+        assertEquals(expectedSymbols, sudoku.getGrille().getSymbolesPossibles());
     }
 }

app/src/test/java/sudoku/TestBloc.java

@@ -0,0 +1,50 @@
+package sudoku;
+
+import org.junit.jupiter.api.Test;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+
+public class TestBloc {
+    @Test
+    public void testBloc() {
+        System.out.println("TEST BLOC : ");
+        System.out.println(new App().getGreeting());
+        // Create a new Sudoku
+        Sudoku sudoku = new Sudoku(9);
+        sudoku.getGrille().setSymbolesPossibles(new ArrayList<>(Arrays.asList(
+                Symbole.of(1),
+                Symbole.of(2),
+                Symbole.of(3),
+                Symbole.of(4),
+                Symbole.of(5),
+                Symbole.of(6),
+                Symbole.of(7),
+                Symbole.of(8),
+                Symbole.of(9))));
+
+        sudoku.getGrille().setCase(0, 0, Symbole.of(1));
+        sudoku.getGrille().setCase(6, 1, Symbole.of(2));
+        sudoku.getGrille().setCase(2, 2, Symbole.of(3));
+        sudoku.getGrille().setCase(0, 3, Symbole.of(4));
+        sudoku.getGrille().setCase(4, 4, Symbole.of(5));
+        sudoku.getGrille().setCase(0, 5, Symbole.of(6));
+        sudoku.getGrille().setCase(5, 6, Symbole.of(7));
+        sudoku.getGrille().setCase(0, 7, Symbole.of(8));
+        sudoku.getGrille().setCase(4, 8, Symbole.of(9));
+        //doublon bloc
+        sudoku.getGrille().setCase(1, 1, Symbole.of(1));
+
+        sudoku.getGrille().createSquareBlocs();
+
+        System.out.println("Sudoku :");
+        System.out.println(sudoku.getGrille().toString());
+
+        System.out.println("Blocs :");
+        sudoku.getGrille().printBlocs();
+
+        System.out.println("Symboles possibles :");
+        sudoku.getGrille().printSymbolesPossibles();
+        System.out.println("FIN TEST BLOC");
+    }
+}

app/src/test/java/sudoku/TestColonne.java

@@ -0,0 +1,50 @@
+package sudoku;
+
+import org.junit.jupiter.api.Test;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+
+public class TestColonne {
+    @Test
+    public void testColonne() {
+        System.out.println("TEST COL : ");
+        System.out.println(new App().getGreeting());
+        // Create a new Sudoku
+        Sudoku sudoku = new Sudoku(9);
+        sudoku.getGrille().setSymbolesPossibles(new ArrayList<>(Arrays.asList(
+                Symbole.of(1),
+                Symbole.of(2),
+                Symbole.of(3),
+                Symbole.of(4),
+                Symbole.of(5),
+                Symbole.of(6),
+                Symbole.of(7),
+                Symbole.of(8),
+                Symbole.of(9))));
+
+        sudoku.getGrille().setCase(0, 0, Symbole.of(1));
+        sudoku.getGrille().setCase(6, 1, Symbole.of(2));
+        sudoku.getGrille().setCase(2, 2, Symbole.of(3));
+        sudoku.getGrille().setCase(0, 3, Symbole.of(4));
+        sudoku.getGrille().setCase(4, 4, Symbole.of(5));
+        sudoku.getGrille().setCase(0, 5, Symbole.of(6));
+        sudoku.getGrille().setCase(5, 6, Symbole.of(7));
+        sudoku.getGrille().setCase(0, 7, Symbole.of(8));
+        sudoku.getGrille().setCase(4, 8, Symbole.of(9));
+        // doublon colonne
+        sudoku.getGrille().setCase(4, 0, Symbole.of(1));
+
+        sudoku.getGrille().createSquareBlocs();
+
+        System.out.println("Sudoku :");
+        System.out.println(sudoku.getGrille().toString());
+
+        System.out.println("Blocs :");
+        sudoku.getGrille().printBlocs();
+
+        System.out.println("Symboles possibles :");
+        sudoku.getGrille().printSymbolesPossibles();
+        System.out.println("FIN TEST COL");
+    }
+}

app/src/test/java/sudoku/TestLigne.java

@@ -0,0 +1,50 @@
+package sudoku;
+
+import org.junit.jupiter.api.Test;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+
+public class TestLigne {
+    @Test
+    public void testLigne() {
+        System.out.println("TEST LIGNE : ");
+        System.out.println(new App().getGreeting());
+        // Create a new Sudoku
+        Sudoku sudoku = new Sudoku(9);
+        sudoku.getGrille().setSymbolesPossibles(new ArrayList<>(Arrays.asList(
+                Symbole.of(1),
+                Symbole.of(2),
+                Symbole.of(3),
+                Symbole.of(4),
+                Symbole.of(5),
+                Symbole.of(6),
+                Symbole.of(7),
+                Symbole.of(8),
+                Symbole.of(9))));
+
+        sudoku.getGrille().setCase(0, 0, Symbole.of(1));
+        sudoku.getGrille().setCase(6, 1, Symbole.of(2));
+        sudoku.getGrille().setCase(2, 2, Symbole.of(3));
+        sudoku.getGrille().setCase(0, 3, Symbole.of(4));
+        sudoku.getGrille().setCase(4, 4, Symbole.of(5));
+        sudoku.getGrille().setCase(0, 5, Symbole.of(6));
+        sudoku.getGrille().setCase(5, 6, Symbole.of(7));
+        sudoku.getGrille().setCase(0, 7, Symbole.of(8));
+        sudoku.getGrille().setCase(4, 8, Symbole.of(9));
+        //doublon ligne
+        sudoku.getGrille().setCase(0, 4, Symbole.of(1));
+
+        sudoku.getGrille().createSquareBlocs();
+
+        System.out.println("Sudoku :");
+        System.out.println(sudoku.getGrille().toString());
+
+        System.out.println("Blocs :");
+        sudoku.getGrille().printBlocs();
+
+        System.out.println("Symboles possibles :");
+        sudoku.getGrille().printSymbolesPossibles();
+        System.out.println("FIN TEST LIGNE");
+    }
+}

gradle/wrapper/gradle-wrapper.properties

@@ -1,6 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.0.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.3-bin.zip
 networkTimeout=10000
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists