Discussion :

[Msysteme]

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 :

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

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 :

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

&& 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 ?

[Guulh]

que pensez vous ?

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

Du style

Code :

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

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.

[Graffito]

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

et verifier en debug!

[Philippe Vialatte]

+1

Sinon, moi, ton :

Code :

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

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 :

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

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

et ca ferait du :

Code :

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

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

[Graffito]

-1

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

pas de return si 0, il faut continuer la boucle

[Philippe Vialatte]

-1

pas de return si 0, il faut continuer la boucle

Vi, c'est vrai (pas vu )

Ceci dit ,cui-la :

Code :

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

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

[Msysteme]

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 :

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

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 ?

[Bluedeep]

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 ?