Использование getter снижает производительность приложения?

Question

[Владимир]

Хорошо ли в контроллерах использовать геттер, не запускает ли он лишний раз цикл дайджеста ?

let services = new WeakMap;

class SomeController {
  constructor(SomeService) {
    services.set(this, SomeService);
  }
  get svcData(){
    return services.get(this).getData();
  }
}
angular.module('app').controller('SomeController', SomeController);

class SomeService {
  constructor() {
  }
  getData(){
    return [
    ...
    ...
    ]
  }
}
angular.module('app').service('SomeService', SomeService);

Answer 1

[Николай]

Ну, а почему должен?
Если там не происходит ничего, что триггерит дайджест - то нет.
Можете кстати проверить, манкипачнув $digest как-то так:

var oldDigest = $scope.$digest;
$scope.$digest = function() { 
    console.log('digest fired'); 
    oldDigest.call(this); 
};

Comments

[Владимир]

Я просто не могу в голове сложить каким образом работает дайджест, по идее чтобы понять что ничего не поменялось, нужно дернуть функцию, а чтобы дернуть, нужно запустить дайджест нет ? Или Вотчеры работают всегда, независимо от дайджеста

Answer 2

[Сергей Протько]

новых циклов $digest конкретно геттер не вызывает.

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

Ну и это не говоря о том что на каждый $digest цикл будет дергаться ваш сервис. Вместо этого стоило дергуть сервис один раз и заменять данные в контроллере только тогда когда нужно.

Если брать современные подходы, то мы вообще должны вынести все получение состояния из контроллеров (компонентов) наружу, например в ресолверы и мэпить все на проперти компонентов. Там уже геттеры/сеттеры хорошо подходят, но только те геттеры/сеттеры которые на уровне языка а не "методы". Это легче тестить, проще поддерживать и в целом открывает большой простор для дальнейшей архитектурной вакханалии.

Comments

[Владимир]

первый подход, который вы описываете это сервис екстендить от ивент эмиттера, и в контроллере ловить сообщения от сервиса про изменение данных, я правильно понял ?
Второй это ng-redux ?