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;