EventEmitter to report activity

alexstrat commented 12 years ago

Bon alors comme on en avait discuter, je pense qu'il sera plus simple de faire un gros "EventManager" pour tout KadOH plutôt que chaque objet (RoutingTable , les KBuckets...) soient chacun un EventEmitter auquel il faudra s'abonner individuellement..

Avantages :

on s'occupe d'ajouter des listeners qu'à un seul endroit, c'est à dire à l'endroit où on en a vraiment besoin : juste avant un console-log, à l'update d'une UI ou avant de l'envoyer dans socket.io pour un report à distance.
pas d'extension de classe qui peut-être badante.
Petits inconvénients
faudrait faire gaffe à faire un namespace des events "intelligent" et à donner des arguments "utiles" aux listeners, avec une bonne doc.
Petits plus

Y'a moyen de faire un systeme de namespace comme dans celui de EventEmitter2 qui permet de s'abonner à des events qui matchent un pattern (eg foo:bar)... Mais je trouve l'intêret finalement limité.

Démystification

Un systeme d'EventEmitter n'est en fait qu'un tableau de callback quoi qu'il arrive. Jette un coup au module event de Node.js, c'est déconcertant de simplicité.

Du coup..

Je vasi essayer de mettre en place ce truc rapidement, comme ça on rajoute des emitions d'event un peu partout rapidement..

alexstrat commented 12 years ago

Bon, j'ai changé d'avis..

Je pencherais maintenant pour avoir des objets qui étendrais/implémentrais une classe EventEmitter car :

c'est quand même un pattern plus répandu et plus pratique lorsqu'on travaille avec des objets
on aurait finalement pas tant d'objets qui aurait besoin d'être des event-emitters, ceux dans lesquels il y a la 'logique'/'story-line' : je pense du coup aux principaux : Node, RoutingTable et Reactor (et par la suite un StorageManager).
au pire on peut parfois binder un EventEmitter sur un autre. Exemple : un objet RPC peut très bien être EventEmitter dont les events sont forwardés (avec adaptation ou pas) par RoutingTable.

Qu'en penses tu ?

jinroh commented 12 years ago

Je pense que c'est mieux d'avoir un système d'extend comme tu le décris, ça me parait plus sain :)

jinroh / kadoh