Hmmm, I guess I have to point it out again.  I'm not saying line count
is the only indicator of design quality.  I'm saying it's a useful
indicator of design quality as a general rule.  There will always be
exceptions.  I concur fully on the "bad smells" concept from the
Refactoring book (an excellent book by the way and highly recommended).


-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On
Behalf Of Doug Keen
Sent: Tuesday, July 13, 2004 11:21 AM
To: [EMAIL PROTECTED]
Subject: Re: [CFCDev] CFC size

I think using only line count as an indicator of design quality is an
oversimplification.  I think the things (or "bad smells", as they'd be
referred to in the seminal work, "Refactoring: Improving the Design of
Existing Code", ISBN:0-201-48567-2) to be looking (sniffing) for are
unusually long methods, an unusually large amount of instance
variables, or classes (CFCs) that have an unusually large amount of
methods, or more accurately, resposibilities.  In those cases, you're
CFC can almost certainly be broken down into smaller CFCs with more
focused responsibility (i.e., better cohesion).  Of course, there's a
certain degree of subjectivity in determining what "unusual" is.

I highly suggest reading through the book I cited above... it really
helps develop your technical instincts, or "sense of smell" used in
detecting places where code/design can be improved.

-Doug Keen
----------------------------------------------------------
You are subscribed to cfcdev. To unsubscribe, send an email
to [EMAIL PROTECTED] with the words 'unsubscribe cfcdev' 
in the message of the email.

CFCDev is run by CFCZone (www.cfczone.org) and supported
by Mindtool, Corporation (www.mindtool.com).

An archive of the CFCDev list is available at
www.mail-archive.com/[EMAIL PROTECTED]

----------------------------------------------------------
You are subscribed to cfcdev. To unsubscribe, send an email
to [EMAIL PROTECTED] with the words 'unsubscribe cfcdev' 
in the message of the email.

CFCDev is run by CFCZone (www.cfczone.org) and supported
by Mindtool, Corporation (www.mindtool.com).

An archive of the CFCDev list is available at www.mail-archive.com/[EMAIL PROTECTED]

Reply via email to