Passage par référence

**the-morpher** · 31/03/2015, 15h44

Bonjour tous le monde,

j'aimerai passer une classe par référence à une méthode qui détruit tous simplement cette classe et ses enfants,

ceci se passe dans un environnement graphique et du coup cette méthode me sert a détruit les objets qui ne sont plus utilisables,

la méthode est :

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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
public static void DisposeIGfxAndChild(IGfx igfx)
        {
            #region
            // supression d'un IGfx et de ses enfants
            if (igfx != null)
            {
                // cast de l'objet
                if (igfx.GetType() == typeof(Bmp))
                {
                    Bmp b = igfx as Bmp;
                    b.Visible = false;
                    b.Child.Clear();
                }
                else if (igfx.GetType() == typeof(Rec))
                {
                    Rec r = igfx as Rec;
                    r.Visible = false;
                    r.Child.Clear();
                }
                else if (igfx.GetType() == typeof(Anim))
                {
                    Anim a = igfx as Anim;
                    a.Visible(false);
                    a.Close();       // arret de l'animation
                    a.img.Child.Clear();
                }
 
                // supression du parent
                if (igfx.GetType() == typeof(Bmp))
                {
                    (igfx as Bmp).Visible = false;
                    (igfx as Bmp).bmp = null;
                    igfx = null;
                }
                else if (igfx.GetType() == typeof(Anim))
                {
                    (igfx as Anim).img.Visible = false;
                    (igfx as Anim).Close();
                    (igfx as Anim).img = null;
                    igfx = null;
                }
                else if (igfx.GetType() == typeof(Rec))
                {
                    (igfx as Rec).Visible = false;
                    igfx = null;
                }
                else if (igfx.GetType() == typeof(Txt))
                {
                    (igfx as Txt).Visible = false;
                    igfx = null;
                }
            }
 
            // netoyage des list
            IGfxList.RemoveAll(f => f == null);
            #endregion
        }

pour IGfx c'est une Interface pour les classes Bmp (Bitmap), Rec (Rectangle), Anim (Animation), Txt (Text) qui hèrite de cette interface

donc quand j'appel la méthode DisposeIGfxAndChild avec le code suivant :

Code :

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

1
2
3
 
IGfx __PlayerStats = IGfxList.FindAll(f => f.GetType() == typeof(Rec) && (f as Rec).name == "__playerStatesInBattleRecParent");
DisposeIGfxAndChild(__PlayerStats);

la y a un truc qui ne passe pas comme je le souhaite, c'est que l'objet passé par référence ne se retire pas de la liste après l'avoir mis sur null
c-à-d le code suivant :
IGfxList.RemoveAll(f => f == null);
ne supprime pas l'objet de la liste, et après débogage, l'objet passé par référence n'a pas subit de changement et n'est null, mais on dirai que c'est l'objet en cours qui est null, du coup aucune correspondance dans la liste IGfxList alors que l'objet existe toujours puisque je le trouve sous le nom "__playerStatesInBattleRecParent" mais pas null

j'ai compris alors qu'il faut passer l'objet pas le mot clef Ref,
chose faite et j'appel avec :

Code :

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

1
2
 
DisposeIGfxAndChild(ref __PlayerStats);

et j'ai créée une surcharge de la méthode de destruction :

Code :

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

1
2
3
4
5
 
public static void DisposeIGfxAndChild(Ref IGfx igfx)
        {
        ...
        }

Erreur de compilation :
Une propriété, un indexeur ou l'accès au membre dynamique ne peut pas être passé en tant que paramètre de sortie (out) ni de référence (ref)

je pige rien, une idée svp ?

**jopopmk** · 31/03/2015, 16h12

Salut,

tu ne peux pas passer by ref un élément d'une collection, et je suppose que IGfxList en est une.

**the-morpher** · 31/03/2015, 18h53

ah ok, bon je vais le supprimer en le cherchons sur la collection et le supprimer directement

merci pour l'info

**MABROUKI** · 31/03/2015, 19h26

bonjour

Est -tu sur que ce code est fiable:

Code :

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

1
2
3
4
5
 
 
 
IGfx __PlayerStats = IGfxList.FindAll(f => f.GetType() == typeof(Rec) && (f as Rec).name == "__playerStatesInBattleRecParent");
DisposeIGfxAndChild(__PlayerStats);

PlayersStats est un List<IGfx> car IGfxList.FindAll() renvoit un list<IGfx)...
Comme ta methode recoit un objet IGFx ca ne marche pas...
j'aurais ecrit plutot:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
             List<IGfx>  __PlayerStats = IGfxList.FindAll(f => f.GetType() == typeof(Rec) && (f as Rec).name == "__playerStatesInBattleRecParent");
 
            for (int i = 0; i < playerStats1.Count;i++ )
            {
                DisposeIGfxAndChild(playerStats1[i]);
            }
            // netoyage des list
            IGfxList.RemoveAll(f => f == null);

et dans ta methode simplement:

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
37
38
 
 
 
public static void DisposeIGfxAndChild(IGfx igfx)
        {
            #region
            // supression d'un IGfx et de ses enfants
            if (igfx != null)
            {
                // cast de l'objet
                if (igfx is Bmp)
                {
                    Bmp b = igfx as Bmp;
                    b.Visible = false;
                    b.Child.Clear();
                    b=null
                }
                else if (igfx is Rec)
                {
                    Rec r = igfx as Rec;
                    r.Visible = false;
                    r.Child.Clear();
                 r=null;
                }
                else if (igfx is Anim)
                {
                    Anim a = igfx as Anim;
                    a.Visible(false);
                    a.Close();       // arret de l'animation
                    a.img.Child.Clear();
                    a=null;
                }
 
 
            }
 
          #endregion
        }

