Angular. Обязательно ли помещать код внутри $apply?

Question

[Егор]

Приветствую.

Есть некоторые места в коде, где необходимо использовать $apply. Чтобы не получать ненужные ошибки, проверяю не выполняется ли сейчас $apply или $digest if(!scope.$phase && !scope.$root.$phase). Из-за этого условия выполнять код внутри $apply становится не очень удобно, т.к приходится дублировать код.

if(!scope.$phase && !scope.$root.$phase){
scope.$apply(function(){
/* some code  */
});
}else{
/* some code */
}

Поэтому сейчас использую такую конструкцию

/* some code */
if(!scope.$phase && !scope.$root.$phase){
scope.$apply();

Какие потенциальные проблемы могут возникнуть, если вызывать $apply после выполнения кода?
Правильно ли я понимаю, что вызов внутри $apply актуален для асинхронного кода, а в случае последовательного выполнения проблем быть не должно?

Answer 1

[kazmiruk]

Так делать плохо https://github.com/angular/angular.js/wiki/Anti-Pa...
Можно сделать проще

$timeout(function() {
код
});

Эффект будет такой же. Ну и все это актуально для кода, который изменяет состояние извне.

Comments

[Егор]

в $timeout вызывается тот же $apply. не получу ли я "$digest already in progress" ?

[Егор]

И не совсем понимаю, что плохого в этом ? "it means your $scope.$apply() isn't high enough in the call stack"

[kazmiruk]

Егор: плохо - нарушение инкапсуляции. Т. е. вольный перевод фразы - не стоит лезть впереди очереди и пытаться прицепиться к текущему циклу, добавьтесь в конец очереди и ждите. Т. е. если не смущает нарушение принципов, которые дает ангуляр - то можно и так (правда читал еще мнение, что есть ненулевой шанс того, что после проверки $phase Вы можете увидеть, что цикл запущен, не запускать свой цикл, произойдет переключение контекста и цикл сразу завершается оставив Ваш код в одиночестве). Про $timeout - это обертка над setTimeout которая всегда запускается извне цикла ангуляра. Передавая задержку в 0 секунд Вы говорите таймауту, что нужно запуститься как только так сразу (но асинхронно). Это "сразу" наступает сразу после завершения цикла. Т. е. фактически Вы дожидаетесь своей очереди.

[Егор]

kazmiruk: заюзал в некоторых местах таймаут, все работает отлично. спасибо за подробное разъяснение.

Answer 2

[Валерий]

Поидее никакой разницы нет. Сам пользуюсь вторым способом уже довольно давно и ни разу небыло из-за этого проблем.

Answer 3

[lega]

Если ваш код не вызовет бесконечную рекурсию/циклы, то проблем не будет.
В отличие от $timeout, будет работать быстрее, т.к. с $timeout будет происходить +1 перерисовка, заново запускается $digest, ну и все это откладывается по времени. Хотя в большинстве случаев это все мелочи.

Comments

[lega]

Кстати с $timeout можно тоже получить рекурсию, а это ещё хуже, т.к. приложение будет просто тормозить и не выдавать ошибок.