You're actually getting the totals for all the files. Try the following:

#!/perl/bin/perl
#
# Author: L.H
# Script Name: lab6-1
# Date Written: March 15, 2005
# Purpose: Basic I/0
#
#

use strict;

foreach my $file (@ARGV) {
    open(FILE, "$file");
    my $sum = 0;
    while (<FILE>) {
        $sum+=$_;
        print "$file $_\n";
    }
    print "$sum\n";
    close( FILE );
}
[download]

The major change is the my $sum = 0; line. It scopes $sum to be within the foreach-loop. This means that it gets reset every time. You'll notice I added strict to my version. This provides some typo-checking, and also requires that you declare your variables (preferably with my) beforehand.

This node was much better than your previous nodes. You posted what you tried and why it didn't work. This means we'll help you out, explaining why what you did didn't work and how to improve it. We'll edit, not create.

wow guys, I really feel good know, actually I am learning lot of stuff, I've been awake from 7 am. and learning pearl all day long. Here is the script tha I modified. The script should do the following: copy all the files that were entered in command line to a folder lab06d which is already created. and add a "b" at the end of the file to tell that is a backup. also it should print the total value added from all the files. but I don't thing is adding the value from all the files together. here is the script again:

#!/perl/bin/perl
#
# Author: L.H
# Script Name: lab6-1
# Date Written: March 15, 2005
# Purpose: Basic I/0
#
#
print "L.H\n";
print "Starting Script\n\n";
print "Filename\tValue\t(print heading)\n";

use File::Copy;
use strict;
my $total;
foreach my $file (@ARGV)
{
open(FILE, "$file");

my $sum = 0;
while (<FILE>)
{
chomp();
$sum+=$_;
print "$file $_\n";
$total+=$sum;
}
print "Total value for the $file : $sum\n";
close( FILE );
}
print "Total value accumulated from all the files
is:$total";

foreach my $file (@ARGV)
{
my $newfile=$file . "b";
copy ($file, "lab06d\\$newfile");
}

The script should do the following: copy all the files that were entered in command line to a folder lab06d which is already created. and add a "b" at the end of the file to tell that is a backup. also it should print the total value added from all the files. but I don't thing is adding the value from all the files together.

Does it do those things? Have you tested it out? It looks good, but the only way to know is by actual testing. Good luck! :-)

---


My wife's blog

I did this:

> cat - >1.pl
1
>
> cat - >2.pl
2
>
> cat - >3.pl
3
>
> perl foo.pl ?.pl
1.pl 1
Total value for the 1.pl : 1
2.pl 2
Total value for the 2.pl : 2
3.pl 3
Total value for the 3.pl : 3
Total value accumulated from all the files
is:6>
[download]

Which is the correct total. So either you didn't test it or you didn't specify what you though the errant behavior was. Either will get you down votes.

If you format your code as follows it will be more readable to you and others. You can use ether tabs or spaces, Perl doesn't care.

foreach my $file (@ARGV)
{
    open(FILE, "$file");
    my $sum = 0;
    while (<FILE>)
    {
        chomp();
        $sum+=$_;
        print "$file $_\n";
        $total+=$sum;
    }
    print "Total value for the $file : $sum\n";
    close( FILE );
}
[download]

Also; you already foreach through the files, why not do the copy in the same loop?

Please when you post code place <code> before and </code> after. These html tags put the code into the pretty little boxes you see in other posts. It also keeps the code from fouling up the browsers for those that read your posts. Please read Writeup Formatting Tips again and take it to heart.

---

s//----->\t/;$~="JAPH";s//\r<$~~/;{s|~$~-|-~$~|||s |-$~~|$~~-|||s,<$~~,<~$~,,s,~$~>,$~~>,, $|=1,select$,,$,,$,,1e-1;print;redo}

I gave you a plus vote for taking your medicine and coming back again. I'd give you another plus for a well written second question if I could. I noticed someone still downvoted you on your second question showing perhaps still some bias towards homework questions. Or it could because of the reference to Perl Monks being too advanced.

There are all levels of programmers here at the Monastery and many basic questions asked and many basic questions answered. We all think this is the best forum for Perl you'll find and would encourage you to keep returning. There also is also a lot of good basic material available here, some of it was already pointed out to you and you have the book. We'll expect you to do your research before posting as you've already seen. Please feel free to come here for help and bring a thick skin.

P.S. You'll get a wider readership on new questions if you start a new node rather than posting as a response to an existing node.