修复合成验证逻辑：避免服务层处理时出错

任务时间: 2025年06月09日 16:52
任务类型: Bug修复
影响模块: 游戏物品合成系统

任务背景

用户反馈合成功能出错，在验证通过后服务层处理时还会出错，验证逻辑不完善。从日志分析发现：

验证阶段没有捕获到材料不足的问题
服务层处理时才发现用户没有足够的材料
错误信息为空字符串，用户体验差

问题分析

1. 核心问题

CraftHandler中没有使用ItemCraftValidation进行完整验证，只是做了基础的参数验证，导致：

材料不足等业务逻辑错误在服务层才被发现
验证逻辑不完善，无法提前捕获问题
错误信息不准确，用户体验差

2. 错误处理问题

发现多个错误处理的问题：

ItemRecipe::canCraftByUser方法中使用了$consumeResult->error而不是!$consumeResult->success
CraftService::craftItem方法中使用了$consumeResult->error而不是!$consumeResult->success
CraftHandler中使用了$result->error而不是!$result->success
异常捕获使用了Exception而不是\Exception

3. 日志分析

从错误日志可以看到：

[2025-06-09T16:43:15.356041+08:00] laravel.ERROR: 合成物品失败 {"user_id":10001,"recipe_id":6,"error":"","trace":"..."}

错误信息为空字符串，说明异常没有被正确捕获和处理。

修复方案

1. 在Handler中添加完整验证

文件: app/Module/AppGame/Handler/Item/CraftHandler.php

// 添加验证类引用
use App\Module\GameItems\Validations\ItemCraftValidation;

// 在处理逻辑中添加验证
// 进行完整的合成验证
$validation = new ItemCraftValidation([
    'user_id' => $userId,
    'recipe_id' => $recipeId,
    'quantity' => $quantity
]);

// 验证数据，如果验证失败会抛出异常
$validation->validated();

2. 修复错误处理逻辑

2.1 修复ItemRecipe模型

文件: app/Module/GameItems/Models/ItemRecipe.php

// 修复第217行
if (!$consumeResult->success) {  // 原来是 $consumeResult->error
    return [
        'can_craft' => false,
        'reason' => '材料不足: ' . $consumeResult->message,
    ];
}

2.2 修复CraftService

文件: app/Module/GameItems/Services/CraftService.php

// 修复第78行
if (!$consumeResult->success) {  // 原来是 $consumeResult->error
    throw new LogicException("消耗失败: " . $consumeResult->message);
}

// 修复异常捕获
} catch (\Exception $e) {  // 原来是 Exception $e

2.3 修复CraftHandler

文件: app/Module/AppGame/Handler/Item/CraftHandler.php

// 修复第61行
if (!$result->success) {  // 原来是 $result->error
    throw new LogicException($result->message);  // 使用具体的错误信息
}

验证结果

1. 修复前的错误

{
  "响应内容": null,
  "日志错误": "合成物品失败 {\"error\":\"\"}",
  "用户体验": "无法获得具体错误信息"
}

2. 修复后的正确响应

{
  "runUnid": "6846a0e96375f",
  "runMs": "275",
  "code": "VALIDATE_ERROR",
  "msg": "物品 5 数量不足，需要 100，实际 0",
  "callpath": "Item-Craft",
  "lastData": []
}

3. 验证效果

✅ 验证阶段正确捕获材料不足问题
✅ 返回具体的错误信息："物品 5 数量不足，需要 100，实际 0"
✅ 错误码为VALIDATE_ERROR，表明是验证阶段的错误
✅ 避免了在服务层事务中出错
✅ 提升了用户体验

技术优势

1. 提前验证

在Handler层进行完整验证，避免在服务层才发现问题
减少不必要的数据库事务开销
提高系统性能

2. 错误信息准确

使用正确的Res对象属性进行错误检查
提供具体的错误信息，便于用户理解
改善用户体验

3. 代码健壮性

修复异常捕获问题，确保异常能被正确处理
统一错误处理逻辑，提高代码一致性
增强系统稳定性

修改文件

app/Module/AppGame/Handler/Item/CraftHandler.php
- 添加ItemCraftValidation引用
- 在处理逻辑中添加完整验证
- 修复错误检查逻辑
app/Module/GameItems/Models/ItemRecipe.php
- 修复canCraftByUser方法中的错误检查
app/Module/GameItems/Services/CraftService.php
- 修复craftItem方法中的错误检查
- 修复异常捕获类型

测试验证

使用复现命令验证修复效果：

php artisan debug:reproduce-error 68469e705a737

修复前：返回null，错误信息为空修复后：返回具体错误信息，用户体验良好

总结

通过在Handler层添加完整的验证逻辑，并修复多个错误处理问题，成功解决了验证通过后服务层还会出错的问题。现在系统能够在验证阶段就捕获到材料不足等问题，避免了在事务中出错，提升了用户体验和系统稳定性。

09日1652-修复合成验证逻辑避免服务层处理时出错.md 4.9 KB Histórico Em bruto