移除未使用的GetFeeStatsByDateRange存储过程

任务时间: 2025-06-25 19:44
任务类型: 代码清理
状态: ✅ 已完成

任务概述

用户询问数据库中的 GetFeeStatsByDateRange 存储过程的作用和使用情况，经过分析发现该存储过程没有实际使用，决定移除。

问题分析

存储过程功能

名称: GetFeeStatsByDateRange
位置: app/Module/Transfer/Database/fee_daily_stats.sql
功能: 获取指定应用和日期范围的手续费统计数据
参数:
- p_app_id: 应用ID（0表示所有应用）
- p_start_date: 开始日期
- p_end_date: 结束日期

使用情况调查

通过全项目搜索发现：

没有任何PHP代码调用该存储过程
没有CALL语句或DB::select调用
功能已被Laravel ORM完全替代

替代方案

实际业务中使用的是 TransferFeeDailyStats 模型的 getByDateRange 方法：

public static function getByDateRange(string $startDate, string $endDate, int $appId = 0)
{
    $query = static::with('transferApp')
        ->whereBetween('stat_date', [$startDate, $endDate]);

    if ($appId > 0) {
        $query->where('transfer_app_id', $appId);
    }

    return $query->orderBy('stat_date', 'desc')
        ->orderBy('total_fee_amount', 'desc')
        ->get();
}

执行的操作

1. 移除SQL文件中的存储过程定义

文件: app/Module/Transfer/Database/fee_daily_stats.sql
删除了完整的存储过程定义（37行代码）

2. 从数据库中删除存储过程

DROP PROCEDURE IF EXISTS `GetFeeStatsByDateRange`;

3. 更新相关文档

文件: AiWork/202506/251039-Transfer模块手续费统计功能.md
移除了存储过程的说明

4. 提交代码更改

提交信息: "移除未使用的GetFeeStatsByDateRange存储过程"
修改文件: 2个
删除行数: 37行
新增行数: 1行

清理结果

✅ 已完成

删除SQL文件中的存储过程定义
从数据库中删除存储过程实例
更新相关文档
提交并推送代码更改
创建任务记录

验证

确认数据库中不再存在该存储过程
确认代码中没有调用该存储过程的地方
确认现有功能正常工作（使用Laravel ORM）

技术说明

为什么移除

冗余代码: 存储过程功能已被Laravel ORM完全替代
维护成本: 存储过程增加了维护复杂度
架构一致性: 项目统一使用Laravel ORM，不使用存储过程
代码清洁: 移除未使用的代码，保持代码库整洁

影响评估

无业务影响: 该存储过程从未被调用
无功能损失: Laravel ORM方法提供相同功能
提升维护性: 减少了冗余代码

总结

成功移除了未使用的 GetFeeStatsByDateRange 存储过程，包括：

SQL定义文件中的代码
数据库中的实际存储过程
相关文档说明

这次清理提升了代码库的整洁度，减少了维护成本，同时保持了功能的完整性。

251944-移除未使用的GetFeeStatsByDateRange存储过程.md 3.0 KB Permalink History Raw