Laravel prepareForValidation新增属性报null读取错误如何解决

你遇到的报错核心原因是链式调用属性时，前置对象返回了null，具体触发场景有三类：

• History::find($this->route('id')) 没有查询到匹配id的数据库记录时，find()方法会返回null，此时直接调用->transaction就会触发该错误。偶发运行成功是因为传入的路由id刚好对应存在的History记录，多数场景报错是因为传入的id无效、对应记录已被删除、或路由参数名不匹配导致$this->route('id')本身取到null值，传入find()后直接返回null。
• 就算查询到了History实例，如果该实例关联的transaction记录不存在（比如关联数据被软删、外键关联断裂无对应数据），->transaction会返回null，再调用->transaction_slug同样会触发同类错误。
• 如果你的路由使用了模型隐式绑定，参数名不是id而是模型名（比如{history}），$this->route('id')会直接返回null，查询结果自然为null。

根据业务场景选择对应写法即可：

1. 常规严谨写法（推荐）：不存在对应记录时自动返回404响应，避免无意义的属性读取报错

protected function prepareForValidation()
{
    // 查询不到History记录时直接抛出ModelNotFoundException，框架自动处理为404响应
    $history = History::findOrFail($this->route('id'));
    // 预加载关联判断是否存在
    if (!$history->relationLoaded('transaction')) {
        $history->load('transaction');
    }
    // 关联记录不存在时按业务需求抛出404或验证错误
    abort_if(empty($history->transaction), 404, '关联交易记录不存在');

    $this->merge([
        'transaction_slug' => $history->transaction->transaction_slug,
    ]);
}

2. 简化写法（PHP8.0+支持）：使用空安全操作符避免报错，配合后续验证规则兜底

protected function prepareForValidation()
{
    $this->merge([
        'transaction_slug' => History::find($this->route('id'))?->transaction?->transaction_slug,
    ]);
}

注意：该写法下如果任意环节拿不到数据，transaction_slug会被赋值为null，你需要在表单请求的rules()方法中给该字段加上required等校验规则，避免后续逻辑拿到null值引发新的错误。
3. 路由已配置模型隐式绑定的优化写法：避免重复查询数据库
如果你的路由参数是绑定History模型的（比如路由定义为/history/{history}/edit），可以直接从路由中取已解析的模型实例，不需要手动调用find查询：

protected function prepareForValidation()
{
    /** @var History $history */
    $history = $this->route('history');
    abort_if(!$history?->transaction, 404, '请求记录不存在');
    
    $this->merge([
        'transaction_slug' => $history->transaction->transaction_slug
    ]);
}

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