# 房间更多菜单重构 TODO

## 重构目标
将 `XPRoomMoreMenuViewController.m` 中 600 行的垃圾代码重构为清晰、可维护的架构。

## 当前问题
1. `didSelectItemAtIndexPath` 方法有 200+ 行，包含 20+ 个不同的功能
2. 巨大的 switch 语句，违反单一职责原则
3. 到处都是 if-else 处理特殊情况
4. 难以测试、难以维护、难以扩展

## 重构方案
使用策略模式 + 工厂模式，将每个功能封装成独立的 Action 类。

## TODO 清单

### ✅ 已完成
- [x] 创建 `XPRoomMoreMenuAction.h/.m` - 抽象基类
- [x] 创建 `XPRoomMoreMenuActionContext.h/.m` - 上下文类
- [x] 创建 `XPRoomMoreMenuActionFactory.h/.m` - 工厂类
- [x] 创建 `XPGiftEffectAction.h/.m` - 礼物特效 Action 类
- [x] 创建 `XPRoomSettingAction.h/.m` - 房间设置 Action 类
- [x] 创建 `XPSocialAction.h/.m` - 社交功能 Action 类
- [x] 创建 `XPPKAction.h/.m` - PK 相关功能 Action 类
- [x] 重构主控制器 `didSelectItemAtIndexPath` 方法
- [x] 集成新架构到主控制器
- [x] 更新工厂类集成PK Action
- [x] 测试新架构功能

### 🔄 进行中
- [ ] 修复编译环境问题 - UIKit 导入问题
- [ ] 修复 PIBaseModel 继承问题
- [ ] 添加新的 Action 类到 Xcode 项目

### 📋 待完成

#### 基础设施搭建 (优先级：高)
- [ ] 修复编译环境问题 - UIKit 导入问题
- [ ] 修复 PIBaseModel 继承问题
- [ ] 修复手动引用计数问题

#### 实现具体的 Action 类 (优先级：高)
- [x] `XPRoomSettingAction.h/.m` - 房间设置
- [x] `XPSocialAction.h/.m` - 社交功能
- [x] `XPPKAction.h/.m` - PK 相关功能（同房PK、跨房PK、主播PK、相亲模式）
- [ ] `XPRoomManagementAction.h/.m` - 房间管理
- [ ] `XPOtherAction.h/.m` - 其他功能

#### 重构主控制器 (优先级：高)
- [x] 重构 `XPRoomMoreMenuViewController.m` - 简化主逻辑
- [x] 集成工厂模式
- [x] 更新数据源创建逻辑
- [ ] 移除巨大的 switch 语句 (保留为 fallback)
- [ ] 进一步优化架构

#### 测试和优化 (优先级：中)
- [ ] 创建单元测试
- [ ] 集成测试
- [ ] 性能测试
- [ ] 代码审查

#### 文档和优化 (优先级：低)
- [ ] 编写重构文档
- [ ] 创建使用示例
- [ ] 进一步优化架构

## 当前遇到的问题
1. **编译环境问题**：UIKit 导入失败，可能是预编译头文件配置问题
2. **PIBaseModel 继承问题**：找不到 PIBaseModel 接口声明
3. **手动引用计数问题**：项目使用 MRC，需要调整属性修饰符

## 下一步计划
1. 解决编译环境问题
2. 完成基础设施类
3. 实现第一个具体的 Action 类
4. 在主控制器中集成测试

## 重构收益
- 代码行数：从 600 行减少到每个类 50-100 行
- 可维护性：每个类职责单一，易于理解和修改
- 可测试性：每个 Action 可以独立测试
- 可扩展性：新增功能只需添加新的 Action 类