回答編集履歴

質問に対する追記を行いました

2017/10/04 05:26

投稿

mingos

スコア4025

test CHANGED Viewed

@@ -397,3 +397,209 @@
 end
 ```
+---
+## 追記
+> 追記：mingosさんのおすすめコードについて
+attr_reader :scoreのscoreとdef initialize(customer:)の（customer:)についてです。
+attr_reader :scoreのscoreですが、mingosさんの例でいくとcommandテーブルのscoreカラムにアクセスするという意味になるのでしょうか？
+attr_reader :scoreは、以下のコードと同じ意味です。
+```ruby
+class UpdateScoreCommand
+  def score
+    @score
+  end
+end
+```
+コマンドクラスはrailsのモデルではありませんので、データベースのカラムとは関係がありません。
+ここでは、UpdateScoreCommandという普通のクラスのインスタンス変数として@scoreを使っているだけです。
+データベースと関係するのは、ActiveRecordあるいはApplicationRecordを継承したクラスのみです。
+railsでは、app/models/以下にあるクラスになります。
+> def initialize(customer:)の（customer:)ですが（customer)と「：」の有無によって何が違ってくるのでしょうか？
+もしよかったら教えていただけないでしょうか？宜しくお願いします。
+rubyの文法で、名前付き引数とかキーワード引数などと呼ばれるものです。
+このようにメソッドを定義した場合、必ず、メソッド名(customer: 値) という風に使わないとエラーとなります。
+引数が２つ、３つとかになって来るとありがたみが出てきます。
+普通のメソッドは引数の順番を覚えておかなければいけませんが、キーワード引数であれば、引数に名前がついているので順序は適当で良いです。
+あまりいい例が思いつきませんが、
+```ruby
+def sum(a:, b:)
+  a + b
+end
+# キーワード引数なら、引数の順序はばらばらでもいい。
+puts sum(b: 10, a: 5)
+puts sum(a: 30, b: 30)
+```
+これを使う理由は、コード自体が説明書になるからですね。
+今回の例では引数が１つなので、メリットはありませんが、常にこう書いたほうが楽なので。
+実際いろいろやってると、メソッドの引数が３つ、４つ、５つって事が出て来ると思います。
+そうなると、順番を確認するために毎回メソッド定義を見なくてはいけないくてしんどいです。
+例えば、コマンド引数を増やしてみます。
+基本的には＋100ですが、任意に追加するポイントを変更できるようにしたい・・と無理やり条件を考えてみます。
+```ruby
+class UpdateScoreCommand
+  def initialize(customer:, add: 100)
+    @customer = customer
+    @score = customer.score
+    @add = add.to_i # 文字列とか来るかもしれないので、数値にする
+  end
+  def run!
+    @score.sc1 = @customer.point + @add
+    @score.save!
+  end
+end
+customer = Customer.find(1)
+c = UpdateScoreCommand.new(customer: customer, add: 500)
+c.run!
+```
+もし、キーワード引数を使わないとすると、こうですね。
+```ruby
+class UpdateScoreCommand
+  def initialize(customer, add)
+    @customer = customer
+    @score = customer.score
+    @add = add.to_i
+  end
+end
+c = UpdateScoreCommand.new(customer, 500)
+```
+仕事でプログラミングをやっていると数年前に書いたものを修正したりすることがよくあるのですが、
+その頃にはもう自分で書いたものでさえ覚えていません。
+まずは、コードの全体を把握しなければいけませんが、各メソッドの詳細を見なくても、
+たぶんこういう事をしているんだろうな、という想像が出来るコードであって欲しいのです。
+```ruby
+c = UpdateScoreCommand.new(customer: customer, add: 500)
+```
+と
+```
+c = UpdateScoreCommand.new(customer, 500)
+```
+では、どっちが良いかという話なんですね。
+時が経っていると、自分で書いたコードでありながら、500ってなんだよ？みたいな気持ちになります。
+前者もベストとは思いませんが、addって書いてあるので、まぁ500を足すんだろうくらいの予想ができます。
+クラス名もUpdateScore。。。って書いてあるし。
+この単純な例では、どっちでも良いという思うかもしれませんが、実際のコードはもっと複雑です。
+うーん、うまい説明が出来なくて申し訳ないんですが、将来自分が困らないために面倒でもキーワード引数を使っているわけです。