refactoring about onlinejudge.service.*.*Content

[kmyk]

model に対する理解の不足から来た設計ミスがあります。適当なタイミングで修正する必要があります。

onlinejudge.type.Submission のようなクラスは、おそらくメソッドの生えた URL 文字列と認識されるべきです。もしそれ以上の役割を持つならすぐに破綻するからです。
特に、何らかの得られたデータのキャッシュを持ってしまうと「どのような session から得られたいつの時点でのキャッシュなのか」「複数の session が与えられた場合どうなるべきか」「通信が発生するのかキャッシュが参照されるのかの判別ができない」などの困難が発生します。

---

たとえば現状では onlinejudge.service.atcoder.AtCoderContest は以下のようにキャッシュを保持してしまっています。そして実際に、「過去のコンテスト一覧 https://atcoder.jp/contests/archive から取得されたデータで半分ぐらいキャッシュが埋まっている状態」と「コンテストページ https://atcoder.jp/contests/abc130 を読んだ後のすべてキャッシュが埋まっている状態」の区別や、その際の言語設定が「英語」か「日本語」かによるデータの違いなどがとても曖昧になっています。
これは単に、文字列 https://atcoder.jp/contests/abc130 と同一視できるクラス AtCoderContest と、ある 1 回の通信の結果を表現する namedtuple AtCoderContestContent のふたつへ分割することで解決できます。

https://github.com/kmyk/online-judge-tools/blob/60a5b937d9978cc96a83d08c7fc6fbeec6d526a4/onlinejudge/service/atcoder.py#L149-L167