Comments on: A Class Isn’t Always a Noun http://www.agileatwork.com/a-class-isnt-always-a-noun/ by Michael Valenty Thu, 05 Jan 2012 07:24:18 +0000 http://wordpress.org/?v=2.9.1 hourly 1 By: Michael Valenty http://www.agileatwork.com/a-class-isnt-always-a-noun/comment-page-1/#comment-248 Michael Valenty Tue, 08 Dec 2009 00:03:18 +0000 http://www.agileatwork.com/?p=230#comment-248 Sure it's possible not to violate the rule, but the point I'm making is that it's not necessary or valuable to follow it dogmatically. Each link in the chain of responsibility implements IFilter<T>, they could all have the suffix 'Filter' and I could prefix member variables with 'm_' too. I think the pomp and circumstance is just noise and forces the reader to do unnecessary mental mapping. Sure it’s possible not to violate the rule, but the point I’m making is that it’s not necessary or valuable to follow it dogmatically. Each link in the chain of responsibility implements IFilter<T>, they could all have the suffix ‘Filter’ and I could prefix member variables with ‘m_’ too. I think the pomp and circumstance is just noise and forces the reader to do unnecessary mental mapping.

]]> By: Asaph http://www.agileatwork.com/a-class-isnt-always-a-noun/comment-page-1/#comment-247 Asaph Mon, 07 Dec 2009 19:02:17 +0000 http://www.agileatwork.com/?p=230#comment-247 I believe it would still be possible to stick to the "all-classes-must-be-nouns" rule without compromising your fluent interface. Your code would still read like a DSL. Consider the following: <pre class="prettyprint"> <code> private void SyncListing() { new LocatorChain<Yextlisting>(locator) .AddNew<DiscontinuedListingReverter>() .AddNew<LinkedBusinessRefresher>() .AddNew<ExistingBusinessLinker>() .AddNew<NewBusinessLinker>() .Process(listing); } </code> </pre> Is that really less readable? I believe it would still be possible to stick to the “all-classes-must-be-nouns” rule without compromising your fluent interface. Your code would still read like a DSL. Consider the following:


    private void SyncListing()
    {
        new LocatorChain<Yextlisting>(locator)
            .AddNew<DiscontinuedListingReverter>()
            .AddNew<LinkedBusinessRefresher>()
            .AddNew<ExistingBusinessLinker>()
            .AddNew<NewBusinessLinker>()
            .Process(listing);
    }

Is that really less readable?

]]>