bon code...

**the-morpher** · 31/03/2015, 19h42

oui en effet c'est une liste, c'est juste une erreur de copie,

mais le problème été pareil, du coup j'ai un autre soucie la

disons que j'ai une liste d'élément, que j'aimerai faire un traitement dessus, et vus que la requête Sql passé est un peut longue j'ai fait une 2éme liste qui contiens les éléments ciblés pour un traitement rapide comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
List<Bmp> allPlayersExceptChallenged = AllPlayers.FindAll(f => f.Pseudo != ChallengeTo && f != myPlayer);
                    if(allPlayersExceptChallenged.Count >0)
                    {
                        for (int cnt = allPlayersExceptChallenged.Count; cnt > 0; cnt--)
                        {
                            allPlayersExceptChallenged[cnt - 1].Visible = false;
                            allPlayersExceptChallenged[cnt - 1] = null
                            allPlayersExceptChallenged.RemoveAt(cnt - 1);
                        }
                    }

du coup la liste AllPlayers n'est pas modifié, ci comme ci la liste allPlayersExceptChallenged est un autre objet a part, je croyé que allPlayersExceptChallenged pointe vers AllPlayers mais visiblement ca créer un nouveau objet ?

**MABROUKI** · 31/03/2015, 21h15

Normalement ce code est bon et ta liste AllPLayers doit etre mise à jour car la 2eme liste pointe vers les memes objets que la liste d'origine...
Je pense que ton probleme ne vient pas de ce bout de code ...
Mais doit se situer dans un autre code ...
Le AllPlayer du code poste n'est pas le bon AllPlayers ...

**the-morpher** · 31/03/2015, 21h50

Merci pour ta réponse MABROUKI mais après une étude sur le bout de code bah
une liste qui pointe vers une autre liste après un trie grâce à une requête SQL ça crée une nouvelle liste mais qui contient les éléments de la liste trié.
c-à-d que
Liste<objet> L2 = L1.FindAll(Condition);

L2 contiens les éléments de L1 selon la condition, un changement d'un élément sur la liste L2 affecte L1 (originale).

MAIS

si on supprime un élément de la lise L2 n'affecte pas la liste L1, du coup L1 contient toujours l'élément
pour cela j'ai du placer ce code qui marche très bien :

Code :

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

1
2
3
 
// on supprime dépendamment du nom de l'objet 
GfxTopList.RemoveAll(f => f.Name() == igfx.Name());

au lieu de celui la

Code :

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

1
2
3
 
// l'objet ne se supprime pas parce que l'élément originale ne se met pas à null,
IGfxList.RemoveAll(f => f == null);

donc la suppression par son nom ma résolue le problème
et se que j'ai conclus c'est qu'une modification des éléments d'une autre liste se fait sans problème à part l'allocation d'un nouveau espace de mémoire (nouvelle instance d'après une doc MSDN) ou le mettre sur null (<-- je ne sais pas pourquoi

)
je cois

en passons mes codes précédents ne sont pas vraiment ceux que j'utilise, c'est juste une ressemblance pour simplifier

**MABROUKI** · 31/03/2015, 22h27

Re
OUPS !!! En fait ce l'absurdite de ce bout de coup de code m'a echappe :

Code :

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

1
2
3
 
// netoyage des list
            IGfxList.RemoveAll(f => f == null);

Car cette instruction recherche des elements qui -doivent etre null- c.à.d qui n'existe pas ce qui est ABSURDO..et ne fait strictement RIEN...
En fait mes habitudes de code m'ont sauve car la reference des elements est "nullifie" dans les if de la methode :

Code :

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

1
2
3
4
5
6
7
8
9
10
 if (igfx != null)
            {
                // cast de l'objet
                if (igfx is Bmp)
                {
                    Bmp b = igfx as Bmp;
                    b.Visible = false;
                    b.Child.Clear();
                    b=null                }

Evidemment si tu supprimes les elements en faisant reference à leur "name" ,alors FindAll recherche des elements qui EXISTENT(ne sont pas null) et qui satisfont la condition...

Dans ton interface IGfx tu du aurais prevoir un prop ToDelete(un marqueur) ...
Ensuite ta methode DisposeIGfxAndChild passe et marque tous les candidats à la mort par ToDelete et au retour de cette methode RemoveAll liquide tous les items marques...
Enfin la manipulation de nombreuses listes qui sont liees doit etre regroupee dans une methode commune pour pouvoir les synchroniser (addition d'element ,suppression)....
Bon code...

**the-morpher** · 31/03/2015, 22h51

Si j'ai bien compris, si je les mis en null il ne sont plus considérés dans la liste ?
donc une recherche ne donnera rien ?

mais si non, faut que j'évite de les passez par null et me contenter de les supprimer par leur nom ?
en faite j'hésite sur le faite de les mettre en null, je sais pas si le GAC va s'en occuper tous seul , je lui fait pas trop confiance

si non ça dois clôturer le sujet.

Passage par référence

C#

Discussions similaires

Partager

Partager