2
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 3 years have passed since last update.

Rubyでチンチロゲームを作る  最終回 レビューを受けての改善

Posted at

レビューをもらおう

 前回の記事で一応は全体のコードは仕上がった。しかし、まだまだアラが多い。そういうわけで、自分が所属しているオンラインサロンでコードレビューをしてもらった。けっこうたくさんの指摘があったので、ひとつずつ対応していった。

指摘事項一覧

文法関連

  • Rubyのインデントは半角スペース2つ

  • 単純な比較は==を使用する/thenも必要なし

  • 変数展開を使ったほうがいい

  • ファイル名がわかりにくい(transfer_money -> player.rbへ)

メソッドの移動

  • サイコロを振る/お金をかけるのはplayerの中にいれたほうがいい
     (playerの行動であるため)

  • check_win_losetransfer_moneyメソッドはPlayerのメソッドではないかも

読みやすくする

  • main.rbはゲームの流れの単位でメソッド化すると全体がみやすくなる
  • 同じ内容が複数ある箇所はまとめる(strength_relationshipなど)
  • roll_diceの引数をなくす
  • メッセージを出す場所と処理する場所を別にする

その他改善

  • メソッドの戻り値が文字列なのはよくない
     定数または列挙子を使用

  • 班長の賭け金の設定がもったいない(所持金に応じて賭け金を変える)

  • 演算子で勝敗やお金の移動ができるといい。

多すぎ!!!

 

指摘事項の対応1:文法関連

 やっていこう。まずは文法関連だが、これはわりとすぐに改善できた。インデントの修正はVSCodeの場合は楽にできた。以下の記事を参考にした。

 VSCodeでインデントのスペース数を変更する

 変数展開はこんなこと。こんなふうにするとすっきり書ける。

dice_hand = '通常の目(' + uniq_value.to_s + ')'

dice_hand = "通常の目(#{uniq_value.to_s})"

対応したissueのURL
https://github.com/kyokucho1989/ruby-game/issues/13

指摘事項の対応2: メソッドの移動とか

 これはもろもろがんばった。まずは勝敗判定や、賭け金の移動を行うのがPlayerではおかしいとのことだったので、あらたにGameクラスを作成した。そのGameクラスが勝敗判定や賭け金の移動を行う。

 また、それを行うのはクラスメソッドでやることにした。gameクラスは状態を持っていないためだ。(これも指摘された)

指摘事項の対応3: 読みやすくする

 文章を出力させるのもひとつにまとめた。message.rbをあらたに作成し、そこから全てのメッセージが出るようにした。こうすることで、文章を修正する時に楽になる。いいね!

指摘事項の対応4: その他

  • メソッドの戻り値が文字列なのはよくない(定数または列挙子を使用)
     
    これが難しかった。たしかに文字列で返すと、ちょっとしたタイプミスでバグが生じる。ちょっとがんばることにした。

 あらたに module-hand-game.rb を作成し、役と勝敗結果を定数にした。

module Match
  WIN  = '勝ち'
  LOSE = '負け'
  DRAW  = '引き分け'
end

参照するときはこんなかんじ。いいね!

if(my_hand_rank > opponent_hand_rank)
      Match::WIN 
     elsif(my_hand_rank == opponent_hand_rank)
      Match::DRAW
     else
      Match::LOSE
     end

「演算子で勝敗やお金の移動ができるといい。」は対応できなかった。むずい。

おわりに

 Rubyだけでこんなにプログラミングができるのが楽しかった。Railsを含めていろいろやると、環境構築のエラーなどでやる気がそがれがち。はじめはRubyだけでなんか作った方が単純に力がつく気がする。

 いつかこれで教材を作りたいなぁ。おわり。
 完成したコードはこちらにあります↓
https://github.com/kyokucho1989/ruby-game

2
0
2

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
2
0

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?