Discussion :

[Ch123]

Bonjour,

Je souhaite créer une procédure dans laquelle j'envoie comme paramètre un objet (ListBox ou Combobox). Car le traitement est identique que ce soit un ListBox ou un ComboBox

Lorsque la procédure reçoit l'objet, je dois l'assigner explicitement à son type.
Exemple qui fonctionne:

Code :

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

1
2
3
4
5
6
7
 
       public static void Test(Object form_box)
        {
            ListBox form_box1 = (ListBox)form_box;
 
            int a=form_box1.Items.Count;
        }

A priori, je ne sais pas si Objet est un ListBox ou un ComboBox, donc je fais un test pour vérifier son type.
Le problème provient de la boucle if qui m'impose de déclarer l'objet en dehors de la boucle.
Exemple qui ne fonctionne pas:

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
 
  public static void Test(Object form_box)
        {
            Object form_box1 = null;
//   ComboBox form_box1 = null;
 
            if (form_box is ListBox)
            {
                form_box1 = (ListBox)form_box;
            }
            else if (form_box is ComboBox)
            {
                form_box1 = (ComboBox)form_box;
            }
            else
            {
                MessageBox.Show("Error_Type"); 
            }
            int a=form_box1.Items.Count;
        }

MErci d'avance pour votre aide !

[Graffito]

Code :

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

1
2
3
4
5
6
7
public static void Test(Object form_box)
{
    int a= 0 ;
    if      (form_box is ListBox ) { ListBox  form_ListBox  = (ListBox )form_box; a=form_ListBox .Items.Count; }            
    else if (form_box is ComboBox) { ComboBox form_ComboBox = (ComboBox)form_box; a=form_ComboBox.Items.Count; }            
    else MessageBox.Show("Error_Type");
}

[TheOnlyMaX]

Salut Ch123,

Il faut penser au plus grand dénominateur commun, qui dépend de ce que tu veux faire dans ton traitement.
En l'occurrence, si le traitement ne concerne que la collection d'items, tu peux faire ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public Form1()
{
    InitializeComponent();
 
    ListBox listBox = new ListBox();
    ComboBox comboBox = new ComboBox();
    test(listBox.Items);
    test(comboBox.Items);
}
 
void test(IList items)
{
    int a = items.Count;
}

Ca s'appelle la loi de Demeter : http://msdn.microsoft.com/en-us/maga...aspx#id0070040

[Ch123]

Bonjour
Merci bcp. pour vos deux propositions.

Super idée d'envoyer en paramètre la Collection des Items. Je ne connaissais pas cette possibilité.
Mais dans mon cas, le traitement que je veux faire ensuite, c'est entre autre de remplir la ListBox ou la ComboBox...

Exemple:

Code :

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

1
2
3
4
5
6
 string[] Fichiers = Directory.GetFiles(myDir, Filtre);
            foreach (string Fichier in Fichiers)
            {
                Console.WriteLine(Fichier);
                form_box1.Items.Add(Fichier);
            }

Dès lors, ca ne suffit pas la Collection des items. Il faut envoyer l'objet complet

Concernant la première proposition, quitte à créer deux variables avec 2 traitements spécifiques, alors autant directement créer deux procédures indépendantes.

Mon souhait est justement de n'avoir qu'une seule variable (et donc un seul traitement commun) que la variable d'entrée soit une ListBox ou une ComboBox

Si vous avez d'autres idées...
MErci d'avance,

[Graffito]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public static void Test(Object form_box)
{
    if      (form_box is ListBox ) ProcessItems((ListBox )form_box.Items) ;            
    else if (form_box is ComboBox) ProcessItems((ComboBox)form_box.Items) ;            
    else MessageBox.Show("Error_Type");
}

public static void ProcessItems(ItemCollection Items)
{
  a=Items.Count;
}

[Graffito]

Sinon voici une solution à éviter :

Code :

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

ItemCollection Items = form_box.GetType().GetProperty("Items").GetValue(form_box,null) ;