Modifier une variable existante en tant que variable dynamique

Version imprimable

Bonjour, le titre est peu clair, mais voici ce que je désire faire.

Je possède des ComboBox (10), j'ai eu envie de faire en 4 lignes ce que je pourrais faire en 40.

Example de nom de ComboBox :
AlteracValleyProfileType, WarsongGulchProfileType...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
                Battlegrounder.Profiletype.ComboboxItem item = new Battlegrounder.Profiletype.ComboboxItem();
                for (int x = 0; x <= ProfileTypeFile.Battlegrounds.Count - 1; x++)
                {
                    for (int y = 0; y <= ProfileTypeFile.Battlegrounds[x].ProfileTypes.Count - 1; y++)
                    {
                        item = new Battlegrounder.Profiletype.ComboboxItem();
                        item.Text = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeName;
                        item.Value = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeId;
                        this[ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeId + "ProfileType"].Items.Add(item);
// Reviendrait à faire: AlteracValleyProfileType.Items.Add(item);
                    }
                }

Ma structure XML (+la classe ComboBoxItem utilisée pour item)

Code:

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
using System;
using System.Collections.Generic;
 
namespace Battlegrounder.Profiletype
{
    public class ComboboxItem
    {
        public string Text { get; set; }
        public object Value { get; set; }
 
        public override string ToString()
        {
            return Text;
        }
    }
    [Serializable]
    public class BattlegrounderProfileType
    {
        public List<Battleground> Battlegrounds = new List<Battleground>();
    }
 
    [Serializable]
    public class Battleground
    {
        public string BattlegroundName = "";
        public string BattlegroundId;
        public List<ProfileType> ProfileTypes = new List<ProfileType>();
    }
 
    [Serializable]
    public class ProfileType
    {
        public string ProfileTypeName = "";
        public string ProfileTypeId = "";
    }
}

Ce que je pourrais faire:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
                for (int x = 0; x <= ProfileTypeFile.Battlegrounds.Count - 1; x++)
                {
                    if (ProfileTypeFile.Battlegrounds[x].BattlegroundId == AlteracValley.ToString())
                    {
                        for (int y = 0; y <= ProfileTypeFile.Battlegrounds[x].ProfileTypes.Count - 1; y++)
                        {
                            item = new Battlegrounder.Profiletype.ComboboxItem();
                            item.Text = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeName;
                            item.Value = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeId;
                            AlteracValleyProfileType.Items.Add(item);
                        }
                    }
                    if (ProfileTypeFile.Battlegrounds[x].BattlegroundId == WarsongGulch.ToString())
                    {
                        for (int y = 0; y <= ProfileTypeFile.Battlegrounds[x].ProfileTypes.Count - 1; y++)
                        {
                            item = new Battlegrounder.Profiletype.ComboboxItem();
                            item.Text = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeName;
                            item.Value = ProfileTypeFile.Battlegrounds[x].ProfileTypes[y].ProfileTypeId;
                            WarsongGulchProfileType.Items.Add(item);
                        }
                    }
                }

Bonjour,

Je ne suis pas certain d'avoir bien compris le problème mais il existe une classe très intéressante pour créer des objets dynamiques, il s'agit de ExpandoObject.

Cette classe fonctionne comme un dictionnaire et permet de créer des objets "dynamic" (cette classe est utilisée pour le ViewBag en MVC).

Voici comment ça s'utilise :
Code:

1 2 3 dynamic contact = new ExpandoObject(); contact.Name = "John Doe"; contact.Phone = "0102030405";
Je pense que ça pourra vous servir.

Pour améliorer votre algorithme (par exemple pour les recherches), vous pouvez utiliser Linq avec les méthodes "Where" ou "Select", comme ceci par exemple :
Code:

1 2 3 4 5 string[] elements = new string[] { "a", "b", "c", "d" }; // Filtrer les éléments, ne garder que ce qui n'est pas égal à "a" string[] elementsNonA = elements.Where(t => t != "a").ToArray();
Autre solution, utiliser les expressions Linq directement pour filtrer et créer de nouveaux objets (dynamiques):
Code:

1 2 3 4 5 string[] elements = new string[] { "a", "b", "c", "d" }; var result = from p in elements where p != "a" select new { UserName = p, Age = 22};
Le résultat sera une liste (IQueryable) qui contient des objets (anonymes) avec une propriété UserName et une propriété Age.
Ce type de mécanisme est souvent utilisé pour construire des listes temporaires pour des Select Box par exemple.

A votre disposition pour des compléments d'informations.
Pascal

Hello,

@PascalL67: il n'y a pas du tout besoin de faire usage de 'dynamic' ici. Seules les bonnes pratiques de programmation sont utiles pour résoudre le problème et le simplifier.

@Sundark: commence déjà par utiliser foreach au lieu de for, le code sera beaaaaucoup plus clair, et les améliorations envisageables par la suite seront plus apparentes. Le code devient alors
Code:

1 2 3 4 5 6 7 8 9 10 11 foreach(var battleground in ProfileTypeFile.Battlegrounds) { foreach(var type in battleground.ProfileTypes) { item = new Battlegrounder.Profiletype.ComboboxItem(); item.Text = type.ProfileTypeName; item.Value = type.ProfileTypeId; this[type.ProfileTypeId + "ProfileType"].Items.Add(item); } }
Tu devrais aussi expliciter ton besoin clairement. Tu as des BattleGrounds qui ont chacun une liste de ProfileType, et tu veux avoir autant de combos qu'il y a de ProfileType, chaque combo ayant pour éléments les battlegrounds qui ont ce ProfileType ?

