Discussion :

[Jean-Marc68]

Bonjour à toutes et à tous.

Soit un List<Vecteur> dont Vecteur contient, entre autre, PtExtremite1 et PtExtremite2.

Je cherche tous les vecteurs dont les extrémités soient 2 points définis, quelle que soient leur position (PtExtremite1 ou PtExtremite2).

J'ai écris

Code :

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

1
2
3
var q = from c in this
        where ((c.PtExtremite1 == pt1 || c.PtExtremite1 == pt2) && (c.PtExtremite2 == pt1 || c.PtExtremite2 == pt2))
        select c;

mais il me sort toujours null.

Il me semble pourtant qu'il devrait sortir les vecteurs dont PtExtremite1 vaut un des 2 points et PtExtremite2 aussi.

Où me suis-je trompé et comment devrais-je faire ?

Merci de vos z'avis z'avisés,
Jean-Marc

[Graffito]

where ((c.PtExtremite1 == pt1 || c.PtExtremite1 == pt2) && (c.PtExtremite2 == pt1 || c.PtExtremite2 == pt2))

where ((c.PtExtremite1 == pt1 || c.PtExtremite2 == pt2) && (c.PtExtremite2 == pt1 || c.PtExtremite1 == pt2))

[ed73170]

D'où viennent pt1 et pt2 ?

Çà ne peut marcher que si pt1 et pt2 sont déjà dans ta liste de vecteurs mais pas si pt1 = new quelquechose et pt2 = new autrechose car la comparaison est faite sur les adresses de pt1 et pt2 et pas sur leur contenu. A moins bien sûr que tu aies redéfini l'opérateur ==.

[Jean-Marc68]

ed73170 : pt1 et pt2 sont bien les mêmes. Il s'agit de points déclarés avant et le test se fait avec les mêmes points.

Pour les tests le crée

Code :

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

1
2
3
                Point pt1 = new Point("1", 10.0, 10.0);
                Point pt2 = new Point("2", 0.0, 0.0);
                Point pt3 = new Point("3", 10.0, 0.0);

et ce sont toujours pt1, pt2 et pt3 qui sont utilisés, donc il s'agit bien des mêmes points.

Graffito : Même si je ne comprend pas pourquoi tu proposes ce remplacement, j'ai bien essayé ce que tu m'as proposé mais ça ne donne rien de mieux. Perso, je testais si c.PtExtremite1 égale pt1 ou pt2, et aussi si c.PtExtremite2 égale pt1 ou pt2. Toi tu testes si c.PtExtremite1 vaut pt1 et si c.PtExtremite2 vaut pt2 ou si c.PtExtremite1 vaut pt2 et c.PtExtremite2 vaut pt1, ce qui me semble revenir au même.

Merci de votre aide à tous les 2. Je ne comprend vraiment pas pq ça me retourne du null alors qu'un élément est bien dans la liste (vérifié par un break).

voici le code de ma méthode
PS : Cette méthode se trouve dans public class ListVecteurs<T> : List<Vecteur>, d'où l'utilisation de this

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 Vecteur Find(Point pt1, Point pt2)        {
 
 
            if (this.Count == 0)
                return null;
            var q = from c in this
                    where ((c.PtExtremite1 == pt1 || c.PtExtremite1 == pt2) && (c.PtExtremite2 == pt1 || c.PtExtremite2 == pt2))
                    select c;
            if (q == null || q.Count() == 0)
                return null;
            else
            {
                foreach (var vect in q)
                {
                    if (vect.GetType() == typeof(Ligne))
                    {
                        Ligne ligne = q as Ligne;
                        return ligne;
                    }
                }
                return null;
            }
        }

Un break sur if (this.Count == 0) m'a permis de vérifier que this contient bien au moins un vecteur contenant les points recherchés, or var q retourne null.

[GuruuMeditation]

Bizarre, un select ne devrait pas renvoyer null, mais un énumérable vide.

Au passage, pou tester si il y a un résultat, c'est mieux

Code C# :

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

1
2
3
4
5
6
7
8
9
 
if (q.Any())
{
 // resultats
}
else
{
// vide
}

Any() n'énumère pas le résultat comme Count et c'est aussi plus clair/concis (a mon avis)

[ed73170]

Si tu crées les objets pt1, pt2, et pt3 juste pour les tests alors c'est bien ce que je disais plus haut, tu n'obtiendras aucun résultat car ces points ne sont pas dans ta liste. Je ne sais pas comment tu crées ta liste de vecteurs mais j'imagine que c'est quelque chose comme :

pta = new Point(...);
ptb = new Point(...);
liste.Add(new Vecteur(pta,ptb));

Si ensuite tu instancies pt1, pt2, pt3 et que tu fais ton select, il est tout à fait normal que le résultat soit vide. Par contre si tu fais ta recherche avec pta et ptb tu obtiendras un résultat. Peu importe que tes points aient les bonnes coordonnées, ce qui compte c'est qu'ils aient le même adresse. Ou bien tu dois surcharger == pour que les tests se fassent sur les données contenues dans les objets Point.

[Graffito]

Pour éviter de selectionner les vecteurs [Pt1,Pt1] et [pt2,Pt2] :

where ((c.PtExtremite1 == pt1 && c.PtExtremite2 == pt2) || (c.PtExtremite2 == pt1 && c.PtExtremite1 == pt2))

Et comme indiqué par ed73170, remplacer "c.PtExtremitéE == ptP" par "(c.PtExtremitéE.X == ptP.X && c.PtExtremitéE.Y == ptP.Y)"

[Jean-Marc68]

Merci à tous pour le temps passé à étudier ma question.

La simple utilisation de q.any() comme le proposait GuruuMeditation a réglé le problème. Je laisse votre science comprendre pourquoi. Je ne comprend pas pourquoi le remplacement de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
            if (q == null || q.Count() == 0)
                return null;
            else
            {
                foreach (var vect in q)
                {
                    if (vect.GetType() == typeof(Ligne))
                    {
                        Ligne ligne = q as Ligne;
                        return ligne;
                    }
                }
                return null;
            }

par

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
            if (q.Any())
            {
                foreach (var vect in q)
                {
                    if (vect.GetType() == typeof(Ligne))
                    {
                        Ligne ligne = q as Ligne;
                        return ligne;
                    }
                }
                return null;
            }
            return null;

règle le problème, mais je dois constater que cela fonctionne.

Encore merci à tous et à chacun,
Jean-Marc