Как пофиксить одновременное выполнение кода?

Question

[Fragman]

Есть вот такой код, который выполняется сразу после подключения игрока. Столкнулся с проблемой, что если игроки подключаются одновременно, функция startGame не срабатывает. Статусы обоих игроков не успевают обновится. Что делать?

if(p1.id == userid)
	p1.status = 'connected';
else
	p2.status = 'connected';
if(status == 'created' && p1.status == 'connected' && p2.status == 'connected')
	startGame();

Answer 1

[Антон Неверов]

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

Отличное видео по теме: https://youtu.be/8cV4ZvHXQL4
А это статья, как пользоваться асинхронностью на практике: https://medium.com/devschacht/ire-aderinokun-async...
Ну и на YouTube есть канал CodeDojo - очень рекомендую для понимания основ JavaScript.

Comments

[Fragman]

Этот код идёт сразу после подключения socket.io, и да он асинхронный. Он выполняется одновременно у каждого пользователя, из-за этого функция не выполняется. Как это изменить?

Answer 2

[Boris Köln]

Подписаться (например, с помощью Promise) на изменение status, p1.status, p2.status.
При любом вышеуказанном событии выполнять проверки и запускать startGame.

Comments

[Fragman]

В моём коде получается, что эта проверка выполняется одновременно, так как статус изменяется в одно и тоже время, а это значит, что у одного пользователя статус противника будет другой всегда. Как выполнять проверку не одновременно, при одновременном подключении? Например, если я между if ставлю console.log(p1.status, p2.status) он выводит connected disconnected, disconnected connected

[Boris Köln]

Fragman, я уже ответил: выполнять проверки только по событию.

[Fragman]

Boris Korobkov, Событие приходит одновременно

[Boris Köln]

https://en.wikipedia.org/wiki/Race_condition