AngularJS JavaScript для предупреждения цикла, который будет работать не более одного раза

Question

Почему WebStorm предупреждает меня, что мой цикл for будет работать не более одного раза?

if (data[i].name) { 
    for (var j = 0; j < data[i].name.length; j++) { 
     hobby = $scope.users[data[i].name].hobby; 
     sport = $scope.users[data[i].name].sport; 
     education = $scope.users[data[i].name].education; 
     break; 
    } 
} 

В чем проблема? Что мне здесь не хватает?

Answer 1

Как уже упоминалось в комментариях, цикл выполняется только один раз максимум, так как вы выполните break выполнение после первой итерации.

Однако, глядя на код, вам может не понадобиться петля здесь вообще. Кажется, что все, что вы хотите сделать, это проверить, установлена ​​ли строка data[i].name, и если да, то получите некоторые свойства на основе этого, правильно?

Если это так, снимите петлю, так как он ничего не способствует публикуемую сниппета:

if (data[i].name) { 
    hobby = $scope.users[data[i].name].hobby; 
    sport = $scope.users[data[i].name].sport; 
    education = $scope.users[data[i].name].education; 
} 

data[i].name Если предположить, что это строка, data[i].name.length только даст вам длину строки.