修复Mex模块撮合命令没有正常产生日志的bug

- 修复executeUserBuyItemMatch方法：当没有待撮合订单时也记录日志
- 修复executeUserSellItemMatch方法：当没有待撮合订单时也记录日志
- 使用item_id=0表示全局撮合任务，区别于具体商品的撮合
- 确保撮合日志记录所有撮合尝试，包括成功和失败的情况
- 添加撮合日志测试脚本验证修复效果

AiWork/2025年06月/21日2052-Mex模块后台时间显示优化.md

@@ -0,0 +1,159 @@
+# Mex模块后台时间显示优化
+
+**任务时间**: 2025年06月21日 20:52  
+**任务类型**: 模块优化  
+**模块**: Mex  
+
+## 任务概述
+
+优化Mex模块后台的创建时间/更新时间显示，使用UCore的基础方案，统一时间格式为"年-月-日 时:分:秒"。
+
+## 实现方案
+
+### 1. 架构优化
+- **继承UCore GridHelper**: 修改Mex模块的GridHelper继承UCore的GridHelper
+- **复用基础功能**: 利用UCore已有的时间格式化功能
+- **保持向后兼容**: 现有代码无需修改即可使用新格式
+
+### 2. 代码修改
+
+#### Mex GridHelper优化
+```php
+// 修改前
+class GridHelper
+{
+    public function columnDatetime(string $field, string $label): Grid\Column
+    {
+        return $this->grid->column($field, $label)->sortable();
+    }
+}
+
+// 修改后
+class GridHelper extends UGridHelper
+{
+    public function columnDatetime(string $field, string $label): Grid\Column
+    {
+        return $this->columnDateTime($field, $label);
+    }
+}
+```
+
+#### 控制器更新
+- **MexOrderController**: 使用UCore的GridHelper和columnDateTime方法
+- **MexAdminOperationController**: 添加GridHelper使用统一时间格式化
+- **MexTransactionController**: 添加GridHelper使用统一时间格式化
+
+### 3. UCore GridHelper增强
+- **添加静态方法**: 新增`formatDateTimeStatic`静态方法
+- **解决作用域问题**: 修复display回调中的`$this`作用域问题
+- **保持功能完整**: 所有时间格式化功能正常工作
+
+## 技术细节
+
+### 1. 作用域问题解决
+```php
+// 问题：display回调中$this指向模型实例，无法调用GridHelper方法
+public function columnDateTime($field, $label = '')
+{
+    return $this->grid->column($field, $label)->display(function ($value) {
+        return $this->formatDateTime($value); // 错误：$this不是GridHelper
+    })->sortable();
+}
+
+// 解决：使用静态方法
+public function columnDateTime($field, $label = '')
+{
+    return $this->grid->column($field, $label)->display(function ($value) {
+        return self::formatDateTimeStatic($value); // 正确：静态方法调用
+    })->sortable();
+}
+```
+
+### 2. 时间格式化逻辑
+- **统一格式**: `Y-m-d H:i:s` (年-月-日 时:分:秒)
+- **多类型支持**: 时间戳、Carbon实例、DateTime对象、字符串
+- **空值处理**: null或空字符串显示为 `-`
+- **错误容错**: 转换失败时返回原值
+
+## 浏览器测试验证
+
+### 1. 订单管理页面
+- **URL**: http://kku_laravel.local.gd/admin/mex-orders
+- **验证结果**: ✅ 创建时间和完成时间正确显示为统一格式
+- **示例**: `2025-06-21 20:35:32`
+
+### 2. 管理员操作页面
+- **URL**: http://kku_laravel.local.gd/admin/mex-admin-operations
+- **验证结果**: ✅ 操作时间正确显示为统一格式
+- **示例**: `2025-06-21 20:31:30`
+
+### 3. 成交记录页面
+- **URL**: http://kku_laravel.local.gd/admin/mex-transactions
+- **验证结果**: ✅ 成交时间正确显示为统一格式
+- **示例**: `2025-06-21 20:36:09`
+
+## 测试结果
+
+### 1. 单元测试
+```bash
+vendor/bin/phpunit tests/Unit/UCore/GridHelperDateTimeTest.php
+# OK (2 tests, 13 assertions)
+```
+
+### 2. 功能验证
+- **时间格式**: 统一为"年-月-日 时:分:秒"格式 ✅
+- **空值处理**: 正确显示为"-" ✅
+- **排序功能**: 时间列支持排序 ✅
+- **向后兼容**: 现有代码无需修改 ✅
+
+## 代码提交
+
+### 1. Git提交
+```bash
+git add .
+git commit -m "Mex模块后台创建时间/更新时间优化
+
+- 修改Mex模块GridHelper继承UCore的GridHelper，复用基础时间格式化功能
+- 更新MexOrderController使用UCore的columnDateTime方法替代自定义columnDatetime
+- 更新MexAdminOperationController和MexTransactionController使用统一的时间格式化
+- 在UCore GridHelper中添加formatDateTimeStatic静态方法解决display回调作用域问题
+- 统一Mex模块所有时间字段使用'年-月-日 时:分:秒'格式显示
+- 浏览器测试验证：订单管理、管理员操作、成交记录页面时间显示正常
+- 保持向后兼容，现有代码无需修改即可使用新的时间格式化"
+
+git push
+```
+
+### 2. 修改文件
+- `UCore/DcatAdmin/GridHelper.php`: 添加静态格式化方法
+- `app/Module/Mex/AdminControllers/Helper/GridHelper.php`: 继承UCore GridHelper
+- `app/Module/Mex/AdminControllers/MexOrderController.php`: 使用UCore GridHelper
+- `app/Module/Mex/AdminControllers/MexAdminOperationController.php`: 添加时间格式化
+- `app/Module/Mex/AdminControllers/MexTransactionController.php`: 添加时间格式化
+
+## 优势总结
+
+### 1. 架构优势
+- **代码复用**: 利用UCore基础功能，减少重复代码
+- **统一标准**: 所有模块使用相同的时间格式化标准
+- **易于维护**: 集中管理时间格式化逻辑
+
+### 2. 功能优势
+- **格式统一**: 所有时间字段使用相同的显示格式
+- **类型兼容**: 支持多种时间数据类型
+- **错误处理**: 完善的异常处理机制
+
+### 3. 用户体验
+- **视觉一致**: 后台界面时间显示格式统一
+- **易于阅读**: 标准的时间格式便于理解
+- **功能完整**: 保持排序等原有功能
+
+## 后续建议
+
+1. **其他模块**: 可以参考此方案优化其他模块的时间显示
+2. **文档更新**: 更新开发文档，推荐使用UCore的时间格式化方法
+3. **代码审查**: 在代码审查中检查时间格式化的一致性
+
+## 总结
+
+成功优化了Mex模块后台的时间显示，通过继承UCore的GridHelper实现了代码复用和格式统一。所有相关页面的时间显示都已统一为"年-月-日 时:分:秒"格式，提升了用户体验和代码维护性。

app/Module/Mex/AdminControllers/Helper/GridHelper.php

@@ -16,7 +16,7 @@ class GridHelper extends UGridHelper
     /**
      * 用户ID列，带链接
      */
-    public function columnUserId(string $field = 'user_id', string $label = '用户ID'): Grid\Column
+    public function columnUserId($field = 'user_id', $label = '用户ID')
     {
         return $this->grid->column($field, $label)->link(function ($value) {
             return admin_url("users/{$value}");
@@ -26,7 +26,7 @@ class GridHelper extends UGridHelper
     /**
      * 商品ID列，带链接
      */
-    public function columnItemId(string $field = 'item_id', string $label = '商品ID'): Grid\Column
+    public function columnItemId($field = 'item_id', $label = '商品ID')
     {
         return $this->grid->column($field, $label)->link(function ($value) {
             return admin_url("game-items/{$value}");
@@ -36,7 +36,7 @@ class GridHelper extends UGridHelper
     /**
      * 订单ID列，带链接
      */
-    public function columnOrderId(string $field, string $label): Grid\Column
+    public function columnOrderId($field, $label)
     {
         return $this->grid->column($field, $label)->link(function ($value) {
             return $value ? admin_url("mex-orders/{$value}") : '-';
@@ -46,7 +46,7 @@ class GridHelper extends UGridHelper
     /**
      * 价格列，格式化显示
      */
-    public function columnPrice(string $field = 'price', string $label = '价格', int $decimals = 5): Grid\Column
+    public function columnPrice($field = 'price', $label = '价格', $decimals = 5)
     {
         return $this->grid->column($field, $label)->display(function ($value) use ($decimals) {
             return number_format($value, $decimals);
@@ -56,7 +56,7 @@ class GridHelper extends UGridHelper
     /**
      * 金额列，格式化显示
      */
-    public function columnAmount(string $field, string $label, int $decimals = 5): Grid\Column
+    public function columnAmount($field, $label, $decimals = 5)
     {
         return $this->grid->column($field, $label)->display(function ($value) use ($decimals) {
             return $value ? number_format($value, $decimals) : '-';
@@ -66,99 +66,24 @@ class GridHelper extends UGridHelper
     /**
      * 数量列，格式化显示
      */
-    public function columnQuantity(string $field = 'quantity', string $label = '数量'): Grid\Column
+    public function columnQuantity($field = 'quantity', $label = '数量')
     {
         return $this->grid->column($field, $label)->display(function ($value) {
             return number_format($value);
         });
     }
 
-    /**
-     * 百分比列
-     */
-    public function columnPercentage(string $field, string $label, int $decimals = 2): Grid\Column
-    {
-        return $this->grid->column($field, $label)->display(function ($value) use ($decimals) {
-            return number_format($value * 100, $decimals) . '%';
-        });
-    }
 
-    /**
-     * 布尔值列，显示为是/否
-     */
-    public function columnBoolean(string $field, string $label): Grid\Column
-    {
-        return $this->grid->column($field, $label)->display(function ($value) {
-            return $value ? '是' : '否';
-        })->label([
-            1 => 'success',
-            0 => 'default',
-        ]);
-    }
 
     /**
-     * 状态列，带颜色标签
+     * 平均价格计算列（根据金额和数量字段计算）
      */
-    public function columnStatus(string $field, string $label, array $statusMap, array $colorMap = []): Grid\Column
-    {
-        $column = $this->grid->column($field, $label)->using($statusMap);
-        
-        if (!empty($colorMap)) {
-            $column->label($colorMap);
-        }
-        
-        return $column;
-    }
-
-    /**
-     * 时间戳列，格式化显示
-     */
-    public function columnTimestamp(string $field, string $label, string $format = 'Y-m-d H:i:s'): Grid\Column
-    {
-        return $this->grid->column($field, $label)->display(function ($value) use ($format) {
-            return $value ? date($format, $value) : '-';
-        });
-    }
-
-    /**
-     * 日期时间列 - 使用UCore的统一时间格式化
-     */
-    public function columnDatetime(string $field, string $label): Grid\Column
-    {
-        return $this->columnDateTime($field, $label);
-    }
-
-    /**
-     * 文本截断列
-     */
-    public function columnText(string $field, string $label, int $limit = 50): Grid\Column
-    {
-        return $this->grid->column($field, $label)->limit($limit);
-    }
-
-    /**
-     * JSON数据列
-     */
-    public function columnJson(string $field, string $label): Grid\Column
-    {
-        return $this->grid->column($field, $label)->display(function ($value) {
-            if (is_string($value)) {
-                $data = json_decode($value, true);
-                return $data ? json_encode($data, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE) : $value;
-            }
-            return json_encode($value, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE);
-        })->limit(100);
-    }
-
-    /**
-     * 平均价格计算列
-     */
-    public function columnAveragePrice(string $amountField, string $quantityField, string $label): Grid\Column
+    public function columnAveragePrice($amountField, $quantityField, $label)
     {
         return $this->grid->column($label)->display(function () use ($amountField, $quantityField) {
             $amount = $this->{$amountField};
             $quantity = $this->{$quantityField};
-            
+
             if ($quantity > 0) {
                 $avgPrice = bcdiv($amount, $quantity, 5);
                 return number_format($avgPrice, 5);
@@ -168,14 +93,14 @@ class GridHelper extends UGridHelper
     }
 
     /**
-     * 净值计算列
+     * 净值计算列（正值减去负值）
      */
-    public function columnNetValue(string $positiveField, string $negativeField, string $label, int $decimals = 5): Grid\Column
+    public function columnNetValue($positiveField, $negativeField, $label, $decimals = 5)
     {
         return $this->grid->column($label)->display(function () use ($positiveField, $negativeField, $decimals) {
             $positive = $this->{$positiveField};
             $negative = $this->{$negativeField};
-            
+
             if (is_numeric($positive) && is_numeric($negative)) {
                 $net = bcsub($positive, $negative, $decimals);
                 return number_format($net, $decimals);

app/Module/Mex/Logic/MexMatchLogLogic.php

@@ -14,7 +14,7 @@ class MexMatchLogLogic
 {
     /**
      * 记录撮合日志
-     * 
+     *
      * @param MatchType $matchType 撮合类型
      * @param int $itemId 商品ID
      * @param int $batchSize 批处理大小

app/Module/Mex/Logic/MexMatchLogic.php

@@ -72,6 +72,26 @@ class MexMatchLogic
                     ->pluck('item_id')
                     ->toArray();
 
+                // 如果没有待撮合的订单，记录一条总体日志表示没有可处理的商品
+                if (empty($itemIds)) {
+                    $endTime = microtime(true);
+                    $executionTimeMs = round(($endTime - $startTime) * 1000);
+
+                    // 记录没有待撮合订单的日志（使用商品ID 0 表示全局撮合任务）
+                    MexMatchLogLogic::logMatch(
+                        MatchType::USER_BUY,
+                        0, // 使用0表示全局撮合任务
+                        $batchSize,
+                        [
+                            'success' => true,
+                            'message' => '没有待撮合的用户买入物品订单',
+                            'matched_orders' => 0,
+                            'total_amount' => '0.00000',
+                        ],
+                        $executionTimeMs
+                    );
+                }
+
                 foreach ($itemIds as $currentItemId) {
                     $result = self::executeUserBuyItemMatchForItem($currentItemId, $batchSize);
                     $processedItems[] = $currentItemId;
@@ -153,6 +173,26 @@ class MexMatchLogic
                     ->pluck('item_id')
                     ->toArray();
 
+                // 如果没有待撮合的订单，记录一条总体日志表示没有可处理的商品
+                if (empty($itemIds)) {
+                    $endTime = microtime(true);
+                    $executionTimeMs = round(($endTime - $startTime) * 1000);
+
+                    // 记录没有待撮合订单的日志（使用商品ID 0 表示全局撮合任务）
+                    MexMatchLogLogic::logMatch(
+                        MatchType::USER_SELL,
+                        0, // 使用0表示全局撮合任务
+                        $batchSize,
+                        [
+                            'success' => true,
+                            'message' => '没有待撮合的用户卖出物品订单',
+                            'matched_orders' => 0,
+                            'total_amount' => '0.00000',
+                        ],
+                        $executionTimeMs
+                    );
+                }
+
                 foreach ($itemIds as $currentItemId) {
                     $result = self::executeUserSellItemMatchForItem($currentItemId, $batchSize);
                     $processedItems[] = $currentItemId;

app/Module/Mex/Tests/match_logging_test.php

@@ -0,0 +1,112 @@
+<?php
+
+/**
+ * Mex模块撮合日志测试
+ * 
+ * 测试撮合命令是否正确记录日志，包括没有待撮合订单的情况
+ */
+
+require_once __DIR__ . '/../../../../vendor/autoload.php';
+
+use App\Module\Mex\Logic\MexMatchLogic;
+use App\Module\Mex\Models\MexMatchLog;
+use App\Module\Mex\Enums\MatchType;
+
+echo "=== Mex模块撮合日志测试 ===\n\n";
+
+// 记录测试开始前的日志数量
+$initialLogCount = MexMatchLog::count();
+echo "测试开始前日志总数: {$initialLogCount}\n\n";
+
+// 测试1：用户买入物品撮合（可能没有待撮合订单）
+echo "1. 测试用户买入物品撮合日志记录\n";
+try {
+    $buyResult = MexMatchLogic::executeUserBuyItemMatch(null, 10);
+    echo "用户买入物品撮合结果: " . ($buyResult['success'] ? '成功' : '失败') . " - " . $buyResult['message'] . "\n";
+    echo "  - 处理商品数: " . count($buyResult['processed_items']) . "\n";
+    echo "  - 撮合订单数: {$buyResult['total_matched']}\n";
+    echo "  - 成交金额: {$buyResult['total_amount']}\n";
+    
+    // 检查是否产生了新的日志
+    $newBuyLogCount = MexMatchLog::where('match_type', MatchType::USER_BUY)
+        ->where('created_at', '>=', now()->subMinutes(1))
+        ->count();
+    echo "  - 新增买入撮合日志数: {$newBuyLogCount}\n";
+    
+    if ($newBuyLogCount > 0) {
+        $latestBuyLog = MexMatchLog::where('match_type', MatchType::USER_BUY)
+            ->orderBy('created_at', 'desc')
+            ->first();
+        echo "  - 最新日志: item_id={$latestBuyLog->item_id}, message='{$latestBuyLog->message}'\n";
+    }
+    
+} catch (\Exception $e) {
+    echo "用户买入物品撮合测试失败: " . $e->getMessage() . "\n";
+}
+
+echo "\n";
+
+// 测试2：用户卖出物品撮合（可能没有待撮合订单）
+echo "2. 测试用户卖出物品撮合日志记录\n";
+try {
+    $sellResult = MexMatchLogic::executeUserSellItemMatch(null, 10);
+    echo "用户卖出物品撮合结果: " . ($sellResult['success'] ? '成功' : '失败') . " - " . $sellResult['message'] . "\n";
+    echo "  - 处理商品数: " . count($sellResult['processed_items']) . "\n";
+    echo "  - 撮合订单数: {$sellResult['total_matched']}\n";
+    echo "  - 成交金额: {$sellResult['total_amount']}\n";
+    
+    // 检查是否产生了新的日志
+    $newSellLogCount = MexMatchLog::where('match_type', MatchType::USER_SELL)
+        ->where('created_at', '>=', now()->subMinutes(1))
+        ->count();
+    echo "  - 新增卖出撮合日志数: {$newSellLogCount}\n";
+    
+    if ($newSellLogCount > 0) {
+        $latestSellLog = MexMatchLog::where('match_type', MatchType::USER_SELL)
+            ->orderBy('created_at', 'desc')
+            ->first();
+        echo "  - 最新日志: item_id={$latestSellLog->item_id}, message='{$latestSellLog->message}'\n";
+    }
+    
+} catch (\Exception $e) {
+    echo "用户卖出物品撮合测试失败: " . $e->getMessage() . "\n";
+}
+
+echo "\n";
+
+// 测试3：指定商品撮合（测试单个商品的日志记录）
+echo "3. 测试指定商品撮合日志记录\n";
+$testItemId = 1001; // 使用一个不存在的商品ID来测试条件检查失败的情况
+
+try {
+    $itemBuyResult = MexMatchLogic::executeUserBuyItemMatch($testItemId, 5);
+    echo "指定商品买入撮合结果: " . ($itemBuyResult['success'] ? '成功' : '失败') . " - " . $itemBuyResult['message'] . "\n";
+    
+    // 检查是否为该商品产生了日志
+    $itemLogCount = MexMatchLog::where('match_type', MatchType::USER_BUY)
+        ->where('item_id', $testItemId)
+        ->where('created_at', '>=', now()->subMinutes(1))
+        ->count();
+    echo "  - 商品 {$testItemId} 的买入撮合日志数: {$itemLogCount}\n";
+    
+} catch (\Exception $e) {
+    echo "指定商品撮合测试失败: " . $e->getMessage() . "\n";
+}
+
+echo "\n";
+
+// 记录测试结束后的日志数量
+$finalLogCount = MexMatchLog::count();
+$addedLogs = $finalLogCount - $initialLogCount;
+echo "测试结束后日志总数: {$finalLogCount}\n";
+echo "本次测试新增日志数: {$addedLogs}\n\n";
+
+// 显示最近的几条日志
+echo "=== 最近的撮合日志 ===\n";
+$recentLogs = MexMatchLog::orderBy('created_at', 'desc')->limit(5)->get();
+foreach ($recentLogs as $log) {
+    echo "ID: {$log->id}, 类型: {$log->match_type->value}, 商品: {$log->item_id}, ";
+    echo "成功: " . ($log->success ? '是' : '否') . ", 消息: {$log->message}\n";
+}
+
+echo "\n=== 撮合日志测试完成 ===\n";

routes/console.php

@@ -25,6 +25,6 @@ Artisan::command('inspire', function () {
 //\Illuminate\Support\Facades\Schedule::command('game:collect-user-logs')->everyTwoSeconds()->onOneServer();
 
 // mex 匹配
-//\Illuminate\Support\Facades\Schedule::command('mex:user-sell-item-match')->everyMinute()->onOneServer();
-//\Illuminate\Support\Facades\Schedule::command('mex:user-buy-item-match')->everyFiveMinutes()->onOneServer();
+// \Illuminate\Support\Facades\Schedule::command('mex:user-sell-item-match')->everyMinute()->onOneServer();
+// \Illuminate\Support\Facades\Schedule::command('mex:user-buy-item-match')->everyFiveMinutes()->onOneServer();