From 2ced1215d807ca444fad7530012a1da1e6d36f3d Mon Sep 17 00:00:00 2001
From: Justine Yannis <yannisjustine91@gmail.com>
Date: Fri, 11 Nov 2022 20:40:36 +0100
Subject: [PATCH] =?UTF-8?q?V=C3=A9rification=20des=20noms=20+=20un=20peu?=
 =?UTF-8?q?=20de=20javadoc?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../Controller/ColorChooserListener.java      |  8 +++++++
 .../Puissance4/View/ColorChooserDialog.java   | 24 +++++++++++++++----
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ColorChooserListener.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ColorChooserListener.java
index 14cd319..4f308c1 100644
--- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ColorChooserListener.java
+++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ColorChooserListener.java
@@ -7,11 +7,19 @@ import javax.swing.event.ChangeListener;
 
 import fr.iutfbleau.projetAgile.Puissance4.Utils.Constants;
 
+/**
+ * Listener des changements de couleurs pour chaque joueur d'un {@link fr.iutfbleau.projetAgile.Puissance4.View.ColorChooserDialog ColorChooserDialog}
+ */
 public class ColorChooserListener implements ChangeListener {
 
     private JComponent composant;
     private int player;
 
+    /**
+     * 
+     * @param c
+     * @param player
+     */
     public ColorChooserListener(JComponent c, int player) {
         this.composant = c;
         this.player = player;
diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/ColorChooserDialog.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/ColorChooserDialog.java
index f19cbcc..fa24e0a 100644
--- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/ColorChooserDialog.java
+++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/ColorChooserDialog.java
@@ -21,7 +21,7 @@ import fr.iutfbleau.projetAgile.Puissance4.Controller.ColorChooserListener;
 import fr.iutfbleau.projetAgile.Puissance4.Utils.Constants;
 
 /**
- * JDialog personnalisé permettant de choisir un nom et une couleur pour deux joueur
+ * JDialog personnalisé permettant de choisir un nom et une couleur pour le puissance 4
  */
 public class ColorChooserDialog extends JDialog{
 
@@ -32,7 +32,7 @@ public class ColorChooserDialog extends JDialog{
 
     /**
      * Crée une fenêtre modale avec pour nom "Choisissez vos noms et couleurs"
-     * @param owner  la {@code Frame} depuis laquelle la fenetre modale est affichée
+     * @param owner la {@code Frame} depuis laquelle la fenetre modale est affichée
      * @param modal specifie si la fenêtre modale bloque les interactions avec les fenêtres mères
      * @see javax.swing.JDialog#JDialog(Frame, boolean)
      */
@@ -226,8 +226,10 @@ public class ColorChooserDialog extends JDialog{
     public boolean isNameValid() {
         String J1 = null;
         String J2 = null;
+        String J3 = null;
         J1 = field1.getText();
         J2 = field2.getText();
+        J3 = field3.getText();
         if(J1.length() > 10) {
             J1 = J1.substring(0, 9);
             this.field1.setText(J1);
@@ -235,13 +237,25 @@ public class ColorChooserDialog extends JDialog{
         if(J2.length() > 10) {
             J2 = J2.substring(0, 9);
             this.field2.setText(J2);
-        }    
-        if(J1.isEmpty() || J2.isEmpty()) {
+        } 
+        if(J3.length() > 10) {
+            J3 = J3.substring(0, 9);
+            this.field3.setText(J3);
+        }   
+        if(J1.isEmpty() || J2.isEmpty() || J3.isEmpty()) {
             JOptionPane.showMessageDialog(this,"Un des noms est vides","Erreur ", JOptionPane.ERROR_MESSAGE);
             return false;
         }
         else if(J1.compareTo(J2) == 0) {
-            JOptionPane.showMessageDialog(this,"Les deux noms sont identiques","Erreur ", JOptionPane.ERROR_MESSAGE);
+            JOptionPane.showMessageDialog(this,"Les deux noms sont identiques (1,2)","Erreur ", JOptionPane.ERROR_MESSAGE);
+            return false;
+        }
+        else if(J1.compareTo(J3) == 0) {
+            JOptionPane.showMessageDialog(this,"Les deux noms sont identiques (1,3)","Erreur ", JOptionPane.ERROR_MESSAGE);
+            return false;
+        }
+        else if(J2.compareTo(J3) == 0) {
+            JOptionPane.showMessageDialog(this,"Les deux noms sont identiques (2,3)","Erreur ", JOptionPane.ERROR_MESSAGE);
             return false;
         }
         return true;