活动#4 来互相 Code Review 吧

网友评论

• 前端GoGoGo7:一个jQuery拖拽组件 https://github.com/timeTravelCYN/Drag- 预览地址 https://timetravelcyn.github.io/Drag-/
• 前端GoGoGo7:一个懂摄影的前端做到 一个全球领先的选片应用 https://github.com/cold-coder/lspp
• 
  Camol:欢迎review我最近整理的刮刮卡的小功能-->https://github.com/kanweiwei/ScratchCard

  Camol:@九彩拼盘

  前端GoGoGo7: 。提几个改进项目
  1. 给img 赋 src 最好写在 img.onload 后面，否则在有的浏览器中可能会出问题。
  2. 把默认配置 imgDefaultOptions 放到更前面，明显的位置。目前有点混在逻辑里。
  3. img.onload 的处理函数做的事情有点多，可以考虑分成几个函数。
  4. 功能上，可以加上，刮完的回调。我来fork个，用空来加加看
• 前端GoGoGo7:https://github.com/lifebeatiful/one-week-one-project/tree/master/week_1_%E5%A4%A7%E8%BD%AC%E7%9B%98%E6%8A%BD%E5%A5%96 的 review
  
  1 配置与代码分离。
  配置包括：包括一等级数量，二等奖数量，三等奖数量等。如
  ```
  const PRIZE_FRIST_NUM = 20 // 一等级数量
  const PRIZE_SECOND_NUM = 30 // 二等级数量
  const PRIZE_THIRD_NUM = 50 // 三等级数量
  
  var totalPrizeNum = PRIZE_FRIST_NUM + PRIZE_SECOND_NUM + PRIZE_THIRD_NUM
  var allPrize = []
  
  for(var i = 0; i < totalPrizeNum; i++) {
  allPrize.push(i + 1)
  }
  ```
  
  
  2 避免污染全局变量
  不要在 script 的顶层直接写， 如
  ```
  <script>
  var a = []
  ...
  </script>
  ```
  
  a 是全局变量。放在自执行函数可以解决这个问题
  ```
  ```
  <script>
  (function(){
  var a = []
  ...
  })()
  </script>
  ```
  
  3 方法名和变量名的优化
  不要起 a，b，c 这种。
  
  4 if ，else if 的优化
  ```
  if (c < 21 && c > 0) {
  price = 60;
  } else if (c > 20 && c < 51) {
  price = 180;
  } else if (c > 50 && c < 100) {
  }
  ```
  
  优化成
  
  ```
  if (c < 21 && c > 0) {
  price = 60;
  } else if (c < 51) {
  price = 180;
  } else if (c < 100) {
  }
  ```
  
  5 把 script 和 style 独立成一个文件
  独立成文件可以缓存和复用。
  
• 姚冰coding:有个能review代码的人，提高能力，付费也是值得的~
• 81f21a05501b:😊😊
• 前端GoGoGo7:欢迎大家来reveiw我的
  一个用Vue.js 写的 todolist-> https://github.com/iamjoel/todolist
  刷的2017 百度前端学院的题 -> https://github.com/iamjoel/baidu-ife-task/tree/master/src/2017