Почему в некоторых случаях карточки не переворачиваются?

Question

[mxkss]

Вот игра Суть игры открывать карточки и если на 2 карточках совпадает слово то убирать их, а если нет - развернуть их обратно. Не знаю почему но когда несколько карточек открываешь, то игра начинат багатся. В чем проблема? (С Классами только начал разбиратся, может быть в них проблема)

Comments

[mxkss]

Возможно поможет то, что счетчик openedCards становится отрицательным когда роисходит баг, я так понимаю он где-то не обнуляется

Answer 1

[Сергей delphinpro]

Я бы сделал проще логику.
Не стал бы отдельно вести учет открытых карточек.
У каждой карточки будут свойства: isOpen, isActive.
isOpen обозначает, открыта ли карточка.
isActive – возможность по ней кликнуть (то есть не скрытая в результате совпадения).
Теперь по каждому клику выполняем все проверки:

const openedCards = cards
  .filter(card => card.isActive) // работаем только с активными
  .filter(card => card.isOpen) // берем все открытые
if (openedCards.length == 2) { // открыто 2 карточки
  if (openedCards[0].text == openedCards[1].text) { // совпали
    openedCards.forEach(card => card.hide()); // скрываем
  } else { // не совпали
    openedCards.forEach(card => card.close()); // переворачиваем обратно
  }
}
if (cards.every(card => !card.isActive)) { // Если все скрыты
  alert('Game over');
}

Comments

[mxkss]

Спасибо, так реально прощ будет. А все же в том коде что я написал можна сделать поправку чтобы все работало? Или много логики нужно дописать?

[Сергей delphinpro]

mxkss, я не сильно вникал в код. логически вроде все верно, но возможно из-за задержки возникает ситуация с неправильным подсчетом.