Memory Pair Game #344

zhenyakornilov · 2022-08-29T11:36:21Z

Memory Pair Game

Demo | Code base

The code is submitted in a dedicated feature branch.

Only code files are submitted.

Please, review.

zhenyakornilov · 2022-09-07T20:30:06Z

Added some changes with fixes and improvements

zhenyakornilov · 2022-09-11T10:19:27Z

Feedback collected, issues fixed accordingly to remarks by other students.

Check here

stale · 2022-09-25T13:13:08Z

This issue has been automatically marked as stale because there were no activity during last 14 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

А. Чому так?
Найбільш розповсюджена причина: Студент не реагує на коментарі змінами коду і не задає запитань через брак часу або зміну життєвих пріоритетів. Покинуті піари відволікають менторів. Коли у студента з'явиться час, він/вона зможе перевідкрити той самий піар і продовжити роботу.

Б. Що робити, якщо в піарі нема оновлень, оскільки не зрозуміло, що треба зробити?
Варіант 1. Задати питання в самому PR.
Варіант 2. Задати питання в студентському чаті.

В. А якщо я все зробив(ла) і це ментор не рев'юває мої зміни?

Переконайся, що ти відреагував(ла) на всі коментарі або кодом, або запитаннями, або відповідями. Напиши в PR і в чаті, що чесно вважаєш, що все зроблено і попроси повторне рев'ю. Якщо щось не зрозуміло, задай запитання.
Реагуй на коментарі як менторів, так і інших учасників, включаючи ботів.
Не ігноруй прохання типу * "Let's do some self-checks ..." * "Go through the checklist below..." * "mark fulfilled requirements..." * "if you would silently ignore this recommendation, a mentor may think that you are still working on fixes"
навіть якщо вони написані ботом. Боти помічники і ментори покладаються на те, що прохання і пропозиції бота дотримуються.
Не лінись піти по лінках в коментарях, погуглити термінологію та скористатись Google Translate.
Можливо, у менторів склалися інші пріоритети через роботу, сімейні обставини і т.п. В такому разі, якщо ти зробив(ла) рекомендоване вище, то волай в чаті, що PR позначений stale, наче, все зроблено, а ментори чомусь не реагують - рятуйте!

Г. Хіба недостатньо того, що я додав(ла) коміт із змінами?
Часто буває так, що бачиш новий коміт, ідеш перевіряти, змін багато, доводиться перечитувати весь код. А потім з'ясовується, що одна невеличка зміна "відкладена на потім" чи з'являється ще один коміт і знов треба перечитувати все. Любіть нас, спілкуйтеся з нами - і ми відповімо повною взаємністю.

Традиційна пропозиція: задай питання по вищенаписаному в студентському чаті.

zhenyakornilov · 2022-09-25T16:08:19Z

Self checks done, please, remove stale label.

kasionio

Good job! Please take a look at the comments.
Small tip - if we want to compare two elements we can simply use
function isElementsMatched (first, second) { return first.name === second.name }

kasionio · 2022-10-02T16:09:55Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+  currentCardPair: [],
+  totalTries: 0,
+};
+const allGameCards = shuffleCards([...cards, ...cards]);


Seems allCards or shuffledCards will be enough here, to make it consistent with shuffleCards

kasionio · 2022-10-02T16:12:13Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+renderCards(allGameCards, cardsMenu);
+
+function shuffleCards(cardsArr) {
+  for (let i = cardsArr.length - 1; i > 0; i--) {


Looks sort array method is the best choice here

Did You mean that Fisher–Yates shuffle is overkill to shuffle game cards? :)

We have a built-in sort method in JS for this purpose, no need to reinvent the wheel, also it is much better for code readability.

But you can leave it, I changed my mind :))

kasionio · 2022-10-02T16:15:37Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+}
+
+function sleep(milliseconds) {
+  return new Promise((resolve) => setTimeout(resolve, milliseconds));


why do we need async operation here?

I wanted to create some kind of custom delay function instead of repeating setTimeout all the time. This is solution I came up with and thats why I use async - await later. I haven't figured out how to do that in another way yet (except using setTimeout in every place where I currently used my sleep() function.

kasionio · 2022-10-02T16:16:31Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+  cardsList.innerHTML = cardsHTML;
+}
+
+function sleep(milliseconds) {


I guess cards won't sleep, please try rename a function :)

kasionio · 2022-10-02T16:19:22Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+}
+
+function checkCardsMatch() {
+  return states.firstFlippedCard.isEqualNode(states.secondFlippedCard);


why so a complicated approach? I guess better compare name of the cards, even for performance reasons

kasionio · 2022-10-02T16:20:23Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+}
+
+function restartGame() {
+  location.reload();


clumsy method :))

kasionio · 2022-10-02T16:24:53Z

submissions/zhenyakornilov/memory-pair-game/js/index.js

