-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am Freitag, 4. Dezember 2009 18:24:01 schrieb Trevor Daniels: > David Kastrup wrote Friday, December 04, 2009 4:24 PM > > > I've unwrapped a bit of code of the form > > > > for (a;b;c) { > > if (condition) > > short something > > else { > > Large something > > } > > z > > } > > > > into > > > > for (a;b;z,c) { > > if (condition) { > > short something > > continue; > > } > > Large something > > } > > > > which helps a bit keeping track of what happens where. > > I disagree. Hiding the z in the iteration clause means > it is likely to be overlooked. Also the use of continue > rather than else does not clarify the structure IMO.
+1 To me continue clauses are like goto statements. I try to avoid them whenever possible. Only in some extreme cases where I would get like 12-times indented code, then its better than else... When reading some code long afterwards, it's quite easy to miss a continue or return (the two pathes of an if are indented and thus easy to spot) and then wonder why some statement apparently has no effect. Cheers, Reinhold - -- - ------------------------------------------------------------------ Reinhold Kainhofer, reinh...@kainhofer.com, http://reinhold.kainhofer.com/ * Financial & Actuarial Math., Vienna Univ. of Technology, Austria * http://www.fam.tuwien.ac.at/, DVR: 0005886 * LilyPond, Music typesetting, http://www.lilypond.org -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iD8DBQFLGVnUTqjEwhXvPN0RAqovAKDS/7/q5S+2Zerx9vlTf2rh9QlYSgCgsDZt tzSVasyeJMgyuH15Qy+aMeI= =T+hB -----END PGP SIGNATURE----- _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel