[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi] Inversion of control [Was: Another large census error with group q
From: |
Greg Chicares |
Subject: |
[lmi] Inversion of control [Was: Another large census error with group quotes] |
Date: |
Fri, 7 Sep 2018 23:49:12 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-08-21 12:24, Vadim Zeitlin wrote:
[...]
> I know we've already discussed this in the past, but I still have an
> impression that you underestimate the benefit of centralising all the logic
> in a single place with the "template method" approach. Yes, it does require
> you to use either OO, with derived classes for each use of the base class,
> or lambdas (preferable nowadays), but IMO it's absolutely not such a huge
> disadvantage as you seem to believe. While the advantage is real, as we
> keep seeing again and again, with bugs due to the differences in the "same
> logic" implemented in different places.
No matter how many times I try to read that GoF stuff, I can't understand it.
But I hope that if we start with something that I certainly do understand and
feel sure you will too--i.e., commit f27b89f1--then you'll be able to show me
how intruding this control structure into page_with_tabular_report::render()
(as I hope we can do) would constitute this kind of "design pattern".
- [lmi] Inversion of control [Was: Another large census error with group quotes],
Greg Chicares <=
- Re: [lmi] Inversion of control, Vadim Zeitlin, 2018/09/08
- Re: [lmi] Inversion of control, Greg Chicares, 2018/09/08
- Re: [lmi] Inversion of control, Vadim Zeitlin, 2018/09/08
- Re: [lmi] Inversion of control, Greg Chicares, 2018/09/12
- Re: [lmi] Inversion of control, Vadim Zeitlin, 2018/09/17
- Re: [lmi] Inversion of control, Greg Chicares, 2018/09/17
- Re: [lmi] Inversion of control, Vadim Zeitlin, 2018/09/17
- Re: [lmi] Inversion of control, Greg Chicares, 2018/09/17
- Re: [lmi] Inversion of control, Vadim Zeitlin, 2018/09/18
- Re: [lmi] Inversion of control, Greg Chicares, 2018/09/18