Post real code, code that compiles , code that runs, code that produces output, and the output, code and data insice <c></c> tags ( How do I post a question effectively? )

Is there a better way of accessing the information in a hash of arrays?

Not really. You could use Data::Diver, but a HoA is shallow enough not to need it

foreach $newdn (sort keys %newgrouphash )  {
      print "Name:$newdn\n\t";
         foreach $group ( sort @{$newgrouphash{$newdn}} )  {
             print "$group, ";
         }
      print "\n";
}
[download]

Thanks, I still get this result: Name: Name:CN=test123,OU= etc etc The group prints fine

How can we know why things get printed twice? The code you showed does not even have a print statement!

I am pushing a user name and an array of AD groups for that user name into a hash of arrays: Code: push (@{$newgrouphash{"$newdn"} },"$newgroup")

You are not pushing an array of AD groups into a HoA. You are pushing the stringified version of a scalar into your HoA. For a start, drop all the double quotes around your variables. They serve no purpose. Then check whether $newgroup indeed contains a reference to your array of AD groups.

And finally show us the code you use to output this data.

CountZero

A program should be light and agile, its subroutines connected like a string of pearls. The spirit and intent of the program should be retained throughout. There should be neither too little or too much, neither needless loops nor useless variables, neither lack of structure nor overwhelming rigidity." - The Tao of Programming, 4.1 - Geoffrey James

My blog: Imperial Deltronics

Thanks, I still get this result: Name: Name:CN=test123,OU= etc etc The group prints fine

Funny :)

If you want help, you'll need to provide sample data (as in Data::Dump, Data::Dumper ... ) as Basic debugging checklist explains

As others have suggested, please post the code where you do the "push", and the "print".

I re-read your statement:


I am pushing a user name and an array of AD groups for that user name into a hash of arrays

So -it sounds like you are pushing both the NAME and the GROUP into the hash value.
This is in addition to the name being in the Key.

If this is the case, it would indeed explain the name coming out twice, since you (or at least my posted code) prints the key and value.

Anyway - this discussion would be a lot less abstract and hypothetical, if you posted your actual code.
In it's absence, interest in assisting you is rapidly waning.

             "By three methods we may learn wisdom: First, by reflection, which is noblest; Second, by imitation, which is easiest; and third by experience, which is the bitterest."           -Confucius

Sorry, new here, I used the print statements from the previous posters suggestions. The actual data being pushed into the HOA is a lot of code and is an AD query so it wouldn't compile. What I end up with is a key of Bob and an array containing groups: domain users, testing, server team etc

push (@{$newgrouphash{"$newdn"} },"$newgroup");  
    foreach $newdn (sort keys %newgrouphash )  {
              print "Name:$newdn\n\t";                  
               foreach $group ( sort @{$newgrouphash{$newdn}} )  {    
+                                   
                  if (!($group =~ /CN=Domain Users/i)) {    
                 # print "$group, ";                                  
+                   
                   $res = `dsmod group $group -rmmbr \"$newdn\"`;     
+         
                   print "$res\n";
          

[download]

The actual data being pushed into the HOA is a lot of code and is an AD query so it wouldn't compile

And that is the setting you chose to learn how to use HOA's? Try this instead: post some code that builds an HOA from the data in a file, and then prints out all the keys and their values. Post the file and your code.

Thanks, I was using %newgrouphash = undefined; to declare my hash instead of %newgrouphash = (); That is where the extra key was coming from. forever learning - thanks for the help

If you had

use strict;
use warnings;
[download]

at the top of your code, it would have pointed out your problem, with the message:

Use of uninitialized value in list assignment at ..
[download]

(This comes from 'use warnings;')

There is overwhelming agreement here that those two directives should be at the top of EVERY script you write, and you have just experienced an instance where it could have saved days of questioning and struggle.

             "By three methods we may learn wisdom: First, by reflection, which is noblest; Second, by imitation, which is easiest; and third by experience, which is the bitterest."           -Confucius