前端 code review

1.demo1

• 1. 样式需要结构化， 公共化， 组件化。
• 2. 需要完善 readMe.md
• 3. 避免使用行类样式
• 4. Sonar 审查跟功能开发并行， 不能测试之后再做
• 5. 移动端用 Flex 布局, 少用浮动布局， 各机型之间兼容性更好
• 6. 页面实现之前考虑是否有必要组件化方便其它模块共用， 尽量不要一个页面一个组件包含所有业务逻辑
• 7. 路由名称跟页面名称匹配， 跟实现功能定义匹配

2.demo

• 代码复用
• 重复造轮子 => 沟通
• template 中复杂的表达式(vue)
• 函数职责不单一
• 文件内行数过多
• 复杂的三目运算符
• 删除不需要的代码
• 缺少必要的注释
• if else 过深
• 容错处理，非空判断