docs(tutorial) Change ctrls to not have name twice in tutorial

[victorb]

Naming both the controller and the function passed in to it with the
controller name both seems unessecery and confusing to beginners. This
commit removes the name from the function, as it is in the rest of the
docs.

[mary-poppins]

Thanks for the PR!

• Contributor signed CLA now or in the past
  • If you just signed, leave a comment here with your real name
• PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

[victorb]

Signed CLA with: Victor Bjelkholm
commit message follows the guidelines.

[btford]

commit message follows the guidelines.

you're missing a colon (:). should be docs(tutorial): change controllers to not have name twice

[victorb]

you're missing a colon (:). should be docs(tutorial): change controllers to not have name twice

Sorry about that, fixed now!

[btford]

No problem. Thanks for correcting it. :)

[petebacondarwin]

The benefit of naming these functions come out when you are debugging, since you then get nice names for the function in the call stack rather than anonymous.

This is good practice in my opinion - minification can remove them if you are worried about file size.

Perhaps, instead, there should be a line in the tutorial explaining why these are here?

[petebacondarwin]

Closing due to lack of activity

[victorb]

I'm not worried about the size of the code, I'm worry about repeating myself in my code. Also, this practice is not used anywhere else so I thought that no one is using this.

Of course it's nice with the name of the function in the call stack but it is really necessary?

Either way, there should be a decision about either using it with the name twice everywhere or nowhere except a section with debugging tips or something similar.

(Missed that there was any reply to this so please open it up again if you can)

[petebacondarwin]

OK, so since we don't seem to do this anywhere else, it is probably best to be consistent. I'm going to merge your changes.

[petebacondarwin]

Now to update the phonecat project itself, accordingly ...

[petebacondarwin]

Done