我有一个 Angular 应用程序。它运行良好，但随着我的应用程序变得越来越大，我担心我必须在每个控制器中注入大量依赖项。
例如
app.controller('viewapps',[
'$scope','Appfactory','Menu','$timeout','filterFilter','Notice', '$routeParams',
function($scope,Appfactory,Menu,$timeout,filterFilter,Notice,$routeParams) {
//controller code..
}])
我确信依赖项列表将来会增加。我在这里做错了吗？这是正确的方法吗？有效处理此问题的最佳方法是什么？
【问题讨论】：
标签： angularjs dependency-injection
尝试将尽可能多的逻辑转移到服务中，甚至只是让控制器方法充当“路由-传递”方法。一段时间后，如果您想在其他控制器/指令中使用类似的方法，您会发现它非常有用。无论如何，我认为7次注射并不多:)
（编辑：见下方 Matt Way 的评论） 另外，提示 - 在较新版本的 Angular 中，您不必编写此数组，只需：
app.controller('viewapps', function($scope,Appfactory,Menu, $timeout,filterFilter,Notice,$routeParams){
//controller code..
}])
【讨论】：
如果没有确切的用例，或者在控制器中查看确切的代码，很难具体说明，但看起来你的控制器可能做的太多了（或者在你以后添加东西时可能最终做的太多了）。你可以做 3 件事：
将更多逻辑委托给注入的服务。
分成不同的控制器，所以每个控制器只有（大约）一项职责。
将输出分成指令，每个指令都有自己的控制器和模板，并允许通过属性和指令的scope 选项传入和输出选项。这通常是我的首选，因为您最终会构建一套可重用的组件，每个组件都有一个迷你 API。
至少在我看来，这样使用指令很好。它们不仅仅用于处理原始 Javascript 事件或直接访问 DOM。
【讨论】：
我一直在玩基于控制器捆绑服务的想法。
所以在你的例子中你会重构你的; AppFactory、Menu、filterFilter 和 Notice 服务合并为一个服务，例如ViewAppsServices。
然后您将使用 ViewAppsServices.AppFactory.yourFunction() 等服务。
在我看来，这样你至少可以将你的注入转移到另一个文件中，清理你的控制器。
我认为可读性会受到一些影响，因为其他开发人员将不得不查看捆绑包而不是控制器本身。
这是我整理的JSFiddle 来演示它的工作原理；这就是我想象的你的工作方式。
.service('ViewAppsServices', ['AppFactory', 'Menu', 'filterFilter', 'Notice',
function (AppFactory, Menu, filterFilter, Notice) {
return {
AppFactory: AppFactory,
Menu: Menu,
filterFilter: filterFilter,
Notice: Notice
};
} ])
【讨论】：
我的方法是使用$injector，当有很多依赖时：
app.controller('viewapps', ['$scope','$injector',function($scope,$injector){
var Appfactory = $injector.get('Appfactory');
var Menu = $injector.get('Menu');
//etc...
}]);
优点：
【讨论】：
$injector 会混淆资源的实际依赖关系（在本例中为controller）并破坏可维护性。 （当心那个最终会维护你的代码和知道你住的地方的精神变态者。他一点也不开心......）
$injector 注入。几乎一样。我假设您不熟悉 Javascript，尤其是 AngularJS。我也知道单一责任原则，但我不知道你为什么在这里提到它。您的评论非常笼统，超出了问题的背景。