refactor(generate): 拆分 useStep5Voice 模式选择逻辑到子Hook（184→165行, -10%） #1089

Merged

xiaoxia merged 2 commits from refactor/use-step5-voice into develop 2026-07-28 16:18:55 +08:00

Conversation 3 Commits 2 Files Changed 2

+94 -43

xiaoxia commented 2026-07-28 13:50:06 +08:00

Owner

Copy Link

Copy Source

变更说明

将 useStep5Voice 中的模式切换、克隆操作等选择逻辑抽离为 useVoiceModeSelection 子 Hook，主文件从 184 行减至 165 行（-10%）。

注：useStep5Voice 之前已拆分为 4 个子 Hook（useVoiceAudio/useVoiceRecommend/useTtsSynthesis/useSaveToLibrary），本此为增量优化。

拆分结构

• useVoiceModeSelection：模式切换、克隆弹窗、音色选择等操作回调
• useStep5Voice：主 Hook，组装各子模块并对外暴露统一接口

兼容性

• 导出路径不变
• 返回值接口完全兼容，无破坏性变更

### 变更说明 将 useStep5Voice 中的模式切换、克隆操作等选择逻辑抽离为 `useVoiceModeSelection` 子 Hook，主文件从 184 行减至 165 行（-10%）。 注：useStep5Voice 之前已拆分为 4 个子 Hook（useVoiceAudio/useVoiceRecommend/useTtsSynthesis/useSaveToLibrary），本此为增量优化。 ### 拆分结构 - **useVoiceModeSelection**：模式切换、克隆弹窗、音色选择等操作回调 - **useStep5Voice**：主 Hook，组装各子模块并对外暴露统一接口 ### 兼容性 - 导出路径不变 - 返回值接口完全兼容，无破坏性变更

xiaoxia force-pushed refactor/use-step5-voice from 35fc644e96 to e2304f316f 2026-07-28 14:42:42 +08:00 Compare

xiaoxia added 2 commits 2026-07-28 16:06:06 +08:00

refactor(generate): 拆分useStep5Voice模式选择逻辑到子Hook（184→165行, -10%） aa36e7ae62

chore: retrigger CI 4750e67c55

xiaoxia force-pushed refactor/use-step5-voice from 2673e71774 to 4750e67c55 2026-07-28 16:06:06 +08:00 Compare

xiaoxia merged commit 15f12ead3f into develop 2026-07-28 16:18:55 +08:00

auto-approve-bot reviewed 2026-07-28 16:31:09 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

gitea-actions bot commented 2026-07-28 16:31:38 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-28 16:33:55 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：2 个

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

无

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

1. [useVoiceModeSelection.ts: 43-47] 逻辑不一致风险

   • 具体内容：handleSelectClonedVoice 函数仅更新了 selectedClonedVoice，但未同步更新 voiceMode 为 "clone"。相比之下，handleSelectPresetVoice 和 handleSelectRecommendedVoice 均在选中音色时同步更新了模式。这种不对称可能导致状态不一致（例如：当前模式为 preset，但 selectedClonedVoice 有值），建议在函数内补充 onVoiceModeChange("clone")。

2. [useStep5Voice.tsx: 104] 性能优化建议

   • 具体内容：重构后的 handleCloneSuccess 是一个普通函数，每次组件重新渲染时都会创建新的引用。如果该函数被传递给子组件或用于 useEffect 依赖项，会导致不必要的重渲染或副作用循环。建议使用 useCallback 包裹，并将 rawCloneSuccess 加入依赖数组。

✅ 良好实践

• 将 useVoiceModeSelection 逻辑抽离为独立的 Hook，符合关注点分离原则，提高了代码的可复用性和可测试性。
• 保留了 message.success 在上层组件调用，使得底层 Hook 不依赖具体的 UI 组件库，保持了良好的纯度。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 08:33:55 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[useVoiceModeSelection.ts: 43-47] 逻辑不一致风险** - 具体内容：`handleSelectClonedVoice` 函数仅更新了 `selectedClonedVoice`，但未同步更新 `voiceMode` 为 `"clone"`。相比之下，`handleSelectPresetVoice` 和 `handleSelectRecommendedVoice` 均在选中音色时同步更新了模式。这种不对称可能导致状态不一致（例如：当前模式为 preset，但 selectedClonedVoice 有值），建议在函数内补充 `onVoiceModeChange("clone")`。 2. **[useStep5Voice.tsx: 104] 性能优化建议** - 具体内容：重构后的 `handleCloneSuccess` 是一个普通函数，每次组件重新渲染时都会创建新的引用。如果该函数被传递给子组件或用于 `useEffect` 依赖项，会导致不必要的重渲染或副作用循环。建议使用 `useCallback` 包裹，并将 `rawCloneSuccess` 加入依赖数组。 ### ✅ 良好实践 - 将 `useVoiceModeSelection` 逻辑抽离为独立的 Hook，符合关注点分离原则，提高了代码的可复用性和可测试性。 - 保留了 `message.success` 在上层组件调用，使得底层 Hook 不依赖具体的 UI 组件库，保持了良好的纯度。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 08:33:55 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

Sign in to join this conversation.

Reviewers

No Reviewers

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#1089