From 3ab665b70d97ba12043661ca36d69a5106d8b6f0 Mon Sep 17 00:00:00 2001 From: stiti Date: Mon, 29 Apr 2024 13:01:40 +0200 Subject: [PATCH] =?UTF-8?q?Am=C3=A9lioration=20MVC=20/=20Responsabilit?= =?UTF-8?q?=C3=A9=20unique=20partie=202?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Makefile | 2 +- src/GridMaker/GridMakeUserInterfaceView.java | 4 +- src/GridMaker/GridMakerCase.java | 46 +--- src/GridMaker/GridMakerCaseListener.java | 54 ++++ src/GridMaker/GridMakerChecker.java | 13 +- src/GridMaker/GridMakerCheckerListener.java | 28 ++ src/GridSolver/GSCase.java | 264 ++++++++++--------- 7 files changed, 233 insertions(+), 178 deletions(-) create mode 100644 src/GridMaker/GridMakerCaseListener.java create mode 100644 src/GridMaker/GridMakerCheckerListener.java diff --git a/Makefile b/Makefile index 285704a..9d20c79 100644 --- a/Makefile +++ b/Makefile @@ -13,7 +13,7 @@ JCFLAGS := -encoding UTF-8 -implicit:none JVM := java JVMFLAGS := -SRCDIR := ./src/GridSolver +SRCDIR := ./src/GridMaker OUTDIR := ./out DOCDIR := ./doc SRC := $(wildcard $(SRCDIR)/*.java) diff --git a/src/GridMaker/GridMakeUserInterfaceView.java b/src/GridMaker/GridMakeUserInterfaceView.java index dc3862c..f98c78a 100755 --- a/src/GridMaker/GridMakeUserInterfaceView.java +++ b/src/GridMaker/GridMakeUserInterfaceView.java @@ -28,7 +28,9 @@ public class GridMakeUserInterfaceView { JMenuItem nouveauItem = createMenuItem("Nouveau", new GridMakerResetGrid(grid)); JMenuItem chargerItem = createMenuItem("Charger", new GridMakerImport(window, grid)); JMenuItem sauvegarderItem = createMenuItem("Sauvegarder", new GridMakerSaver(window, grid)); - JMenuItem verifierItem = createMenuItem("Vérifier", new GridMakerChecker(grid)); + GridMakerChecker checker = new GridMakerChecker(grid); // Créez une instance de GridMakerChecker + GridMakerCheckerListener checkerListener = new GridMakerCheckerListener(checker); // Créez une instance de GridMakerCheckerListener en passant GridMakerChecker en argument + JMenuItem verifierItem = createMenuItem("Vérifier", checkerListener); JMenuItem aideGrilleItem = createMenuItem("Comment créer une grille", new GridMakerHowToCreateController()); JMenuItem reglesSudokuItem = createMenuItem("Règles du Sudoku", new GridMakerRules()); diff --git a/src/GridMaker/GridMakerCase.java b/src/GridMaker/GridMakerCase.java index 6260b91..27d4c59 100755 --- a/src/GridMaker/GridMakerCase.java +++ b/src/GridMaker/GridMakerCase.java @@ -4,12 +4,12 @@ import javax.swing.*; /** * GridMakerCase représente une case dans une grille éditable. - * Cette classe étend JPanel et implémente KeyListener pour permettre à l'utilisateur de saisir des valeurs dans la case. + * Cette classe étend JPanel et utilise un GridMakerCaseListener pour gérer les événements de touche. * @version 1.0 * @author Moncef STITI * @author Marco ORFAO */ -public class GridMakerCase extends JPanel implements KeyListener { +public class GridMakerCase extends JPanel { private int cellValue = 0; // Valeur actuelle de la case private String displayText = ""; // Texte à afficher dans la case @@ -17,56 +17,23 @@ public class GridMakerCase extends JPanel implements KeyListener { /** * Constructeur par défaut de GridMakerCase. - * Initialise le bouton avec les propriétés nécessaires, ajoute un KeyListener et configure le layout. + * Initialise le bouton avec les propriétés nécessaires, crée un GridMakerCaseListener et l'ajoute au bouton. */ public GridMakerCase() { actionButton.setOpaque(false); actionButton.setContentAreaFilled(false); actionButton.setBorderPainted(false); actionButton.setPreferredSize(new Dimension(50, 50)); - actionButton.addKeyListener(this); // Ajout du KeyListener au bouton BorderLayout gestionnaire = new BorderLayout(); setLayout(gestionnaire); add(actionButton, BorderLayout.CENTER); setPreferredSize(new Dimension(60, 60)); - } - /** - * Méthode appelée lorsqu'une touche est enfoncée. - * Si la touche est un chiffre, met à jour la valeur de la case avec ce chiffre. - * @param e L'événement KeyEvent généré lorsque la touche est enfoncée. - */ - @Override - public void keyPressed(KeyEvent e) { - int keyChar = e.getKeyChar(); - if (Character.isDigit(keyChar)) { - int num = Character.getNumericValue(keyChar); - if (num >= 0 && num <= 9) { - setCellValue(num); - } - } - } - - /** - * Méthode appelée lorsqu'une touche est relâchée. - * Aucun traitement n'est nécessaire dans ce cas. - * @param e L'événement KeyEvent généré lorsque la touche est relâchée. - */ - @Override - public void keyReleased(KeyEvent e) { - // Pas de traitement nécessaire - } - - /** - * Méthode appelée lorsqu'une touche est tapée (enfoncée puis relâchée). - * Aucun traitement n'est nécessaire dans ce cas. - * @param e L'événement KeyEvent généré lorsque la touche est tapée. - */ - @Override - public void keyTyped(KeyEvent e) { - // Pas de traitement nécessaire + // Crée un GridMakerCaseListener et l'ajoute au bouton + GridMakerCaseListener listener = new GridMakerCaseListener(this); + actionButton.addKeyListener(listener); } /** @@ -96,4 +63,3 @@ public class GridMakerCase extends JPanel implements KeyListener { return this.cellValue; } } - diff --git a/src/GridMaker/GridMakerCaseListener.java b/src/GridMaker/GridMakerCaseListener.java new file mode 100644 index 0000000..69d6496 --- /dev/null +++ b/src/GridMaker/GridMakerCaseListener.java @@ -0,0 +1,54 @@ +import java.awt.event.KeyEvent; +import java.awt.event.KeyListener; + +/** + * GridMakerCaseListener est un KeyListener utilisé pour écouter les événements de touche dans GridMakerCase. + */ +public class GridMakerCaseListener implements KeyListener { + + private GridMakerCase gridMakerCase; + + /** + * Constructeur de GridMakerCaseListener. + * @param gridMakerCase La case à écouter. + */ + public GridMakerCaseListener(GridMakerCase gridMakerCase) { + this.gridMakerCase = gridMakerCase; + } + + /** + * Méthode appelée lorsqu'une touche est enfoncée. + * Si la touche est un chiffre, met à jour la valeur de la case avec ce chiffre. + * @param e L'événement KeyEvent généré lorsque la touche est enfoncée. + */ + @Override + public void keyPressed(KeyEvent e) { + int keyChar = e.getKeyChar(); + if (Character.isDigit(keyChar)) { + int num = Character.getNumericValue(keyChar); + if (num >= 0 && num <= 9) { + gridMakerCase.setCellValue(num); + } + } + } + + /** + * Méthode appelée lorsqu'une touche est relâchée. + * Aucun traitement n'est nécessaire dans ce cas. + * @param e L'événement KeyEvent généré lorsque la touche est relâchée. + */ + @Override + public void keyReleased(KeyEvent e) { + // Pas de traitement nécessaire + } + + /** + * Méthode appelée lorsqu'une touche est tapée (enfoncée puis relâchée). + * Aucun traitement n'est nécessaire dans ce cas. + * @param e L'événement KeyEvent généré lorsque la touche est tapée. + */ + @Override + public void keyTyped(KeyEvent e) { + // Pas de traitement nécessaire + } +} diff --git a/src/GridMaker/GridMakerChecker.java b/src/GridMaker/GridMakerChecker.java index 8097a42..5f0cdab 100755 --- a/src/GridMaker/GridMakerChecker.java +++ b/src/GridMaker/GridMakerChecker.java @@ -1,5 +1,4 @@ import javax.swing.*; -import java.awt.event.*; /** * La classe GridMakerChecker est utilisée pour vérifier la cohérence de la grille. @@ -8,9 +7,10 @@ import java.awt.event.*; * @author Moncef STITI * @author Marco ORFAO */ -public class GridMakerChecker implements ActionListener { +public class GridMakerChecker { private GridMakerGrid grid; + private GridMakerCheckerListener checkerListener; /** * Constructeur pour créer une instance de GridMakerChecker. @@ -18,16 +18,9 @@ public class GridMakerChecker implements ActionListener { */ public GridMakerChecker(GridMakerGrid grid) { this.grid = grid; + this.checkerListener = new GridMakerCheckerListener(this); } - /** - * Méthode invoquée lorsqu'un bouton est cliqué. - * Lance la vérification de la grille. - * @param e L'événement d'action - */ - public void actionPerformed(ActionEvent e) { - checkGrid(); - } /** * Vérifie si la grille est correcte. diff --git a/src/GridMaker/GridMakerCheckerListener.java b/src/GridMaker/GridMakerCheckerListener.java new file mode 100644 index 0000000..f81fa2f --- /dev/null +++ b/src/GridMaker/GridMakerCheckerListener.java @@ -0,0 +1,28 @@ +import java.awt.event.ActionEvent; +import java.awt.event.ActionListener; + +/** + * La classe GridMakerCheckerListener est utilisée pour gérer les événements liés à la vérification de la grille. + */ +public class GridMakerCheckerListener implements ActionListener { + + private GridMakerChecker gridMakerChecker; + + /** + * Constructeur de GridMakerCheckerListener. + * @param gridMakerChecker L'instance de GridMakerChecker à utiliser. + */ + public GridMakerCheckerListener(GridMakerChecker gridMakerChecker) { + this.gridMakerChecker = gridMakerChecker; + } + + /** + * Méthode invoquée lorsqu'un bouton est cliqué. + * Lance la vérification de la grille. + * @param e L'événement d'action + */ + @Override + public void actionPerformed(ActionEvent e) { + gridMakerChecker.checkGrid(); + } +} diff --git a/src/GridSolver/GSCase.java b/src/GridSolver/GSCase.java index 8c79633..955615b 100755 --- a/src/GridSolver/GSCase.java +++ b/src/GridSolver/GSCase.java @@ -2,29 +2,41 @@ import java.awt.Color; import javax.swing.JPanel; import javax.swing.JLabel; +/** + * GSCase représente une case dans une grille de jeu. + * @version 1.0 + * @author Moncef STITI + * @author Marco ORFAO + */ public class GSCase extends JPanel { - private int primaryValue; - private int secondaryValue = 0; - private int tertiaryValue = 0; - private int quaternaryValue = 0; + private int primaryValue; // Valeur principale de la case + private int secondaryValue = 0; // Deuxième valeur de la case (optionnelle) + private int tertiaryValue = 0; // Troisième valeur de la case (optionnelle) + private int quaternaryValue = 0; // Quatrième valeur de la case (optionnelle) - private String text = ""; - protected boolean isInitial = false; - protected boolean isActive; - private JLabel label = new JLabel(); - private byte digitCount = 0; - protected int positionX; - protected int positionY; - protected GSGrid parentGrid; - private GSCaseMouseListener mouseListener; // Instance de la classe qui gère les événements de souris + private String text = ""; // Texte affiché dans la case + protected boolean isInitial = false; // Indique si la valeur de la case est initiale + protected boolean isActive; // Indique si la case est active + private JLabel label = new JLabel(); // Étiquette pour afficher le texte + private byte digitCount = 0; // Compteur du nombre de valeurs insérées dans la case + protected int positionX; // Position X de la case dans la grille + protected int positionY; // Position Y de la case dans la grille + protected GSGrid parentGrid; // Grille parente de la case + private GSCaseMouseListener mouseListener; // Écouteur pour les événements de souris + /** + * Constructeur de la classe GSCase. + * @param grid La grille parente de la case. + * @param x La position X de la case dans la grille. + * @param y La position Y de la case dans la grille. + */ public GSCase(GSGrid grid, int x, int y) { this.positionX = x; this.positionY = y; - this.primaryValue = 0; + this.primaryValue = 0; // Valeur par défaut - this.setBackground(Color.white); + this.setBackground(Color.white); // Couleur de fond par défaut this.parentGrid = grid; this.add(label); @@ -35,126 +47,126 @@ public class GSCase extends JPanel { this.addMouseListener(this.mouseListener); } - /** - * Configure l'apparence initiale de la case. - */ - public void layoutSetup() { - this.label.setText(this.primaryValue != 0 ? this.text : ""); - this.label.setVisible(true); - this.repaint(); - } + /** + * Configure l'apparence initiale de la case. + */ + public void layoutSetup() { + this.label.setText(this.primaryValue != 0 ? this.text : ""); + this.label.setVisible(true); + this.repaint(); + } - /** - * Initialise la case avec une valeur initiale. - */ - public void initializeCell() { - label.setText(this.text); - this.setBackground(this.isInitial ? Color.lightGray : Color.white); - layoutSetup(); - } + /** + * Initialise la case avec une valeur initiale. + */ + public void initializeCell() { + label.setText(this.text); + this.setBackground(this.isInitial ? Color.lightGray : Color.white); + layoutSetup(); + } - /** - * Définit la valeur de la case. - * @param value La valeur à définir pour la case. - */ - public void setValue(int value) { - this.isInitial = (value != 0); - this.text = (value != 0) ? String.valueOf(value) : ""; - this.primaryValue = value; - initializeCell(); - } + /** + * Définit la valeur de la case. + * @param value La valeur à définir pour la case. + */ + public void setValue(int value) { + this.isInitial = (value != 0); + this.text = (value != 0) ? String.valueOf(value) : ""; + this.primaryValue = value; + initializeCell(); + } - /** - * Obtient la valeur de la case. - * @return La valeur de la case. - */ - public int getValue(){ - return this.primaryValue; - } + /** + * Obtient la valeur de la case. + * @return La valeur de la case. + */ + public int getValue(){ + return this.primaryValue; + } - /** - * Désactive la case. - */ - public void deactivateCell(){ - this.isActive = false; - this.setBackground(Color.white); - } + /** + * Désactive la case. + */ + public void deactivateCell(){ + this.isActive = false; + this.setBackground(Color.white); + } - /** - * Obtient l'état d'activité de la case. - * @return True si la case est active, sinon False. - */ - public Boolean getActivity(){ - return this.isActive; - } + /** + * Obtient l'état d'activité de la case. + * @return True si la case est active, sinon False. + */ + public Boolean getActivity(){ + return this.isActive; + } - /** - * Met à jour la valeur de la case. - * @param value La nouvelle valeur de la case. - */ - public void updateValue(int value){ - if (value == 0) { - this.primaryValue = 0; - this.digitCount = 0; - layoutSetup(); - } else { - if (!checkInputValue(value)) { - if (this.primaryValue != value ) { - if (this.digitCount == 0) { - this.digitCount++; - this.primaryValue = value; - this.text = String.valueOf(this.primaryValue); - } else if (this.digitCount == 1 && value != this.primaryValue) { - this.secondaryValue = value; - this.digitCount++; - this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue); - } else if (this.digitCount == 2 && value != this.primaryValue && value != this.secondaryValue) { - this.tertiaryValue = value; - this.digitCount++; - this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue + ", " + this.tertiaryValue); - } else if (this.digitCount == 3 && value != this.primaryValue && value != this.secondaryValue && value != this.tertiaryValue) { - this.quaternaryValue = value; - this.digitCount++; - this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue + ", " + this.tertiaryValue + ", " + this.quaternaryValue); + /** + * Met à jour la valeur de la case. + * @param value La nouvelle valeur de la case. + */ + public void updateValue(int value){ + if (value == 0) { + this.primaryValue = 0; + this.digitCount = 0; + layoutSetup(); + } else { + if (!checkInputValue(value)) { + if (this.primaryValue != value ) { + if (this.digitCount == 0) { + this.digitCount++; + this.primaryValue = value; + this.text = String.valueOf(this.primaryValue); + } else if (this.digitCount == 1 && value != this.primaryValue) { + this.secondaryValue = value; + this.digitCount++; + this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue); + } else if (this.digitCount == 2 && value != this.primaryValue && value != this.secondaryValue) { + this.tertiaryValue = value; + this.digitCount++; + this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue + ", " + this.tertiaryValue); + } else if (this.digitCount == 3 && value != this.primaryValue && value != this.secondaryValue && value != this.tertiaryValue) { + this.quaternaryValue = value; + this.digitCount++; + this.text = String.valueOf(this.primaryValue + ", " + this.secondaryValue + ", " + this.tertiaryValue + ", " + this.quaternaryValue); + } + } + this.setBackground(Color.WHITE); + layoutSetup(); + } else if (checkInputValue(value)){ + this.setBackground(Color.RED); } } - this.setBackground(Color.WHITE); - layoutSetup(); - } else if (checkInputValue(value)){ - this.setBackground(Color.RED); - } } - } - /** - * Vérifie si la valeur entrée est valide pour la case. - * @param val La valeur à vérifier. - * @return True si la valeur est valide, sinon False. - */ - public Boolean checkInputValue(int val){ - int temp = this.primaryValue; - this.primaryValue = val; - GSTest test = new GSTest(this.parentGrid); - Boolean isValid = test.test(); - this.primaryValue = temp; - return isValid; - } + /** + * Vérifie si la valeur entrée est valide pour la case. + * @param val La valeur à vérifier. + * @return True si la valeur est valide, sinon False. + */ + public Boolean checkInputValue(int val){ + int temp = this.primaryValue; + this.primaryValue = val; + GSTest test = new GSTest(this.parentGrid); + Boolean isValid = test.test(); + this.primaryValue = temp; + return isValid; + } - /** - * Obtient le nombre de valeurs insérées dans la case. - * @return Le nombre de valeurs insérées. - */ - public int getDigitCount(){ - return this.digitCount; - } + /** + * Obtient le nombre de valeurs insérées dans la case. + * @return Le nombre de valeurs insérées. + */ + public int getDigitCount(){ + return this.digitCount; + } - /** - * Insère une valeur dans la case. - * @param value La valeur à insérer. - */ - public void insertValue(int value){ - this.primaryValue = value; - this.text = String.valueOf(this.primaryValue); - layoutSetup(); - } -} \ No newline at end of file + /** + * Insère une valeur dans la case. + * @param value La valeur à insérer. + */ + public void insertValue(int value){ + this.primaryValue = value; + this.text = String.valueOf(this.primaryValue); + layoutSetup(); + } +}