Как оптимизировать этот код?

Question

[Web Lizard]

var intervalTimer;
    intervalTimer = setTimeout((function() {
      return setTimeout((function() {
        console.log(1);
        return setTimeout((function() {
          console.log(2);
          return setTimeout((function() {
            return console.log(3);
          }), 1000);
        }), 1000);
      }), 1000);
    }), 1000);

Суть в том что после каждой альтерации должна выполнятся определенная программа, консоль логи я вывел просто для примера.
Запихнуть это все в setInterval здесь не вариант

Answer 1

[Risent Veber]

Чтобы избежать callback hell используйте promise и setTimeout. В таком случае код превращается в линейный:

new Promise(function(resolve){
  setTimeout(function() {
    console.log(1);
    resolve();
  }, 1000);
}).then(function() {
  let nextPromise = new Promise(function(resolve){
    setTimeout(function() {
      console.log(2);
      resolve();
    }, 1000);
  });
  return nextPromise;
}).then(function() {
  let nextPromise = new Promise(function(resolve){
    setTimeout(function() {
      console.log(3);
      resolve();
    }, 1000);
  });
  return nextPromise;
}).then(function() {
   setTimeout(function() {
      console.log(4);
   }, 1000);
});

Comments

[Web Lizard]

Не, мне это не подходит. Суть в том что после каждого таймаута должна выполнятся определенная программа.. консоль логи я просто для демонстрации того что там что то будет происходить написал

[Risent Veber]

Web Lizard: Понял изменил ответ.

[Ильдар Гизетдинов]

Risent Veber: какое то не читабельное решение)

[Risent Veber]

Ильдар Гизетдинов: очень даже читабильное - при беглом простмотре очень даже понятно что функция then делает, при этом степень влженности функций не превосходит 2, очень гибко - можно задать любые интервалы. Вообще promises в современных проектах де факто становится стандартом индустрии.

Answer 2

[Ильдар Гизетдинов]

var Steps = {
	0: function () {
		console.log(1);
	},
	1: function () {
		console.log(2);
	}
};

var counter = 0;
var intervalTimer = setInterval(function () {
	if (!Steps[counter]) {
		clearInterval(intervalTimer);
		return;
	}

	Steps[counter]();
	counter++;
}, 1000);

Comments

[Web Lizard]

Спасибо! А Steps - это прототипом называется, да?

[Ильдар Гизетдинов]

Web Lizard: В данном случае это просто объект, который содержит функции, можно использовать обычный массив, в данном случае это ничего не изменит

[Web Lizard]

Ильдар Гизетдинов: только вот что то конструкция
if (!Steps[counter]) {
clearInterval(intervalTimer);
}
не работает.. когда таймаут доходит до конца объекта ошибка возникает

Steps[counter] is not a function

[Ильдар Гизетдинов]

Web Lizard: а return куда вы дели?

Answer 3

[Кирилл]

Вы уверены, что setInterval вам не подойдет?

var state = 0;
var intervalTimer = setInterval(function () {
  switch(state){
    case 0:
      console.log(1);
      break;
    case 1:
      console.log(2);
      break;
    case 2:
      console.log(3);
      break;
    default:
      clearInterval(intervalTimer);
  }
  state++;
}, 1000);

Comments

[Web Lizard]

не подойдет, ибо значение таймера может быть разное от альтерации к альтерации

[Анатолий Сидоров]

Web Lizard: итерация, ну совсем не альтерация)