Bonjour !

J'ai un problème avec la condition suivante :

if(joueurActuel.getPosJoueur() == 1 && (!damier[xCaseDépart][yCaseDépart].equals("b") || !damier[xCaseDépart][yCaseDépart].equals("B")))
			return false;

Il me retourne faux alors qu'il ne doit normalement pas executer l'instruction.

En effet getPosJoueur renvoi bien 1 et et damier[xCaseDépart][yCaseDépart] renvoi bien un string "b"...

Des idées ?

Bonjour,
Si "damier[xCaseDépart][yCaseDépart] renvoi bien un string "b" " comme tu l'indiques,
Alors

(!damier[xCaseDépart][yCaseDépart].equals("b") || !damier[xCaseDépart][yCaseDépart].equals("B"))

renverra true. (false || true).
Par conséquent ton if ressemblera à : (true && (false ||true)) càd true!
D'où ton return false

En vrac :

• Ton histoire de damier qui peut contenir une lettre en majuscule ou minuscule me semble cacher un problème de conception
  
• Les String ont une méthode .equalsIgnoreCase(String)
  
• En général on préfère un "Yoda-test" dans ce genre de cas, pour se prémunir des NullPointerException :
  

  String s = getMyString();
  // Risque de NPE si s est null :
  s.equals("X");
  // Aucun risque de NPE quel que soit s,
  // Renverra false si s est null :
  "X".equals(s);
  

D'autre part, damier[xCaseDépart][yCaseDépart] renvoie "b", qui n'est pas "B" donc tu valides la dernière condition.
En réalité, tu renvoies "false" si :

• getPosJoueur vaut 1
  
• et damier[xCaseDépart][yCaseDépart] n'est pas "b" ou n'est pas "B"

Or si damier[xCaseDépart][yCaseDépart] est "b" il n'est pas "B" et inversement. Le deuxième membre du "ou" est toujours vrai.
Donc ton test peut se résumer à :

if(joueurActuel.getPosJoueur() == 1 && (true))

Donc :

if(joueurActuel.getPosJoueur() == 1)

Ce que tu veux se résume à :

return joueurActuel.getPosJoueur() != 1 || "b".equalsIgnoreCase(damier[xCaseDépart][yCaseDépart])

Merci beaucoup! Cela m'a beaucoup aidé

Pense à indiquer que le sujet est résolu