$ rootScope не обновляется с помощью forEach

Question

$ rootScope не обновляется с помощью forEach

0

Смешное поведение? Объект my menus содержит пункты меню, которые в этот момент кода равны = true. этот фрагмент работает на заводе.

angular.forEach($rootScope.menus, function(menu){
   menu = false;
   console.log(menu); // outputs false
})
console.log($rootScope.menus); //values are still equal to true

Казалось бы, хотя значение в цикле устанавливается в false, оно не применяется к $ rootScope или, скорее, не обновляется? Нужно ли применять новое значение?

С уважением,

Dbrandt 18 май 2016, в 03:28

Источник

0

Почему вы привязываете menus к $rootScope а не к экземпляру $ scope на уровне контроллера?
Haymaker87 18 май 2016, в 00:49
0

@ Haymaker87 в идеале это будет директива в любом случае. Не имеет значения, так как поведение сейчас под вопросом, а не архитектура. Мне просто любопытно, почему вышеупомянутое делает то, что делает.
Dbrandt 18 май 2016, в 00:51
0

Ладно, было просто любопытно. Я не уверен, что это хорошая практика или нет. В любом случае, я второй ответ Райана Дженкина.
Haymaker87 18 май 2016, в 00:59
0

@ Haymaker87 Причиной этого является экран загрузки, загружаемый во время поиска, который должен закрыть все открытые меню. Это должно быть сделано с помощью директивы, определяющей внешние клики, что, вероятно, является лучшей практикой. Буквально просто быстрое подтверждение концепции, чтобы показать поведение.
Dbrandt 18 май 2016, в 01:03

Показать ещё 2 комментария

Теги:

javascript

angularjs

foreach

rootscope

1 ответ

Ещё вопросы

Почему вы привязываете menus к $rootScope а не к экземпляру $ scope на уровне контроллера?
@ Haymaker87 в идеале это будет директива в любом случае. Не имеет значения, так как поведение сейчас под вопросом, а не архитектура. Мне просто любопытно, почему вышеупомянутое делает то, что делает.
Ладно, было просто любопытно. Я не уверен, что это хорошая практика или нет. В любом случае, я второй ответ Райана Дженкина.
@ Haymaker87 Причиной этого является экран загрузки, загружаемый во время поиска, который должен закрыть все открытые меню. Это должно быть сделано с помощью директивы, определяющей внешние клики, что, вероятно, является лучшей практикой. Буквально просто быстрое подтверждение концепции, чтобы показать поведение.

Ryan Jenkin · Accepted Answer · 2016-05-17T22-12-00.000Z

ForEach имеет копию переданного элемента массива (переданного по значению вместо переданного по ссылке). Для обновления исходного массива используйте $ rootScope.menus [i]:

angular.forEach($rootScope.menus, function (menu, i) {
    $rootScope.menus[i] = false;
});

Спасибо, это работает и действительно имеет смысл сейчас, когда я думаю об этом. Также спасибо, что ответили на вопрос, объясняя поведение.