At 03:13 PM 5/9/2005, you wrote: > > At 01:25 PM 5/9/2005, you wrote: > >>How about we look at what makes a programmer look low > >>level and work our > >>way up. Two things that come right to mind are: > >>1. Improper use of pound signs in evaluation zones > >>2. Improper usage of IF clauses (not using short circuited > >>Boolean evaluation) > > > A few things that bug me: > > > Non-descriptive variable names. "X" is a lot less > > descriptive than > > "CurrentAuthor" or "AuthorIndex" for example. > >Can I use X as a loop index? I do that a lot. :)
Sometimes it makes debugging harder to do down the line. But, it is fairly common. I prefer to use more descriptive names if at all possible. > > Breaking encapsulation In CFCs (or custom tags or UDFs) > > by referencing shared scope variable (application, > > session, request, etc.. ). > >Does that include when the shared scope variables are part of a >wholistic framework used to develop the application? If I have a >well-structured set of function libraries in the request scope is it >bad form for one function in these libraries to reference another >function in these libraries from the request scope? I'll call this a gray area. To properly encapsulate, this would be wrong. However, within the context of an application and the framework it's probably okay. -- Jeffry Houser, Web Developer, Writer, Songwriter, Recording Engineer AIM: Reboog711 | Phone: 1-203-379-0773 -- My Company: <http://www.dot-com-it.com> My Books: <http://www.instantcoldfusion.com> My Recording Studio: <http://www.fcfstudios.com> Connecticut Macromedia User Group: <http://www.ctmug.com> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Logware (www.logware.us): a new and convenient web-based time tracking application. Start tracking and documenting hours spent on a project or with a client with Logware today. Try it for free with a 15 day trial account. http://www.houseoffusion.com/banners/view.cfm?bannerid=67 Message: http://www.houseoffusion.com/lists.cfm/link=i:11:2545 Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/11 Subscription: http://www.houseoffusion.com/lists.cfm/link=s:11 Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.11 Donations & Support: http://www.houseoffusion.com/tiny.cfm/54