【Ruby】each_with_index について

Question

A = ['a', 'b', 'c', 'd', 'e']
B = [[], [], [], [], []]

という二つの配列において、Aのそれぞれの要素をBに代入し

B = [['a'], ['b'], ['c'], ['d'], ['e']] としたかったので

A.each_with_index do |n, i|
B[i] << n
end
としたところ、

irb(main)> B
=> [["a", "b", "c", "d", "e"], ["a", "b", "c", "d", "e"], ["a", "b", "c", "d", "e"], ["a", "b", "c", "d", "e"], ["a", "b", "c", "d", "e"]]

になってしまいました。

何故だか理解ができません。
原因がおわかりになる方いらっしゃいましたら、教えていただけますでしょうか。
よろしくお願いします。

Answer 1

B配列をB = Array.new(5, [])というコードで作っていませんか？
これをB = [[], [], [], [], []]もしくはB = Array.new(5){[]}に書き換えるとうまくいくと思います。
指摘がずれていたらすみません……。

Comments

[nimbus1978]

まさに、ご指摘のとおりに Bを作っていました。
書き換えたらうまくいきました。　　しかし、何故でしょうか？　: D
不思議です

[jun68ykt]

なるほど！勉強になりました。

[nimbus1978]

A, Bの生成からコードを書いておくべきでした。
お手数をおかけしました。

[maisumakun]

配列、ハッシュ、文字列など、書き換え可能なオブジェクトをArrayの初期値にセットすると、コピーではなくて、同じオブジェクトに対する参照が入ってしまいます。

上の回答にあるようにブロックで指定すると、要素1つごとに新たな配列を生成するようになります。

[penguinshunya]

うーん、この辺は説明が難しいんですが、
B = Array.new(5, [])
というコードだと、要素としての配列は1個しか作られません。
たとえ 5 を 100 や 200 に書き換えたとしても、作られる配列は1個だけです。
しかし、
B = Array.new(5){[]}
というコードでは、要素としての配列が5個作られます。
そして 5 を 100 に書き換えると、配列も100個作られます。

要素として作られる配列が1つだけなので B[0] も B[1] も同じ配列を指します。
つまりnimbus1978さんの書いたコードと次のコードは全く同じ意味になります。

A.each_with_index do |n, i|
B[0] << n
end

[nimbus1978]

なるほど、そういうことだったんですね。
参照する要素がひとつだけだったとは。
とてもわかりやすかったです。　
ありがとうございました。

[jun68ykt]

マニュアル
http://docs.ruby-lang.org/ja/2.0.0/method/Array/s/new.html
に
「要素毎に val が複製されるわけではないことに注意してください。」
と書いてあることが、まさにこの質問で確認できたわけですよね。

[nimbus1978]

改めてマニュアルを見ると、同じnewでも、各々の記法で、それぞれ仕様が違うんですね。
勉強になりました。

Answer 2

こんにちは。

私のほうの手元にある環境で、当該のソース、question9676.rb
を作成して実行してみたログが以下です。

[ykt68@sakura-vps] cat /etc/redhat-release
CentOS release 6.6 (Final)
[ykt68@sakura-vps] date
2015年 5月 11日 月曜日 14:23:10 JST
[ykt68@sakura-vps] cat question9676.rb

a = ['a', 'b', 'c', 'd', 'e'] 
b = [[], [], [], [], []] 

p b

a.each_with_index do |n, i| 
  b[i] << n 
end 

p b

[ykt68@sakura-vps] ruby -v
ruby 2.1.2p95 (2014-05-08 revision 45877) [x86_64-linux]
[ykt68@sakura-vps] ruby question9676.rb
[[], [], [], [], []]
[["a"], ["b"], ["c"], ["d"], ["e"]]
[ykt68@sakura-vps]

恐縮ながら念のため、ひとこと申し添えますと、上記のコードの

  b[i] << n 

の n をうっかり a にしてしまっていて、

  b[i] << a

となっていると、結果は

[ykt68@sakura-vps] ruby question9676.rb

[[], [], [], [], []]
[[["a", "b", "c", "d", "e"]], [["a", "b", "c", "d", "e"]], [["a", "b", "c", "d", "e"]], [["a", "b", "c", "d", "e"]], [["a", "b", "c", "d", "e"]]]

となります。
ご質問の状況とは少し違いますが、
参考になれば幸いです。

Comments

[nimbus1978]

ご回答ありがとうございます。　参考にさせていただき、再度たしかめたのですが、
やはり同じ結果になりました。　

Answer 3

私の環境では以下のように意図された通りの動作になります。

$ pry
[1] pry(main)> A = ['a', 'b', 'c', 'd', 'e']
=> ["a", "b", "c", "d", "e"]
[2] pry(main)> B = [[], [], [], [], []]
=> [[], [], [], [], []]
[3] pry(main)> A.each_with_index do |n, i|
[3] pry(main)*   B[i] << n
[3] pry(main)* end
=> ["a", "b", "c", "d", "e"]
[4] pry(main)> B
=> [["a"], ["b"], ["c"], ["d"], ["e"]]

A,Bのように大文字で記述されたものは変数ではなく定数として扱われるため、恐らくこの操作の前に既にBに値が設定されていたのではないでしょうか？

プログラム内で値を書き換える変数にはa,bのように小文字から始まるものを使用しましょう。

Comments

[nimbus1978]

アドバイスありがとうございます。　変数は小文字でしたね。
しかしA, Bを小文字に置き換えてみても、同じ結果になります。

[nimbus1978]

私のrubyのバージョンは　2.2.0p0 です