Rubyの引数（配列）の受け渡しについて

Question

現在CODEPREPというサイトでRubyの学習を行なっています。
（現在のブック：Ruby演習　配列・Hash・Class編）
現在の課題で書いたプログラムが以下のようになっていますがどこを直せば良いのかわかりません。
アドバイスをください。

演習内容
クラスを継承する
Arrayクラスを継承したMyArrayクラスを実装しましょう。
MyArrayクラスにはインスタンスメソッドmultiplyを実装しましょう。
-----実装内容-----
Arrayを継承したMyArrayを実装する
MyArrayにインスタンスメソッドmultiplyを実装する。
multiplyメソッドは引数に整数nをとり、MyArrayの全ての要素をn倍して返すメソッドです。

class Array
end
class MyArray < Array
  def initialize(array)
    @ary=array
  end
  def multiply(n)
    s=@ary.map{|i|i*n}
    return s
  end
  attr_accessor :ary
end

実行例は以下のようになるそうです。

my_array = MyArray[1, 2, 3, 4]
p my_array.class.superclass
# => "Array"

p my_array.multiply(5)
# => [5, 10, 15, 20]

おそらく最初の@aryに配列を入れられない状態だと認識している？のですが
修正方法がわかりません。どなたかアドバイスをください、よろしくお願いします。

Comments

[maisumakun]

現在のコードでは、どのようになるのですか？

[soheil]

実行例のコードを入力すると以下のようになります。
Array
/root/codecheck/main.rb:8:in `multiply': undefined method `map' for nil:NilClass (NoMethodError)
from /root/codecheck/main.rb:16:in `<top (required)>'
from /usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
from /usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
from test.rb:1:in `<main>'

Answer 1

おそらく最初の@aryに配列を入れられない状態だと認識している？のですが

別に@aryを作るのは適当ではありません。自分自身の持っているmapそのまま呼べばいいだけです。

class MyArray < Array
  def multiply(n)
    map{|i|i*n}
  end
end

Comments

[soheil]

ありがとうございます！！上記の記述だけで無事動作しました！
しかし、このプログラムの場合、インスタンス変数は必要ない（むしろ書かないほうが良い）のでしょうか？
書いた場合でも動作する方法はあるのでしょうか。（学習中にインスタンス変数を作成するところがあり、その流れから描くべきものなのかと思っていました）

[maisumakun]

> インスタンス変数は必要ない（むしろ書かないほうが良い）のでしょうか？

書いてはいけません。本来の配列と別にデータを持つことになってしまい、Arrayを継承した意味の大半が失われます。

[soheil]

なるほど、ありがとうございます！ちなみに書いたほうが良いケースはあるのですか？今の所どのような場合に書くべきなのかいまいち理解できていません。

[maisumakun]

そもそも論として、「標準で存在するクラス」を継承すること自体が実用上はレアケース（多くの場合は、別な手段を取るのが適切）なので、あまり気にしなくて構いません。

[soheil]

そうなんですね、わかりました。ご回答ありがとうございます！！