Как остановить выполнение setTimeout на 2 шаге?

Question

[viktorulyushev]

Есть скрипт,который проходится по всем .progress-page-tabs-col и добавляет им классы,можно ли как то сделать чтобы скрипт стопался на 2 шаге т.е на втором progress-page-tabs-col?пробовал так же через setinterval.И нормально ли написан скрипт?или можно было сделать проще?

function highlightcurrent(items, index) {

    items.removeClass("highlightcurrent");
    items.eq(index).addClass('highlight');  
    items.eq(index).addClass('highlightcurrent');   
    setTimeout(function() {
    	highlightcurrent(items, index + 1)
    }, 1000);
}

highlightcurrent($('.progress-page-tabs-col'), 0);

Answer 1

[alexalexes]

Можно.

function highlightcurrent(items, index) 
{
  if(index != 2) // выходим из функции, если достигли определенного индекса
  {
    items.removeClass("highlightcurrent");
    items.eq(index).addClass('highlight');  
    items.eq(index).addClass('highlightcurrent');   
    setTimeout(function()
    {
      highlightcurrent(items, index + 1)
    }, 1000);
  }
}

highlightcurrent($('.progress-page-tabs-col'), 0);

Comments

[viktorulyushev]

да точно,не догадался до простого :/

[Вася]

Плохая практика размножать точки выхода, return, лучше одно условие
if (index != 2)

[alexalexes]

Ладно, переправил.
А вообще, я не люблю когда большой кусок кода обернут в if-блок.
Если if несколько, потом догадайся по табуляции, к какому условию относится этот блок.
А так сделал return в нужном месте, и табуляция не страдает.