チェッカーまわりリファクタリングしたい about private-isu HOT 4 CLOSED

とりあえずcli.goのチェック処理を全部scenario.goにリファクタリングして移植した。

一つ一つのシナリオが上から一直線になって読みやすくはなった。しかし、同じような処理が複数箇所に書かないといけなかったのは今後のリファクタリング課題。

それよりも大きな問題は、 PlayWithURL のインターフェイス。

    _, err = postImage.PlayWithURL(s1)
    if err != nil {
        return // TODO: どういうエラーハンドリングが適切か考える
    }

CheckFunc の外でActionをエラーにすることはできないので、ここに来た時点でエラーであってもActionはエラーにならないし、競技者にもエラーが伝わらない。

やはり PlayWithURL は廃止して CheckFunc の引数にリダイレクト先URLを受け取るか、ExpectedLocation で正規表現や前方一致を受け付けるようにするかのどちらかにするべき。

from private-isu.

現状の実装だと PlayWithURL と Play の差は返り値しかないので、返り値を受け取らないなら使う必要は無いです。

PlayWithURL はすごくイケてないので廃止したいですが、リダイレクト先のURLを受け取る方法がないのでとりあえず作りました。

from private-isu.

ExpectedLocationを正規表現にしてPlayWithURLを廃止することにします。

from private-isu.

PlayとCheckFuncを分けずにの第二引数にCheckFuncを渡すスタイルでいい気がする

これだけ残ってるがあとは終わった。

CheckFuncを書いた時点でチェックできてる気になってるから、Playを呼び忘れてしまう。実際2回ぐらい気づいて直した。

from private-isu.