Mailing List Archive: 49091 messages
  • Home
  • Script library
  • AltME Archive
  • Mailing list
  • Articles Index
  • Site search

[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.