Thanks for your tips. I actually kinda forgot to polish code before uploading. There was still some debugging code in there and indeed all the “var”‘s and semicolons were missing. I’ll add them in a minute.
What do you mean with “every load”? There is only one load event for each object right?
D5 Creation
< ![CDATA[
Nice idea!
My tips to improve:
1. You declare all variables global, so they may interfere with other scripts. (especially names like script or event are used often)
-> declare with “var”
2. Instead of using runscript as interface, you could set a script in every load event, which defines functions like subscribe and unsubscribe.
3. If you use semicolons to end your lines the code is easier to read and won’t break when format is lost. (E.g. when copy/pasting from some browsers)
]]>
< ![CDATA[
Thanks for your tips. I actually kinda forgot to polish code before uploading. There was still some debugging code in there and indeed all the “var”‘s and semicolons were missing. I’ll add them in a minute.
What do you mean with “every load”? There is only one load event for each object right?
]]>
< ![CDATA[
The polishing is finally done. Fixed some bugs meanwhile too!
]]>
< ![CDATA[
The getOb function isn’t using the passed parameter. Maybe use two return statements instead of ob variable?
]]>
< ![CDATA[
Oops little derp. It’s fixed now.
]]>