vim-jp / slacklog-generator

Archive generator for Slack
https://vim-jp.org/slacklog/
Creative Commons Attribution 4.0 International
26 stars 9 forks source link

generator と message と store が密結合してる #109

Open koron opened 4 years ago

koron commented 4 years ago

理想的な構成としては、 messageは単なる器でモデルにとどまるべき。 storeは名前とDBというかモデルをどう構造化して可能するかに注力するべき(repositoryってやつ)。 generatorはstoreからmessageなどのモデルを適宜取り出して、HTMLを作ることに注力すべき。

なのにいまはそれぞれがお互いの責任境界を越えていらんことしてる。 そのため、たとえばmessageがチャンネルのメッセージデータを読むときに スレッドを構築しているため日付順にファイルを読まないとならない みたいなおかしな制約が存在している。

ちなみにこれは store に message を突っ込むときに store が構造化の面倒をみるべきはなし。


これらをちゃんとあるべきところに分解してやる必要がある