{"action":"created","issue":{"url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/3","repository_url":"https://api.github.com/repos/JunichiIto/train-ticket-rails","labels_url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/3/comments","events_url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/3/events","html_url":"https://github.com/JunichiIto/train-ticket-rails/pull/3","id":242210650,"number":3,"title":"人柱","user":{"login":"yhirano55","id":15371677,"avatar_url":"https://avatars0.githubusercontent.com/u/15371677?v=3","gravatar_id":"","url":"https://api.github.com/users/yhirano55","html_url":"https://github.com/yhirano55","followers_url":"https://api.github.com/users/yhirano55/followers","following_url":"https://api.github.com/users/yhirano55/following{/other_user}","gists_url":"https://api.github.com/users/yhirano55/gists{/gist_id}","starred_url":"https://api.github.com/users/yhirano55/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yhirano55/subscriptions","organizations_url":"https://api.github.com/users/yhirano55/orgs","repos_url":"https://api.github.com/users/yhirano55/repos","events_url":"https://api.github.com/users/yhirano55/events{/privacy}","received_events_url":"https://api.github.com/users/yhirano55/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-07-11T23:10:31Z","updated_at":"2017-07-12T10:54:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/pulls/3","html_url":"https://github.com/JunichiIto/train-ticket-rails/pull/3","diff_url":"https://github.com/JunichiIto/train-ticket-rails/pull/3.diff","patch_url":"https://github.com/JunichiIto/train-ticket-rails/pull/3.patch"},"body":"回答します。\r\n\r\n---\r\n\r\n> セットアップが難しくないか\r\n\r\n- 難しくないです。\r\n- DBのセットアップが `bin/setup` に書かれていてもよさそう。\r\n- READMEに明示されているのはありがたい。\r\n- chromedriver使ったsystemテストは着手していない人もいるのでありがたい気がしました。\r\n\r\n> 問題の難易度は適切か\r\n\r\n- 区間距離を求めるときに、ちょっと考えました。\r\n- Ex1が実装できれば、後は簡単な気がします。\r\n- GateにFARESが紐づいているのか...うーん、これは修正してもよいのかな? など、迷いました（別に変えても問題ないと思うのですが、こういう問題に不慣れなのもあって、パッと対応できなかったところはあります）\r\n- 適切な命名をするのが難しかったです ><\r\n\r\n> 回答までの手順が難しくないか\r\n\r\n- テストにskipが書かれていて、それをpassさせる、というTDDの手法が素敵です。\r\n- 逆にEx1をもっと分割してもよいかもしれません。\r\n- Ex1で実装したメソッドを以降で使う予定だったと思うのですが、使わずにEx2以降が回答できたので、Ex1の実装が後の工程に影響した方が流れとしては良い気がしました。\r\n\r\n> そもそもこのRailsアプリの設計が適切か？\r\n\r\n- 問題としては十分だと思います。\r\n- アプローチが回答者によってバラつきそうではあります。"},"comment":{"url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/comments/314731541","html_url":"https://github.com/JunichiIto/train-ticket-rails/pull/3#issuecomment-314731541","issue_url":"https://api.github.com/repos/JunichiIto/train-ticket-rails/issues/3","id":314731541,"user":{"login":"JunichiIto","id":1148320,"avatar_url":"https://avatars1.githubusercontent.com/u/1148320?v=3","gravatar_id":"","url":"https://api.github.com/users/JunichiIto","html_url":"https://github.com/JunichiIto","followers_url":"https://api.github.com/users/JunichiIto/followers","following_url":"https://api.github.com/users/JunichiIto/following{/other_user}","gists_url":"https://api.github.com/users/JunichiIto/gists{/gist_id}","starred_url":"https://api.github.com/users/JunichiIto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JunichiIto/subscriptions","organizations_url":"https://api.github.com/users/JunichiIto/orgs","repos_url":"https://api.github.com/users/JunichiIto/repos","events_url":"https://api.github.com/users/JunichiIto/events{/privacy}","received_events_url":"https://api.github.com/users/JunichiIto/received_events","type":"User","site_admin":false},"created_at":"2017-07-12T10:54:20Z","updated_at":"2017-07-12T10:54:20Z","body":"フィードバックもありがとうございました。\r\n\r\n> アプローチが回答者によってバラつきそうではあります。\r\n\r\nここがちょっとネックですね。\r\n実は社内のメンバーにも解いてもらったんですが、僕の解答例を見て「えー、そんなふうに実装するの－！？」みたいな（不満に近い）声が上がりました。\r\nなので、もうちょっとバラ付かないように明文化すべきところはしておいた方がいい気がします。\r\n\r\nもちろん、「俺だったらこう設計/実装する！」という意見はウェルカムなのですが、それはこの例題とはまた別の「Ex 5」として参加者のみなさんで議論してもらうのがいいかもです。"}}
{"actor":{"display_login":"JunichiIto"}}