[REBOL] Re: Downloading patents?
From: arolls:bigpond:au at: 2-Sep-2001 16:41
Well, it's working isn't it? :) Good job.
You don't need to convert CurrentPageNumber
to an integer inside the repeat loop, as
you can see, the integer gets converted
automatically by rejoin:
>> rejoin ["astring" 23]
== "astring23"
I am also of the opinion that such long variable
names end up not being that helpful. I find that
it gets hard to read because the code is too long.
Obviously, you should keep it how it looks best
to you, but I would name my variables much shorter:
LastPage -> pages (or totpg, showing where it came from)
CurrentPageNumber -> page (or pg)
Also, once you have figured out how to print PG
and TOTPG, you can remove the print statement and
use probe instead:
LastPage: second parse probe copy/part find page "TOTPG=" 10 "=&"
^
Probe is cool, it allows you to print the value to the right
but returns that same value to the left, so it has no effect
on the function of the code, it just prints out stuff at a
particular point, or "probes" a point. I use it all the time.