Невозможно переназначить переменную в CoffeeScript

Question

Я новичок в CoffeeScript и попытаюсь написать угловой контроллер. Однако я думаю, что у меня проблема с переменным охватом. Мой код:

app.controller 'IndexController', ['$scope', 'Authentication', 'Items', 'Snackbar', ($scope, Authentication, Items, Snackbar) -> 
    activate = -> 
     itemsSuccessFn = (data, status, headers, config) -> 
      vm.items = data.data 
      return 

     itemsErrorFn = (data, status, headers, config) -> 
      Snackbar.error data.error 
      return 

     Items().all().then itemsSuccessFn, itemsErrorFn 
     return 

    vm = this 
    vm.items = [] 
    vm.isAuthenticated = Authentication.isAuthenticated() 
    activate() 
    return 
] 

Проблема с переменными элементов. При запуске itemsSuccessFn в activate переменная заполняется данными, поступающими из службы. Но в основной области переменная снова пуста. Что я делаю не так? Как мне его решить?

Если в коде есть что-то еще, это должно быть сделано лучше, пожалуйста, дайте мне знать, потому что, как я уже говорил, я только начал учиться.

Answer 1

Вместо объявления локальной переменной вы можете напрямую ссылаться на $scope.

app.controller 'IndexController', ['$scope', 'Authentication', 'Items', 'Snackbar', ($scope, Authentication, Items, Snackbar) -> 
    activate = -> 
     itemsSuccessFn = (data, status, headers, config) -> 
      $scope.items = data.data 
      return 

     itemsErrorFn = (data, status, headers, config) -> 
      Snackbar.error data.error 
      return 

     Items().all().then itemsSuccessFn, itemsErrorFn 
     return 

    $scope.items = [] 
    $scope.isAuthenticated = Authentication.isAuthenticated() 
    activate() 
    return 
] 

Однако, если вы используете синтаксис controllerAs, то вы должны иметь правильную ссылку на this с помощью жира стрелки. Таким образом, вам не нужна переменная vm.

app.controller 'IndexController', ['$scope', 'Authentication', 'Items', 'Snackbar', ($scope, Authentication, Items, Snackbar) -> 
    activate = => 
     itemsSuccessFn = (data, status, headers, config) => 
      @items = data.data 
      return 

     itemsErrorFn = (data, status, headers, config) -> 
      Snackbar.error data.error 
      return 

     Items().all().then itemsSuccessFn, itemsErrorFn 
     return 

    @items = [] 
    @isAuthenticated = Authentication.isAuthenticated() 
    activate() 
    return 
]