2048 #1053
2048 #1053
Conversation
Dushkaaa-BabyBoss
commented
Nov 28, 2024
•
Dushkaaa-BabyBoss
commented
| this.status = 'idle'; // Початковий статус гри | ||
| } | ||
|
|
||
| moveLeft() { |
There was a problem hiding this comment.
it works even before I click "start" button - it should be fixed:
src/modules/Game.class.js
Outdated
| this.board[i] = newRow; | ||
| } | ||
|
|
||
| this.addRandomTile(); |
There was a problem hiding this comment.
You should not add a new tile if nothing changed:
Then I clicked left, as result - the new tile appeared:
|
|
||
| /** | ||
| * Resets the game. | ||
| */ | ||
| restart() {} | ||
| restart() { |
There was a problem hiding this comment.
after clicking restart, the button text is "Start", but new tiles appeared and I can play
src/modules/Game.class.js
Outdated
| } | ||
|
|
||
| moveLeft() { | ||
| this.checkLose(); |
There was a problem hiding this comment.
it should be done in after move - same as checkWin (no need to do it in each "move" method)
| .reverse() | ||
| .filter((value) => value !== 0); | ||
|
|
||
| for (let j = 0; j < newRow.length - 1; j++) { |
There was a problem hiding this comment.
code in this method is almost the same, but you do reverse before and after manipulations. Try to add reusable code to a separate function and reuse it instead of duplication
src/modules/Game.class.js
Outdated
| @@ -23,22 +23,201 @@ class Game { | |||
| constructor(initialState) { | |||
| // eslint-disable-next-line no-console | |||
| console.log(initialState); | |||
|
|
|||
| // Ініціалізуємо гральну дошку | |||
There was a problem hiding this comment.
Remove all comments
| // Ініціалізуємо гральну дошку |
