TS Conversion

[wilbennett]

Mark,

I think there is an easier way to do the conversion. You mentioned yesterday that you think the cost of creating a bug is much higher than not writing code. In the same vein, I argue that trying to change everything at once is much more costly than making incremental changes. Here is my suggestion:

• make the Javascript files compile with Typescript
• run the app
• visit each .js file
  • rename to .ts
  • fix any issues
  • convert to a class if needed
  • run the app

This way you can address any errors as they come up and you are not bombarded with a bazzilion error messages. If you encounter an error that you don't know how to fix, you can easily keep that file as .js until you figure it out. The way you are doing it now, if it works, it'll be great. If it doesn't, you may be spending a lot of time trying to figure out what is wrong.

Just my $0.02

Wil

[wilbennett]

... Also, as I was trying to mention in the chat, you are doing more than needed. For example, when you have a parameter in a constructor that you need to be a field, you don't have to explicitly create the field and assign it.

Instead of:

private myField: number;

constructor(myField: number) {
    this.myField = myField;
}

You can simply do:

constructor(private myField: number) {
}

It's hard to make these suggestions in real-time if you're not paying attention to the chat ;)