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

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

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

[StormimOn]

Tu devrais commencer par supprimer ceci

Code :

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

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 :

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

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

Sinon, utilise plutôt

Code :

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

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.

[Msysteme]

ok, mais j'ai pas compris ca :

Code :

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

1
2
3
4
5
6
7
..
if (test)
{
    exist = 1;
    break;
}
...

[StormimOn]

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

[sisqo60]

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 :

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

1
2
3
4
5
 
if (testFaux() && testVrai())
{
 
}

et ailleurs tu déclares les méthodes suivantes

Code :

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

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

[Bluedeep]

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

Code :

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

1
2
3
4
5
 
if (testFaux() && testVrai())
{
 
}

Petite précision : ca marche si on écrit

Code :

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

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

[sisqo60]

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

[Bluedeep]

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.

[sisqo60]

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!

[Bluedeep]

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.

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.