js_2048_game #61

IvanShulhan · 2022-02-14T12:41:24Z

varseniuk

Загалом - варіант робочий. Варто лише внести деякі незначні поправки, на які я вказав.

Також, зверни увагу, що можна було б реалізувати трошки з іншою логікою:

Створюєш метод (наприклад render), який відмальовує дані з масиву на ігровому полі після всіх операцій з матрицею
Створюєш метод, наприклад, rotateMatrix, розвертаєш матрицю в правильну сторону і тоді можеш завжди робити виклик лише одної функції (наприклад moveUp, тобто реалізацію логіки для одного напрямку) і працювати в одну сторону. Тобто ти розвертаєш матрицю в правильну сторону, мерджиш клітинки - розвертаєш матрицю в початкове положення і тим само скорочуєш код в чотири рази (мінус три функції)

varseniuk · 2022-03-14T15:32:32Z

src/scripts/main.js

+  });
+};
+
+const renderCube = (arr) => {


Suggested change

const renderCube = (arr) => {

const renderField = (arr) => {

Це скоріше не куб, а квадрат (square) або просто поле (field)

varseniuk · 2022-03-14T15:34:37Z

src/scripts/main.js

+  }
+};
+
+const areWin = () => {


Suggested change

const areWin = () => {

const winHandler = () => {

isWin буде правильніше

varseniuk · 2022-03-14T15:35:43Z

src/scripts/main.js

+    return;
+  }
+
+  if (ev.key === 'ArrowUp') {


switch...case тут підійде краще

solution

3fa3fa7

varseniuk approved these changes Mar 14, 2022

View reviewed changes

Provide feedback