A. This code is broken - badly - as has been pointed out by Les and Allen. The time you take to fix it could be spent creating managable code.
B. Hard coding site & computer data into your scripts is a bad idea. Every time a printer, server, or location changes, you need to modify your script, and every time you edit a script, you introduce the possibility for errors.
Please take a look at this post, which was discussed just this week and illustrates a method for mapping printers from a configuration file. This can reduce your script to about 2-dozen lines. The example is actually shorter than that, but could use some error checking, and you'll need to tie it into your processing logic. Still, much simpler code and less prone to failure when your environment changes.
Glenn
_________________________
Actually I
am a Rocket Scientist!
