Code Review 是一种通过复查代码,提高代码质量的过程。通过 Review 别人的代码,学习别人写的好的地方。别人 Review 自己的代码,别人可以指出自己的不足。
Code Review 有一堆检查点,我们不用严格地按照那检查点去检查。我们互相 Code Review 主要学习别人写的好的,指出别人写的不好的。
想被 Review 的,把代码地址回复在本文评论区~
所有活动见这里。
Code Review 是一种通过复查代码,提高代码质量的过程。通过 Review 别人的代码,学习别人写的好的地方。别人 Review 自己的代码,别人可以指出自己的不足。
Code Review 有一堆检查点,我们不用严格地按照那检查点去检查。我们互相 Code Review 主要学习别人写的好的,指出别人写的不好的。
想被 Review 的,把代码地址回复在本文评论区~
所有活动见这里。
本文标题:活动#4 来互相 Code Review 吧
本文链接:https://www.haomeiwen.com/subject/twpwtxtx.html
网友评论
1. 给img 赋 src 最好写在 img.onload 后面,否则在有的浏览器中可能会出问题。
2. 把默认配置 imgDefaultOptions 放到更前面,明显的位置。目前有点混在逻辑里。
3. img.onload 的处理函数做的事情有点多,可以考虑分成几个函数。
4. 功能上,可以加上,刮完的回调。我来fork个,用空来加加看
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 独立成一个文件
独立成文件可以缓存和复用。
一个用Vue.js 写的 todolist-> https://github.com/iamjoel/todolist
刷的2017 百度前端学院的题 -> https://github.com/iamjoel/baidu-ife-task/tree/master/src/2017