> I thought there was some consensus that attributes were tried
> to be casted intelligently, so setXxxx should be:
>
> void setXxxx(<ObjectType> value); // attributes
>
> where the String value of the attribute is tried to be casted
> to <ObjectType> (by using a constructor accepting a String, etc).
> Note that <ObjectType> in the setXxxx case might be a primitive
> type.
Yes.
> I also prefer the "addText(String value)" method over the
> "setText(String value)" because of the separate text chunks. It's
> up to the task to decide whether it wants it as a whole (just
> use StringBuffer.append() in addText) or as separate chunks.
Yes. I'm convinced.. :)
|