Question

我对您对以下代码的看法感兴趣：

            this.skipAdmCharge = function (tot) {
                if (FR.rynTag.hasOwnProperty('prodfix')) {
                    var prodfix = FR.rynTag.prodfix;
                    for (var prop in prodfix) {
                        if (prodfix.hasOwnProperty(prop)) {
                            if (prodfix[prop].key === 'ADM') {
                                tot.amt -= prodfix[prop].price * prodfix[prop].quantity
                            }
                        }
                    }
                }
            }

以下这个：

            this.skipAdmCharge = function (tot) {
                var prodfix = FR.rynTag.prodfix;
                if (prodfix) {
                    for (var prop in prodfix)
                        if (prop && (prop.key === 'ADM'))
                           tot.amt -= prop.price * prop.quantity
                }
            }

你认为哪一个更好？如果是这样，为什么？

谢谢！

Answer 1

两者都没有多少不必要的支票..

第一个if (prodfix.hasOwnProperty(prop))的情况下的示例不是必需的如果是第二个，if (prop &&没有必要。

 this.skipAdmCharge = function (tot) {
    try {
        var prodfix = FR.rynTag.prodfix;
        if (prodfix) {
            for (var prop in prodfix)
                if (prop.key === 'ADM'))
                    tot.amt -= prop.price * prop.quantity
        }
    } catch (e) {
        console.log("failed to calculate total amount in skipAdmCharge" + e.message);
    }
}

PS：我也是JavaScript的新手！

javascript中的编码风格

1 个答案: