Abdelrazak Younes <[EMAIL PROTECTED]> writes: > Text is a Paragraph container. Actually most of the methods there > should become helper functions or Cursor methods. Text::changeCase() > for example have no business at all in Text.
Agreed. It could be an helper function, as you write. > So is getWord(), selectWord(), etc. If you want to complain about > class being fatter, begin by erasing those methods. I agree with these points. > Year and that's exactly my point and why it is stupid, because it uses > public interface whereas it can do things much more efficiently by > accessing directly to the data. I agree it would be more efficient. But I am not sure it is worth writing a new piece of code accessing the internals just for an efficiency nobody cares about. > I am. "optimisation" is maybe not the good word but a good programmer > should always have efficiency in mind when programming, *always*. Of course, but maintainability too. I prefer to have as little code as possible to access the implementation details of ours structures. JMarc