如何将循环中的循环变量传递至Controller？

看起来你是在Rails视图里把业务逻辑和展示代码混在一起了，还卡在不知道怎么把循环里的group变量传递到复用方法里对吧？别慌，咱们一步步把逻辑抽离出来，让视图变得干净清爽。

第一步：把查询逻辑封装到模型里 #

视图里不该直接写UserGroup.where(...)这种底层查询，咱们把这些业务逻辑放到对应的模型里，既让视图更简洁，还能在其他地方复用代码。

在User模型里添加「判断用户是否属于某组」的方法 #

打开app/models/user.rb，添加这个方法：

def member_of?(group)
  UserGroup.exists?(group_id: group.id, user_id: self.id)
end

现在你在视图里就可以直接用current_user.member_of?(group)替代原来的查询，循环变量group直接作为参数传进去就行，完全不用纠结传参问题。

在Group模型里添加「判断是否为创建者」的方法 #

假设你的groups表用created_by_id字段存储创建者的用户ID，打开app/models/group.rb加：

def created_by?(user)
  created_by_id == user.id
end

如果你的字段名直接是created_by（存用户ID），把方法里的created_by_id改成created_by就行。

第二步：重构你的视图代码 #

现在把视图里的硬编码逻辑替换成咱们刚才写的模型方法，代码会清爽很多：

- @groups.in_groups_of(2, false).each do |groups|
  .row
    - groups.each do |group|
      .box.col-md-5{id: "group_#{group.id}"}
        %h3= group.group_name
        - if current_user.member_of?(group)
          %p= link_to 'Show', group
        - unless group.created_by?(current_user)
          # 这里放你原来要写的非创建者专属逻辑

你看，现在视图里只剩下和展示相关的代码，所有业务逻辑都封装到模型里了，循环变量group也顺理成章地传给了模型方法，完美解决你的问题。

进阶：用Partial拆分重复代码 #

如果每个.box里的内容还比较多，你可以把这部分抽成独立的partial，让视图结构更清晰：

创建app/views/groups/_group_box.html.erb：

.box.col-md-5{id: "group_#{group.id}"}
  %h3= group.group_name
  - if current_user.member_of?(group)
    %p= link_to 'Show', group
  - unless group.created_by?(current_user)
    # 非创建者逻辑

然后原来的视图就可以简化成：

- @groups.in_groups_of(2, false).each do |groups|
  .row
    - groups.each do |group|
      = render 'group_box', group: group

这样后续维护起来会更方便，代码可读性也更高。

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