代码review记录
| 项目 | 提出日期 | 解决日期文件 | 问题 | line | 解决方法 |
|---|---|---|---|---|---|
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 参数表明是否需要传 | 全局代码 | 表明参数的注释 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | url 链接 注意位置 | 全局代码 | 统一的文件来存放URL |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 数据模型封装 | 全局代码 | 网络数据使用封装模型来处理数据 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 字典 value nil 问题 加下判空 | 全局代码 | 对字典值的使用注意判空处理 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 数据处理 | 全局代码 | 数据处理方面需要注意不要重复的创建无用对象和判断 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | if 等 语句判断 | 全局代码 | if使用注意使用的判断是否合理和正确 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 数据处理放在单独类 不要在controller | 全局代码 | 统一数据处理类来管理 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 遍历 找到对应数据 记得break | deviceviewcontroller 866行 | 使用break跳出 |
| 魔法家lite（ios） | 2018/12/29 | 2019/1/4 | 控制返回获取对应pwr 判断 | deviceviewcontroller 868 | 要增加判断如果没有pwr以后下面取值的逻辑判断 |
###新版大韦项目（iOS） 项目 | 提出日期|解决日期文件|问题|line|解决方法 —|—|—|—|—|—| 新版大韦(ios)| 2019/1/9|2019/1/9|配网成功后直接从sdk移除设备|BLConfigOneViewController 124|配网SDK返回成功后，马上移除SDK中的设备| 新版大韦(ios)| 2019/1/9|2019/1/9|-2014 设置常量表示|全局代码|用宏表示-2014| 新版大韦(ios)| 2019/1/9|2019/1/9|添加家庭成功showmessage为空|BLSelectRoomViewController 126行|增加创建成功提示| 新版大韦(ios)| 2019/1/9|2019/1/9|默认家庭问题（无家庭，家庭重名）|全局代码|重新设计交互| 新版大韦(ios)| 2019/1/9|2019/1/9|修改家庭名称失败，需要把家庭名称再设置回|BLFamilyRoomViewController 107行|修改失败后，家庭名称改回原来的值| 新版大韦(ios)| 2019/1/9|2019/1/9|名称判断空在重名判断之前|BLAddRoomViewController 59行|修改判断顺序|