From 94ebb17d19b0b19a3826dbdb518bfbe61d8a04c3 Mon Sep 17 00:00:00 2001 From: Morph01 Date: Tue, 20 May 2025 16:20:02 +0200 Subject: [PATCH] refactor: clean up unused imports and improve reinitialization call --- app/src/main/java/org/App.java | 3 -- app/src/main/java/org/Controllers/IO.java | 2 +- .../Controllers/TetrisBandeauControleur.java | 4 +- app/src/main/java/org/Models/Grille.java | 46 +------------------ app/src/main/java/org/Models/Jeu.java | 33 ++++++------- app/src/main/java/org/Models/Musique.java | 8 ++-- app/src/main/java/org/Views/VueGrille.java | 5 -- 7 files changed, 22 insertions(+), 79 deletions(-) diff --git a/app/src/main/java/org/App.java b/app/src/main/java/org/App.java index 6f11107..80f2ee7 100644 --- a/app/src/main/java/org/App.java +++ b/app/src/main/java/org/App.java @@ -34,7 +34,4 @@ public class App { // musique // instanciée } - - // TODO: RESTE A IMPLEMENTER - // ext possible: conserver un historique de score dans un fichier txt ? } \ No newline at end of file diff --git a/app/src/main/java/org/Controllers/IO.java b/app/src/main/java/org/Controllers/IO.java index 7296e5d..d8e09bd 100644 --- a/app/src/main/java/org/Controllers/IO.java +++ b/app/src/main/java/org/Controllers/IO.java @@ -18,7 +18,7 @@ public class IO implements KeyListener { @Override public void keyPressed(KeyEvent e) { if (jeu.getGrille().estEnPause()) { - System.err.println("Grille est en pause"); + return; } switch (e.getKeyCode()) { diff --git a/app/src/main/java/org/Controllers/TetrisBandeauControleur.java b/app/src/main/java/org/Controllers/TetrisBandeauControleur.java index 201ab3a..746cb3b 100644 --- a/app/src/main/java/org/Controllers/TetrisBandeauControleur.java +++ b/app/src/main/java/org/Controllers/TetrisBandeauControleur.java @@ -3,7 +3,6 @@ package org.Controllers; import org.Models.Grille; import org.Models.Jeu; import org.Models.Musique; -import org.Models.PieceCourante; import org.Views.VueBandeauControle; import javax.swing.*; @@ -15,7 +14,6 @@ public class TetrisBandeauControleur { private VueBandeauControle vueControle; private Grille grille; private Jeu jeu; - private JButton aideButton; public TetrisBandeauControleur(VueBandeauControle vueControle, Musique musique, Grille grille, Jeu jeu) { this.vueControle = vueControle; @@ -34,7 +32,7 @@ public class TetrisBandeauControleur { this.vueControle.getAideButton().addActionListener(e -> afficherAide()); // Listener pour le bouton recommencer this.vueControle.getRelancerButton().addActionListener(e -> { - jeu.reinitialiserPartie(); + this.jeu.reinitialiserPartie(); vueControle.getPauseButton().setText("PAUSE"); partieEnPause = false; partieTerminee = false; diff --git a/app/src/main/java/org/Models/Grille.java b/app/src/main/java/org/Models/Grille.java index 07754e5..3e5a168 100644 --- a/app/src/main/java/org/Models/Grille.java +++ b/app/src/main/java/org/Models/Grille.java @@ -10,7 +10,7 @@ import java.util.List; import java.util.Observable; @SuppressWarnings("deprecation") -public class Grille extends Observable { // TODO: ?? implements Runnable { +public class Grille extends Observable { private boolean[][] grille; public int nbLignes; public int nbColonnes; @@ -127,29 +127,6 @@ public class Grille extends Observable { // TODO: ?? implements Runnable { } } - // TODO : ENLEVER ? - // public int getMaxYPieceCouranteColoree(List motifPieceColoree) { - // int maxY = 0; - // for (Point caseColoree : motifPieceColoree) { - // if (caseColoree.y > maxY) { - // maxY = caseColoree.y; - // } - // } - - // return maxY; - // } - - // public int getMaxXPieceCouranteColoree(List motifPieceColoree) { - // int maxX = 0; - // for (Point caseColoree : motifPieceColoree) { - // if (caseColoree.x > maxX) { - // maxX = caseColoree.x; - // } - // } - - // return maxX; - // } - public int getNbLignes() { return nbLignes; } @@ -206,7 +183,7 @@ public class Grille extends Observable { // TODO: ?? implements Runnable { setCase(caseColoree.y, caseColoree.x, true); } - System.err.println("post fixage de piece"); + verifierEtSupprimerLignesSiBesoin(); } @@ -281,25 +258,6 @@ public class Grille extends Observable { // TODO: ?? implements Runnable { this.nbLignesSupprimees = nbLignesSupprimees; } - // // TODO : EUHHHHHHHHHH JE CROIS PAS que ça marche comme ça c'est pas en mode - // gravité récursive, c'est juste tout descend de n lignes cassées - // // dès qu'on a fixe la pièce courante on verifie le nombre de lignes a - // supprimes - // // dans la grille - - // // -> - // // faire tomber JUSQU'EN bas la ligne X au dessus des n lines supprimes - // // PAR CONTRE faire tomber de n lignes toutes les lignes au dessus de X. - // while (peuxBouger(Direction.BAS, getLignePoints(i))) { - // System.err.println("testtttttttttt descnete"); - // descendreLigne(i - 1); - // } - - // // for (int j = i - 2; j > 0; j--) { - // // descendreLigne(j); - // // } - // } - public void supprimerLigne(int i) { for (int ligne = i; ligne > 0; ligne--) { for (int j = 0; j < nbColonnes; j++) { diff --git a/app/src/main/java/org/Models/Jeu.java b/app/src/main/java/org/Models/Jeu.java index 5672d8f..cb514f9 100644 --- a/app/src/main/java/org/Models/Jeu.java +++ b/app/src/main/java/org/Models/Jeu.java @@ -7,10 +7,7 @@ import java.awt.Point; import java.util.Observable; import java.util.Random; -import java.util.Random; -import org.Models.Pieces.PieceI; -import org.Models.Pieces.PieceJ; import org.Models.Pieces.PieceI; import org.Models.Pieces.PieceJ; import org.Models.Pieces.PieceL; @@ -19,7 +16,6 @@ import org.Models.Pieces.PieceS; import org.Models.Pieces.PieceT; import org.Models.Pieces.PieceZ; import org.Views.VueGameOver; -import org.Views.VueTetris; @SuppressWarnings("deprecation") public class Jeu extends Observable implements Runnable { @@ -33,16 +29,16 @@ public class Jeu extends Observable implements Runnable { public boolean jeuEnCours = true; public Jeu(Grille grille, Musique musique) { - System.err.println("init jeu"); + this.grille = grille; this.musique = musique; this.musique = musique; - System.err.println("init nouvelle piece courante"); - System.err.println("init nouvelle piece courante"); + + this.grille.setPieceCourante(getNouvellePiece()); - System.err.println("init nouvelle piece suivante"); - System.err.println("init nouvelle piece suivante"); + + this.pieceSuivante = getNouvellePiece(); this.ordonnanceur = new Ordonnanceur(this, 1000); @@ -52,38 +48,38 @@ public class Jeu extends Observable implements Runnable { private PieceCourante getNouvellePiece() { Random random = new Random(); int randomiiii = random.nextInt(7); - System.err.println("randomiiiii : " + randomiiii); + PieceCourante nouvellePiece; switch (randomiiii) { case 0: nouvellePiece = new PieceI(); - System.err.println("piece I"); + break; case 1: nouvellePiece = new PieceL(); - System.err.println("piece L"); + break; case 2: nouvellePiece = new PieceJ(); - System.err.println("piece J"); + break; case 3: nouvellePiece = new PieceO(); - System.err.println("piece O"); + break; case 4: nouvellePiece = new PieceS(); - System.err.println("piece S"); + break; case 5: nouvellePiece = new PieceT(); - System.err.println("piece T"); + break; case 6: nouvellePiece = new PieceZ(); - System.err.println("piece Z"); + break; default: nouvellePiece = new PieceL(); @@ -111,7 +107,7 @@ public class Jeu extends Observable implements Runnable { public boolean estFinPartie() { for (Point caseColoree : this.grille.motifPieceCouranteColoriee()) { if (this.grille.getCase(caseColoree.y, caseColoree.x)) { - VueGameOver gameOver = new VueGameOver(grille.getScore(), e -> System.exit(0), () -> { + new VueGameOver(grille.getScore(), e -> System.exit(0), () -> { System.out.println("\"rejouer\"clique"); reinitialiserPartie(); }); @@ -162,7 +158,6 @@ public class Jeu extends Observable implements Runnable { @Override public void run() { - // TODO: game logic here if (!jeuEnCours || grille.estEnPause()) { return; } diff --git a/app/src/main/java/org/Models/Musique.java b/app/src/main/java/org/Models/Musique.java index dcacbe4..d15544c 100644 --- a/app/src/main/java/org/Models/Musique.java +++ b/app/src/main/java/org/Models/Musique.java @@ -30,7 +30,7 @@ public class Musique { try { InputStream audioSrc = getClass().getResourceAsStream("/TetrisMusic.wav"); if (audioSrc == null) { - System.err.println("Erreur : le fichier TetrisMusic.wav est introuvable."); + return; } @@ -48,11 +48,11 @@ public class Musique { System.out.println("Clip audio initialisé avec succès."); } catch (UnsupportedAudioFileException e) { - System.err.println("Format audio non supporté : " + e.getMessage()); + } catch (LineUnavailableException e) { - System.err.println("Ligne audio indisponible : " + e.getMessage()); + } catch (IOException e) { - System.err.println("Erreur d'E/S lors de la lecture audio : " + e.getMessage()); + } } diff --git a/app/src/main/java/org/Views/VueGrille.java b/app/src/main/java/org/Views/VueGrille.java index 1735326..b87f5b2 100644 --- a/app/src/main/java/org/Views/VueGrille.java +++ b/app/src/main/java/org/Views/VueGrille.java @@ -128,10 +128,6 @@ public class VueGrille extends JPanel implements Observer, Runnable { repaint(); } - private void afficherFinPartie() { - System.err.println("FIN PARTIE"); // TODO : gerer affichage ? - } - /** * * Met à jour la vue de la grille lorsque l'état de la grille change. */ @@ -144,7 +140,6 @@ public class VueGrille extends JPanel implements Observer, Runnable { if (o instanceof Jeu && !afficherFenetreFinPartie) { if (!this.jeu.jeuEnCours) { afficherFenetreFinPartie = true; - afficherFinPartie(); } } }