1

Есть массив числ, хочу четные элементы из одного массива перекинуть в другой. Есть код:

numbers=[3,543,25,6,753,96,100,53,50]
newlist=[]
newlist=numbers
newlist.each do |numb|
  ind=newlist.index(numb)
  if numb.odd?
    newlist.delete_at(ind)
  end
end
puts newlist

При выводе второго массива будут числа: 543 6 96 100 50

Вопрос: как здесь оказалось число 543?

1
  • 1
    ответ не касается задания, но алгоритмически ваш код неверен, т.к проще не удалять элемент из 2 массива если он нечетный, а наоборот добавлять во второй если он четный
    – finally
    13 окт 2019 в 13:02

2 ответа 2

3

Во-первых, зачем вы используете each и тут же получаете индекс? Вы понимаете, что вы сами себе O(n2) из ничего сделали?

Во-вторых, никогда, повторяю, никогда не удаляйте элементы из итератора, по которому проходитесь. У вас же мясо получится:

Iteration start:
  3  543  25  6  753  96  100  53  50
  ^

First element deleted:
  543  25  6  753  96  100  53  50
  ^

Second iteration:
  543  25  6  753  96  100  53  50
       ^

Понимаете, в чём дело? Элементы сдвигаются, но итератор об этом ничего не знает.

3

Зачем такие сложности? Есть же готовый метод filter

numbers = [3, 543, 25, 6, 753, 96, 100, 53, 50]
even_numbers = numbers.filter(&:even?)

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.