refactor: clean up unused imports and improve reinitialization call

This commit is contained in:
2025-05-20 16:20:02 +02:00
parent 3232cd136f
commit 94ebb17d19
7 changed files with 22 additions and 79 deletions

View File

@@ -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 ?
}

View File

@@ -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()) {

View File

@@ -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;

View File

@@ -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<Point> motifPieceColoree) {
// int maxY = 0;
// for (Point caseColoree : motifPieceColoree) {
// if (caseColoree.y > maxY) {
// maxY = caseColoree.y;
// }
// }
// return maxY;
// }
// public int getMaxXPieceCouranteColoree(List<Point> 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++) {

View File

@@ -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;
}

View File

@@ -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());
}
}

View File

@@ -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();
}
}
}