search

darwinop-ens / darwin-op

This is an unofficial repository containing all the changes we made on the original code base from http://sourceforge.net/project/darwinop
27 stars 19 forks source link

mouvements, la structure n'est pas claire... #2

Open ghost opened 11 years ago

ghost commented 11 years ago

L'interface des mouvements est bizarre...

MotionModule représente non le "module qui gère le mouvement"... mais un mouvement abstrait... elle devrait s'appeler "AbstractMotion", ou tout simplement "Motion"

Walking hérite de MotionModule : ok

MotionManager semble être une succession de mouvements... mais dans ce cas, .... c'est un mouvement !! Pourquoi MotionManager ne s'appelle pas MotionSequence (séquence de mouvements) ? Pourquoi Pourquoi MotionManager (MotionSequence) n'est pas un mouvement, (c'est à dire n'hérite pas de MotionModule (ou Motion) ?)

Ca manque un peu de cohérence tout ça...

ghost commented 11 years ago

Et Head ne représente pas la tête... mais bien quelques mouvements de la tête... donc ça devrait être MotionHead (ou HeadMotion)...

ghost commented 11 years ago

Et la classe "Action" est la pire... on ne comprend pas ce qu'elle fait !! "Action" c'est trop vague...

ghost commented 11 years ago

Aussi la classe Action a l'air très grosse... et même la classe Head... la classe Head fait "hocher la tête", "suivre un point" etc. avec des méthodes différentes etc. Ce n'est pas terrible car si on veut améliorer la fonction "hocher la tête" il faut aller aller modifier le fichier....

Je propose quelque chose comme une interface qui s'occupe des mouvements de BAS NIVEAU de la tête... et on l'implémente pour DarwinOP puis des comportements plus haut niveau comme "hocher la tête", "suivre un point" qui elle utilise l'interface... à voir....

Faut que je dessine de l'UML :)