Ruby动态生成变量名及Rails视图`lan`未定义报错咨询

报错原因 #

你遇到的undefined local variable or method 'lan'错误，是因为视图代码里直接使用了lan.id和lan.name，但根本没定义过lan变量——循环里只处理了lan_role，没有把对应的Lan对象赋值给lan。

修复步骤+优化方案 #

1. 先解决变量未定义的问题 #

在循环内部先获取Lan对象，就能直接使用lan的属性：

<% LanRole.where("lan_id = ?", requestable.id).each do |lan_role| %>
  <% # 先获取对应的Lan对象 %>
  <% lan = Lan.find(lan_role.lan_id) %>
  <li><a data-toggle="tab" href="#lan_info_<%= lan.id %>"><%= lan.name %> INFO</a></li>
<% end %>

2. 替代动态实例变量的合理方式 #

你想通过lan_role.id动态命名实例变量（比如@lan_123）的思路不推荐，这属于Rails里的反模式，后续维护和调试都会很麻烦。更合适的是用哈希来存储Lan对象：

第一步：初始化哈希存储Lan对象

在视图开头添加：

<% @lan_map = {} %>

第二步：循环时把Lan对象存入哈希

<% LanRole.where("lan_id = ?", requestable.id).each do |lan_role| %>
  <% lan = Lan.find(lan_role.lan_id) %>
  <% @lan_map[lan.id] = lan %>
  <li><a data-toggle="tab" href="#lan_info_<%= lan.id %>"><%= lan.name %> INFO</a></li>
<% end %>

第三步：标签页部分调用哈希中的Lan对象

如果是批量渲染标签页，直接循环哈希即可：

<div class="tab-content">
  <% @lan_map.each do |lan_id, lan| %>
    <div id="lan_info_<%= lan_id %>" class="tab-pane fade">
      <h4><%= lan.name %> INFO</h4>
      <!-- 其他业务代码 -->
    </div>
  <% end %>
</div>

如果是单独渲染某个标签页，也可以通过@lan_map[目标lan_id]获取对应的Lan对象。

3. 进阶优化：避免N+1查询 #

如果你的LanRole模型里定义了belongs_to :lan关联，那可以直接用lan_role.lan获取关联对象，还能在控制器里预加载Lan数据，避免循环里多次查询数据库：

控制器代码：

# 预加载关联的Lan对象，避免N+1查询
@lan_roles = LanRole.where(lan_id: requestable.id).includes(:lan)

视图代码：

<% @lan_map = {} %>
<% @lan_roles.each do |lan_role| %>
  <% lan = lan_role.lan %>
  <% @lan_map[lan.id] = lan %>
  <li><a data-toggle="tab" href="#lan_info_<%= lan.id %>"><%= lan.name %> INFO</a></li>
<% end %>

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