发表于: 2017-05-14 21:52:07

0 989


1.今天完成的事情

今天比较水,昨晚喝断片了,今天很难受,感觉身体被掏空。。。。

过来给师弟做codereview,

王帅js-task4

1. 文件命名不要用拼音,太low啦。
2.注释需要写的更详细些,方便后来人理解维护,例如文件开头可以写上这个文件更新时间,文件的用处是什么?
3.变量命名不是很规范,尽量不要用节点名字来作为名字,最好也不要使用array作为变量名。我的思路是JS会用到的比如object,array等这一些关键词,别拿来命名。
4.for (var i = 0;i<array.length;i++ ) {
    box[i].index = i;
    box[i].onclick = function() {
    picture[i].style.display = "block”;
    }
这一块其实可以使用闭包解决,看看怎么解决?
5.状态机其实可以使用,为每个对象新建一个状态机实例就可以。

6.大面积改变节点样式干嘛不在css里面写好呢?除非必要,少用js修改style,直接改class,应用不同的样式要好的多。 


刘思言js-task1

1.变量名最好驼峰大小写,如boxNum;
2.最好写个onload。
3.定义全局变量最好都放在代码顶部写。 


剩珺琢和小胖的明天看。

2.明天要做的事情

做前台的数据get,调试页面。

3.遇到的问题


4.收获

review前又看了下js代码规范。



返回列表 返回列表
评论

    分享到