Discussion :

[r0d]

Bonjour,

Imaginons que j'ai une classe Mere, ainsi qu'une classe Fille qui herite de Mere.
Maintenant j'ai une collection de Mere, qui contient des Mere et des Fille.
Et je voudrais effectuer un traitement sur chaque element de cette collection mais seulement sur les objets de type Fille.

Un exemple concret: j'ai une classe MyControlType qui herite de Control.
Ma Form possede une collection de Control, dans lequel j'insere des MyControlType, mais qui possede egalement des controles "normaux" (boutons etc.). Et je voudrais faire une boucle sur Controls, de facon a appeler une fonction DoSomething() de MyControlType. Pour l'instant, j'ai le code suivant:

Code :

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

1
2
3
foreach (Control c in Controls)
    if (c is MyControlType)
        ((MyControlType)c).DoSomething();

Ca marche, mais le compilateur me lance un "performance warning". Car effectivement, il y a des casts dans tous les sens dans cette boucle. J'imagine par exemple que le is effectue un "cast cache".

Comment faire, donc, pour faire cela proprement?

[Er3van]

Et comme ça ?

Code c# :

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

1
2
3
4
 
foreach (MyControlType mc in Controls.Where(c => c is MyControlType)) {
     mc.DoSomething();
}

[MaximePalmisano]

Bonjour,

tu peux créer une méthode virtuelle DoSomething() dans ta classe mère que tu override dans ta classe fille avec les opérations à faire. Tu ne mets rien dans la méthode de la classe mère et ensuite tu boucles juste en appelant la méthode. Si c'est une mère, la méthode est vide, si c'est une fille, la méthode contient les opérations à faire.

[Er3van]

tu peux créer une méthode virtuelle DoSomething() dans ta classe mère que tu override dans ta classe fille avec les opérations à faire. Tu ne mets rien dans la méthode de la classe mère et ensuite tu boucles juste en appelant la méthode. Si c'est une mère, la méthode est vide, si c'est une fille, la méthode contient les opérations à faire.

Si tu peux éditer la classe mère c'est sans doute le plus simple en effet pour éviter les cast.

[r0d]

Merci pour vos reponses.
Malheureusement non, je ne peux pas editer la classe mere, qui est une classe du framework. Je ne peux meme pas modifier la collection, qui appartient a la classe Windows.Form.

Je pense donc que la solution va etre celle de Er3van.

[Sehnsucht]

Bonjour,

Tout en restant en Linq et basé sur la solution d'Er3van il faut savoir qu'il existe déjà exactement ce qui est demandé :

Code :

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

1
2
3
foreach (MyControlType mc in Controls.OfType<MyControlType>()) {
     mc.DoSomething();
}

Cordialement !

[antoine.debyser]

Bonjour,

Sauf erreur de ma part, l'IL généré par le compilateur à partir du code d'Er3van
sera du même acabit que celui issu de ton premier code.

En effet il y a un cast caché dans le foreach, on fait dans tout les cas le test avec "is".
Pour moi ça reviens strictement a la même chose, sauf que dans un cas le compilateur est capable de t'indiquer qu'il y a un potentiel problème de performance, et pas dans l'autre.

[Pol63]

moi je ne vois rien qui mène à se poser des questions de performance là dessus

[r0d]

Oui aparamment c'est juste que dans un cas il parvient a trouver qu'il y a un probleme, et dans l'autre non.
Je pensais qu'on aurait pu eviter ces casts caches, sans toutefois avoir la moindre idee de comment faire concretement. Mais theoriquement, il me semble qu'en c# les types ont un id, donc je pensais a une solution se basant sur cet id, plutot que de tenter le cast.

[Guulh]

moi je ne vois rien qui mène à se poser des questions de performance là dessus

Absolument. Si y'a des problèmes de perf pour ça, c'est qu'il ya des milliers (des millions ?) de contrôles, et s'il y a des milliers de contrôles, c'est qu'il y a un souci de conception...

Mais pour une problématique plus générale, j'appuie la suggestion de Sehnsucht. Linq propose une jolie méthode OfType, qui fait ce qu'on lui demande, de façon efficace.

[tomlev]

Ca marche, mais le compilateur me lance un "performance warning".

Ca m'étonnerait que ce soit le compilateur... tu n'utiliserais pas un outil comme Resharper ou autre ?

Sinon, tu peux utiliser l'opérateur as qui est parfaitement adapté à ce genre de chose :

Code :

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

1
2
3
4
5
6
foreach (Control c in Controls)
{
    MyControlType x = c as MyControlType;
    if (x != null)
        x.DoSomething();
}

Sinon, si tu es en 3.5 ou supérieur, la meilleure solution est celle de Sehnsucht avec la méthode OfType

[Pol63]

