Вложенная петля - эта плохая практика

Question

Было бы плохой практикой выполнять такой цикл? Я перебираю все машины и добавляю текст «новый» к имени каждого цвета. Каждый автомобиль может иметь несколько цветов.

var cars = [ 
    { 
     company: "honda", 
     colors: [ 
      { 
       name="burntRed" 
      }, 
      { 
       name: "springGreen" 
      } 
     ] 
    }, 
    { 
     company: "ford", 
     colors: [ 
      { 
       name="burntOrange" 
      }, 
      { 
       name: "black" 
      } 
     ] 
    } 
]; 
     for (var c = 0; c < cars.length; p++) { 

      var car = cars[c]; 

      for (var i = 0; i < cars.colors.length; i++) { 
       car.color[i].name += 'new ' + car.color[i].name; 
      }; 
     }; 

Answer 1

Поскольку структура данных, в которой вы выполняете итерирование, вложен, я не вижу другого (более совершенного) способа архивирования вашей задачи.
Вложенные итерации не обязательно являются плохими, даже многие известные алгоритмы полагаются на них. Но вы должны быть крайне осторожны в том, что вы выполняете в самом глубоком цикле, поскольку эти команды будут выполняться очень часто.

Answer 2

Вы можете подумать о другом стиле, который должен пересекать массивы. Javascript предлагает множество методов циклирования arrays.

Вот пример предлагаемой вами функции, здесь вы можете определить более чистый стиль из-за не более необходимой индексной переменной.

Кстати, вложенные петли неизбежны.

var cars = [{ company: "honda", colors: [{ name: "burntRed" }, { name: "springGreen" }] }, { company: "ford", colors: [{ name: "burntOrange" }, { name: "black" }] }]; 
 

 
cars.forEach(function (car) { 
 
    car.colors.forEach(function (color) { 
 
     color.name = 'new ' + color.name; 
 
    }); 
 
}); 
 

 
document.write('<pre>' + JSON.stringify(cars, 0, 4) + '</pre>');

Answer 3

Немного поздно, но вложенные циклы, как правило, плохо, потому что их трудно проверить и рассуждать о. Я бы пошел более декларативный подход:

const appendStringToColor = (string) => (color) => ({ name: `${color.name} ${string}` }); 
const appendNewToColor = appendStringToColor('new'); 
const updateCarWithColor = (car) => ({ ...car, colors: car.colors.map(appendNewToColor) }); 

const updateCarsColors = (cars) => cars.map(updateCarWithColor);