Condition if non pris en considération

Bonjour, j'ai créer une fonction qui me retourne soit 1 soit 0selon l'existence des parametres passés, cette fonction test s'il y a intersection ou inclusion entre les paramètres passés et les ligne d'une table :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

public int isExist(int SN, int EN, char design) { int exist = 0;   for (int i = 0; i < this.Count; i++) { if (((SN <= this[i].EndNumber && SN >= this[i].StartNumber) || (EN <= this[i].EndNumber && EN >= this[i].StartNumber) || (this[i].StartNumber <= EN && this[i].StartNumber >= SN) || (this[i].EndNumber <= EN && this[i].EndNumber >= SN)) && this[i].Design.Equals(design)) { exist = 1; } } return exist; }

---

le problème c'est que cette fonction ne prend pas la derniere condition en considération :

Code:

---

&& this[i].Design.Equals(design)

---

par exemple : si je donne : D 1 500 et dans la table il ya S 1 500, la fonction doit me retourné 0, ce qui permet d'insérer dans la table car S 1 500 # D 1 500, que pensez vous ?

Citation:

---

Envoyé par Msysteme

que pensez vous ?

---

Que quand on a un if aussi maousse, on a intérêt à le découper :)

Du style

Code:

---

1 2 3

if (gros machin de la mort avec des || de partout) if (autre gros bidule) ....

---

Aussi, passe ton code en pas à pas, sélectionne une expression, rajoute un espion dessus, et tu verras sa valeur (ici true ou false) dans la fenêtre d'espion.

Citation:

---

if (le gros machin de la mort avec les || de partout)
exist=this[i].Design.Equals(design))?1:0 ;

---

et verifier en debug!

+1

Sinon, moi, ton :

Code:

---

1 2 3 4 5 6 7 8

for (int i = 0; i < this.Count; i++) { if ((this[i].EndNumber >= SN && SN >= this[i].StartNumber) || (this[i].EndNumber >= EN && EN >= this[i].StartNumber) || (EN >= this[i].StartNumber && this[i].StartNumber >= SN) || (EN >= this[i].EndNumber && this[i].EndNumber >= SN))   return this[i].Design.Equals(design) ? 1 : 0; }

---

J'ajouterais une fonction, genre IsInBetween

Code:

---

1 2 3 4

private bool IsInBetween (int val, int min, int max){ return min <= val&& val<= max; }

---

et ca ferait du :

Code:

---

1 2 3 4 5 6 7

for (int i = 0; i < this.Count; i++) { if (IsInBetween(SN , this[i].StartNumber, this[i].EndNumber) || IsInBetween(EN , this[i].StartNumber, this[i].EndNumber) || IsInBetween(this[i].StartNumber, SN, EN) || IsInBetween(this[i].EndNumber , SN, EN)) return this[i].Design.Equals(design) ? 1 : 0; }

---

-1 ;)

Citation:

---

return this[i].Design.Equals(design) ? 1 : 0;

---

pas de return si 0, il faut continuer la boucle

Citation:

---

Envoyé par Msysteme

Bonjour, j'ai créer une fonction qui me retourne soit 1 soit 0selon l'existence des parametres passés,

---

Les booléens te posent un problème de conscience ?

Citation:

---

Envoyé par Graffito

-1 ;)

pas de return si 0, il faut continuer la boucle

---

Vi, c'est vrai (pas vu ;) )

Ceci dit ,cui-la :

Code:

---

1 2	if (le gros machin de la mort avec les || de partout) exist=this[i].Design.Equals(design))?1:0 ;

---

il est faux aussi :aie:

Mais normalement je dois vérifier si this[i].Design.Equals(design)) si oui je fais le test sinon j'incremente ma boucle, parce que si la ligne a controlé n'a pas la meme designation que celle dans la table je dois pas faire de test ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

