search

Aqua3k / RandomTestCaseDebugger

ランダムテストケースの作成と実行が簡単にできる環境
0 stars 1 forks source link

WIP HTML出力の検討 #18

Closed Aqua3k closed 2 years ago

Aqua3k commented 2 years ago

概要

6 で検討していたHTMLで結果出力の検討したソース

@dorapon2000 とりあえずこんな方針でいいか簡単にレビューお願いできる？ 細かいコードのレビューというよりは方針がこんな感じでいいのか的なところを確認してほしい 機能拡張のため少し大きめに変更加えたいのでここで1回確認しておきたい

出力ファイル

result.html ・ここに結果(AC, WA, REとか)とかも載せて表形式にしたい 1

caseX.html　(result.htmlのリンクからたどれる) ・REの場合はdiffの結果ではなくエラーメッセージの内容を表示するようにしたい ・(こっちだけじゃなくてresult.htmlもだけど)cssできれいにしたい 2

その他

動作確認は最低限しかしてない

Aqua3k commented 2 years ago

よさそう。もう一つ案として、result.htmlにすべての結果を書いてしまってもいいんじゃないかなとも思う。1つ1つ詳細を見るためにリンクを辿るのは大変そうに感じた。一般的なテストもめっちゃたくさんのエラーを一度に吐き出すし、1ページにしてしまってもそこまでおかしくないと思う。

さらに別案として、アコーディオンみたいなデザインを使って、見せる情報をスリムにする方法もありそう。ただし、jsでの実装が必要。

なるほどー。 例えば出力がN行必要な問題で100ケースとか作って結果表示したらめちゃくちゃ見にくそうだなーって思ってリンクにしてる。 やるとしたらアコーディオンかなって思ってる。

dorapon2000 commented 2 years ago

例えば出力がN行必要な問題で100ケースとか作って結果表示したらめちゃくちゃ見にくそうだなーって思ってリンクにしてる。

そのケースは一度にバッと表示されても困るな。ただあんまり多いケースでもないから、アコーディオンにするか、あるいはオプション引数で出力形式を変える対応もありそう。

@Aqua3k 変更LGTM

Aqua3k commented 2 years ago

さんきゅー

そのケースは一度にバッと表示されても困るな。ただあんまり多いケースでもないから、アコーディオンにするか、あるいはオプション引数で出力形式を変える対応もありそう。

オプション引数で出力形式を変えるのはよさそう

Aqua3k commented 2 years ago

大きな変更点

・実行結果の情報をクラスResultStatusで管理するように変更 ・実行結果情報のクラス化に伴い、txtファイルやHTMLファイル、標準出力の出力方法を調整 ・実行時のエラー情報の取得方法をtracebackモジュールを使うように変更 ・HTMLの出力形式を表形式に変更