Rails开发：如何仅展示Shopify中未导入本地库的商品

绝对推荐在控制器层处理数据过滤，而不是在视图里加判断——这不仅符合Rails MVC的设计原则（视图只负责展示，逻辑交给控制器/模型），还能提升性能、让代码更易维护。

具体实现步骤 #

1. 先获取本地数据库中已存在的商品ID集合：

   existing_product_ids = Product.pluck(:id)
   

   pluck(:id)会直接从数据库查询所有商品ID，返回一个数组，效率比加载整个对象集合高得多。

2. 过滤Shopify商品，只保留本地没有的：

   # 拉取所有Shopify商品后，排除已存在于本地的ID
   @products = ShopifyAPI::Product.find(:all).reject { |shopify_product| existing_product_ids.include?(shopify_product.id) }
   

   如果你确认Shopify API支持远程过滤（比如通过id_not_in参数），可以进一步优化，减少数据传输量：

   # 直接让Shopify返回不在本地ID列表中的商品（需确认Shopify API是否支持该参数）
   @products = ShopifyAPI::Product.find(:all, params: { id_not_in: existing_product_ids.join(',') })
   

3. 本地商品查询可以简化：

   # 原来的where.not(id: nil)等价于Product.all，因为id是主键不可能为nil
   @myproducts = Product.all
   

为什么不推荐在视图里加判断？ #

如果硬要在视图里处理，代码会变成这样：

<% @products.each do |product| %>
  <% unless @myproducts.pluck(:id).include?(product.id) %>
    <tr>
      <td><%= product.id %></td>
      <td><%= product.title %></td>
    </tr>
  <% end %>
<% end %>

但这种方式有几个明显的问题：

• 视图中混入业务逻辑，违反MVC职责分离，后续维护难度大；
• 每次循环都调用pluck(:id)会重复查询数据库，性能低下；
• 会把所有Shopify商品拉到服务器内存中再过滤，数据量大时浪费资源。

总结 #

把数据过滤逻辑放在控制器（甚至封装到模型层），让视图只专注于渲染内容，是更优雅、高效的解决方案。

内容的提问来源于stack exchange，提问作者rlsaj