search

mikiya771 / tetris-CLI

this repo serve Tetris, the most famous game, on CLI
1 stars 0 forks source link

Refactor: add event listener for view #22

Closed mikiya771 closed 4 years ago

mikiya771 commented 4 years ago

https://github.com/mikiya771/tetris-CLI/issues/15#issue-614711322

view用のdispatcherを投げる部分を分離した. viewControllerにCallback関数を登録して, そこでstore更新のイベントを受けるようにした

progfay commented 4 years ago

Dispatcherの実装が重複しているのが気になりますね structとして作って、StoreとViewControllerに一つづつ持たせるのはどうでしょうか

mikiya771 commented 4 years ago

event listenerとして扱えるようにしていると解釈して，event listenerからdispatcherを作る，event listenerからview-contollerを作るのが良さそうですね．

progfay commented 4 years ago

Dispatcher をインスタンスのまま保持するというアイデアには賛成です。

mikiya771 commented 4 years ago

https://github.com/mikiya771/tetris-CLI/pull/22#discussion_r423600752 dispatcherが本来の用途なので残そうと思っていました．しかしインスタンス名がDispatcherになっているので，event listenerの書き方に習う方が良さそうですね

mikiya771 commented 4 years ago

わざわざeventListener作らずに，普通にobserverを作ればいいかもしれない