>------------------------------
>
>Date: Sun, 08 Jul 2001 13:08:41 -0400
>From: [EMAIL PROTECTED] (Judith Dinowitz)
>To: [EMAIL PROTECTED]
>Subject: Re: Bad code.. (Inefficient templates)
>Message-ID: <01b201c107d0$a3c6eec0$43bd61cf@judith>
>
>LOL
>
>Well, sometimes it's code for code's sake, right? : ) Any clue what he
>thought he was doing?
>
>Judith

  Not really, I believe the intention was to add some adds to the site, 
based on the comment (which I removed from the code below), but...  I'm 
seeing stuff like that all over the place.  I found one template that had 
two queries that weren't even used.  I also found an if statement with the 
if and else part being identical.  I'm going to have to recommend that we 
(And by we, I mean I) go over the templates with a fine-tooth comb to 
evaluate them for inefficient code. I'm sure he's going to say "Aren't you 
already supposed to be doing that." and I'm gonna have to say "no, we only 
have a contract for these specific changes."


> >   I did a project for a client over a year ago.  I'm currently working
>with
> > the same client.  He fancies himself as an beginner intermediate
> > developer.  I just found this code in one of the templates:
> >
> >
> > <cfoutput query="queryname">
> >          <CFIF isDefined("CompanyID")>
> >                  <!--- Comment here ! --->
> >          <CFELSE>
> >          </cfif>
> > </cfoutput>
> >
> >
> >   I modified it to remove all incriminating site references, but...
>That's
> > a whole lot of processing to accomplish nothing.
> >
> >




--
Jeffry Houser | mailto:[EMAIL PROTECTED]
AIM: Reboog711  | ICQ: 5246969 | Phone: 860-229-2781
--
I'm looking for a room-mate in the Hartford CT area, starting in August
--
Instant ColdFusion 5.0  | ISBN: 0-07-213238-8
--
DotComIt, LLC
database driven web data using ColdFusion, Lotus Notes/Domino
--
Far Cry Fly, Alternative Folk Rock
http://www.farcryfly.com | http://www.mp3.com/FarCryFly


Archives: http://www.mail-archive.com/cf-community@houseoffusion.com/
Unsubscribe: http://www.houseoffusion.com/index.cfm?sidebar=lists

Reply via email to