Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > comp.lang.javascript > #18043
| From | "Leonardo Azpurua" <leonardo@exmvps.org> |
|---|---|
| Newsgroups | comp.lang.javascript |
| Subject | Re: Request for opinions on my newbie approach |
| Date | 2013-01-09 18:10 -0430 |
| Organization | A noiseless patient Spider |
| Message-ID | <kckrav$41u$1@dont-email.me> (permalink) |
| References | <kcilrc$vt8$1@dont-email.me> <10136443.OnBuAidq6I@PointedEars.de> <a2hre85m1aojpu1l2a46uqbklvnhq7ih3h@4ax.com> |
"Jim T." <x@y.z> escribió en el mensaje news:a2hre85m1aojpu1l2a46uqbklvnhq7ih3h@4ax.com... > On Wed, 09 Jan 2013 04:31:58 +0100, Thomas 'PointedEars' Lahn >>> for (var i = 0; i < f.elements.length; i++) >> >> for (var i = 0, len = f.elements.length; i < len; ++i) > > Why? Because it's faster? First, modern JS interpreters will probably > already do this optimization. Second, unless the form has a million > elements it will make no noticeable difference. Cleaner code is better > than pointless "optimization". Hi, In this particular case, Thomas is absolutely right. Good style -hence good code- comes from good habits. And good habits come from the strict observance of best coding rules: unless a function value may change during the loop execution, avoid using the function as a limit for the loop. Thomas suggestion is an improvement independently of the context. My original code may be fine in the given context, but is bad code. And bad code must be corrected. It is not a trivial optimization, but an important correction of the style. --
Back to comp.lang.javascript | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Request for opinions on my newbie approach "Leonardo Azpurua" <leonardo@exmvps.org> - 2013-01-08 22:24 -0430
Re: Request for opinions on my newbie approach Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2013-01-09 04:31 +0100
Re: Request for opinions on my newbie approach "Leonardo Azpurua" <leonardo@exmvps.org> - 2013-01-08 23:34 -0430
Re: Request for opinions on my newbie approach Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2013-01-09 05:14 +0100
Re: Request for opinions on my newbie approach Jim T. <x@y.z> - 2013-01-09 14:32 -0500
Re: Request for opinions on my newbie approach Cezary Tomczyk <cezary.tomczyk@gmail.com> - 2013-01-09 20:41 +0100
Re: Request for opinions on my newbie approach Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2013-01-09 21:03 +0100
Re: Request for opinions on my newbie approach Jim T. <x@y.z> - 2013-01-09 15:11 -0500
Re: Request for opinions on my newbie approach Luc Yen <luc@goal.tw> - 2013-01-09 14:38 -0800
Re: Request for opinions on my newbie approach "Leonardo Azpurua" <leonardo@exmvps.org> - 2013-01-09 18:10 -0430
Re: Request for opinions on my newbie approach Thomas 'PointedEars' Lahn <PointedEars@web.de> - 2013-01-10 01:36 +0100
csiph-web