How to declare the controller in angularjs

Question

I had one question during interviews.

"Which of the following code snippets is more efficient?"

AppModule.controller('homeController', ['$scope', 'dep1', 'dep2', function($scope, dep1, dep2) {
...
    $scope.aMethod = function() {
       ...
    }
...
}]);

AppModule.controller('homeController', function($scope, dep1, dep2) {
...
    $scope.aMethod = function() {
        ...
    }
...
});

• Both are equally efficient.

• The second code is more efficient as it contains less code.

• The first code is more efficient because it holds names of dependencies.

• The second code is more efficient because it not contains additional an array.

I think the 3rd answer is right but not sure I'm right.

Please help me.

None of the above. Read this : https://github.com/johnpapa/angular-styleguide/blob/master/a1/README.md#controllers — Pritam Banerjee, Mar 28 '18 at 08:01
The second will work as the first one if you inject separately: `homeController.$inject = ['$scope', 'dep1', 'dep2'];` — Aleksey Solovey, Mar 28 '18 at 08:14
https://stackoverflow.com/questions/14184656/angularjs-different-ways-to-create-controllers-and-services-why — Manimaran P, Mar 28 '18 at 09:39

score 2 · Answer 1 · answered Mar 28 '18 at 08:02

The first one will work as expected after applying some uglification/minification while the second one won't work after that since the variable names will change during this process and angularjs will have no way to know what to inject since the variable names changed.

score 1 · Answer 2 · answered Mar 28 '18 at 08:12

1

Third is right, Because when you will uglification/minification the code by using any task runner then angular have no way to know the dependency. so third is better.

answered Mar 28 '18 at 08:12

Shohel

3,886
4
38
75

How to declare the controller in angularjs

2 Answers2