umm - yup!

Simplifies the logic and provides easier growth or adaptation by placing the control (group names) in the data instead of the code. \:\)

One note about your code - if a user is a member of multiple groups, then the printer for each group will be mapped. This may or may not be desireable. If it isn't, simply list the groups in order of importance. The code change is simple - Change the For loop slightly, using:
 Code:
$Tag = 1
For Each $Group in $Groups
  If InGroup($Group) And $Tag
    $Tag = 0       ; don't allow additional group comparison actions

    :

  EndIf
Next
The $Tag at the start allows an InGroup comparison to continue.. the first group match clears $Tag, preventing further group comparison If statements to succeed and process the printer map.

Glenn
_________________________
Actually I am a Rocket Scientist! \:D