我正在重构一些代码。我试图在我的视图中使用数组作为一个for循环的一部分,该循环在表中生成列。
我在控制器中定义了数组:
subjects_controller.rb
def index
...
@CRFS_TO_VIEW = [Baseline, TreatmentCompletion]
@CRF_PATH = {Baseline => 'baseline_path', TreatmentCompletion => tc_path}
end因此,我的目标是:当函数在@CRFS_TO_VIEW上迭代时,从@CRF_PATH中选择正确的路径并附加到link_to函数中。
indext.html.erb
<% @CRFS_TO_VIEW.each do |crf| %>
<% path = @CRF_PATH[crf] %>
<%= link_to "edit", path(crf.where(subject_id: sub.subject_id).first %>
<% end %>我也试过:
<%= link_to "edit", @CRF_PATH[crf](crf.where(subject_id: sub.subject_id).first %>但没起作用。我觉得我一定是接近了,任何帮助或洞察力都将不胜感激。
谢谢。
发布于 2016-09-08 18:55:19
有几件事:
您应该节省一些时间,循环使用字典而不是数组:
<% @CRF_PATH.each do |crf, path| %>
...
<% end %>从循环中获得一个字符串??您可以使用send调用等效方法。
<%= send(path, ...) %>c.您可以使用以下方法简化对象的检索:
crf.find_by(subject_id: sub.subject_id)也就是说,这似乎是一种非常糟糕的做事方式。我建议添加一个视图助手:
def crf_path(crf)
case crf
when Baseline then baseline_path(crf)
...
end在这种情况下,您可以使用(为了安全起见,请注意将find_by改为find_by! ):
<% @CRFS_TO_VIEW.each do |crf| %>
<%= link_to "edit", crf_path(crf.find_by!(subject_id: sub.subject_id) %>
<% end %>最后,不应将实例变量命名为大写。如果要使用常量,请将其定义为常量(否则使用小写名称)。
https://stackoverflow.com/questions/39398062
复制相似问题