> 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. :) > 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? > In databases, not defining relationships or keys. > This one really bugs me. You're talking about lack of constraints. That's one of my pet peeves too... it's bugged me at just about every place I've worked to date. :-/ s. isaac dealey 954.522.6080 new epoch : isn't it time for a change? add features without fixtures with the onTap open source framework http://www.fusiontap.com http://coldfusion.sys-con.com/author/4806Dealey.htm ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| 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:2542 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