From a4fb1ee2c6146fa55bec794e38bd488addebc1f0 Mon Sep 17 00:00:00 2001 From: Justine Yannis Date: Sat, 12 Nov 2022 13:09:09 +0100 Subject: [PATCH] =?UTF-8?q?Utilisation=20des=20fonctions=20d=C3=A9j=C3=A0?= =?UTF-8?q?=20existantes=20pour=20=C3=A9viter=20la=20duplication=20de=20co?= =?UTF-8?q?de?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../Puissance4/Controller/ModelEventListener.java | 4 ++-- .../Puissance4/Event/AbstractGridInitiater.java | 6 +++--- .../Puissance4/Event/GridChangedListener.java | 4 ++-- .../projetAgile/Puissance4/Event/PlayerEvent.java | 8 +------- .../projetAgile/Puissance4/Model/GrilleModel.java | 13 +++++++------ .../projetAgile/Puissance4/View/Grille.java | 14 +------------- .../Puissance4/View/Puissance4Panel.java | 7 +++---- 7 files changed, 19 insertions(+), 37 deletions(-) diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ModelEventListener.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ModelEventListener.java index b1aadd7..8d4169e 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ModelEventListener.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Controller/ModelEventListener.java @@ -31,7 +31,7 @@ public class ModelEventListener implements GridChangedListener { } @Override - public void playerRemove(PlayerEvent e){ - this.panel.removePlayer(e.getJ(), e.getGrille()); + public void playerWin(PlayerEvent e){ + this.panel.playerWin(e.getJ()); } } diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/AbstractGridInitiater.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/AbstractGridInitiater.java index 5542d81..16ec791 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/AbstractGridInitiater.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/AbstractGridInitiater.java @@ -65,12 +65,12 @@ public abstract class AbstractGridInitiater { } } - protected void firePlayerChanged(int playerRemove, int[][] grille){ + protected void firePlayerWin(int playerRemove){ PlayerEvent event=null; for(GridChangedListener listener : getGridListeners()) { if(event == null) - event= new PlayerEvent(playerRemove, grille); - listener.playerRemove(event); + event= new PlayerEvent(playerRemove); + listener.playerWin(event); } } diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/GridChangedListener.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/GridChangedListener.java index a07c853..940fbcc 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/GridChangedListener.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/GridChangedListener.java @@ -19,9 +19,9 @@ public interface GridChangedListener extends EventListener{ void statusChanged(StatusEvent e); /** - * Méthode invoqué lors de la suppréssion d'un joueur + * Méthode invoqué lorsqu'un joueur gagne * @param e evenement du joueur a supprimé */ - void playerRemove(PlayerEvent e); + void playerWin(PlayerEvent e); } diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/PlayerEvent.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/PlayerEvent.java index 19fb482..cbf2e1e 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/PlayerEvent.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Event/PlayerEvent.java @@ -5,17 +5,11 @@ package fr.iutfbleau.projetAgile.Puissance4.Event; */ public class PlayerEvent { private int J; - private int[][] grille; - public PlayerEvent(int j0, int[][] grille0){ + public PlayerEvent(int j0){ this.J=j0; - this.grille=grille0; } public int getJ() { return J; } - - public int[][] getGrille() { - return grille; - } } diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Model/GrilleModel.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Model/GrilleModel.java index 8a767fc..f671cba 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Model/GrilleModel.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/Model/GrilleModel.java @@ -99,7 +99,7 @@ public class GrilleModel extends AbstractGridInitiater{ private void removePlayer(){ this.increaseScore(); this.removePlayerPion(); - this.firePlayerChanged(this.playerTurn, this.grille); + this.firePlayerWin(this.playerTurn); this.playersList.pollLast(); this.switchPlayer(); this.verifyNewWin(); @@ -244,21 +244,22 @@ public class GrilleModel extends AbstractGridInitiater{ private void removePlayerPion(){ for(int i=0; i=0; i--){ if(i>0){ - col[i]=col[i-1]; + this.grille[col][i]=this.grille[col][i-1]; }else{ - col[0]=Constants.EMPTY_PLAYER; + this.grille[col][0]=Constants.EMPTY_PLAYER; } + this.fireGridChanged(col, i, this.grille[col][i]); } } diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Grille.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Grille.java index 9fb752c..ff24721 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Grille.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Grille.java @@ -79,19 +79,7 @@ public class Grille extends JPanel{ } } this.repaint(); - } - - public void removePlayer(int Joueur, int[][] j){ - for (int x = 0; x < this.column; x++) { - for (int y = 0; y < this.row; y++) { - this.grille[x][y].setPlayer(j[x][y]); - this.grille[x][y].setHover(false); - } - } - this.repaint(); - } - - + } /** * Change la couleur de survol des pions selon le tour du joueur diff --git a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Puissance4Panel.java b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Puissance4Panel.java index 9578c4b..945f747 100644 --- a/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Puissance4Panel.java +++ b/projetAgile/src/fr/iutfbleau/projetAgile/Puissance4/View/Puissance4Panel.java @@ -124,6 +124,7 @@ public class Puissance4Panel extends JPanel{ gbc.insets = new Insets(10, 0, 0, 0); this.add(this.reset, gbc); + reset.setBackground(new Color(255,64,64)); reset.setForeground(Color.WHITE); @@ -153,7 +154,6 @@ public class Puissance4Panel extends JPanel{ case Constants.PLAYER_ONE: this.playerTurn.setText("Tour de : " + this.playerOneName); break; - case Constants.PLAYER_TWO: this.playerTurn.setText("Tour de : " + this.playerTwoName); break; @@ -230,9 +230,8 @@ public class Puissance4Panel extends JPanel{ this.changeScore(playerOneScore, playerTwoScore, playerThreeScore); } - public void removePlayer(int player, int[][] grille){ - this.grille.removePlayer(player, grille); - JOptionPane.showMessageDialog(this, "Le joeur "+String.valueOf(player)+" a gagné"); + public void playerWin(int player){ + JOptionPane.showMessageDialog(this, "Le joueur "+String.valueOf(player)+" a gagné"); } /**