Skip to content


adventures in bad code

I’m going through an app that someone built in a very short period of time. That’s no excuse for things like this:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<body onLoad="CloseWindow()">
<div class="mainContainer">
		<div class="mainApp">
			<div class="header">
                <cfinclude template="includes/help.cfm">
            </div>
            <div class="meat">
		<h2>Thank You!</h2>
                <p>Thanks for viewing the webcast. Have a wonderful day!</p>
		<p>&nbsp;</p>
		<p>&nbsp;</p>
		<p>&nbsp;</p>
		<p>&nbsp;</p>
		<p>&nbsp;</p>
            </div>

Sorry man, but it’s like that throughout. No MVC. No SVN used for deployment even though we have full Admin access to the production env. No load testing. No peer review of code. System Registry used for client variable storage. No understanding of how thread pools work. No template caching. SVN used infrequently and incorrectly anyway. Using hand-coded AJAX requests instead of a Javascript API like Prototype or jQuery. BAFFLING.

So much ugly code. Oh, and that nesting is exactly how it looks in TextMate.

Posted in coding, coldfusion.


0 Responses

Stay in touch with the conversation, subscribe to the RSS feed for comments on this post.



Some HTML is OK

or, reply to this post via trackback.