refactor: Формат хранения кирпичей в классе Game переписан с вложенного массива на обычный массив. Входные данные для значений каждого кирпича берутся из карты уровней

src/entities/brick/layBricks/layBricks.js

@@ -1,37 +1,35 @@
 import { Brick } from '../brick';
 
 /**
- * Ложит кирпичи по заданым размерам
- * @param {number} columnAmount количество кирпичей по оси X, неотрицателное, целое число
- * @param {number} rowAmount количество кирпичей по оси Y, неотрицателное, целое число
+ * Создает массив кирпичей проинициализированных значениями в зависимости от расположения на карте уровня
+ * @param {number[][]} levelMap карта уровня в формате массива
  * @param {number} brickWidth ширина кирпича, неотрицателное число
  * @param {number} brickHeight длина кирпича, неотрицателное число
- * @returns {Brick[][]} массив с массивами кирпичей
+ * @returns {Brick[]} массив кирпичей
  */
-export function layBricks(columnAmount, rowAmount, brickWidth, brickHeight) {
+export function layBricks(levelMap, brickWidth, brickHeight) {
   try {
-    const someArgsArentNumbers = [columnAmount, rowAmount, brickWidth, brickHeight].some(
-      (arg) => typeof arg !== 'number',
-    );
-    if (someArgsArentNumbers) {
-      throw new Error('Параметры кладки кирпичей должны являться числами');
+    if (!(Array.isArray(levelMap) && levelMap.every(Array.isArray))) {
+      throw new Error('Значение карты уровня должно являться вложенным масивом чисел глубины 2');
     }
 
-    const someArgsLessThanZero = [columnAmount, rowAmount, brickWidth, brickHeight].some((arg) => arg < 0);
-    if (someArgsLessThanZero) {
-      throw new Error('Параметры кладки кирпичей должны являться положительными целыми числами');
+    if (!(typeof brickWidth === 'number' && typeof brickHeight === 'number')) {
+      throw new Error('Значения ширины и высоты кирпича должны являться числами');
     }
 
-    const rowsOrColumnsAreFloat = [columnAmount, rowAmount].some((amount) => amount % 1 !== 0);
-    if (rowsOrColumnsAreFloat) {
-      throw new Error('Размеры рядов и колонок должны являться целыми числами');
+    if (brickWidth < 0 || brickHeight < 0) {
+      throw new Error('Значения ширины и высоты кирпича должны являться положительными целыми числами');
     }
 
-    const bricks = Array.from({ length: rowAmount }).map((_, rowIndex) =>
-      Array.from({ length: columnAmount }).map(
-        (_, columnIndex) => new Brick(columnIndex * brickWidth, rowIndex * brickHeight, brickWidth, brickHeight),
-      ),
-    );
+    const bricks = [];
+
+    for (let i = 0; i < levelMap.length; i++) {
+      for (let j = 0; j < levelMap[i].length; j++) {
+        if (levelMap[i][j] !== 0) {
+          bricks.push(new Brick(j * brickWidth, i * brickHeight, brickWidth, brickHeight));
+        }
+      }
+    }
 
     return bricks;
   } catch (err) {

src/entities/brick/layBricks/layBricks.test.js

@@ -5,54 +5,63 @@ describe('layBricks', () => {
   it('Возвращает корректное значение в случае неверного типа параметра', () => {
     const wrongType = 'wrong';
 
-    expect(layBricks(wrongType, 1, 1, 1)).toBeNull();
-    expect(layBricks(1, wrongType, 1, 1)).toBeNull();
-    expect(layBricks(1, 1, wrongType, 1)).toBeNull();
-    expect(layBricks(1, 1, 1, wrongType)).toBeNull();
+    expect(layBricks(wrongType, 1, 1)).toBeNull();
+    expect(layBricks(1, wrongType, 1)).toBeNull();
+    expect(layBricks(1, 1, wrongType)).toBeNull();
   });
 
   it('Возвращает корректное значение в случае неверных размеров кирпича', () => {
     const wrongWidth = -1;
     const wrongHeight = -1;
 
-    expect(layBricks(1, 1, wrongWidth, 1)).toBeNull();
-    expect(layBricks(1, 1, 1, wrongHeight)).toBeNull();
+    expect(layBricks(1, wrongWidth, 1)).toBeNull();
+    expect(layBricks(1, 1, wrongHeight)).toBeNull();
   });
 
-  it('Возвращает корректное значение в случае неверных значений рядов и колонок', () => {
-    const negativeRow = -1;
-    const negativeColumn = -1;
-    const decimalRow = 1.5;
-    const decimalColumn = 1.5;
+  it('Возвращает корректное значение в случае некоректной карты уровня', () => {
+    const levelMap = ['brick', 'brick'];
 
-    expect(layBricks(negativeRow, 1, 1, 1)).toBeNull();
-    expect(layBricks(1, negativeColumn, 1, 1)).toBeNull();
-    expect(layBricks(decimalRow, 1, 1, 1)).toBeNull();
-    expect(layBricks(1, decimalColumn, 1, 1)).toBeNull();
+    expect(layBricks(levelMap, 1, 1)).toBeNull();
   });
 
   it('Возвращает массив корректных размеров', () => {
-    const columnAmount = 10;
-    const rowAmount = 5;
+    const levelMap = [
+      [1, 1, 1, 1, 1],
+      [1, 1, 1, 1, 1],
+      [1, 1, 1, 1, 1],
+    ];
 
-    const bricks = layBricks(columnAmount, rowAmount, 1, 1);
-    expect(bricks.length).toEqual(rowAmount);
-    for (const row of bricks) {
-      expect(row.length).toEqual(columnAmount);
-    }
+    const bricks = layBricks(levelMap, 1, 1);
+    expect(bricks.length).toBe(levelMap.flat().length);
+
+    const emptyLevelMap = [
+      [0, 0, 0, 0, 0],
+      [0, 0, 0, 0, 0],
+      [0, 0, 0, 0, 0],
+    ];
+    const emptyBricksArray = layBricks(emptyLevelMap, 1, 1);
+    expect(emptyBricksArray.length).toBe(0);
   });
 
   it('Задает корректные координаты кирпичам', () => {
-    const bricks = layBricks(3, 3, 1, 1);
-    const columnAmount = 10;
-    const rowAmount = 5;
+    const levelMap = [
+      [1, 1],
+      [1, 1],
+    ];
+    const brickWidth = 1;
+    const brickHeight = 1;
 
-    const brickss = layBricks(columnAmount, rowAmount, 1, 1);
+    const bricks = layBricks(levelMap, brickWidth, brickHeight);
+    let index = 0;
 
-    for (let i = 0; i < rowAmount; i++) {
-      for (let j = 0; j < columnAmount; j++) {
-        expect(brickss[i][j].x).toEqual(j);
-        expect(brickss[i][j]?.y).toEqual(i);
+    for (let i = 0; i < levelMap.length; i++) {
+      for (let j = 0; j < levelMap[i].length; j++) {
+        if (levelMap[i][j] === 0) {
+          continue;
+        }
+        expect(bricks[index].x).toBe(j * brickWidth);
+        expect(bricks[index].y).toBe(i * brickHeight);
+        index++;
       }
     }
   });