1) It seems to violate a number of traditional programming principles (e.g. variable scope)
You need to declare variables using var
, else it will go into the global scope.
2) Undefined variables seem to appear out of nowhere and already have values associated with them (how did this happen?)
This is possibly related to 1) and/or 4).
3) There seems to be bunch of random rules (e.g. return false to stop a default action, but sometimes this doesn’t work?)
You need to let the handler return false as well. E.g. form onsubmit="return functionname()"
. You also need to return from the “main” function, not only from a closure (a function inside a function), referring to your previous question. It would only return into the “main” function and continue on.
4) Non-deterministic behavior when debugging. (e.g. I refresh the browser, try something and get result X for JS variables I’m watching in Firebug. I refresh again and I get result Y?)
Probably the code was executed before the HTML DOM was finished populating. You need to hook on window.onload
or $(document).ready()
whenever you want to execute stuff during page load.
5) Very messy looking code that is hard to follow. What happens when? I’m using Firebug and Chrome Developer Tools, but I’m not getting enough visibility.
I bet that you’re talking about jQuery source? It’s just a large library. You should after all not worry about this when debugging. Rather worry about your own code. However, make sure that you’re looking at the unminified version of jQuery’s source code.
See also:
- JavaScript: the bad parts
- What should every JavaScript programmer know