Discussion :

[rinuom99]

salut

je sais que c bête comme question, mais je vais la posé comeme

SVP dis moi est ce qu'il y a un problème dans ce test puisque malgré que les condition sont toujours vrai , mais le flag reste toujours false

Code :

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

1
2
3
4
5
6
7
8
string x1= textBox1.Text;
string x2 = textBox2.Text;
bool flag = false;
 
if ((x1 == dataGridView1[1,i].Value.ToString()))  && x2 == dataGridView1[4,i].Value.ToString()))
                 {
                     flag = true;
                 }

merci d'avance

[SaumonAgile]

Il y a un truc qui s'appelle un debogueur dans tout bon environnement de développement. Je te conseille de l'utiliser. Il sert à répondre à ce genre de questions.

[ZuoYue]

salut,
C'est claire, dans ce cas vaut mieux mettre un point d'arrêt en utilisant le débogueur,
mais je pense vu que tu compare des chaines de caractères, il y a souvent des problèmes liés à la casse et aux espaces du début et fin de chaine.
il est conseillé d'améliorer un peu ton code en ajoutant trim et ToUpper:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
string x1= textBox1.Text.trim().ToUpper();
string x2 = textBox2.Text.trim().ToUpper();
bool flag = false;
 
if ((x1 == dataGridView1[1,i].Value.ToString().trim().ToUpper()))  && x2 == dataGridView1[4,i].Value.ToString().trim().ToUpper()))
                 {
                     flag = true;
                 }

[bilb0t]

Bonjour,

Quand tu accedes à des tableau, ça vaut aussi toujours la peine de tester l'existance des "cases".

par exemple: dataGridView1[4,i] Es-tu toujours sur que tu auras 5 lignes ds ton datagrid ?

[ced600]

Si c'est bien ton code et pas une erreur de copier coller, tu as des problèmes de parenthèses :

Code :

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

1
2
3
4
if ((x1 == dataGridView1[1,i].Value.ToString()))  && x2 == dataGridView1[4,i].Value.ToString()))
                 {
                     flag = true;
                 }

J'ai mis des couleurs pour se repérer.

A mon avis voici la solution :

Code :

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

1
2
3
4
if ((x1 == dataGridView1[1,i].Value.ToString())  && (x2 == dataGridView1[4,i].Value.ToString()))
                 {
                     flag = true;
                 }

[rinuom99]

Merci pour la réponse

le probleme des parenthese c juste une faute de copier coller

merci

[SaumonAgile]

il est conseillé d'améliorer un peu ton code en ajoutant trim et ToUpper

Peux tu expliquer en quoi ToUpper() est capable d'améliorer une comparaison de chaînes ?

[ced600]

En fait cela dépend si l'on veut être case sensitive ou non dans la comparaison.

J'ai fait pas mal de comparaison de chaine de caractères où je souhaitais validé que les chaines sont égales qu'elles soient en minuscules ou majuscules. Donc sans être case sensitive.
Et j'ai souvent constaté que ce n'est pas toujours écrit en majuscules ou minuscules, des fois un peu des deux.
Cela faussait mon test, alors pour résoudre ce problème j'ai utilisé ToUpper sur les deux chaines à tester.

Voilà pourquoi ZuoYue a conseillé ToUpper (du moins je crois).

C'est utile à condition de ne pas vouloir différencier deux chaines où les seules différences seraient au niveau des majuscules.

[rinuom99]

Peux tu expliquer en quoi ToUpper() est capable d'améliorer une comparaison de chaînes ?

merci a tous, avec le débogueur et Trimstar() et Trimend(), j'ai réglé le probleme

merci encor une fois

[SaumonAgile]

En fait cela dépend si l'on veut être case sensitive ou non dans la comparaison
[...]

Je ne vois toujours pas l'intérêt de passer tout en majuscule pour comparer sans tenir compte de la casse, alors qu'il existe des méthodes faites pour ça :

Code :

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

maChaine.Equals(maDeuxiemeChaine, StringComparison.InvariantCultureIgnoreCase)

[ced600]

Ha je ne la connaissais pas