est ce qu'il y a un probleme dans cette instruction

Version imprimable

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:

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

16/01/2008, 08h44
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.

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:

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; }

16/01/2008, 10h56
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 ?

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

Code:

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:

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

16/01/2008, 12h33
rinuom99

Merci pour la réponse

le probleme des parenthese c juste une faute de copier coller

merci
16/01/2008, 12h36
SaumonAgile

Citation:

Envoyé par ZuoYue

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 ?
16/01/2008, 12h44
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.
16/01/2008, 12h45
rinuom99

Citation:

Envoyé par SaumonAgile

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
16/01/2008, 12h52
SaumonAgile

Citation:

Envoyé par ced600

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:

maChaine.Equals(maDeuxiemeChaine, StringComparison.InvariantCultureIgnoreCase)
16/01/2008, 14h02
ced600

Ha je ne la connaissais pas :mrgreen: