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

Reply via email to