Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

スレッドの dat 取得処理は thread.js から分離するべき #15

Open
nodaguti opened this issue Jun 26, 2013 · 1 comment
Open
Milestone

Comments

@nodaguti
Copy link
Contributor

thread.js はスレッドの表示に関わる処理のみが存在するべき。
dat の取得とスレッドの表示が分離されれば、現状の複雑な挙動がかなり改善されるはず。

期待される効果

  • dat 取得先について表示部分は気にする必要がなくなり、以下の実装が可能になる
    • dat の動的追加(手動追加)
    • dat 落ちの処理
    • ローカルスレッド
  • 未読件数、取得済み件数、スレのレス件数などと dat 本体との結びつきが疎になり、より柔軟な管理が可能になる
@nodaguti
Copy link
Contributor Author

nodaguti commented Jan 4, 2015

現在は Thread2ch#datLineParse -> ThreadConverter.getResponse という流れだが、

  1. URL を解析: ChaikaThread などの仕事?
  2. 該当する dat を取得: 別クラスの担当
  3. dat データにスキンを適用し, HTML データを得る: 別クラスの担当
  4. HTML データをブラウザに出力する: ChaikaServer の仕事

という流れにし、thread.js は各クラスを利用して処理の橋渡しをするような感じでリファクタリングする。

また 3. について、新しいフローでは dat を解析するクラスと HTML への変換をするクラスの2つに分ける. なぜなら、 #9 などで dat の解析だけを利用するユースケースがあると考えられるからである.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant