Правильно ли я выстроил Promise?

Question

[justifycontent]

Здравствуйте, подошел к изучению Promise, подскажите пожалуйста правильно ли я выстроил Promise и можно ли его записать как-нибудь короче? Может быть можно не возвращать каждый раз new Promise?
Код:

console.log('Запрос данных...');

const req = new Promise((resolve, reject) => {

    setTimeout(() => {
        console.log('Подготовка данных...');

        const data = {
            name: 'Personal computer',
            model: 'MSI',
            price: 120000,
            weightKG: 10,
        };

        resolve(data);
    }, 2000);
}).then((data) => {
    return new Promise ((resolve, reject) => {

        setTimeout(() => {
            console.log('Обработка данных...');
            data.status = 'order';
            data.discount = '20%';
            data.price -= 120000 * parseInt(data.discount, 10) / 100;

            resolve(data);
        }, 2000);
    });
}).then((data) => {
    return new Promise((resolve, reject) => {

        setTimeout(() => {
            console.log('Выписка чека...');
            data.check = `№${Math.floor(Math.random() * 100000000)}`;

            resolve(data);
        }, 2000);
    });
}).then((data) => {
    return new Promise((resolve, reject) => {

        setTimeout(() => {
            console.log(`Данные получены:\n1.Наименование: ${data.name}\n2.Модель: ${data.model}\n3.Цена: ${data.price}\n4.Вес: ${data.weightKG}\n5.Статус: ${data.status}\n6.Скидка: ${data.discount}\n7.Чек: ${data.check}\n`);
            resolve();
        }, 2000);
    });
}).then(() => {
    setTimeout(() => {
        console.log('...:::С новым годом!:::...');
    }, 2000);
});

Answer 1

[Aetae]

Технически правильно, но база программирования - вынесение повторяющихся действий в функции, условно как-то так:

function wait(delay) {
  let promise;
  const fn = function(result) {
    return promise || (promise = new Promise(resolve => setTimeout(resolve, delay, result)));
  };
  fn.then = resolve => fn().then(resolve); // использование функции как thennable
  return fn;
}

const req = wait(2000)
.then(() => {
  console.log('Подготовка данных...');

  const data = {
    name: 'Personal computer',
    model: 'MSI',
    price: 120000,
    weightKG: 10,
  };

  return data;
})
.then(wait(2000))
.then(data => {
  console.log('Обработка данных...');
  data.status = 'order';
  data.discount = '20%';
  data.price -= 120000 * parseInt(data.discount, 10) / 100;

  return data;
})
.then(wait(2000))
.then(data => {
  console.log('Выписка чека...');
  data.check = `№${Math.floor(Math.random() * 100000000)}`;

  return data;
})
.then(wait(2000))
.then(data => {
  console.log(`Данные получены:
1.Наименование: ${data.name}
2.Модель: ${data.model}
3.Цена: ${data.price}
4.Вес: ${data.weightKG}
5.Статус: ${data.status}
6.Скидка: ${data.discount}
7.Чек: ${data.check}`);
})
.then(wait(2000))
.then(() => {
  console.log('...:::С новым годом!:::...');
});

Понятно что у вас там не просто setTimeout, ну так и выносить можно что угодно.)

Comments

[justifycontent]

Спасибо вам большое, если у вас есть немного времени, не могли бы ответить, что делает эта функция?:

function wait(delay) {
  let promise;
  const fn = function(result) {
    return promise || (promise = new Promise(resolve => setTimeout(resolve, delay, result)));
  };
  fn.then = resolve => fn().then(resolve); // использование функции как thennable
  return fn;
}

[Aetae]

justifycontent, по сути у неё 2 цели, одна:

function wait(delay) {
  return function(result) {
    return new Promise(resolve => setTimeout(resolve, delay, result)));
  }
}

При вызове создаёт новую функцию, которая создаёт промис с таймаутом:

function(result) {
    return new Promise(resolve => setTimeout(resolve, delay, result)));
}

замыкая значение delay,
Это нужно чтобы прямо передавать эту функцию в then:
.then(wait(2000))
в данном случае равносильно:

.then(function(result) {
    return new Promise(resolve => setTimeout(resolve, 2000, result)));
})

Вторя часть, обеспечиваемая строчками fn.then = resolve => fn().then(resolve);, добавляет взвращаемой функции собственный метод then, который на самом деле просто вызывает саму эту возвращаемую функцию. Это нужно, чтобы можно было делать так:
wait(2000).then(...)
или так:

.then(data => {
  return wait(2000)
})

[justifycontent]

Aetae, Блин, спасибо огромное, желаю тебе успехов в 2к21. Я до замыканий не дошел еще, поэтому в замешательстве )

[justifycontent]

Aetae, Но уже становится все ясно, спасибо большое тебе, еще раз

[Aetae]

Ну да, такие трюки с 2 в 1 лучше начинающему не делать.)
Лучше завести 2 отдельные функции по 1 на задачу:

function waitFunction(delay) {
  return function(result) {
    return new Promise(resolve => setTimeout(resolve, delay, result)));
  }
}

function waitPromise(delay) {
    return new Promise(resolve => setTimeout(resolve, delay)));
}

[justifycontent]

Aetae, Ты лучший )