Re: Perl Hash Script

One thing I can spot right away regards these lines.

foreach my $empnum(sort keys %empwork)
{
 my $periods=0;
 $periods++;
 my $hours = 0;
 $hours += $empwork{$empnum};
[download]

Your code boils down to

foreach my $empnum(sort keys %empwork)
{
  my $periods = 1;
  my $hours = $empwork{$empnum}
[download]

I doubt that is what you really want.

Did you mean this instead?

my $periods=0;
my $hours = 0;
foreach my $empnum(sort keys %empwork)
{
 $periods++;
 $hours += $empwork{$empnum};
[download]

Comment on Re: Perl Hash Script Select or Download Code

Replies are listed 'Best First'.
Re^2: Perl Hash Script by prithviraj (Novice) on Jan 08, 2013 at 16:10 UTC
I am not getting the exact output for your code Sir.	[reply]
Re^3: Perl Hash Script by muba (Priest) on Jan 08, 2013 at 16:16 UTC
I never claimed ~~you~~ it would. I was merely questioning whether your script might suffer from a logical error, and explaining why I think it does. Does it? Edit: s/you/it/	[reply]
Re^4: Perl Hash Script by prithviraj (Novice) on Jan 08, 2013 at 16:18 UTC
Yes. You are right.	[reply]