directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Trustin Lee <trus...@gmail.com>
Subject Re: [seda] Coding standards and commit 56039 (by Trustin)
Date Sun, 07 Nov 2004 05:28:12 GMT
Oh I didn't know that MX4J styles doesn't word-wrap.  I think
80-column rules should be kept. :)


On Sat, 06 Nov 2004 09:52:48 -0500, Vincent Tence <vtence@videotron.ca> wrote:
> I second Niclas
> 
> [ ] Jakarta standards
> [ ] SUN Java recommended standards (might be the same as Jakarta)
> [x] Alex's coding standards
> [ ] Geronimo coding standards
> [ ] mx4j standards but use 4 spaces instead of 3 for tabs
> [x] invent our own
> 
> - Vincent
> 
> 
> 
> On Sat, 2004-11-06 at 13:30 +0800, Niclas Hedhman wrote:
> > On Saturday 06 November 2004 06:29, Enrique Rodriguez wrote:
> >  [ ] Jakarta standards
> >  [ ] SUN Java recommended standards (might be the same as Jakarta)
> >  [x] Alex's coding standards
> >  [ ] Geronimo coding standards
> >  [ ] mx4j standards but use 4 spaces instead of 3 for tabs
> >  [x] invent our own
> >
> > There are a lot in the Mx4j style that are making reading code harder, and
> > should expressly be avoided. Especially;
> >
> >   * single line if() constructs or potentially very dangerous
> >   * Not following Law of Demeter[1]
> >
> > Other things that are more personal choice;
> >
> >   * Consistency in spacing around "(" and ")". The MX4J has space before "("
> > in keywords but not for methods. Personally, I prefer space after "(" and
> > before ")" ALL.
> >
> >   *  throws clauses on new line for methods.
> >
> >
> > Most importantly, it is a lot harder to read code than write it and hopefully
> > it is also read many more times than it is written, so reading must always
> > take precendence over writing.
> >
> >
> > Cheers
> > Niclas
> >
> > [1]
> > http://www.ccs.neu.edu/research/demeter/demeter-method/LawOfDemeter/general-formulation.html
> 
> 


-- 
what we call human nature in actually is human habit
--
http://gleamynode.net/

Mime
View raw message