петля внутри петли activerecord

Question

У меня есть таблица category_aliases, в которой есть столбец aliase (тип массива) и столбец category_id, что указывает на таблицу categories.

У меня есть products стол, который имеет category column.

есть categories стол.

Я хочу просмотреть все продукты (много), которые имеют старые категории и сопоставить их с новыми категориями.

Поэтому мне нужно проверить, если какой-либо из category_aliases.aliase (aliase массив старых категорий) включает в себя это product «s category, и если да, то я хочу, чтобы отобразить его в новую категорию из categories таблицы.

То, что я попытался это:

Product.all.each do |p| 
    CategoryAlias.all.each do |ca| 
    if ca.aliase.include? p.category 
     p.update_column(:category, Category.find(ca.category_id).name) 
    else 
     p.update_column(:category, 'undefined') 
    end 
    end 
end 

Но я, конечно, что-то отсутствует, потому что даже если я знаю, что там почти всегда должно быть совпадение, он обновляет p.category на 'undefined' все время.

Answer 1

Вам не нужно перебирать все категории («каждый») в каждом контуре продукта, вам просто нужно найти подходящую категорию detect. Кроме того: если вам нужно перебрать много продуктов, это может быть лучше использовать find_each:

category_aliases = CategoryAlias.pluck(:name, :aliase) 

Product.find_each do |product| 
    category = category_aliases.detect { |ca| ca.aliase.include? product.category } 

    if category 
    p.update_column(:category, cateory.name) 
    else 
    p.update_column(:category, 'undefined') 
    end 
end 

Answer 2

я вижу ряд проблем в коде:

1. Вы изменили category поле на каждой итерации. Это приводит к обновлению до 'undefined', если запись CategoryAlias не содержит категории. Тогда все остальные итерации не имеют смысла.
2. Вам необходимо разбить внутренний цикл сразу после того, как обновлен category.
3. Вам необходимо обновить category до 'undefined' во внешнем цикле, если в списке нет категории псевдонимов category_aliases.