public int isExist(int SN, int EN, char design) { int exist = 0;   for (int i = 0; i < this.Count; i++) { Console.WriteLine("Design " + this[i].Design + " EN " + this[i].EndNumber); if (design.Equals(this[i].Design)) {   if (((SN <= this[i].EndNumber && SN >= this[i].StartNumber) || (EN <= this[i].EndNumber && EN >= this[i].StartNumber) || (this[i].StartNumber <= EN && this[i].StartNumber >= SN) || (this[i].EndNumber <= EN && this[i].EndNumber >= SN))) { exist = 1; } } else i++; } return exist; }

---

Mais meme celle la ne donne pas de bons résultat ?

Tu devrais commencer par supprimer ceci

Code:

---

1 2

else i++;

---

Parce qu'avec ça tu vas sauter un élément à chaque fois que le test sera faux. L'incrémentation du compteur est déjà faite un niveau de la boucle for

Code:

---

for (int i = 0; i < this.Count; i++)

---

Sinon, utilise plutôt

Code:

---

1 2 3 4 5 6 7

... if (test) { exist = 1; break; } ...

---

pour éviter des parcours inutiles. Une fois la variable à 1 rien ne peut la faire passer à 0, donc ça ne sert à rien de continuer.

ok, mais j'ai pas compris ca :

Code:

---

1 2 3 4 5 6 7

.. if (test) { exist = 1; break; } ...

---

Aide-toi et la MSDN t'aidera ... Instruction break

salut, c'est bien normal qu'il ne rentre pas dans la deuxième condition :
false && ? = false
true || ? = true

la deuxième condition n'est pas executée parce que la première est fausse, sachant que false ET un autre prédicat renvoie toujours false le compilateur optimise et n'execute pas le deuxième prédicat.

cas concret, essaye de passer en pas à pas avec cet essai :

Code:

---

1 2 3 4 5

if (testFaux() && testVrai()) {   }

---

et ailleurs tu déclares les méthodes suivantes

Code:

---

1 2 3 4 5 6 7 8 9

private bool testFaux() { return false; } private bool testVrai() { return true; }

---

Tu verra que c'est tout à fait normal.
Maintenant je suis assez d'accord avec les précédents message elle est plutot balèze ta fonction... et tu devrais certainement suivre les conseils qu'on t'a donné, ils sont importants

Citation:

---

Envoyé par sisqo60

cas concret, essaye de passer en pas à pas avec cet essai :

Code:

---

1 2 3 4 5

if (testFaux() && testVrai()) {   }

---

---

Petite précision : ca marche si on écrit

Code:

---

1 2 3 4 5

if (testFaux() & testVrai()) {   }

---

Avec un seul '&'; dans ce cas les deux membres sont évalués. (règle valable aussi pour le "|" et pour le "^"). (j'ai remarqué que pas mal de développeurs C# ignoraient cela, même des "pas débutants").

oui bien sur il fait un "et bit à bit". là c'est encore autrechose...
Mais c'est certainement important de le rappeler.

Citation:

---

Envoyé par sisqo60

oui bien sur il fait un "et bit à bit". là c'est encore autrechose....

---

Non, pas du tout, tu fais une confusion avec le C++. (et tu confortes ce que j'écrivais supra).

En C#, les opérateurs n'opérent en "bit a bit" que sur des variables de type char, byte, int ou long, pas sur des variables bool (ça n'aurait aucun sens).

La syntaxe "un seul &" (et "|", "^") sur des booléens n'est pas une opération bit-a-bit mais une opération booléenne sans optimisation de la part du compilo, entrainant une évaluation de tous les membres avant calcul du résultat.

discussion fort intéressante, mais tu devrais faire un petit post pour rappeler tout ça. et regarde par la même occasion http://msdn.microsoft.com/fr-fr/libr...1c(VS.80).aspx .

C'est plus du tout en rapport avec le question de départ!

Citation:

---

Envoyé par sisqo60

regarde par la même occasion http://msdn.microsoft.com/fr-fr/libr...1c(VS.80).aspx .

---

Euh .. c'est exactement ce que j'écrivais.

Citation:

---

C'est plus du tout en rapport avec le question de départ!

---

Peut être mais c'est en rapport avec ce qui était écrit trois poste plus haut.