This is all stuff you learn to grok quickly as you gain javascript experience. When you think in closures, only then can you attain inner peace :)
-1 here... keep it concise. On 10/4/07, Sven Klose <[EMAIL PROTECTED]> wrote: > > > I´m new to the source code, and had big trouble reading it. With the > right indentation one could spot constructs immediately, but actually, > the code is so compressed, it takes ages to even figure out where a > function starts and where it ends, to spot if-statements and variable > declarations, and so on. > > Some suggestions in detail: > > - dedicate a whole line for an opening/ending brace of a function if > it doesn't contain just one statement, > - keep the width of lines small, > - declare only a single variable in a var-statement, > - have indented conditional blocks, instead of squeezing an if- > statement into a single line, > - use distinct variable names, not 'Method' and 'method' in the same > function for example, > - use TABs, so all this all can be done without increasing the code > size, and one can set the indentation width to suit his/her needs, > - add more comments :) > > Your visual perception can spot the overall structure of code at once. > Please keep it stupid for the likes of me. ;) I'd do the reformatting, > if desired. > However, I find the use of spaces and empty lines very good already. > > Best regards. > > > > > -- Ryan Gahl Manager, Senior Software Engineer Nth Penguin, LLC http://www.nthpenguin.com -- Architect WebWidgetry.com / MashupStudio.com Future Home of the World's First Complete Web Platform -- Inquire: 1-262-951-6727 Blog: http://www.someElement.com LinkedIn Profile: http://www.linkedin.com/in/ryangahl --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Prototype: Core" group. To post to this group, send email to prototype-core@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/prototype-core?hl=en -~----------~----~----~----~------~----~------~--~---