add task solution #426
add task solution #426
Conversation
src/scripts/Game.js
Outdated
| this.message.setMessage(this.MESSAGES.START); | ||
| } | ||
|
|
||
| load() { |
There was a problem hiding this comment.
The load() method returns this without doing anything, which seems unnecessary. If this is not being used, consider removing it.
src/scripts/Field.js
Outdated
| addTile(value) { | ||
| const emptyCells = []; | ||
|
|
||
| for (let row = 0; row < this.cells.length; row++) { |
There was a problem hiding this comment.
Consider using 'forEach' or 'map' instead of the 'for' loop for better readability and performance. You could use a nested 'forEach' loop to iterate over the 2D 'cells' array.
src/scripts/Field.js
Outdated
| } | ||
|
|
||
| hasAvailableMoves() { | ||
| for (let row = 0; row < this.cells.length; row++) { |
There was a problem hiding this comment.
Consider using 'forEach' or 'map' instead of the 'for' loop for better readability and performance. You could use a nested 'forEach' loop to iterate over the 2D 'cells' array.
src/scripts/Field.js
Outdated
|
|
||
| const rows = element.querySelectorAll('.field-row'); | ||
|
|
||
| for (let i = 0; i < rows.length; i++) { |
There was a problem hiding this comment.
Consider using 'forEach' or 'map' instead of the 'for' loop for better readability and performance. For instance, you can refactor this part like 'rows.forEach((row, i) => {...})'.
|
sorry, i thought for some reason that any number can appear) |
DEMO LINK