29

我正在尽力构建一个帮助器，它输出一个由集合的所有成员组成的 <'ul> 。对于集合的每个成员，我想打印出一个 <'li> ，它有一个标题，以及一个指向该成员的 CRUD 链接的 div。这与 Rails 为索引视图搭建脚手架的输出非常相似。

这是我的助手：

def display_all(collection_sym)
  collection = collection_sym.to_s.capitalize.singularize.constantize.all

  name = collection_sym.to_s.downcase

  html = '' 

  html << "<ul class=\"#{name}-list\">"

  for member in collection do
    html << content_tag(:li, :id => member.title.gsub(' ', '-').downcase.strip) do
     concat content_tag(:h1, member.title, :class => "#{name}-title")
     concat link_to 'Edit', "/#{name}/#{member.id}/edit"
     concat "\|"
     concat link_to 'View', "/#{name}/#{member.id}"
     concat "\|"
     concat button_to 'Delete', "/#{name}/#{member.id}", :confirm => 'Are you sure?  This cannot be undone.', :method => :delete
    end
   end

   html << '</ul>'

 return html
end

而那个输出正是我想要的。首先，如果有人认为有更好的方法可以做到这一点，请随时纠正我，我怀疑我是在用低音做这件事，但目前这是我知道的唯一方法。

然后我尝试将链接包装在 div 中，如下所示：

def display_all(collection_sym)
  collection = collection_sym.to_s.capitalize.singularize.constantize.all

  name = collection_sym.to_s.downcase

  html = '' 

  html << "<ul class=\"#{name}-list\">"

  for member in collection do
     html << content_tag(:li, :id => member.title.gsub(' ', '-').downcase.strip) do
     concat content_tag(:h1, member.title, :class => "#{name}-title")
     concat content_tag(:div, :class => "links-bar") do
       concat link_to 'Edit', "/#{name}/#{member.id}/edit"
       concat "\|"
       concat link_to 'View', "/#{name}/#{member.id}"
       concat "\|"
       concat button_to 'Delete', "/#{name}/#{member.id}", :confirm => 'Are you sure?  This cannot be undone.', :method => :delete
     end
   end
 end

 html << '</ul>'

 return html
end

但是，我现在不再将 div.links-bar 输出中的任何标记发送到视图。我确信这一定与块和绑定有关，但我可以在我的一生中弄清楚要修复它的内容或方法。任何人都可以提供任何帮助吗？

4

3 回答 3

46

我同意上面建议使用部分的评论...但是如果您确实需要在助手中执行此操作，这是一种更清洁的实现方式：

def display_all(collection)
  content_tag(:ul, class: "list") do
    collection.collect do |member|
      concat(content_tag(:li, id: member.name.gsub(' ', '-').downcase.strip) do
        member.name
      end)
    end
  end
end

我会明确地传入一个集合，而不是传入一个符号来创建一个集合，因此您并不总是需要一次显示特定表中的所有记录。您可以添加分页等。

于 2010-09-04T19:50:51.613 回答
25

@Joe，您仍然可以使用您的方法display_all(collection_sym) 只需使用： return html.html_safe 而不是： return html

我仍然发现在许多情况下，最好从帮助程序生成 HTML，而不是使用部分。因此， Rails 3html_safe中的函数将确保您生成 HTML，而不是将其转换为.String

于 2011-03-11T22:45:29.370 回答
2

正如@TheDelChop 所说，您需要 aconcat用于 inner content_tag，否则输出只是<ul></ul>

看起来是这样的：

def display_all(collection)
  content_tag(:ul, :class => "list") do
    collection.collect do |member|
      concat(
        content_tag(:li, :id => member.name.gsub(' ', '-').downcase.strip) do
          member.name
        end
      )
    end
  end
end

更多解释：在 Rails 3 中嵌套 content_tag

于 2015-12-10T11:12:54.257 回答