Angularjs: Controller Функция: переменные: «var» или контроллеры propetsies

Question

В приложении AngularJS внутри функции, которая снова находится внутри контроллера, , что будет лучшим подходом при создании переменной.

• обычно создает с var ключевым словом

  function getAnswer() { 
       //Some code here 
       var persistedDateValue = templateFactory.getAnswer(ctrl.item.ItemId); 
       //Some other code here 
       return persistedDateValue; 
      } 
  

• добавить его в области контроллера

  function getDateAnswer() { 
      //Some code here 
      ctrl.persistedDateValue = templateFactory.getAnswer(ctrl.item.ItemId); 
      //Some other code here 
      return ctrl.persistedDateValue; 
  } 
  

Здесь ctrl является контроллером, как и в директиве controllerAs: 'ctrl',.

Мое преуменьшение в том, что вариант один лучше, поскольку нам не нужна эта переменная, кроме этой функции.

Просьба предложить.

Answer 1

Javascript очень разрешительный. Если вы использовали более строгий язык как C или любой из его вариантов, вы не могли бы сделать то, что вы раскрываете во втором варианте. Этот способ доступа к переменным из внешней области (из-за пределов функции) называется «закрытием» и может быть полезен в некоторых ситуациях, но, конечно же, не в том, что вы показываете, это только испортит ваш код, поскольку сама функция может возвращать это результаты, это результат stament в «нормальном» моды, сохраняя один из самых важных принципов программирования науки: разделение интересов (SoC для друзей)

Так придерживаться вариант 1

Answer 2

Лучшей практикой будет вариант 1, если, как вы сказали, вы не собираетесь использовать эту переменную в другом месте. Таким образом, вы избегаете использования неиспользуемых данных в своей области.

Answer 3

Я согласен с ответом ZSnake, но если вам нужно сделать эту переменную доступной в остальной части вашего контроллера на экране, тогда вам понадобится

.controler('ControllerName', function() { 
    var _this = this; 

    function getDateAnswer() { 
    //Some code here 
    _this.persistedDateValue = templateFactory.getAnswer(ctrl.item.ItemId); 
    }