Answer by DaiAoki

[DaiAoki]

こだわった点/アピールポイントなど

• メソッド名が汎用的なものとなるようにこころがけた。

# exitedではなく、返金済などの可能性を考慮しusedとする
def redirect_if_used_ticket
    redirect_to root_path, alert: '降車済みの切符です。' if @ticket.used?
end

• 再利用できそうな処理(ex. calculate_distanceなど)は、なるべく切り出すようにした。

[JunichiIto]

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

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

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