Ruby on Rails报错：MainController#home中NilClass调用id方法无定义

问题根源 #

wp = WorkflowProcess.find_by(workflow: 'Timesheet')或wpr= WorkflowProcess.find_by(workflow: 'Petty Cash Request')未找到对应数据库记录，返回nil，后续调用.id触发异常。

解决方案 #

1. 确认数据库存在必要记录 #

• 打开Rails控制台执行以下命令，检查目标记录是否存在：

  WorkflowProcess.where(workflow: ['Timesheet', 'Petty Cash Request'])
  

• 若记录不存在，在db/seeds.rb中添加初始化代码（替换示例字段为模型实际必填项）：

  # 创建Timesheet流程
  WorkflowProcess.find_or_create_by(workflow: 'Timesheet') do |process|
    process.description = '工时单审批流程'
  end
  
  # 创建Petty Cash Request流程
  WorkflowProcess.find_or_create_by(workflow: 'Petty Cash Request') do |process|
    process.description = '小额现金申请审批流程'
  end
  

  执行rails db:seed完成数据初始化。

2. 代码层面添加nil防护 #

方案A：强制检查记录存在（适合必须有记录的场景）

用find_by!替代find_by，未找到记录时会抛出ActiveRecord::RecordNotFound异常，快速定位数据缺失问题：

wp = WorkflowProcess.find_by!(workflow: 'Timesheet')
wpr= WorkflowProcess.find_by!(workflow: 'Petty Cash Request')

# 后续查询代码保持不变

方案B：优雅处理nil情况

添加条件判断避免nil调用方法，同时用pluck替代collect提升查询性能：

wp = WorkflowProcess.find_by(workflow: 'Timesheet')
wpr= WorkflowProcess.find_by(workflow: 'Petty Cash Request')

by_s = if wp
         WorkflowStateTransition.joins("INNER JOIN workflow_states ON workflow_states.workflow_state_id = workflow_state_transitions.workflow_state_id")
                                .where("workflow_states.workflow_process_id = ? AND workflow_state_transitions.by_supervisor = ?", wp.id, true)
                                .pluck(:workflow_state_id)
       else
         [] # 无记录时返回空数组
       end

by_s_r = if wpr
           WorkflowStateTransition.joins("INNER JOIN workflow_states ON workflow_states.workflow_state_id = workflow_state_transitions.workflow_state_id")
                                  .where("workflow_states.workflow_process_id = ? AND workflow_state_transitions.by_supervisor = ?", wpr.id, true)
                                  .pluck(:workflow_state_id)
         else
           [] # 无记录时返回空数组
         end

3. 优化关联查询（可选） #

如果模型已定义关联（WorkflowProcess has_many :workflow_states、WorkflowState has_many :workflow_state_transitions），用Rails安全导航操作符&.简化代码，避免手动写SQL：

wp = WorkflowProcess.find_by(workflow: 'Timesheet')
by_s = wp&.workflow_states&.joins(:workflow_state_transitions)&.where(workflow_state_transitions: { by_supervisor: true })&.pluck(:workflow_state_id) || []

wpr = WorkflowProcess.find_by(workflow: 'Petty Cash Request')
by_s_r = wpr&.workflow_states&.joins(:workflow_state_transitions)&.where(workflow_state_transitions: { by_supervisor: true })&.pluck(:workflow_state_id) || []

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