G'day sunil9009,

I strongly suspect poorly named variables are one of your biggest problems here:

• %hash — we already know it's a hash (by the % sigil), beyond that the name is meaningless: it neither reflects the keys (which are customer names) nor the values (which are times).
• $word1, $word2, $word3 — all meaningless and tells us nothing about the data they hold.
• $user — started off as customer names, now they're users.
• $loc — fairly common as an abbreviation for "location" except it holds customer names not locations.

Not surprisingly, when you got to printing your results, you had no idea what data was held where!

You probably want something closer to this:

$ perl -Mstrict -Mwarnings -le '
    my @file_data = (
        "12:10 a  america",
        "12:11 b  bombay",
        "12:12 c  calcutta",
        "12:13 a  australia",
        "2:30   b  bhutan",
        "3:40  n    neterland"
    );

    my %name_data;

    for (@file_data) {
        my ($time, $name, $loc) = split;
        push @{$name_data{$name}{time}} => $time;
        push @{$name_data{$name}{loc}} => $loc;
    }

    for my $name (sort keys %name_data) {
        print "$name: @{$name_data{$name}{time}} : @{$name_data{$name}
+{loc}}";
    }
'
a: 12:10 12:13 : america australia
b: 12:11 2:30 : bombay bhutan
c: 12:12 : calcutta
n: 3:40 : neterland
[download]

Other issues:

• You haven't used strict.
• You haven't used warnings.
• You've used the 2-argument form of open.
• You call die without a message. If you don't want to hand-craft a message, use autodie instead.

Thank you guys for the solutions. Its amazing to get such knowledge from gurus of perl.

I see two options:

1. As per your script, you populate %hash and %population%location in such a way that they have the same keys. So you could just move the print statement from your "locations" loop into the "users" loop (it would also make sense to rename %hash to %users).
2. Alternatively, you could add another dimension to your %hash such that you store both times and locations in the hash. For example:

      push (@{$hash{$word2}{'times'}},$word1);
      push (@{$hash{$word2}{'locations'}},$word3);
   [download]

   This way you would keep the data closer together.

EDIT: Changed %population to %location (thanks kcott)

So here's a solution using a single data structure with each customer referencing 2 arrays, one for time and the other for location.

~/$ cat tmp.pl
#!/usr/bin/perl
use strict;
use warnings;

open(my $customers, $ARGV[0])|| die "Couldn't open $ARGV[0] $!\n";
my %cust;
while(<$customers>){
    chomp;
    my @rec=split(/\s+/, $_);
    push @{$cust{$rec[1]}->{time}},$rec[0];
    push @{$cust{$rec[1]}->{location}},$rec[2];
}

for my $cust (sort keys %cust){
    print "$cust: ",join(" ",@{$cust{$cust}->{time}})," : ",join(" ",@
+{$cust{$cust}->{location}}),"\n";
}
~/$ perl tmp.pl tmp.dat
a: 12:10 12:13 : america australia
b: 12:11 2:30 : bombay bhutan
c: 12:12 : calcutta
n: 3:40 : neterland
[download]

Your choice of data structure is poor. Whenever you are using to hashes with the same keys, you should think about using a hash of hashes, or hash of arrays, or whatever suits uour needs the best.

However, since you have the same keys you could do something like this:

for $user (sort keys %hash) {
    print "$user: @{$hash{$user}} \t";
     print "$loc: @{$location{$loc}} \n";
}
[download]

EDIT: my copy and paste was too hasty (the train from which I posted it was arriving at my destination). It should be something like this:

for $user (sort keys %hash) {
    print "$user: @{$hash{$user}} \t";
    print "$user: @{$location{$user}} \n";
}
[download]

I do not think the latter will work as $loc will be undefined...

Yes, right, my answer was too hasty, I updated my post now.