发表于: 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代码规范。
评论