05/02/2013, 16h22
Sundark
Je vous remerci pour votre aide, mais je vais abandonner cette idée, les for/if deviennent dans la suite de mon code, je ne pourrais pas construire toutes les variables automatiquement.

Note: je ne voulais pas un type "dynamic", mais utiliser la variable "dynamiquement" construite, ce sont donc 2 problèmes bien distinct.

Example en php :
Code:

1 2 3 4 5 6 $a = "b"; ${$a} = "c"; echo $a; // b echo $b; // c // La variable $b a été construite de manière dynamique.

Note: voici mon code final

Code:

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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
        private void RefreshProfileListType()
        {
            try
            {
                AlteracValleyProfileType.Items.Clear();
                WarsongGulchProfileType.Items.Clear();
                ArathiBasinProfileType.Items.Clear();
                EyeoftheStormProfileType.Items.Clear();
                StrandoftheAncientsProfileType.Items.Clear();
                IsleofConquestProfileType.Items.Clear();
                BattleforGilneasProfileType.Items.Clear();
                TwinPeaksProfileType.Items.Clear();
                TempleofKotmoguProfileType.Items.Clear();
                SilvershardMinesProfileType.Items.Clear();
                var item = new ComboboxItem();
                foreach(var battleground in ProfileTypeFile.Battlegrounds)
                {
                    for (var i = 0; i <= battleground.ProfileTypes.Count - 1; i++)
                    {
                        item = new ComboboxItem
                                   {
                                       Text = battleground.ProfileTypes[i].ProfileTypeName,
                                       Value = battleground.ProfileTypes[i].ProfileTypeId
                                   };
                        if (battleground.BattlegroundId == AlteracValley.Name)
                        {
                            AlteracValleyProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.AlteracValleyProfileType || AlteracValleyProfileType.SelectedIndex == -1)
                            {
                                AlteracValleyProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == WarsongGulch.Name)
                        {
                            WarsongGulchProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.WarsongGulchProfileType || WarsongGulchProfileType.SelectedIndex == -1)
                            {
                                WarsongGulchProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == ArathiBasin.Name)
                        {
                            ArathiBasinProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.ArathiBasinProfileType || ArathiBasinProfileType.SelectedIndex == -1)
                            {
                                ArathiBasinProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == EyeoftheStorm.Name)
                        {
                            EyeoftheStormProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.EyeoftheStormProfileType || EyeoftheStormProfileType.SelectedIndex == -1)
                            {
                                EyeoftheStormProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == StrandoftheAncients.Name)
                        {
                            StrandoftheAncientsProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.StrandoftheAncientsProfileType || StrandoftheAncientsProfileType.SelectedIndex == -1)
                            {
                                StrandoftheAncientsProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == IsleofConquest.Name)
                        {
                            IsleofConquestProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.IsleofConquestProfileType || IsleofConquestProfileType.SelectedIndex == -1)
                            {
                                IsleofConquestProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == BattleforGilneas.Name)
                        {
                            BattleforGilneasProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.BattleforGilneasProfileType || BattleforGilneasProfileType.SelectedIndex == -1)
                            {
                                BattleforGilneasProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == TwinPeaks.Name)
                        {
                            TwinPeaksProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.TwinPeaksProfileType || TwinPeaksProfileType.SelectedIndex == -1)
                            {
                                TwinPeaksProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == TempleofKotmogu.Name)
                        {
                            TempleofKotmoguProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.TempleofKotmoguProfileType || TempleofKotmoguProfileType.SelectedIndex == -1)
                            {
                                TempleofKotmoguProfileType.SelectedIndex = i;
                            }
                        }
                        else if (battleground.BattlegroundId == SilvershardMines.Name)
                        {
                            SilvershardMinesProfileType.Items.Add(item);
                            if (battleground.ProfileTypes[i].ProfileTypeId ==
                                BattlegrounderSetting.CurrentSetting.SilvershardMinesProfileType || SilvershardMinesProfileType.SelectedIndex == -1)
                            {
                                SilvershardMinesProfileType.SelectedIndex = i;
                            }
                        }
                    }
                }
            }

Bonsoir,

Étant donné que tu recherches un contrôle, l'indexeur de Control.Controls qui attend un string (le nom du contrôle à chercher) ne pouvait pas convenir ?
Code:

1 2 3 4 5 6 7 8 //remplacer this par le contrôle contenant les ComboBox (Panel etc.) ComboBox cb = this.Controls[battlegroundId] as ComboBox; //cb sera null si le controle n'a pas été trouvé (à vérifier sinon exception sûrement) et si le controle n'est pas un ComboBox (ça j'en suis sûr) if (cb != null) { cb.Items.Add(...); }
Si les ComboBox ne se trouvent pas forcément dans le même conteneur on doit pouvoir s'en sortir avec Control.Controls.FindControl mais là ça renverra un tableau de Control.

Bref comme je suis pas sûr d'avoir tout pigé je sais pas si ça servira mais au cas où ;)
lien vers la doc de Control.ControlCollection (le type de Controls)

Cordialement !