This screenshot more or less perfectly illustrates what I consider wrong with the Udacity HTML5 game development class. We're in the middle of a routine for drawing sprites to the screen, and so far this particular lesson, on asset atlasses and sprite packing and all the rest, has been fine, but there's something in this particular code that makes me twitch hard. Take a look at the "hlf" object created in the middle of this function. What is it doing?
It's doing nothing. It's not a transformation. It's not analysis. It's not _functional. _ It's just a renaming of one variable ("center of x") to another ("half of x"). It doesn't even help the user understand what's going on.
But it's also doing something else. It's creating a new object in the context of this function. That object goes onto the heap until the Javascript VM performs garbage collection, at which point the entire_ VM_ pauses to make sure no references are tampered with as it cleans up. Different VMs (V8, SpiderMonkey, Chakra, etc.) perform garbage collection on different criteria, but all monitor memory growth, watch for times when memory usage exceeds a certain point, and perform routine garbage collection on an interval. Every HTML5 game designer in her right mind should be doing absolutely everything she can to prevent the garbage collector from triggering, and if it must, giving it as little as possible to clean up. A long VM pause is going to make one hell of a mess of your jealously protected 30 frames per second, and depending upon the tightness of your code it may throw off timing routines as well. Remember: setTimeout(func, 60) doesn't promise to run func exactly 60 milliseconds later; it promises to run func "as soon as possible" after 60 milliseconds has passed. Relying on that while throwing in possible ~100 millisecond garbage collection pauses is a sure recipe for failure.
I understand that this is a basic course, and it's going to be missing a lot of details. Memoization and hyper-functional up-front calculations are fairly advanced subjects. But there's a difference between not covering advanced topics and actively teaching students a bad habit.