rewrite _find_sample_tags of AtCoder

[kmyk]

close #618

次をします:

• if elif elif の構造から単一の `for の構造へ変更する
• download: no samples found in AtCoder tenka1_2014_qualA_e #618 のためのテストを足す
• (ついでに) コメントを足しておく

#618 の原因は h3+pre.prettyprint であるのに h3+section pre のための分岐に食われてしまって h3 のために違う場所を探していたためだったようです。
同様の理由で h3+pre.literal-block なども (もしあれば) 検出に失敗します。

---

実装ですが、大きく構造を変えるのが楽なのでそうしました。
ついでに tag_plus がよくない (無闇に yield してるのとか外側の変数 pre を読んでるのとか) も修正されました。バグ修正のついでの refactoring はよくないのですが、書き直しが多いので巻き込みました。

ところで、単なる h3+section>pre を足すと <h3>Sample Output 1</h3><section><pre>13</pre>構成できる文字列は<pre><code>abc</code></pre>や…… で誤検出になります。これは意図的に回避しています。

(修正量が大きくて嫌な感じがしますが、元がまずかったのでしかたないやつだと思っています)

[kawacchu]

コメントの変更に伴い、
tests/service_atcoder.py内の

• test_from_html_first_format
• test_from_html_second_format
• test_from_html_third_format
  をそれぞれ
• test_from_html_very_old
• test_from_html_old
• test_from_html_standard
  のように変えて欲しいです。

[kawacchu]

ところで、今回のPRのようなtestを今までtests/service_atcoder.pyに追加していましたが、tests/command_download_atcoder.pyに将来的に移動させるべきと感じました。

[kmyk]

tests/service_atcoder.py 内の関数の rename をしました。

---

ところで間違えて master に push してしまいました: 7db05b0
ごめんなさい。原因は次の 2 点です:

1. git checkout fix/618 と叩くべきところを git checkout -b fix/618 としており、失敗して branch が master のままで切り替わっておらず、そのまま作業してしまった
2. 作業環境を一新したときに pre-push hook の設定をし忘れていた

[kmyk]

master に push した commit は削除はせずにそのままにしてある (force push はできない設定になってる) ので、他に問題がなければこのままマージしてください。

テストを tests/service_atcoder.py と tests/command_download_atcoder.py のどちらに置くべきかについては、自明なものではないと思っています。
tests/command_download_atcoder.py のようにすると oj 全体に対する大きなテストになり、tests/service_atcoder.py はサンプル解析部分のみに関する小さなテストになります。大きなテストは実際の使用例に即したある種の網羅性を持つテストになりますが、一方で小さなテストは細かい部分 (たとえばサンプルの <pre> の上の <h3> に書いてある中身など) もテストでき、一長一短なためです。

[kawacchu]

masterにpushされた内容を確認しましたが、大丈夫だと思います。
先ほど変更を希望した箇所以外は問題がないと思いますので、マージします。
ただ、#618で少し言及したA問題のバグ修正は将来なされるべきと考えています。

より見通しのよいコードに仕上がっていると感じています。ありがとうございました。

[kawacchu]

テストについてですが、そのような視点があるのですね。たしかにおっしゃるとおりです。

command_download_atcoder.pyをあまり読まずにtests/service_atcoder.pyで同じ問題についてのテストを作成してしまった経緯があるので、単一化できそうな箇所があれば片方を削除しようかと考えています。

[kmyk]

https://atcoder.jp/contests/tenka1-2014-quala/tasks/tenka1_2014_qualA_a の <h3> 内で <var> が利用されているという問題は、たしかに注意は必要ですが、実際に問題となるケースが存在するとは思えません。今回の例の場合は入力が存在しない問題であるため発生していますが、通常の「入力例1」「出力例1」と並ぶ形式で <var> が出現するとは思えません。どちらかといえば、コードの簡潔さを優先してこのままにしておきたさがあります。ですので対応はひとまず見送りました。

[kawacchu]

そうですね。
現行のコンテストでは問題が発生していないようなので、このことについてはひとまず見送ろうと思います。