javascript - function moveElement(elementID,final_x,final_y,interval)

Question

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，此处不理解，请各位老师指教，谢谢！

Answer 1

不声明也没啥影响

Answer 2

~,为了让看代码的人知道，这个变量存储的数据类型是数字~~

Answer 3

没有意义。

事实上，放在今天的角度看，贴出来的这段 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。