I don't know if this is the right place to provide feedback on the Standard Asset Characters, but here we go.



First of all, it could be useful to have a version number in a comment in the top of the scripts. I started using Unity October 22 2018, and soon after installed whatever version of the Standard Assets there was at a time. I'm not sure though if this is the most current version, or the version other devs use. Maybe such version number already exist for the asset as a whole, I don't know.



Regarding the scripts it would help to understand if things were better named, or otherwise explained in a code comment. As an example, what is the difference between m_Jump and m_Jumping? After some reading through the code (and some months of C# and Unity experience) it appears m_Jump is true from the player presses the jump key, until the first fixed update when the characters takes off, while jumping is true for as long the player is in the air. It would be nice though to easier see this, and not have to dissect the code to understand it, especially when you aren't used to the language or the engine yet. If my conclusion is correct, perhaps one variable could be called m_AboutToJump.



In general I think it would be less distracting and easier to understand if some extra attention was put into the polishing of the scripts, e.g. make sure comments are actual sentences starting with a capital letter and ending with some sort of punctuation. There are other things like inconsistent double line breaks that just makes the code feel noisy and makes it harder to read.



Since this is examples I think it's extra important to show best practices, and increasing readability would definitely help new users.

Click to expand...