[Ruby]ご教示下さい: @の有無について

Question

(※当方プログラミングを勉強して約1ヶ月になる初心者です。)
質問をご覧いただきありがとうございます。
あるRuby入門書を勉強している中で不明点が生じたため、質問いたしました。
Rubyバージョンは ruby 2.6.5です。

不明点のあるコードは以下の通りです。

class Product
  attr_reader :name, :price
  
　def initialize(name, price)
    @name = name
    @price = price
  end

  def to_s
    "name: #{name}, price: #{price}"
  end
end

product = Product.new('A great movie', 1000)
product.to_s  #=> "name: A great movie, price: 1000"

以上のコードは継承が説明された章に書かれています。

私は、Productメソッドがto_sメソッドをオーバーライドしているのは理解しています。
しかし、to_sメソッドの中で行われている、
"name: #{name}, price: #{price}"
でなぜ引数がname,priceとなっているのでしょうか。@name,@priceではないのでしょうか。

色々調べてみたのですが、分かりませんでした。
よろしくお願いいたします。

Comments

[raccy]

https://ja.stackoverflow.com/questions/61877/
にも投稿されているようですが、ベストアンサーを決めたのであれば、スタック・オーバーフロー側には、解決した回答を自分で書いて承認してください。

[退会済みユーザー]

失礼致しました。
スタックオーバーフロー側でコメントを記載しました。

Answer 1

@を取っただけの名前に見えますが、そうではないです。
他の言語風に書くとこんな感じです。

class Product
  def get_name
    @name
  end
  def get_price
    @price
  end

　def initialize(name, price)
    @name = name
    @price = price
  end

  def to_s
    "name: #{self.get_name()}, price: #{self.get_price()}"
  end
end

attr_readerについては、入門書のそれまでの部分に書いてあるはずです。インスタンス変数に対して、インスタンス変数名から@を取った名前の、読み取りメソッドをパブリックに定義してくれます。

attr_readerやattr_writerやattr_accessorで不用意にアクセッサーを定義するのは、オブジェクト指向のカプセル化を壊すことになるので、利用には慎重さが必要ですが、安易にattr_accessorしているケースも見受けられます。

この例は、to_sのオーバーライドとともに、アクセッサーの使い方を教えるためのサンプルだと思うのでアクセッサーを使っているのでしょうが、to_sの中では直接インスタンス変数を参照すべきですね。

Comments

[退会済みユーザー]

ご回答ありがとうございます。

attr_readerのname,priceだったのですね。self.が省略されていたため分かりませんでした

また、アクセッサーの利用時の注意まで書いてくださってありがとうございます。
大変勉強になりました。

Answer 2

なぜ引数がname,priceとなっているのでしょうか。@name,@priceではないのでしょうか。

@name, @priceを使っても動きますが、Productクラスを定義するコードの冒頭にあるようにattr_reader :name, :price と、attr_readerを使ってインスタンス変数の読み取りメソッド(アクセサ)を定義して、これでインスタンス変数にアクセスできる為です。インスタンス変数に対して読み取りのみのアクセスを行っていると明示的にする意味もあるでしょう。

Module#attr_reader(*name) - Ruby 2.6

Comments

[退会済みユーザー]

ご回答ありがとうございます。

attr_readerについて詳しく書いていただき、ありがとうございます。

無事解決いたしました。