Discussion :

[thibaut7013]

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

[theMonz31]

d'abord,
il faut surement taper

Code :

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

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 :

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

1
2
3
4
5
if (colonne == ("bepa1" && ligne) = 1)
 
ou autre tapes:
 
if( (colonne == "bepa1") && (ligne == 1))

Ca risque de mieux marcher...

[Monkeyget]

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

Code :

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

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 :

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

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

Code :

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

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 :

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

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.

[Chubyone]

Code :

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

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...

[Thomas Lebrun]

d'abord,
il faut surement taper

Code :

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

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 :

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

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