> 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.. :)
