在javascript中将常量移动到方法的顶部是一种好的做法吗?

在javascript中将常量移动到方法的顶部是一种好的做法吗?,javascript,coding-style,Javascript,Coding Style,在下面的代码片段中,我设置了 privates.RADIX = 10; 而不是在succeedinf代码中仅使用10。这是一个良好的做法还是肛门保持 function peakOut = function (elem, offset, delay, callback) { var privates = {}; // constants initialization privates.RADIX = 10; privates.GRAN_TIME = 15;

在下面的代码片段中,我设置了

privates.RADIX = 10;
而不是在succeedinf代码中仅使用
10
。这是一个良好的做法还是肛门保持

function peakOut = function (elem, offset, delay, callback) {
    var privates = {};

    // constants initialization

    privates.RADIX = 10;
    privates.GRAN_TIME = 15;
    privates.GRAN_DIST = 1;
    privates.UNITS = 'px';

    // privates initialization

    privates.el = elem;
    privates.start = parseInt($P.getComputedStyle(privates.el).getPropertyValue("top"),
            privates.RADIX);

    privates.status = 'down';
    privates.end = privates.start + offset;
    privates.current = privates.start;
    privates.id = null;

    (function next() {
        if ((privates.status === 'down') && (privates.current < privates.end)) {
            privates.current += privates.GRAN_DIST;
            privates.el.style.top = privates.current + privates.UNITS;
            if (!privates.id) {
                privates.id = $P.setInterval(next, privates.GRAN_TIME);
            }
        } else if ((privates.status === 'down') && (privates.current === privates.end)) {
            privates.status = 'up';
            $R.resetInterval(privates);
            $P.setTimeout(next, delay);
        } else if ((privates.status === 'up') && (privates.current > privates.start)) {
            privates.current -= privates.GRAN_DIST;
            privates.el.style.top = privates.current + privates.UNITS;
            if (!privates.id) {
                privates.id = $P.setInterval(next, privates.GRAN_TIME);
            }
        } else if ((privates.status === 'up') && (privates.current === privates.start)) {
            $R.resetInterval(privates);
            callback();
        }
    }());
};
function peakOut=函数(元素、偏移、延迟、回调){
var privates={};
//常量初始化
privates.RADIX=10;
privates.GRAN_时间=15;
privates.GRAN_DIST=1;
privates.UNITS='px';
//私人初始化
privates.el=元素;
privates.start=parseInt($P.getComputedStyle(privates.el).getPropertyValue(“top”),
根茎);
privates.status='down';
privates.end=privates.start+偏移量;
privates.current=privates.start;
privates.id=null;
(函数next(){
如果((privates.status=='down')&&(privates.currentprivates.start)){
privates.current-=privates.GRAN_DIST;
privates.el.style.top=privates.current+privates.UNITS;
如果(!privates.id){
privates.id=$P.setInterval(下一步,privates.GRAN_时间);
}
}else if((privates.status=='up')&&(privates.current===privates.start)){
$R.resetInterval(私人);
回调();
}
}());
};

我不相信这样做会提高JavaScript的性能,但为了可读性和可维护性,这无疑是一种很好的做法


分离常量是很好的,因为如果在多个位置使用该值,那么如果需要更新该值,只需在一个位置对其进行更改。它还可以作为一个信号,告诉其他程序员(或者你自己,将来)该值意味着什么,而不是代码中出现的任意数字。

一般来说,在编写代码时,你永远都不能太直截了当。我同意,还有什么其他方法可以让代码更强大?一般来说,你永远都不能太拘谨。虽然我同意@voithos的观点,但这是一个很好的做法,我必须说……这是一个主观问题,在被标记的边缘摇摇欲坠。@Swarm…谢谢。(在这里插入讽刺)。好吧,投票重新开放,smarty mcGee。