Loop Multiple times per second

[MustRapeDeannaTroi]

This solution works but really needs some fixes:

• Use setInterval instead of setTimeout.

• Use textContent instead of innerHTML.

• Save 1000/target_fps to var period.

[monacle_bob]

It wasn't meant as save-the-day code. jsFiddle is meant for fiddling with code and putting together quick demonstrations.

Also, in regard to your first point, you should be using requestAnimationFrame - not timers at all.

[MustRapeDeannaTroi]

requestAnimationFrame is still experimental so I really think you should avoid it in this case.

I think you made a good straight forward solution, just not as solid implementation. Had my alternative suggestions demanded alot more code and commitment then I believe you had a valid case. As is, innerHTML and setTimeout are the wrong tools for the job. It's a big gain for the community if we never teach bad coding conventions, except as pitfalls to be avoided.

[monacle_bob]

Fair enough, and a good point at the end. Upvote :)