Angular

Forward references in Angular

In our article on Dependency Injection in Angular we explored what dependency injection actually is, and how it is implemented in the Angular framework. If you haven’t read that article yet, I highly recommend you doing so, since this article is based on it.

In a another article we even learned about host and visibility of dependencies as another aspect of Angular’s DI system. But that doesn’t mean that we’ve already discovered all features of the machinery yet. In this article we’ll take a look at forward references. Another tiny, yet useful feature of the DI system in Angular.

Understanding the problem

As a small recap, here we have an AppComponent that relies on DI to get a NameService injected. As we are using TypeScript, all we need to do is to annotate our constructor parameter nameService with the NameService type. This gives Angular all the relevant info to correctly resolve the dependency at runtime.

app.ts

import { Component } from '@angular/core';
import { NameService } from './name.service';

@Component({
  selector: 'my-app',
  template: '<h1>Favourite framework: {{ name }}</h1>'
})
class AppComponent {
  name: string;

  constructor(nameService: NameService) {
    this.name = nameService.getName();
  }
}

nameService.ts

export class NameService {
  getName () {
    return "Angular";
  }
}

This works well, but let’s see what happens when we inline the contents of nameService.ts directly in app.ts. In this case, you probably wouldn’t want to do that but bear with me as I’m trying to make my point.

import { Component } from '@angular/core';

@Component({
  selector: 'my-app',
  template: '<h1>Favourite framework: {{ name }}</h1>'
})
class AppComponent {
  name: string;

  constructor(nameService: NameService) {
    this.name = nameService.getName();
  }
}

class NameService {
  getName () {
    return "Angular";
  }
}

When we try to run this code we notice that it stopped working. In my case, I wasn’t even able to get an error reported to the console which I assume boils down to some glitch with debugging TypeScript code with source maps. Anyways, when we use the debuggers “Pause on exceptions” feature we can follow the rabbit into it’s hole somewhere deep down inside the Angular framework.

Cannot resolve all parameters for AppComponent(undefined). Make sure they all have valid type or annotations.

Ok, this gives us a little hint. It seems NameService is undefined in the constructor of AppComponent. This makes sense if you look at the flow of the code because we already used NameService in the constructor of AppComponent before we actually declared it. But on the other hand, using regular ES5 constructor functions that would be totally valid because function declarations get hoisted to the top by the JavaScript interpreter behind the scenes. And then, aren’t ES2015 classes just sugar on top of regular ES5 functions after all?

Let’s see what happens when we move NameService to the top so that it’s declared before it’s first usage.

import { Component } from '@angular/core';

class NameService {
  getName () {
    return "Angular";
  }
}

@Component({
  selector: 'my-app',
  template: '<h1>Favourite framework: {{ name }}</h1>'
})
class AppComponent {
  name: string;

  constructor(nameService: NameService) {
    this.name = nameService.getName();
  }
}

Ok, this seems to work just fine. But why doesn’t the JavaScript interpreter do that for us in the first place as it does for regular ES5 constructor functions?

Classes aren’t hoisted for a good reason

Let’s step back from Angular for a moment in order to understand the bare mechanics of the JavaScript language in this regard.

The JavaScript interpreter doesn’t hoist class declarations because it may lead to unsound behavior when we have a class that uses the extend keyword to inherit from something. In particular, when it inherits from an expression which is absolutely valid.

Consider this ES6 code:

class Dog extends Animal {

}

function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

var defaultMove = "moving";

var dog = new Dog();
dog.move();

This alerts moving just fine because what happens behind the scenes is that the JavaScript interpreter restructures the code to this.

var defaultMove, dog;

function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

class Dog extends Animal {

}

defaultMove = "moving";

dog = new Dog();
dog.move();

However, try making Animal an expression rather than a function declaration.

class Dog extends Animal {

}

var Animal = function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

var defaultMove = "moving";

var dog = new Dog();
dog.move();

Again, this will be hoisted but now it becomes this.

var Animal, defaultMove, dog;

class Dog extends Animal {

}

Animal = function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

defaultMove = "moving";

dog = new Dog();
dog.move();

At the point where class Dog extends Animal is interpreted Animal is actually undefined and we get an error. We can easily fix that by moving the Animal expression before the declaration of Dog.

var Animal = function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

class Dog extends Animal {

}

var defaultMove = "moving";

var dog = new Dog();
dog.move();

This works just fine again. Now think about what would actually happen if the JavaScript interpreter hoisted Dog just like a regular ES5 constructor function? We would end up with this code:

var Animal, defaultMove, dog;

// Dog is now hoisted above `Animal = function Anim...`
class Dog extends Animal{

}

Animal = function Animal() {
  this.move = function () {
    alert(defaultMove);
  }
}

defaultMove = "moving";

dog = new Dog();
dog.move();

Now that Dog is hoisted to the top the code breaks at the moment where the extends Animal is interpreted because Animal is undefined at that moment. The important thing to note here is that the extends part has to be evaluated at the right point in time. Therefore classes aren’t hoisted.

So the class must always be declared before it’s usage?

Ok, now that we understood why classes aren’t hoisted what does that mean for our earlier Angular example where we had to move the NameService to the very top? Is this the only way to get things working?

Turns out there is a solution we can reach for. Instead of annotating our nameService parameter with the NameService type which we learned evaluates to undefined at this point in time, we can use the @Inject annotation in conjunction with the forwardRef function as demonstrated here.

import {Component, Inject, forwardRef} from '@angular/core';

@Component({
  selector: 'my-app',
  template: '<h1>Favourite framework: {{ name }}</h1>'
})
class AppComponent {
  name: string;

  constructor(@Inject(forwardRef(() => NameService)) nameService) {
    this.name = nameService.getName();
  }
}

class NameService {
  getName () {
    return "Angular";
  }
}

What forwardRef does is, it takes a function as a parameter that returns a class. And because this function isn’t immediately called but instead is called after NameService is declared it is safe to return NameService from it. In other words: At the point where () => NameService runs NameService isn’t undefined anymore.

Conclusion

The described scenario isn’t something that one has to deal with too often. This only becomes a problem when we want to have a class injected that we created in the same file. Most of the time we have one class per file and import the classes that we need at the very top of the file so we won’t actually suffer from the fact that classes aren’t hoisted.

Written by  Author

Christoph Burgdorf