Discussion :

[joel.drigo]

pour le cas de mat non détecté,y'avait un bug : dans le test de blocage du moment je testais aussi le roi c'est sur que le roi peut pas bloquer l'echec

voici la correction

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
parcourColonnes: for(int colonne=0; colonne<8;colonne++) {
  for(int ligne=0; ligne<8;ligne++) {
    if (e.getCase(colonne,ligne).estOccupe(couleurControle) && !(e.getCase(colonne,ligne).getPiece() instanceof Roi) ) {
      // on parcourt toutes les cases du déplacement :
      for(Position pos : deplacementMenace.getPositions() ) {
        Deplacement simuleDeplacement=new Deplacement(new Position(colonne, ligne),pos);
        if ((e.getCase(colonne,ligne).getPiece().estValide(simuleDeplacement) && e.cheminPossible(simuleDeplacement)) ){
          mat=false;
          break parcourColonnes; // pas la peine de continuer
          }
      }
    }
  }
}

[joel.drigo]

sinon tu fais peut etre une manip spéciale : tu a bien fait cette partie en F2F4, E7E6, G2G4, D8H4 ?

Oui


J'ai juste mis l'onglet Annuler coup avant terminé , rien de plus.

bizarre j'ai aussi des différences de numéro de lignes dans Deplacement et Echiquier

Si chez vous sa marche, pouvez vous me renvoyé le fichier s'il vous plais ?

Merci

je te joins le source avec le menu annuler avant terminer des fois que

[Jordan62]

Merci sa fonctionne désormais

[joel.drigo]

je viens de trouver un autre problème : j'avais pas vu que tu avais duppliqué les constantes COULEUR_JOUEUR_BLANC et COULEUR_JOUEUR_NOIR dans la classe Echiquier, du coup lors de mon clean j'ai pas traité ce cas !

ci-joins la classe corrigée

quand tu fais des constantes, faut les faire dans une classe uniquement et les appeler partout, surtout pas les dupliquer : il y a risque de les changer à un endroit sans les changer à un autre, on perd le bénéfice de la constante

par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
public class FenetreJeu {
 
...
 
public final static String COULEUR_JOUEUR_BLANC="rouge et jaune";
 
...
}

pour faire référence dans une autre classe :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
public class Echiquier {
 
...
 
public void maMethode() {
     if ( FenetreJeu.COULEUR_JOUEUR_BLANC.equals(...
}
...
}

tu peux faire aussi un import statique de la constante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
import static projet.FenetreJeu.COULEUR_JOUEUR_BLANC;
public class Echiquier {
 
...
 
public void maMethode() {
     if ( COULEUR_JOUEUR_BLANC.equals(...
}
...
}

ou éventuellement

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
public class Echiquier {
 
...
 
public final static String COULEUR_JOUEUR_BLANC=FenetreJeu.COULEUR_JOUEUR_BLANC;
 
public void maMethode() {
     if ( COULEUR_JOUEUR_BLANC.equals(...
}
...
}

[Jordan62]

Merci BEAUCOUP pour tout !

[joel.drigo]

je viens de tester l'annulation du roque : ça bug

je corrige le problème et je te renvois ça

[joel.drigo]

voici la correction du bug de l'annulation du roque

en plus j'ai l'impression que j'ai oublié de t'envoyer une modif de la classe pièce

[Jordan62]

Merci

[joel.drigo]

ta présentation c'est noté ? on va regarder tes sources ? te poser des questions sur pourquoi tu as fait comme ça ? comment tu as fait ça ? etc...

tu es sur d'avoir bien compris tout ce qu'on a fait ? je parle pas du java lui mais niveau algo ?

si tu as la moindre question, n'hésite pas. c'est fastidieux, mais il faut que tu relises tout le code, pour bien repérer tous les concepts algorithmiques utilisés. c'est vrai pour le code qu'on a fait "ensemble", mais aussi pour le code d'origine (tous les algo de validation de mouvement par exemple)

à mon avis l'un de concept algorithmique qu'on utilise le plus est la simulation de déplacement : le fait de créer des faux déplacement pour toutes les pièces du joueur ou du joueur adverse vers une ou plusieurs positions données
le but : pouvoir utiliser les méthodes de validation de mouvement ou de prise pour des mouvements possibles mais non effectués, afin de prévoir des situations possibles (comme la prise d'un pion, le déplacement possible, le roque possible, l'echec au roi, l'echec et mat...)

pour l'annulation de coup, le concept est facile : à chaque coup joué, on sauvegarde le coup (la pièce qui bouge, les positions de départ et d'arrivée, la pièce prise pour nous faciliter l'annulation), ceci nous permet non seulement de sauvegarder la partie, mais également d'annuler en rejouant le coup à l'envers : il suffit de remettre la pièce jouée dans sa position de départ, la piece éventuellement prise dans la position d'arrivée (la supprimer du panel des pièces prises),
pour le roque, on ne sauvegarde que le mouvement du roi, pour éviter d'avoir à gérer un cas particulier, qui n'est pas indispensable comme on peut facilement connaitre le mouvement inverse de la tour, mais au niveau de l'annulation comme pour le chargement, il faut traiter ce cas de manière particulière

[Jordan62]

Merci
J'ai du mal au niveau des List <deplacement>

[joel.drigo]

c'est à dire ?

du mal avec quoi ?

avec la notation ? la manipulation ?

[Jordan62]

private List<Deplacement> chercheEchec(Position positionRoiJoue, Couleur couleurAdverse) {
List<Deplacement> menaceechec=new ArrayList<Deplacement>();

Ici on créer une liste de deplacement ?

[joel.drigo]

private List<Deplacement> chercheEchec(Position positionRoiJoue, Couleur couleurAdverse) {
List<Deplacement> menaceechec=new ArrayList<Deplacement>();

Ici on créer une liste de deplacement ?

java.util.List est une interface qui hérite de java.util.Collection qui en gros définit une classe qui représente une liste d'élément auquels on peut accèder par leur index.

on déclare une variable de ce type (List) et on crée une instance de ArrayList qui est une classe qui implémente l'interface List en stockant les données de la liste dans un tableau.

la notation <x> est juste une notation qui permet de manipuler lors de la programmation les éléments de la liste comme étant de classe x. ceci n'intervient pas lors de l'exécution du programme : une ArrayList ne contient toujours que des java.lang.Object.

quand on écrit

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

List<Deplacement> menaceechec

on permet simplement à ce que le compilateur reconnaisse des écritures du type :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Deplacement deplacement = menaceechec.get(0);

sans signaler d'erreur

en quelque sorte on "dit" au compilateur "oui je sais une List contient des java.lang.Object mais je sais que je vais y mettre que des project.Deplacement, alors je veux me simplifier l'écriture du code"

avant l'introduction de cette notation, en java 1.5, on aurait écrit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

List menaceechec=new ArrayList();

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Deplacement deplacement = (Deplacement)menaceechec.get(0);

on peut d'ailleurs continuer à le faire

le but logique final de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

List<Deplacement> menaceechec=new ArrayList<Deplacement>();

étant bien entendu de créer une liste de déplacement,
la méthode collectant tous les déplacements de pièces adverses qui résulteraient en une prise du roi de la couleur qui joue, afin de pouvoir les analyser ensuite

[Jordan62]

Merci