problème de syntaxe (if)

salut tlm,

j'ai un petit problème de syntaxe dans mon programme, lorsque je compile j'ai alors cette erreur :
L'opérateur '&&' ne peut pas être appliqué aux opérandes de type 'bool' et 'int'

et voici ma prog :

if (this.listBox1.Text == "Sept S36")
{

if (colonne == "bepa1" && ligne = 1)
{

this.bepa11.Text = "X";

}

après avoir testé avec plusieurs messagebox l'erreur vient probablement du 2ème if.

Merci d'avance de m'aider

d'abord,
il faut surement taper

Code:

---

ligne == 1 et non pas ligne = 1

---

et ensuite, il est "judicieux" voir conseillé de mettre des paranthèses pour eviter que t'as ligne soit interpretéé comme suit :

Code:

---

1 2 3 4 5

if (colonne == ("bepa1" && ligne) = 1)   ou autre tapes:   if( (colonne == "bepa1") && (ligne == 1))

---

Ca risque de mieux marcher...

Citation:

---

Envoyé par theMonz31

[...]ensuite, il est "judicieux" voir conseillé de mettre des paranthèses pour eviter que t'as ligne soit interpretéé comme suit :

Code:

---

1 2 3 4 5

if (colonne == ("bepa1" && ligne) = 1)   ou autre tapes:   if( (colonne == "bepa1") && (ligne == 1))

---

Ca risque de mieux marcher...

---

Je ne suis pas d'accord. Mettre des parenthèses systématiquement pour tous les élements d'une condition rend le code moins lisible.
Qu'est-ce qui est le plus lisible :

Code:

---

if((a=="1") && (b=="2") && (!monbooleen) && ((c=="4") || (c=="5")))

---

Code:

---

if(a=="1" && b=="2" && !monbooleen &&(c=="4" || c=="5"))

---

Pour moi, le deuxième exemple est plus simple à lire et modifier.
Par contre je suis d'accord sur le fait qu'il puisse être parfois utile de mettre des parenthèses dans certains cas pour éviter de se tromper à cause de la priorité des opérations mais de là a le faire systématiquement...


Tant que j'y suis. Quand une condition deviens un peu complexe, un bon truc est de créer une méthode rien que pour la condition afin de rendre le code nettement plus lisible:

Code:

---

1 2 3 4 5 6 7

public boolean EstUnEnvoitPrioritaire(){ return (a=="1" && b=="2" && !monbooleen &&(c=="4" || c=="5")); }   if(EstUnEnvoitPrioritaire()){ //... }

---

De cette façon, une personne qui lis le code n'a pas besoin de décortiquer une expression tordue pour savoir quel est la condition.

Code:

---

1 2 3 4 5 6 7

if( ( truc == machin ) && ( bidule != chose ) || ( !etmoi() ) ) { ... }

---

Je préfères nettement la propreté des parenthèses... Alliée à une lecture Verticale...
La lecture horizontale c'est atroce... En plus en verticale on peut commenter une condition rapidement...

Au fait est ce que c'est résolu? Parce que ca va partir en troll sur la conv de codage...

Citation:

---

Envoyé par theMonz31

d'abord,
il faut surement taper

Code:

---

ligne == 1 et non pas ligne = 1

---

et ensuite, il est "judicieux" voir conseillé de mettre des paranthèses pour eviter que t'as ligne soit interpretéé comme suit :

Code:

---

1 2 3 4 5

if (colonne == ("bepa1" && ligne) = 1)   ou autre tapes:   if( (colonne == "bepa1") && (ligne == 1))

---

Ca risque de mieux marcher...

---

Je confirme, c'est cela ;)