Discussion :

[nedved_01]

Bonjour,

j’essaye de trier un arraylist qui contient 2 types d'objets différents (villa, usine qui sont 2 classes dérivées de la classe mère abstraite Batiment qu'elle a comme attributs proprietaire, adresse et surface), le problème c'est quand j’essaye de trier juste les villa selon la surface ça ne fonctionne pas correctement et je ne sais pas ou est le problème.

Voila l'extrait de code ou je fais le trie par surface :

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
24
25
26
27
28
29
30
31
32
33
34
 
class TrierSurface : IComparer
    {
 
        public int Compare(object a, object b)
        {
            int res = 0;
 
                Batiment a1 = (Batiment)a;
                Batiment a2 = (Batiment)b;
 
                if (a1 is Villa && a2 is Villa)
                {
                    if (a1.Surface > a2.Surface)
                    {
                        res = 1;
                    }
 
                    if (a1.Surface < a2.Surface)
                    {
                        res = -1;
                    }
                    if (a1.Surface == a2.Surface)
                    {
                        res = 0;
                    }
 
 
                }
 
                return res;
 
 
        }

au moment d'appel :

liste.Sort(new TrierSurface());

le contenu d'arraylist avant trie :

Dsty fr250 1600 AQW 160 140 26800 //Usine : proprietaire adresse surface...
Sir df5 500 9 True 750 //Villa
Zes w598 100 20 False 300 //Villa
Saoma szs987 1500 GFR 300 120 36000 //Usine
Ader az12 700 12 False 250 //Villa
Klot drb02 250 8 True 800 //Villa

arraylist après trie :

Zes w598 100 20 False 300 //Villa
Saoma szs987 1500 GFR 300 120 36000 //Usine
Sir df5 500 9 True 750 //Villa
Ader az12 700 12 False 250 //Villa
Klot drb02 250 8 True 800 //Villa
Dsty fr250 1600 AQW 160 140 26800 //Usine

[neilbgr]

Je pense qu'il faut aussi traiter le cas où une villa et une usine sont à comparer.

[tomlev]

Quel est le résultat que tu veux obtenir ? Tu dis que tu ne veux trier par surface que pour les villas, mais dans ce cas, que fais-tu des usines ?

[nedved_01]

tomlev je veux obtenir un résultat à peu près comme celui-là :

Dsty fr250 1600 AQW 160 140 26800 //Usine
Zes w598 100 20 False 300 //Villa
Klot drb02 250 8 True 800 //Villa
Saoma szs987 1500 GFR 300 120 36000 //Usine
Sir df5 500 9 True 750 //Villa
Ader az12 700 12 False 250 //Villa

je ne sais pas si c'est possible mais je veux que les usines restent à leurs places sinon le plus important est que les villas seront triées

[tomlev]

C'est peut-être possible, mais c'est complexe, et je ne pense pas qu'on puisse le faire en implémentant juste un IComparer... Mais de toute façon je ne vois pas très bien l'intérêt de faire ça, ce n'est pas très pratique pour la lecture des résultats

[nedved_01]

neilbgr ta réponse m'a donné une idée que j'ai testé et ça marche du premier coup, voilà ce que j'ai fait :

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
24
25
26
27
28
29
30
31
32
33
34
35
36
 
class TrierSurface : IComparer
    {
 
        public int Compare(object a, object b)
        {
            int res = 0;
 
            if (a is Usine || b is Usine) // je commence par tester si a ou b est une usine
            {
                return res; // retourne 0
            }
 
            Batiment a1 = (Batiment)a;
            Batiment a2 = (Batiment)b;
 
            if (a1.Surface == a2.Surface)
            {
                res = 0;
            }
 
            if (a1.Surface > a2.Surface)
            {
                res = 1;
            }
 
            if (a1.Surface < a2.Surface)
            {
                res = -1;
            }
 
 
 
            return res;
 
        }

le contenu avant trie :

Dsty fr250 1600 AQW 160 140 26800 //Usine : proprietaire adresse surface...
Sir df5 500 9 True 750 //Villa
Zes w598 100 20 False 300 //Villa
Saoma szs987 1500 GFR 300 120 36000 //Usine
Ader az12 700 12 False 250 //Villa
Klot drb02 250 8 True 800 //Villa

après trie :

Zes w598 100 20 False 300 //Villa
Klot drb02 250 8 True 800 //Villa
Sir df5 500 9 True 750 //Villa
Ader az12 700 12 False 250 //Villa
Dsty fr250 1600 AQW 160 140 26800 //Usine
Saoma szs987 1500 GFR 300 120 36000 //Usine

merci pour vos réponses

[tomlev]

A vrai dire je ne comprends pas très bien comment ça peut donner un résultat différent, parce qu'en supposant que tu n'aies que des villas et des usines dans la liste, ton nouveau code est exactement équivalent à l'ancien...

En plus ça ne correspond pas à ce que tu disais : tu voulais que les usines restent à la même place, mais là ce n'est pas le cas.

[nedved_01]

tomlev ta raison ça à marché juste car j'ai fait l'erreur suivant :

if (a is Usine || a is Usine)
{
return res;
}
j'ai testé sur a mais je ne sais pas pourquoi ça marche avec ce test

[nedved_01]

J'ai utilisé :

liste.Sort(0, list.Count, new TrierSurface());

et ça ma marche cette fois

Ze w 598 100 20 False 300 //villa
Dsty fr 25000 1600 AQW 160 140 26800 //usine
Khalid derb 02 250 8 True 800 //villa
Saoma szs 987 1500 GFR 300 120 36000 //usine
Said df 5 500 9 True 750 //villa
Ahmed az 12 700 12 False 250 //villa

la méthode ignore les usines et trie les villas correctement