refactor(products): 拆分 useProductList 为3个子Hook（202→91行, -55%） #1155

Merged

auto-approve-bot merged 4 commits from refactor/product-list-hook into develop 2026-07-30 01:15:19 +08:00

Conversation 5 Commits 4 Files Changed 4

+150 -78

xiaoxia commented 2026-07-29 19:08:26 +08:00

Owner

Copy Link

Copy Source

变更内容

将 useProductList.ts（202行）拆分为目录结构，按职责分离：

• useProductFiltering - 筛选逻辑（搜索/状态/时间/时长/项目/复核）+ 项目选项
• useBatchSelection - 批量选择逻辑（全选/切换/清空）
• index.ts - 主入口，组合子Hook + useQuery数据获取

文件变化

• 删除：hooks/useProductList.ts (202行)
• 新增：hooks/useProductList/index.ts (91行)
• 新增：hooks/useProductList/useProductFiltering.ts (127行)
• 新增：hooks/useProductList/useBatchSelection.ts (42行)
• 修改：test/pages/products/smoke.test.tsx +子模块导入

兼容性

• 原导入路径 @/pages/products/hooks/useProductList 保持不变（目录解析到 index.ts）
• Filters 类型从 index.ts re-export，保持向后兼容

### 变更内容 将 `useProductList.ts`（202行）拆分为目录结构，按职责分离： - **useProductFiltering** - 筛选逻辑（搜索/状态/时间/时长/项目/复核）+ 项目选项 - **useBatchSelection** - 批量选择逻辑（全选/切换/清空） - **index.ts** - 主入口，组合子Hook + useQuery数据获取 ### 文件变化 - 删除：`hooks/useProductList.ts` (202行) - 新增：`hooks/useProductList/index.ts` (91行) - 新增：`hooks/useProductList/useProductFiltering.ts` (127行) - 新增：`hooks/useProductList/useBatchSelection.ts` (42行) - 修改：`test/pages/products/smoke.test.tsx` +子模块导入 ### 兼容性 - 原导入路径 `@/pages/products/hooks/useProductList` 保持不变（目录解析到 index.ts） - `Filters` 类型从 index.ts re-export，保持向后兼容

xiaoxia added 1 commit 2026-07-29 19:08:26 +08:00

refactor(products): 拆分 useProductList 为3个子Hook（202→91行, -55%） ... 39ff3c48ae

- useProductFiltering: 筛选逻辑（搜索/状态/时间/时长/项目/复核）
- useBatchSelection: 批量选择逻辑（全选/切换/清空）
- index.ts: 主入口，组合子Hook并获取数据
- 保持原路径向后兼容（目录index.ts）
- 补充 smoke test 子模块导入

xiaoxia added 1 commit 2026-07-29 21:50:56 +08:00

fix(lint): 移除未使用的 import 6204d28780

gitea-actions bot commented 2026-07-29 21:53:01 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1155
预览链接	https://pr-1155.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1155 | | 预览链接 | [https://pr-1155.preview.xiaoxiajianji.com](https://pr-1155.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot approved these changes 2026-07-29 22:25:44 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-29 22:25:45 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia added 1 commit 2026-07-30 00:57:59 +08:00

fix: 全选/取消全选仅操作当前可见项，避免筛选状态下数据丢失 e204b91102

xiaoxia added 1 commit 2026-07-30 00:58:00 +08:00

fix: products计算使用useMemo避免重复排序的性能回归 6526c56bf5

auto-approve-bot commented 2026-07-30 01:04:04 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：否
• 阻塞级问题数量：0 个

📊 审查概览

• 整体评价：有建议
• 建议级问题数量：3 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

1. [apps/web/src/pages/products/hooks/useProductList/useBatchSelection.ts: 11] 性能优化：allSelected 计算建议使用 useMemo

   • 具体内容：当前 allSelected 是一个直接计算的派生状态，在组件每次渲染时都会执行 filteredProducts.every 遍历。如果产品列表较长，这会造成不必要的计算开销。建议使用 useMemo 包裹，依赖项为 [selectedIds, filteredProducts]。

