Как я могу написать этот цикл 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(), поэтому я уменьшаю значение переменной, чтобы сохранить такое же поведение.

Обратите внимание, что вы, вероятно, может улучшить производительность вашего петли даже лучше, но вы не включили в свой цикл в ОП.