LeeWitherington/xp123 — comment on commit 7587c0041a9f75cf82c6dce4d72f19bd19c7bcf2
File:  line 0
By LeeWitherington:

hehe, i have remove inheritance now in latest commit ( http://tinyurl.com/c43y8vd ).

I used a report_formatter to encapsulate the report algorithme rather than template method using inheritance. I am of the opinion that encapsulating the algorithme in this case is overengineering and actually costs to much in terms of ease of understanding and extra knowledge needed to use the classes or write new reports. Would you usually go this far? or even further? Is there any other elegant patterns im missing?

Cheers, Lee