exercim演習(メソッドとデフォルトの引数)

[kei-600th]

分報

exercim問題演習
メソッドとデフォルトの引数
※詳細なアウトプット内容についてはプルリクページ内の"File changed"タブを参照

学習したこと

• デフォルト引数の設定

学習を通して考えたこと

• デフォルトの値を設定せず引数のみをメソッドに設定した場合、そのメソッドに対して引数が渡されないとエラーが出てしまう
• デフォルトの値を設定して引数をメソッドに設定した場合、そのメソッドに対して引数が渡されなかった場合、設定された引数に対してデフォルト値を当てはめて処理を行う
• 最初この性質に気づいてなかったのでどのように実装すればいいかわからなかったが、以下のリンクにヒントが書いてあったため、これを参考にさせてもらった
  Implement new exercise, two-fer #757

[kei-assistant]

[良い点]
[1] クラスメソッドとして定義されており、全体的にコードがシンプルで読みやすいです。
[2] メソッドの引数にデフォルト値を設定することで、引数がない場合でもメソッドが呼び出せるようになっています。
[3] 三項演算子を使って条件分岐を行うことで、コードが短くてスマートです。

[改善点]
[1] "default"という文字列を使って名前が与えられていない場合を表現していますが、ここではnilを使う方が適切かもしれません。nilは「何もない」状態を表現するのに適しており、そのためにRubyで用意されています。また、"default"という具体的な文字列だと、将来的に"default"という名前の人が現れた時に問題が起きる可能性があります。
[2] メソッド名はRubyの慣習に従ってスネークケースにするとより良いです。例えば、two_ferではなくtwo_forとするなどです。

[kei-600th]

レビューbotがいい指摘を返してくれており、文字列"default"を設定するよりはnilを設定する方が適している

class TwoFer
  def self.two_fer(name = nil)
    name ? "One for #{name}, one for me." : "One for you, one for me."
  end
end

この方が、変数が与えられていない場合の処理をわかりやすくキレイに書けていると思う