à tous les coups le oftype parcourt la collection, vérifie le type, et cast pour remplir un list<T> ou autre
linq apporte un gain de lisibilité mais dans ce cas pas beaucoup plus je pense

[tomlev]

à tous les coups le oftype parcourt la collection, vérifie le type, et cast pour remplir un list<T> ou autre
linq apporte un gain de lisibilité mais dans ce cas pas beaucoup plus je pense

A priori l'implémentation de OfType doit être de cet accabit :

Code :

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

1
2
3
4
5
6
7
8
public static IEnumerable<TResult> OfType<TResult>(this IEnumerable source)
{
    foreach(object o in source)
    {
        if (o is TResult)
            yield return (TResult)o;
    }
}

Donc effectivement ça pose le même problème de performance que le code d'origine. Mais bon, je pense que pour la plupart des cas la différence de perf n'aura pas beaucoup d'impact, et le gain de lisibilité est appréciable

En tous cas ce qui est sûr c'est que ça ne remplit pas une liste avant de la renvoyer ; ça renvoie les résultats au fur et à mesure qu'ils sont lus

[antoine.debyser]

Bon je viens de verifier avec un ILSpy

Code :

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

1
2
3
4
foreach (MyControlType mc in Controls.Where(c => c is MyControlType)) 
{
     mc.DoSomething();
}

produit bien un cast.
Ca n'apporte donc rien par rapport au code de base.

Comme le dit tomlev, l'implementation de OfType (toujours d'apres ILSpy) est la suivant

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 IEnumerable<TResult> OfType<TResult>(this IEnumerable source)
{
	if (source == null)
	{
		throw Error.ArgumentNull("source");
	}
	return Enumerable.OfTypeIterator<TResult>(source);
}
 
private static IEnumerable<TResult> OfTypeIterator<TResult>(IEnumerable source)
{
	foreach (object current in source)
	{
		if (current is TResult)
		{
			yield return (TResult)current;
		}
	}
	yield break;
}

Quand à l'utilisation de "as"
Et bien "as" utilise "is", la difference se fait au niveau du cast il n'y en a pas, c'est un retour conditionnel.


Pour ce qui est du warning evoqué dans le sujet de base.
Ce n'est pas resharper qui le génére, mais l'analyse de code fournit avec Visual: warning CA1800.
Ce warning indique au developpeur qu'il peut éliminer son cast, notament avec "as".
L'analyse de code oublie completement de traiter le cas de Linq.


conclusion pour un parcours simple comme celui-ci, "as" c'est bien.

[Guulh]

Ce warning indique au developpeur qu'il peut éliminer son cast, notament avec "as".

Et tant qu'à en#@er les mouches: OfType ne peut pas utiliser 'as', car 'as' ne fonctionne qu'avec les types référence, et que le paramètre générique de OfType n'a pas de contrainte le forçant à être un type référence.(par exemple, int i = obj as int n'aurait pas de sens ; que renvoyer si ce n'est pas un int ? Par contre, le compilo gère les nullable, qui sont techniquement des structs, mais pour lesquels l'opérateur 'as' est dispo.)

On aurait donc le meilleur des deux mondes avec ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public static class MexExtensionsDeLaMort
{
   public static IEnumerable<T> OfTypeRef<T>(this IEnumerable source) where T : class
   {
     foreach(object obj in source)
     {
        T casté = obj as T; // on peut car T est un type référence
        if (casté != null)
          yield return casté;
     }
   }
}

Mais bon, y'a trèèèèès souvent mieux à faire que s'interroger sur les perfs des casts sur une GUI, dont les méthodes qui accèdent aux contrôles ne sont pas appelées ni assez souvent, ni avec assez de volumes pour que ça vaille la peine.

[brachior]

Et un :

Code :

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

1
2
3
4
 
foreach (Control c in Controls)
    if (c.GetType().Equals(typeof(MyControlType)))
        ((MyControlType)c).DoSomething();

[Guulh]

Et un :

Code :

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

1
2
3
4
 
foreach (Control c in Controls)
    if (c.GetType().Equals(typeof(MyControlType)))
        ((MyControlType)c).DoSomething();

Marche pas : tu refuses ici les objets d'un type dérivé de MyControlType. Et je doute fort que GetType() suivi d'un cast soit plus rapide que 'as'.

[brachior]

Tu refuses ici les objets d'un type dérivé de MyControlType.

Et je voudrais effectuer un traitement sur chaque element de cette collection mais seulement sur les objets de type Fille.

Faudrait savoir ^^

C'est sur qu'un Equals doit être plus couteux ...
Mais ça retire le double cast

Soit dit en passant ...
Je sais très bien que c'est super laid ce que j'ai écrit ^^

[tomlev]

Faudrait savoir ^^

Bah les objets dérivés de Fille sont aussi des Fille...

[brachior]

Rah ça chipote là ^^