Rubyにおいてある配列に繰り返しの処理をした時に1つ飛ばしになってしまう

Question

前提・実現したいこと

Rubyである配列から一文字の文字列を削除したく、for文、each文で反復的に削除する処理をしたところ一文字の文字列が１つ飛ばしに削除されてしまいます。下にソースコードを乗せますのでよろしければご回答お願いします。

each文による処理

array = ["初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]

array.each{|var|
  if var.length == 1
    array.delete(var)
  end
}
puts array
=>
初期登録
漢字氏名
姓
1
3

each文による処理

array = ["初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]
array.each{|var|
  if var.length == 1
    array.delete(var)
  end
}
puts array
=>
初期登録
漢字氏名
姓
1
3

実現したいこと

配列から一文字のものを全て削除し、以下のように結果を出したいです。

〜〜〜
なんらかの処理
〜〜〜〜
puts array
=>
初期登録
漢字氏名

Answer 1

x.rb

array = ["0", "初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]

p array.select{|x| x.length > 1}
p array.map{|x| x if x.length > 1}.compact

p array.delete_if{|x| x.length == 1}
p array

array = ["初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]
(0...array.size).reverse_each do |idx|
  array.delete_at(idx) if array[idx].length == 1
end
p array

実行例

4 つの方法を書いてみました。
後ろの２つの方法は array 自身を変化させます。

Comments

[shuyaaa]

同じ結果でもいくつものアプローチがあるのですね！
また、ピリオド３つで(array.length-1)と記述しなくて良いことも初めて知りました。ありがとうございます^^

Answer 2

以下でどうでしょう？

array = ["初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]


p array.reject { |s| s.length == 1 }  # => ["初期登録", "漢字氏名"]

Comments

[shuyaaa]

ご回答ありがとうございます！
rejectによる削除方法を知りませんでしたので勉強になりました！

[jun68ykt]

どういたしまして。
なお上記の回答は、「長さが 1 の文字列を除外する」という考え方ですが、「長さが 1 より長い文字列だけを集める」という考え方ならば以下のように select を使えば出来ます。

array.select { |s| s.length > 1 }

[shuyaaa]

これから「集める」という処理も必要になりますので感謝です^^

Answer 3

削除することで、それ以降は前に詰められますので、その次の場所を見ると1つ飛んでしまうのは当然です。
eachしながら要素を追加・削除することは普通はしません。

条件によって要素を削除するのは専用のメソッドがあります。

array = ["初期登録", "漢字氏名", "※", "姓", "名", "1", "2", "3"]

array.reject!{|var| var.length == 1 }
puts array

ほぼ同機能のdelete_ifもあり。

Comments

[shuyaaa]

削除したために前に詰められる、単純なことでした。。。
そのようなメソッドを知りませんでしたので勉強になりました。
ありがとうございます！