Зачем при создании промиса нужно передавать resolve?

Question

[zlodiak]

Помогите пожалуйста понять почему при создании промиса нужно передавать в функцию-executor аргумент resolve:

const p = new Promise((resolve) => resolve(2));
p.then(resp => {
	console.log(resp)
});

Это рабочий и правильный код.

Мне непонятно почему нельзя записать так:

const p = new Promise(() => resolve(2));
p.then(resp => {
	console.log(resp)
});

Ведь аргумент resolve непонятно откуда появляется(соответственно непонятно чему он равен). Явно его никто не передаёт. Однако без него код не работает...

Answer 1

[Павло Пономаренко]

Думаю, понятнее будет, если записать функцию в полном виде.

const p = new Promise(
  function (resolve) {
    resolve(2)
  }
);

// vs

const p = new Promise(
  function () {
    resolve(2)
  }
);

Вы видите, что в первом случае resolve - это аргумент коллбека. А во втором - вообще непонятно откуда берется. Вот вы написали как альтернативу второй код.

Comments

[Павло Пономаренко]

Как вариант - мог бы существовать какой-то такой интерфейс, ведь необходимо указать, какой именно промис необходимо резолвить.

const p = new Promise(
  function () {
    p.resolve(2)
  }
);

[zlodiak]

Павло Пономаренко, но ваш код не работает. Это выражается в том, что консоль выводит:

[Павло Пономаренко]

zlodiak, да, я сказал, что такой API МОГ бы существовать.

[zlodiak]

Павло Пономаренко, А не могли бы вы привести полный код. Пусть в нём будут любые упрощения и имитации. Мне именно и непонятно что откуда приходит. И ещё, я тупой)

[Павло Пономаренко]

zlodiak, давайте так. Попробуйте написать свою реализацию промисов. Как можно проще, но похоже на то, что есть в JS. Киньте ее сюда. Это поможет вам понять. Если у вас не получится - я потом вам напишу. Но серьезно попробуйте.

[zlodiak]

Павло Пономаренко, ...вот что у меня получилось: JSFIDDLE

class Promise {

	constructor(callback) {
		this.state = null;
		callback(this.resolve.bind(this), this.reject.bind(this));
		return this;
	}

	resolve() {
		this.state = 'resolve';
	}

	reject() {
		this.state = 'reject';
	}

	then(result, error) {
		if (this.state === 'resolve') {
			result('it result!!');
			return this;
		} else if (this.state === 'reject') {
			error('it error!!');
			return new Error("error message");
		}
	}

}

const promise = new Promise((resolve, reject) => {
	resolve('is resolved');
});

promise.then(
	result => {
		console.log('result', result);
	},
	error => {
		console.log('error', error);
	}
)
.then(
	result => {
		console.log('result2', result);
	},
	error => {
		console.log('error2', error);
	}
);

[Павло Пономаренко]

zlodiak, во! Отличный результат (правда). Теперь вы поняли, откуда берутся resolve и reject в аргументах колбека, который передается в конструктор и почему нельзя их не указывать?

[Павло Пономаренко]

Как второе задание - подумайте, как дописать так, чтобы работал и следующий код:

const promise = new Promise((resolve, reject) => {
  setTimeout(function () {
    resolve('is resolved');
  }, 1000);
});

[zlodiak]

Павло Пономаренко, тогда от вас нужна такая подсказка - можно ли пользоваться библиотекой zoneJS или существуют более нативные способы решить проблему?

[Павло Пономаренко]

zlodiak, нет, конечно никаких библиотек. Подход очень простой - если промис еще state = null, то вы функции, которые приходят в then просто сохраняете в какой-то кеш. Когда промис резолвится или реджектится - собираете весь набор из кеша и выполняете. Дальше просто сразу выполняете то, что приходит.

Answer 2

[Dmitry Alekseev]

Я думаю, что следует почитать про промисы тут

Comments

[zlodiak]

тут я читал прежде чем писать сюда

[Dmitry Alekseev]

zlodiak, тогда, что именно не понятно? resolve нужен для разрешения промиса.

const p = new Promise(() => resolve(2));
p.then(resp => {
  console.log(resp)
});

Так писать нельзя, потому что resolve должен быть возвращен внутри Promise. Сам же Promise возвращает 2 метода resolve и reject. resolve инициирует переход в .then, а reject в .catch

[Dmitry Alekseev]

вот тут накидал просто пример

[Даша Циклаури]

Dmitry Alekseev, можно было ответить проще, что в данном случае функция resolve is undefined и нужно просто внутри скобок поставить resolve

const p = new Promise((resolve) => resolve(2));
p.then(resp => {
  console.log(resp)
});

[Даша Циклаури]

Dmitry Alekseev, а если человек пишет .then(()=>{}, err=>{}) то в .catch он не попадает. Так что ты обманул автора ;)

[Dmitry Alekseev]

Даша Циклаури, я тоже самое написал, просто другими словами. Я думаю, дело в том, что автор не совсем понимает, с чем это едят.

[Даша Циклаури]

Dmitry Alekseev, ты написал ровно то, что написал

resolve инициирует переход в .then, а reject в .catch

. в .catch ты не попадешь, если верно обработаешь во втором аргументе

[Dmitry Alekseev]

Даша Циклаури, я не могу согласится с такой записью, не нравится мне передавать error внутри resolve, отдельным параметром, когда для этого есть reject. Тем более, что если .then несколько, при такой записи, ошибки придётся отлавливать в каждом, а при reject, они все будут в одном месте.

[Даша Циклаури]

Dmitry Alekseev,

не нравится мне передавать error внутри resolve, отдельным параметром

внутри колбэка на resolve ты НЕ передаешь error. Как можно не увидеть что это 2 разных аргумента колбэка?
1й - колбэк на resolve
2й - колбэк на reject

const successCb = ()=>{};
const errorCb = ()=>{};

new Promise().then(successCb, errorCb);

[Dmitry Alekseev]

Даша Циклаури, видимо 2 часа ночи можно. Сути это не меняет. Автор не понимал, как работать с промисами - я написал. И я все равно считаю, что отлавливать reject внутри then - это антипатерн. Для этого есть catch.

Answer 3

[Владимир]

Ты когда хендлер для ивента на странице пишешь, ивент тоже берешь из аргумента, так и тут. Конструктор Промиса принимает в качестве аргумента функцию, и при вызове передает туда два аргумента: функцию resolve и функцию reject

let pr = new Promise(function(){
	console.log("Arguments", arguments);
})

Попробуй так написать и посмотри на вывод консоли.