+
+let cardsBlocked = false;
+
+cardsMenu.addEventListener("click", async function flipCards({ target }) {


please explain why do you need async here

I'm not sure I unterstood remark properly. Without async-await it all breaks and does not work as expected.

zhenyakornilov · 2022-10-02T19:33:44Z

I tried to account all the recommendations and remarks, but some questions still left (answered in comments above).

stale · 2023-04-14T17:03:31Z

This issue has been automatically marked as stale because there were no activity during last 14 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

А. Чому так?
Найбільш розповсюджена причина: Студент не реагує на коментарі змінами коду і не задає запитань через брак часу або зміну життєвих пріоритетів. Покинуті піари відволікають менторів. Коли у студента з'явиться час, він/вона зможе перевідкрити той самий піар і продовжити роботу.

Б. Що робити, якщо в піарі нема оновлень, оскільки не зрозуміло, що треба зробити?
Варіант 1. Задати питання в самому PR.
Варіант 2. Задати питання в студентському чаті.

В. А якщо я все зробив(ла) і це ментор не рев'юває мої зміни?

Переконайся, що ти відреагував(ла) на всі коментарі або кодом, або запитаннями, або відповідями. Напиши в PR і в чаті, що чесно вважаєш, що все зроблено і попроси повторне рев'ю. Якщо щось не зрозуміло, задай запитання.
Реагуй на коментарі як менторів, так і інших учасників, включаючи ботів.
Не ігноруй прохання типу * "Let's do some self-checks ..." * "Go through the checklist below..." * "mark fulfilled requirements..." * "if you would silently ignore this recommendation, a mentor may think that you are still working on fixes"
навіть якщо вони написані ботом. Боти помічники і ментори покладаються на те, що прохання і пропозиції бота дотримуються.
Не лінись піти по лінках в коментарях, погуглити термінологію та скористатись Google Translate.
Можливо, у менторів склалися інші пріоритети через роботу, сімейні обставини і т.п. В такому разі, якщо ти зробив(ла) рекомендоване вище, то волай в чаті, що PR позначений stale, наче, все зроблено, а ментори чомусь не реагують - рятуйте!

Г. Хіба недостатньо того, що я додав(ла) коміт із змінами?
Часто буває так, що бачиш новий коміт, ідеш перевіряти, змін багато, доводиться перечитувати весь код. А потім з'ясовується, що одна невеличка зміна "відкладена на потім" чи з'являється ще один коміт і знов треба перечитувати все. Любіть нас, спілкуйтеся з нами - і ми відповімо повною взаємністю.

Традиційна пропозиція: задай питання по вищенаписаному в студентському чаті.

zhenyakornilov added 3 commits August 29, 2022 14:31

Add all task files for review

a0c5453

Formatting

5eb6a29

Remove duplicate DOM call

fa0f577

OleksiyRudenko added the task-MPG label Aug 29, 2022

Formatting, some improvements and fixes

5227049

zhenyakornilov added 2 commits September 8, 2022 19:41

Fix bug - prevent selecting back image of the card

b081f7d

Webkit disable drag image

45869ee

stale bot added the 💤 Stale label Sep 25, 2022

stale bot removed the 💤 Stale label Sep 25, 2022

OleksiyRudenko added UAT-done Student confirmed User Acceptance Tests are done and collected feedback is processed self-check-done Student confirmed that self-checks against requirements/common-mistakes are done Stage0.1 labels Oct 1, 2022

kasionio self-assigned this Oct 2, 2022

kasionio self-requested a review October 2, 2022 16:05

kasionio requested changes Oct 2, 2022

View reviewed changes

zhenyakornilov added 2 commits October 2, 2022 21:49

Fix mistakes

b5dcd32

Rename function

0d0ddd4

zhenyakornilov requested a review from kasionio October 2, 2022 19:34

Fix shuffle cards on restart game

0a1f484

stale bot added the 💤 Stale label Apr 14, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Memory Pair Game #344

Memory Pair Game #344

zhenyakornilov commented Aug 29, 2022 •

edited

Loading

zhenyakornilov commented Sep 7, 2022

zhenyakornilov commented Sep 11, 2022 •

edited

Loading

stale bot commented Sep 25, 2022

zhenyakornilov commented Sep 25, 2022

kasionio left a comment

kasionio Oct 2, 2022

kasionio Oct 2, 2022

zhenyakornilov Oct 2, 2022 •

edited

Loading

kasionio Oct 2, 2022

kasionio Oct 2, 2022

kasionio Oct 2, 2022

zhenyakornilov Oct 2, 2022

kasionio Oct 2, 2022

kasionio Oct 2, 2022

kasionio Oct 2, 2022

kasionio Oct 2, 2022

zhenyakornilov Oct 2, 2022

zhenyakornilov commented Oct 2, 2022 •

edited

Loading

stale bot commented Apr 14, 2023


		let cardsBlocked = false;

		cardsMenu.addEventListener("click", async function flipCards({ target }) {

Memory Pair Game #344

Are you sure you want to change the base?

Memory Pair Game #344

Conversation

zhenyakornilov commented Aug 29, 2022 • edited Loading

Memory Pair Game

zhenyakornilov commented Sep 7, 2022

zhenyakornilov commented Sep 11, 2022 • edited Loading

stale bot commented Sep 25, 2022

zhenyakornilov commented Sep 25, 2022

kasionio left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

zhenyakornilov Oct 2, 2022 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

zhenyakornilov commented Oct 2, 2022 • edited Loading

stale bot commented Apr 14, 2023

zhenyakornilov commented Aug 29, 2022 •

edited

Loading

zhenyakornilov commented Sep 11, 2022 •

edited

Loading

zhenyakornilov Oct 2, 2022 •

edited

Loading

zhenyakornilov commented Oct 2, 2022 •

edited

Loading