前一篇的功能已经正常,代码很少,很简单。但是还有不少错误。用jslint扫描了一下,有下列错误:
1.window变量要用注释说明,严格来说这不是错误,是jslint的问题
2.parseInt第二个参数必须指定10,代表10进制
3.parsetInt里面的dragObj应该是window.dragObj
4.init函数应该放在最后,因为它用到的几个函数要在前面定义
5.使用use strict语句严格检查
6.多了两个空格
再打开Chrome的调试模式(F12),看到很多error在mouseMove中,原来init函数里面忘记初始化对象为null了。
加上后错误消失。
好,现在代码漂亮多了。
/*global window */
function mouseDown(e) {
'use strict';
window.dragObj = e.currentTarget;
if (window.dragObj !== null) {
window.clickLeft = window.event.x - parseInt(window.dragObj.style.left, 10);
window.clickTop = window.event.y - parseInt(window.dragObj.style.top, 10);
window.dragObj.style.zIndex += 1;
}
}
function mouseStop() {
'use strict';
window.event.returnValue = false;
}
function mouseMove() {
'use strict';
if (window.dragObj !== null) {
window.dragObj.style.left = window.event.x - window.clickLeft;
window.dragObj.style.top = window.event.y - window.clickTop;
}
}
function mouseUp() {
'use strict';
window.dragObj = null;
}
function init() {
'use strict';
window.dragObj = null;
window.document.onmousemove = mouseMove;
window.document.onmouseup = mouseUp;
window.document.ondragstart = mouseStop;
}