2. [apps/web/src/pages/products/hooks/useProductList/useProductFiltering.ts: 40] 潜在的空指针异常风险

   • 具体内容：在搜索过滤逻辑中使用了 p.name.toLowerCase()。如果 ProductItem 中的 name 字段为 null 或 undefined，这将导致运行时报错。尽管这是重构代码，但建议增加防御性判断，如 p.name?.toLowerCase() ?? ""。

3. [apps/web/src/pages/products/hooks/useProductList/index.ts: 22] 日期解析的健壮性

   • 具体内容：在排序逻辑中，new Date(b.date).getTime() 假设日期字符串是有效的。如果 date 是无效格式（既不是空也不是 "—"），getTime() 将返回 NaN，导致排序结果不稳定。建议增加对 isNaN 的检查，确保无效日期被统一处理。

✅ 良好实践

• 关注点分离：将原本庞大的 useProductList 拆分为 useProductFiltering 和 useBatchSelection，逻辑清晰，易于维护。
• 状态管理优化：使用 Set<string> 管理选中 ID，查找和删除操作的时间复杂度为 O(1)，性能优于数组。
• 交互逻辑完善：handleSelectAll 实现了在筛选状态下“全选/反选”时保留筛选外选中项的逻辑，符合复杂列表操作的用户体验预期。
• 防御性编程：在 index.ts 中对 apiProducts 进行了数组类型检查，防止非数组数据导致的崩溃。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 17:04:03 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：3 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/web/src/pages/products/hooks/useProductList/useBatchSelection.ts: 11] 性能优化：allSelected 计算建议使用 useMemo** - 具体内容：当前 `allSelected` 是一个直接计算的派生状态，在组件每次渲染时都会执行 `filteredProducts.every` 遍历。如果产品列表较长，这会造成不必要的计算开销。建议使用 `useMemo` 包裹，依赖项为 `[selectedIds, filteredProducts]`。 2. **[apps/web/src/pages/products/hooks/useProductList/useProductFiltering.ts: 40] 潜在的空指针异常风险** - 具体内容：在搜索过滤逻辑中使用了 `p.name.toLowerCase()`。如果 `ProductItem` 中的 `name` 字段为 `null` 或 `undefined`，这将导致运行时报错。尽管这是重构代码，但建议增加防御性判断，如 `p.name?.toLowerCase() ?? ""`。 3. **[apps/web/src/pages/products/hooks/useProductList/index.ts: 22] 日期解析的健壮性** - 具体内容：在排序逻辑中，`new Date(b.date).getTime()` 假设日期字符串是有效的。如果 `date` 是无效格式（既不是空也不是 "—"），`getTime()` 将返回 `NaN`，导致排序结果不稳定。建议增加对 `isNaN` 的检查，确保无效日期被统一处理。 ### ✅ 良好实践 - **关注点分离**：将原本庞大的 `useProductList` 拆分为 `useProductFiltering` 和 `useBatchSelection`，逻辑清晰，易于维护。 - **状态管理优化**：使用 `Set<string>` 管理选中 ID，查找和删除操作的时间复杂度为 O(1)，性能优于数组。 - **交互逻辑完善**：`handleSelectAll` 实现了在筛选状态下“全选/反选”时保留筛选外选中项的逻辑，符合复杂列表操作的用户体验预期。 - **防御性编程**：在 `index.ts` 中对 `apiProducts` 进行了数组类型检查，防止非数组数据导致的崩溃。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-29 17:04:03 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit f15bc2f2a2 into develop 2026-07-30 01:15:19 +08:00

auto-approve-bot deleted branch refactor/product-list-hook 2026-07-30 01:15:19 +08:00

gitea-actions bot commented 2026-07-30 01:18:40 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1155 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1155 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1155