pushメソッドが動かない

Question

an = [1, 2, 3]

an.each do |a|
an.push(a)
end

p an

ターミナルで上記を動かすと、止まってしまいます。
pushメソッドはあまり使わない方がいいのでしょうか。
改善点あれば教えてください！！

Answer 1

an = [1, 2, 3]

an.each do |a|
  an.push(a)
end

の処理ですが、anにpushしてしまうと(破壊的メソッドなので)anの要素が一つ後ろに追加されます。
ですので、eachの1週目の処理は

an.each do |a|
  an.push(a) # aは１
  # => an == [1, 2, 3, 1]
end

2週目の処理は

an.each do |a|
  an.push(a) # aは２
  # => an == [1, 2, 3, 1, 2]
end

と、ループする要素自体がループを回るごとに増えるため、ループから抜けなくなります。
実際にされたい処理がどういうものかわかりませんが、以下サンプルです

an = [1, 2, 3]
an2 = an.dup

an.each do |a|
  an2.push(a)
end

p an2
#=> [1, 2, 3, 1, 2, 3]

Answer 2

an = [1,2,3]
an.each{|it|
  an.push(10) if it == 1
  p it
}
p an

結果は

1
2
3
10
[1, 2, 3, 10]

となります。

つまり、eachで回している配列を内部で拡張すると
その分までループされるので、無条件に要素が1増えるとループが終わらなくなります。

---

ちなみに、要素を2倍にしたいだけならば

an = [1,2,3]
p(an * 2)
# => [1, 2, 3, 1, 2, 3]

掛け算してやればいいです。