Skip to content

Answer - #16

Open
ota42y wants to merge 5 commits into
JunichiIto:mainfrom
ota42y:answer
Open

Answer#16
ota42y wants to merge 5 commits into
JunichiIto:mainfrom
ota42y:answer

Conversation

@ota42y

@ota42y ota42y commented Aug 1, 2017

Copy link
Copy Markdown

READMEの問題に従って回答を作りました。
レビューをよろしくお願いいたします。

  • Railsの機能をできる限り利用しました
  • 区間表を作ったのはオーバースペックだったかもしれません
  • 運賃が足りない場合と同じ駅で降りる場合はエラーメッセージを変えたい気持ちがありました

ota42y added 5 commits August 1, 2017 19:09
Ex1で運賃表を作ったので実装は不要
エラーにしていたものを実装
エラーメッセージ分けたい
@JunichiIto

Copy link
Copy Markdown
Owner

すでにチェック済みかもしれませんが、こちらの動画でコメントさせてもらいました!
よろしくお願いします。

【前編】全部見ます!プロを目指すRailsエンジニアのための公開コードレビュー #railsdm - YouTube
https://www.youtube.com/watch?v=xZk4f4PPUhU&t=1s

【後編】全部見ます!プロを目指すRailsエンジニアのための公開コードレビュー #railsdm - YouTube
https://www.youtube.com/watch?v=yDxg0sXHXOo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants