Как правильно загрузить зависимости в суперклассе контроллера приложения AngularJS?

Question

Я создаю приложение AngularJS, которое предоставляет основные функции CRUD. Это приложение построено с использованием TypeScript, чтобы использовать преимущества типов и классов. У меня есть базовый контроллер, которому нужен доступ к нескольким зависимостям AngularJS. Первоначально я ввел зависимости следующим образом

Файл: base-controller.ts

class BaseController {
    private $location;
    private $routeParams;
    constructor($location, $routeParams) {
        /* Manually inject dependencies */
        this.$location = $injector.get('$location');
        this.$routeParams = $injector.get('$routeParams ');
    }
}

Файл: list-controller.ts

class WorkRequestListController extends BaseController {
    /* Write code to override base controller or implement local methods */
}

angular.module('app.workRequests')
    .controller('WorkRequestListController', ['$location', '$routeParams',
        ($location, $routeParams) => new WorkRequestListController($location, $routeParams)
]);

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

Файл: base-controller.ts

class BaseController {
    private $location;
    private $routeParams;
    constructor($injector) {
        /* Load dependencies */
        this.$location = $injector.get('$location');
        this.$routeParams = $injector.get('$routeParams');
    }
}

Файл: list-controller.ts

class WorkRequestListController extends BaseController {
    /* Write code to override base controller or implement local methods */
}

angular.module('app.workRequests')
    .controller('WorkRequestListController', ['$injector',
        ($injector) => new WorkRequestListController($injector)
]);

Вот мой вопрос: это правильный способ загрузки зависимостей в суперклассе, не заставляя подкласс или другой код знать о зависимостях?

Answer 1

На самом деле существует очень простой способ реализовать базовый контроллер с помощью службы $controller. Я недавно написал в блоге об этом, вот фрагмент кода, показывающий, как это работает:

'use strict';

angular.module('Diary')

// base controller containing common functions for add/edit controllers
.controller('Diary.BaseAddEditController',
    ['$scope', 'DiaryService',
    function ($scope, DiaryService) {
        $scope.diaryEntry = {};

        $scope.saveDiaryEntry = function () {
            DiaryService.SaveDiaryEntry($scope.diaryEntry);
        };

        // add any other shared functionality here.
    }])

.controller('Diary.AddDiaryController',
    ['$scope', '$controller',
    function ($scope, $controller) {
        // instantiate base controller
        $controller('Diary.BaseAddEditController', { $scope: $scope });
    }])

.controller('Diary.EditDiaryController',
    ['$scope', '$routeParams', 'DiaryService', '$controller',
    function ($scope, $routeParams, DiaryService, $controller) {
        // instantiate base controller
        $controller('Diary.BaseAddEditController', { $scope: $scope });

        DiaryService.GetDiaryEntry($routeParams.id).success(function (data) {
            $scope.diaryEntry = data;
        });
    }]);

Answer 2

Это правильный способ загрузки зависимостей в суперклассе, не заставляя подкласс или другой код знать о зависимостях?

Это способ. Angular не рекомендует наследование в контроллерах. Предпочитает композицию. Общая функциональность должна идти в сервис/фабрику, которая вводится в ваши контроллеры.