Alain,

my complaint wasn't about the single line, only about the "newValue" (a 
matter of style).

But I prefer one style for all methods -- even in single or zero line methods:

   public void mouseMoved(MouseEvent ev) {
   }

   public void mouseDragged(MouseEvent ev) {
     doSomething();
   }

Why?
1) It makes code more readable.
2) I can write code and do not to take care, whether it's a single line 
method or not.

I do not try to start a flame war -- just wanted to explain you my opinion. 
Maybe the OpenAPI would allow you to develop your own layout code mechanism.

Tom


At 10:36 18.03.02 +0100, you wrote:
>Tom,
>
>In a typical persisted class (mapped to a sql db), you can have 20+ fields
>
>=> 20 getters + 20 setters
>
>
>my "flat" notation =
>       20 * (2 + 1)                => 60 lines.
>
>if you kill the white line I add for readability,
>                                   => 40 lines.
>
>IDEA default notation = 40 * 5    => 200 lines.
>
>Do I need to say more?
>
>Alain
>
>
>
>----- Original Message -----
>"Thomas Singer" wrote
>
> > At 09:00 18.03.02 +0100, you wrote:
> >
> > >"Thomas Singer" wrote
> > > > Why not use SUN's coding conventions?
> > >
> > >Why not use SUN's Forte?
> >
> > Because it produces a terrible mix of tabs and spaces.
> >
> > Tom
> >
> >
> > >Alain
> > >
> > >
> > > > At 14:05 17.03.02 +0100, you wrote:
> > > > >I keep reformating getters/setters, flattening the 4 lines into 1,
> > > > >from :
> > > > >
> > > > >     public Date getDate()
> > > > >     {
> > > > >         return __date;
> > > > >     }
> > > > >
> > > > >     public void setDate( Date i_date )
> > > > >     {
> > > > >         __date = i_date;
> > > > >     }
> > > > >
> > > > >
> > > > >to
> > > > >
> > > > >     public Date     getDate()                   {   return 
> __date   ;  }
> > > > >     public void     setDate( Date newVal )      {   __date = 
> newVal ;  }
> > > > >
> > > > >
> > > > >I had to
> > > > >   - merge the lines
> > > > >   - align the elements
> > > > >   - rename the parameter (to 'newVal')
> > > > >   - stick the 2 lines together
> > > > >
> > > > >
> > > > >
> > > > >IDEA could/should help me here.
> > > > >
> > > > >Alain Ravet


_______________________________________________
Eap-features mailing list
[EMAIL PROTECTED]
http://www.intellij.com/mailman/listinfo/eap-features

Reply via email to