ruby delete_at tune up

Question

У меня есть рабочий код, но он мне не нравится, он мне очень не нравится.У меня есть две разные коллекции, и я хочу удалить элементы с соответствующими ключами из одной коллекции.Я «выбрал», что пользователь выбрал из «выбора».Поэтому я хочу отображать «варианты», которые еще не выбраны.Вот код:

@picked.each do |p|
  i = 0
  @choices.each do |c|
    if p.choices_id == c.id 
      @choices.delete_at(i)
      break
    end
    i += 1
  end
end

end

Мне не нравится переменная "i", в частности.Просто подумал, что увижу, что другие могут сделать с этим.

Answer 1

@choices.reject! { |c| @picked.index { |p| p.choices_id == c.id } }

Answer 2

each_with_index и with_index для Ruby 1.9 +

И улучшения для вашего кода

choices_ids = @picked.map(&:choices_id)
@choices.delete_if{ |c| choices_ids.include? c.id }

api: delete_if

UPD

Также вы всегда можете просто позвонить по этому номеру:

@choices - @picked.map(&:choices).flatten

или

@choices -= @picked.map(&:choices).flatten

Answer 3

Ruby поддерживает пересечение и разность для множеств:

picked  = %w[1 2 3]
choices = %w[1 2 3 4 5]

choices - picked # => ["4", "5"]

---

EDIT:

class Pick
  attr_reader :choices_id
  def initialize(id)
    @choices_id = id
  end
end

class Choice
  attr_reader :id
  def initialize(id)
    @id = id
  end
end

# cobble up some collections of objects 
picked = %w[1 2 3].map{ |p| Pick.new(p) }
choices = %w[ 1 2 3 4 5].map{ |c| Choice.new(c) }

choices.map{ |c| c.id } - picked.map{ |p| p.choices_id } # => ["4", "5"]