Its nice to just wrap it up in a cfc. It makes it more readable and a bit
more reeuseable(just a wee bit).
DRE
On 11/14/06, Dan Plesse <[EMAIL PROTECTED]> wrote:
>
> DRE,
>
> I agree that's some bad stuff. It looks like when I first started
> coding
> 2 years ago.
>
> I would readLine() first
>
> Check Null true/false
>
> Enter the loop
>
> readLine again on the bottom of the loop
>
> End loop
>
> close() objects
>
> null them force gc()
>
> Look like this.
>
> <cfset variable.line = java_io_LineNumberReader.readline()>
>
> <cfloop condition="#IsDefined("variable.line")#">
>
> <cfif find(findThis, variable.line) GT 0>
>
> <cfset variable.line = Replace(variable.line, "
> http://www.opticalalert.com/r.cfm?urlvalue=","","ONE";)>
> <cfset variable.line = Replace(variable.line, findThis, replacethis,
> "ONE")>
>
> <cfset start = REFind("[[:digit:]]", variable.line, 1, true).pos[1]+5>
>
> <cfset some_text = left(variable.line,start) & '" target="new"
> style="font:none;">'>
>
> <cfset bigstring = bigstring & some_text>
>
> <cfelse>
> <cfset bigstring = bigstring & variable.line>
> </cfif>
>
> <cfset variable.line = java_io_LineNumberReader.readline()>
>
> </cfloop>
>
> I had to do a big switch / remove text from all my pages. And this is
> inside
> another recursive loop.
>
> Big job.
>
>
>
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Introducing the Fusion Authority Quarterly Update. 80 pages of hard-hitting,
up-to-date ColdFusion information by your peers, delivered to your door four
times a year.
http://www.fusionauthority.com/quarterly
Archive:
http://www.houseoffusion.com/groups/CF-Talk/message.cfm/messageid:260401
Subscription: http://www.houseoffusion.com/groups/CF-Talk/subscribe.cfm
Unsubscribe:
http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4
