我怎样才能更好地编写这个javascript循环

Question

我有这个 JS 代码

if (parseInt(counter.text(), 10) > 0) {
    counter.text(parseInt(counter.text(), 10) - 1);
}

if (parseInt(counter.text(), 10) < 1) {
    counter.attr('data-active', true);
}

这只是检查计数是否 > 0 然后递减它，如果递减后 == 0 然后隐藏它。

我正在寻找更好的方法来编写它。我总是陷入关于非高效编码的代码审查;)

Answer 1

我认为这将为您提供相同的行为，而无需花费太多时间进行解析。

var value = parseInt(counter.text(), 10);
if(value > 0) {
    value--;
    counter.text(value);
}
if(value < 1) {
    counter.attr('data-active', true);
}

• 创建一个变量来保存解析结果。这样可以减少代码重复，并避免重复执行相同的工作。
• 您在检查值之前减少了在 counter.text() 中找到的值，因此我正在减少变量的值以保持相同的行为。

请注意，您可能可以更好地提高 循环 的性能，但您尚未将循环包含在 OP 中。