Commentaires

1 Le mercredi, février 15 2006, 14:41 par Aggelos

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

2 Le mercredi, février 15 2006, 15:16 par zyegfryed

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...

3 Le mercredi, février 15 2006, 16:15 par LAlex

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

4 Le mercredi, février 15 2006, 18:23 par klee

ouais j'ai trouver cette astuce ya pas longtemps.

ce genre de petite astuce simplifie la vie !

5 Le jeudi, février 16 2006, 08:55 par Jep

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

6 Le jeudi, février 16 2006, 12:48 par ekameleon

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+

7 Le jeudi, février 16 2006, 16:36 par LAlex

@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é

++ ^^

8 Le jeudi, février 16 2006, 17:09 par ekameleon

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+

9 Le jeudi, février 16 2006, 22:12 par Philippe

Je trouve ça limite tordu mais magnifique

10 Le vendredi, février 17 2006, 23:42 par SamsamTS

Enorme ! Je ne connaissai pas merci

11 Le samedi, février 18 2006, 17:58 par liguorien

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()

12 Le mardi, février 21 2006, 11:28 par Fred

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+

13 Le mardi, février 21 2006, 13:46 par ekameleon

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

14 Le mardi, février 21 2006, 14:38 par liguorien

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)

@++

15 Le mardi, février 21 2006, 15:04 par LAlex

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... :\

16 Le mardi, février 21 2006, 15:14 par liguorien

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

17 Le mardi, février 21 2006, 15:34 par LAlex

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

18 Le mercredi, février 22 2006, 11:12 par Fred

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()

19 Le mardi, mai 16 2006, 14:50 par caron

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.

Fil des commentaires de ce billet