Laravel 5.6模型事件：能否合并creating与updating事件或复用代码？

嘿，这个问题问得很实在！针对你这种在creating和updating事件里有重复代码的场景，抽离出可复用的代码片段是更优的选择，当然也可以根据具体场景考虑合并事件，我给你详细拆解：

优先推荐：抽离重复代码为独立方法 #

这种方案的优势非常明显：

• 维护成本更低：以后要修改这段逻辑时，只需要改一处，不用在两个事件回调里重复修改，避免出现漏改或者逻辑不一致的问题。
• 职责更清晰：事件回调只负责触发逻辑，具体的业务逻辑放在单独的方法里，代码结构更清爽，可读性更强。
• 扩展性更好：如果后续其他事件（比如created、updated甚至其他模型）需要用到这段逻辑，直接调用方法即可，复用性拉满。

具体实现示例：

protected static function boot() {
    parent::boot();

    static::creating(function ($questionnaire) {
        self::processQuestionnaireCommonLogic($questionnaire);
    });

    static::updating(function ($questionnaire) {
        self::processQuestionnaireCommonLogic($questionnaire);
    });
}

/**
 * 处理问卷的通用业务逻辑
 * @param $questionnaire
 */
private static function processQuestionnaireCommonLogic($questionnaire) {
    // 这里放置你原来重复的代码
    // 示例：生成slug、处理状态、关联数据等
    // $questionnaire->slug = Str::slug($questionnaire->title);
}

可选方案：用saving事件合并处理 #

如果你这段重复逻辑完全适用于所有模型保存场景（包括创建和更新），那可以直接用Laravel提供的saving事件来替代creating和updating——这个事件会在模型即将保存时触发，不管是新模型创建还是已有模型更新。

实现示例：

protected static function boot() {
    parent::boot();

    static::saving(function ($questionnaire) {
        // 这里放置你原来重复的代码
    });
}

不过要注意：saving事件的触发范围更广，比如当你调用save()但模型没有任何属性变更时，它也可能会触发（取决于Laravel版本和模型设置）。如果你的逻辑只需要在模型创建或属性更新时执行，那抽离独立方法的方案会更稳妥。

总结 #

• 如果你需要灵活控制逻辑的触发场景，或者未来有复用需求：选抽离独立方法。
• 如果逻辑完全适配所有保存场景，追求代码简洁：选saving事件合并。

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