Discussion :

[rockley]

Bonjour à tous
Je pense que le titre est assez expressif.

J'ai une méthode :

Code :

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

1
2
3
4
5
6
 
private void traitement_des_lecture(ArrayList tab_lecture)
foreach (Type_tab_lecture _my_enreg in tab_lecture)
{
...
}

Si les traitements sont les mêmes, les arrayList ne le sont pas.
Comment je peux dynamiser le Type_tab_lecture de mon foreach.

[rockley]

Ok j'ai trouvé ça.

Code :

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

Tab_lecture[0].GetType()

c'est pas super propre vu que je pense que je serai obligé de gérer les arraylist null mais bon.

[tomlev]

Ta question n'est pas très claire... Qu'est-ce que tu cherches à faire exactement ?

Soit dit en passant, à moins que tu n'utilises encore .NET 1.x, il vaut mieux éviter d'utiliser ArrayList. Il vaut mieux utiliser les collections génériques comme List<T> (List<Type_tab_lecture> dans ton cas)

[rockley]

Disons que dans mon

Code :

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

private void traitement_des_lecture(ArrayList tab_lecture)

Je peux envoyer des arrayList de différents Objets.

Donc pour mes

Code :

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

1
2
3
4
5
 
foreach (Type_tab_lecture _my_enreg in tab_lecture)
{
...
}

Pourrai se transformer dynamiquement en

foreach (Type_Voiture _my_enreg in tab_lecture)
{
...
}

ou

foreach (Type_Maison _my_enreg in tab_lecture)
{
...
}

Mais tu m'as dis quelque chose de plus intéressant.

Soit dit en passant, à moins que tu n'utilises encore .NET 1.x, il vaut mieux éviter d'utiliser ArrayList. Il vaut mieux utiliser les collections génériques comme List<T> (List<Type_tab_lecture> dans ton cas)

Pourquoi ? (Je suis au 3.5)

[BenoitM]

parce que tu peux spécialiser les List<T>
et donc ca empeche de faire des erreurs mettre des Voitures dans une liste de personne par exemple

Avec une List<Voiture> tu es sur d'avoir que des voitures de plus tu évites de devoir caster

Code :

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

1
2
listVoiture[0].Moteur;
(Voiture)arrayList[0].Moteur;

Pour plus d'info ici (note)

[tomlev]

Mais tu m'as dis quelque chose de plus intéressant.


Pourquoi ? (Je suis au 3.5)

Parce que c'est fortement typé ; au lieu d'une liste de object qui peuvent être n'importe quoi, tu as une liste de T (T étant un type précisé dans la déclaration). Donc plus besoin de faire des cast, pas de risque d'ajouter à la liste un objet d'un type incorrect, etc.

En plus ça résout ton problème, en rendant la méthode générique :

Code :

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

1
2
3
4
5
6
7
private void traitement_des_lecture<T>(IList<T> tab_lecture)
{
    foreach (T _my_enreg in tab_lecture)
    {
        ...
    }
}

Ca s'utilise comme ça :

Code :

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

1
2
3
4
5
6
7
8
List<Type_tab_lecture> list = new List<Type_tab_lecture>();
list.Add(...);
 
...
 
traitement_des_lecture<T>(list);
// ou même, en profitant de l'inférence de type générique :
// traitement_des_lecture(list);

[rockley]

Je suis désolé mais je ne vois pas la différence.

En gros je lis plusieurs table oracle.
Pour chaque table, j'ai une classe.

par exemple pour voiture

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Classe Voiture
{
String Mark;
String Prix;
String Couleur;
   public Voiture(OracleDataReader dr){
       _Mark = dr["Mark"].ToString();
       _Prix =  dr["Prix"].ToString();
       _Couleur = dr["Couleur"].ToString();
   }
   + les getters
}

Pour crée mon arrayList de Voiture je lance

Code :

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

1
2
3
4
5
 
            while (dr.Read())
            {
                res.Add(new Voiture(dr));
            }

Je vois pas comment je peux me retrouver avec d'autres objets dans mon arrayList que des voitures.

Je n'ai peu être pas été claire sur ce que je voulais, dsl.

[BenoitM]

mais rien n'empeche à faire

Code :

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

 res.Add(new Maison(dr));

or si tu utilise une List<Voiture> tu ne pourra pas le faire

[tomlev]

Je vois pas comment je peux me retrouver avec d'autres objets dans mon arrayList que des voitures.

Bah avec une ArrayList, tu pourrais le faire par erreur, sans réaliser que c'est une liste qui ne devrait contenir que des voitures. Dans ce cas précis tu as effectivement peu de chances de te planter, mais dans certains cas c'est moins évident. Alors qu'avec une liste générique, le compilateur détectera l'erreur...

Autres avantages :
- comme mentionné précédemment, tu n'as plus besoin de faire de cast des éléments de la liste vers le type réel de l'objet (avec encore une fois le risque que ça plante si l'objet n'est pas du type attendu).
- en voyant le type de la variable liste, tu sais tout de suite ce qu'elle est supposée contenir, même si le nom n'est pas clair
- avantage aussi en termes de performance, pour les types valeurs (int, DateTime, structures, enums, etc) : il n'y a plus de boxing (mécanisme qui consiste à encapsuler un type valeur dans une instance d'object, et qui peut être assez couteux en ressource)

Bref, d'une manière générale, tu as toujours intérêt à préférer le typage fort, pour des raisons de clarté du code, de fiabilité et de performance. Les génériques sont souvent le moyen le plus simple de profiter du typage fort.

[rockley]

Merci pour votre aide et pour vos explications.

Je pense que je ferais des tests pour savoir quelle est la solution la plus rapide et que je choisirai en conséquence.

[kheironn]

D'après ce que j'ai compris, rockley utilise justement des listes hétérogènes. Donc avoir une liste qui contient des voitures, des maisons et des personnes ne le gêne pas.

Mais Si comme tu le dis, tu récupères en fait un certains type, venant d'une requête sur ton SGBD, alors oui, tu devrais retourner des listes de type unique donc List<T> Ta requête "Select * from personne...." te retourneras des personnes donc ArrayList n'as pas de raison d'être List<Personne> est plus propre et évite le panier de crabes.