クソコードレビュー会なるものに参加してきた - そんな今日この頃でして、、、

転職活動でもお世話になったCodeIQで↓のような問題があったので回答を出したところ、 レビュー会への招待が来ていたので行ってみた！ codeiq.jp 問題 出題はじゃんけんを行うプログラムをリファクタリングするという問題。 「標準入力として与えられた文字列を2人分のじゃんけんの手として解釈して勝敗を標準出力する」というもの。 元はnode.jsだったのだが、僕はjsが不慣れだったりもするのでperlで再現。 sub func02 { my ( $a, $b ) = @_; if ( $a eq 'g' ) { if ( $b eq 'c' ) { print "win"; } elsif ( $b eq 'p' ) { print "lose"; } else { print "draw"; } } if ( $a eq 'c' ) { if ( $b eq 'p' ) { print

[yoshiki_coji]

http://blue1st.hateblo.jp/entry/2016/08/23/235506

[nukonukope]

いいなあ

[lyiase]

こういうの楽しそう！私なら2つの変数をinsideとoutsideにして、2次元テーブル引きかなあ。定義しやすいし、判断のロジックも分かりやすい。

[nacika_inscatolare]

楽しそう。

[kazuhooku]

面白そうな会。引数をplayerとcomputerにするのは、ジャンケンのプレイヤーが可換であることを考えると筋悪だと思う

[bunnyhop]

あえてのクソコード / ruby -e '$*.tap{|p,c|p==c ?$><<:draw:%w(g c p).tap{|h|$><<(h[h.index(p)-1]==c ?:lose: :win)}}' g c

[masuidrive]

参加いただきありがとうございましたー。本当、予想してたより幅広いコードが集まって面白かったです

[tatuyan]

うーん、自分なら3x3二次元配列でテーブル引きにしちゃうかなぁ

[tofu-kun]

自分ならどう書くかなーと想像したら面白かった。勝敗の組み合わせをリストとして定義する感じ。

[vanbraam]

読みやすさ,テストのしやすさ,適切なエラー・ハンドリング<-この3つが客観的に評価できたら苦労も少ないのだが..