问题描述
app.contronller(’xxx’, [’$scope’, ’service’, function($scope, service) { $scope.xxx = ’xxxx’; service.say($scope);}]);app.service(’xxx’, [function() { this.say = function($scope) { }}]);
现在维护的代码里好多这样的情形，这是不是错误的用法？感觉用起来很爽，但是代码好混乱
问题解答
回答1：这明显是偷懒的做法，造成service的方法的含义非常模糊，你传$scope进去，真正用哪个参数完全体现不出来。而且造成service方法对$scope不必要的依赖。
回答2：严格来说，service里面不应该处理非全局性的状态值，尤其是修改。这里就是$scope。换个意思就是说，这里service类似于接口，应该明确函数的参数列表，而不是笼统的参数。如
app.contronller(’xxx’, [’$scope’, ’service’, function($scope, service) { $scope.xxx = ’xxxx’; service.say($scope.xxx);}]);app.service(’xxx’, [function() { this.say = function(word) {}}]);
当然类似$location，$rootScope 之类的全局变量，因为不需要通过其他组件传递过来，自身就可以通过注入获取得的到，是可以使用的。
回答3：去看我的博客 用回调参数传递
回答4：其实我测试的时候发现如果你在服务中返回值给控制器中的$scope ，闭包貌似不起作用，然而传$scope进去直接在服务中更新控制器中的$scope， 这样很好用
