Re: 'Mistakes' in Clean Code Episode 19 Part 1

271 views
Skip to first unread message

Uncle Bob

unread,
Jun 7, 2013, 10:36:08 AM6/7/13
to clean-code...@googlegroups.com
Winner #3

Sean Moir

unread,
Jun 7, 2013, 12:37:56 PM6/7/13
to clean-code...@googlegroups.com
Yay!

On Thursday, 6 June 2013 10:43:06 UTC+1, Sean Moir wrote:
In Episode 19, Uncle Bob invites us to point out his - no doubt intentional - mistakes made while refactoring the code.

I have spotted a few - not sure if I should post them one at a time, all at once, or whether there is another route that I should have taken?

I will say that I think it violates the Single Responsibility Principle to allow the Invert Names class to be responsible for Honorifics and Post Nominals since this functionality could *change for different reasons* than the main Invert Names class itself e.g. for different cultures, or if new titles emerge.

I have a few other spots if you need them, or if this one doesn't qualify for the correct type of correction?

best regards

Sean
Reply all
Reply to author
Forward
0 new messages