-
Notifications
You must be signed in to change notification settings - Fork 1
[代码质量] App.jsx 文件过大 (612行),建议拆分为独立组件 #79
Copy link
Copy link
Open
Description
问题描述
frontend/src/App.jsx 文件已达到 612 行,接近可维护性阈值。这类似于后端的 handlers.rs 文件过大问题 (#69)。
影响
- 代码难以导航和理解
- 组件职责不清晰
- 测试难度增加
- 代码复用困难
建议
考虑将 App.jsx 拆分为更小的组件:
- FileList 组件 - 文件列表相关逻辑 (~150 lines)
- FileDetail 组件 - 文件详情侧边栏 (~200 lines)
- PublishModal 组件 - 发布对话框 (~50 lines)
- MapPreview 组件 - 地图预览相关 (~100 lines)
- AuthComponents - 认证相关组件 (~50 lines)
- 保留 App.jsx 作为主布局和路由 (~62 lines)
参考模式
可参考后端 handlers.rs 的拆分方式:
- handlers/list.rs
- handlers/preview.rs
- handlers/tiles.rs
- handlers/publish.rs
优先级
P2 - 中优先级,不影响功能但影响长期可维护性
相关 Issue
- [代码质量] handlers.rs 文件过大 (882行),建议进一步拆分 #69 - 后端 handlers.rs 文件过大问题
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels