LAlex devblog v6 - Switch/case original : tester la classe d'une variable - Commentaires

Switch/case original : tester la classe d'une variable - caron

caron — Tue, 16 May 2006 14:50:38 +0000

Cette astuce n'est pas valable en .NET, car ce qui suit le case doit être une constante.
Mais je retiens cette astuce pour d'autres langages.

Switch/case original : tester la classe d'une variable - Fred

Fred — Wed, 22 Feb 2006 11:12:46 +0000

Il y a plusierus raisons pour lesquelles l'usage de instanceof est deconseillee et plus particulierement dans un case:
ta fonction getEnvironement doit connaitre de maniere exhaustive tous les types de tous les objets. Au niveau maintenance c'est pas super. Si tu rajoutes un type tu vas etre oblige de repercuter cette creation dans des fonctions qui n'ont rien a voir avec ton design. Je te raconte pas la prise de tete.
au niveau encapsulation c'est pas super non plus. Si durant ton developement tu veux specialiser par exemple bus et velo pour un env qui pourrait s'appeller road_plane tu dois encore modifier ta fonction, pas bien non plus. Un objet est le seul a savoir a son etat.

Les seuls cas que je connais d'utilisation de instanceof sont pour tester des sur-type d'un objet qui aurait un pere abstrait (interface). Comme par exemple pour tester l'existence d'une fonction.
if (instanceof vehicule) engine->getPetrol()
else if (instanceof velo) engine->getShoes()

Switch/case original : tester la classe d'une variable - LAlex

LAlex — Tue, 21 Feb 2006 15:34:49 +0000

Je vais m'abstenir à l'avenir.Ca, ce serait dommage.... :'(
C'est juste que la rethorique, ca va un moment, restons dans le sujet! ^^

Switch/case original : tester la classe d'une variable - liguorien

liguorien — Tue, 21 Feb 2006 15:14:53 +0000

Mon dernier message n'était qu'une réponse à la question d'eka... Je vais m'abstenir à l'avenir.

Switch/case original : tester la classe d'une variable - LAlex

LAlex — Tue, 21 Feb 2006 15:04:04 +0000

Ehoh les copains!!! :o Ce que je démontre là, c'est juste l'utilisation d'un switch...case pour autre chose qu'une comparaison de valeurs...

Si on ne veut pas utiliser de instanceof, on peut aussi se la jouer sur le cast:switch(true) { case (!!Boat(eng)): // blablabla... case (!!Car(eng) || !!Motorcycle(eng)): // blablabla... }

Il doit bien exister a peu prés 10.000 manière de retourner cet environnement, plus ou moins recommandables, plus ou moins "clean", mais c'est une astuce que je donne, pas une leçon de POO...

Evidemment qu'il faut créer une méthode getEnvironment... :\

Switch/case original : tester la classe d'une variable - liguorien

liguorien — Tue, 21 Feb 2006 14:38:47 +0000

Il ne fait répêter ce que tous les profs de POO disent L'utilisation du instanceof est pratique dans certain cas, mais ce n'est pas très souple et on peu très bien s'en passer avec un bon design OO. Si un comportement change en fonction d'un type et que l'on a beaucoup de types et de comportements, il est préférable d'encapsuler le comportement dans le type. (cf. rajouter une méthode getEnvironement() à l'interface Engine)

@++

Switch/case original : tester la classe d'une variable - ekameleon

ekameleon — Tue, 21 Feb 2006 13:46:12 +0000

Hello
Fred pourquoi tu déconseille l'utilisation de instanceof ?
EKA+

Switch/case original : tester la classe d'une variable - Fred

Fred — Tue, 21 Feb 2006 11:28:03 +0000

Salut Alex,
je suis pas super d'accord avec cette solution.
En prog object tu n'as pas a tester le type d'object auquel tu as affaire.
L'utilisation de instanceof est vraiment res deconseillee a part ds certains cas.
La vraie solution elegante et orientee object est de creer pour chaque calsse d'objet : Car, Bus etc.. une methode getEnv par exmple qui est appellee sur eng.
Ta methode getEnvironement se resume a
return eng->getEnv();
a+

Switch/case original : tester la classe d'une variable - liguorien

liguorien — Sat, 18 Feb 2006 17:58:47 +0000

Je trouve ça un peu bizarre comme technique, mais j'avoue que ça peut-être pratique lorsque l'on travail avec une base de code qui provient de l'extérieur. Il aurait été mieu à la base de rajouter une méthode getEnvironement() à l'interface Engine
À noter que cette technique ne s'applique pas aux languages qui n'acceptent que des nombres entier comme valeur dans un switch()

Switch/case original : tester la classe d'une variable - SamsamTS

SamsamTS — Fri, 17 Feb 2006 23:42:19 +0000

Enorme ! Je ne connaissai pas merci

Switch/case original : tester la classe d'une variable - Philippe

Philippe — Thu, 16 Feb 2006 22:12:57 +0000

Je trouve ça limite tordu mais magnifique

Switch/case original : tester la classe d'une variable - ekameleon

ekameleon — Thu, 16 Feb 2006 17:09:15 +0000

Oui c'est clair que le principe est peu commun (je n'avais pas pensé à faire cela ;)) et pour un pattern Strategy en AS2 cela semble une bonne chose, faudrait voir ce que cela donne avec au passage une petite reflexion sur les classes directement et en testant tout simplement le namespace de la fonction constructeur de l'instance
EKA+

Switch/case original : tester la classe d'une variable - LAlex

LAlex — Thu, 16 Feb 2006 16:36:12 +0000

@eka> Si en effet, voila ce que c'est de faire un exemple trop vite fait sans le compiler... En même temps, le but était surtout d'illustrer le principe...

Corrigé

++ ^^

Switch/case original : tester la classe d'une variable - ekameleon

ekameleon — Thu, 16 Feb 2006 12:48:59 +0000

Hello
Dans ton exemple au dessus, la variable env dans le switch ne te renvois pas d'erreur à la compilation ?
Pour ma par j'ai un joli :
Les instructions précédant la première instruction 'case' d'une instruction 'switch' sont ignorées
var env:String = "Twilight zone";
EKA+

Switch/case original : tester la classe d'une variable - Jep

Jep — Thu, 16 Feb 2006 08:55:26 +0000

Génial... je vais réexploiter ça en PHP !

Switch/case original : tester la classe d'une variable - klee

klee — Wed, 15 Feb 2006 18:23:32 +0000

ouais j'ai trouver cette astuce ya pas longtemps.

ce genre de petite astuce simplifie la vie !

Switch/case original : tester la classe d'une variable - LAlex

LAlex — Wed, 15 Feb 2006 16:15:57 +0000

Oups!
Autant pour moi, j'avais mis au début des return, et donc le break était inutile... :p

Switch/case original : tester la classe d'une variable - zyegfryed

zyegfryed — Wed, 15 Feb 2006 15:16:19 +0000

Question bête : ne vaut il pas mieux rajouter un break après l'initialisation de la variable (suite à la détection de type), ceci afin d'éviter d'éxécuter de manière séquentielle tous les tests case ? Histoire de gagner (un petit petit) en temps d'éxécution...

Switch/case original : tester la classe d'une variable - Aggelos

Aggelos — Wed, 15 Feb 2006 14:41:06 +0000

Effecvivement, c'est une utilisation du switch que l'on trouve dans le moock. C'est un bon exemple d'application