function moveElement(elementID,final_x,final_y,interval) {
if (!document.getElementById) return false;
if (!document.getElementById(elementID)) return false;
var elem = document.getElementById(elementID);
if (elem.movement) {
clearTimeout(elem.movement);
}
if (!elem.style.left) {
elem.style.left = "0px";
}
if (!elem.style.top) {
elem.style.top = "0px";
}
var xpos = parseInt(elem.style.left);
var ypos = parseInt(elem.style.top);
var dist = 0;
if (xpos == final_x && ypos == final_y) {
return true;
}
if (xpos < final_x) {
var dist = Math.ceil((final_x - xpos)/10);
xpos = xpos + dist;
}
if (xpos > final_x) {
var dist = Math.ceil((xpos - final_x)/10);
xpos = xpos - dist;
}
if (ypos < final_y) {
var dist = Math.ceil((final_y - ypos)/10);
ypos = ypos + dist;
}
if (ypos > final_y) {
var dist = Math.ceil((ypos - final_y)/10);
ypos = ypos - dist;
}
elem.style.left = xpos + "px";
elem.style.top = ypos + "px";
var repeat = "moveElement('"+elementID+"',"+final_x+","+final_y+","+interval+")";
elem.movement = setTimeout(repeat,interval);
}
在此函数中,为什么声明变量 var dist = 0; 初学JavaScript,此处不理解,请各位老师指教,谢谢!
不声明也没啥影响
~,为了让看代码的人知道,这个变量存储的数据类型是数字~~
没有意义。
事实上,放在今天的角度看,贴出来的这段 JavaScript 跟屎一样。
我就随便批评几点:
dist = 0
确实没意义。多处 if 里面分别
var dist
是错误的。JavaScript 的变量只有函数作用域。字符串作为
setTimeout
的参数是非常不好的写法。parseInt
应该指定第二个参数。至少 JsLint 和 JsHint 都那么要求。不应该使用
==
,应该使用===
。JsLint 和 JsHint 都那么要求。每一祯都执行
getElementById
,非常效率低下。为什么不执行一次存到变量里,之后每一祯直接取?同样,每一祯取
elem.style.left
和elem.style.top
再parseInt
完全是浪费性能。直接放到变量里下一祯读好得多。做动画的话,
setTimeout
性能上不如requestAnimationFrame
。所以现在已经没有理由再用setTimeout
做动画了,除非是作为 fallback。