Perl calculate and average problem

Discussion in 'Perl Misc' started by ed, Sep 30, 2005.

1. edGuest

I'm trying to understand perl and I'm pretty much confused. I'm trying
to write a script that takes an array of numbers (arbitrary in size).
The function will calculate the average of the numbers, the total of
all the numbers added together, and a new array of numbers which is the
other numbers divided by 2. It will then return a new list with all of
the information, and a new array of number, the total. The script
file, should get a list of numbers from the user (either via STDIN or a
list of arguments.)

ed, Sep 30, 2005

ed wrote:
> I'm trying to understand perl and I'm pretty much confused. I'm trying
> to write a script that takes an array of numbers (arbitrary in size).
> The function will calculate the average of the numbers, the total of
> all the numbers added together, and a new array of numbers which is the
> other numbers divided by 2. It will then return a new list with all of
> the information, and a new array of number, the total. The script
> file, should get a list of numbers from the user (either via STDIN or a
> list of arguments.)

Thanks for letting us know.

Where is the code you've written?

Have you read the posting guidelines for this group?

--
Email: http://www.gunnar.cc/cgi-bin/contact.pl

3. edGuest

Well here is what I have so far:

@params = (1,2,3,4,5,7,8,9);

foreach \$digits(@params){
\$total = numbers(\$digits);
print "\$numbers\n";
}

sub numbers (){
my \$digits = shift(@_);

foreach \$params(@params){
\$digits =+ @params;
}

return \$total
}

I can't get the average of the array to even get to the other part of
the code to divide the total.

ed, Sep 30, 2005
4. Josef MoellersGuest

ed wrote:
> Well here is what I have so far:

# Tell us something more ...
use warnings;
use strict;

> @params = (1,2,3,4,5,7,8,9);
>
> foreach \$digits(@params){
> \$total = numbers(\$digits);
> print "\$numbers\n";

Where does \$numbers come from?
The above pragmas would have told you it isn't defined.

> }
>
> sub numbers (){
> my \$digits = shift(@_);
>
> foreach \$params(@params){
> \$digits =+ @params;

You _do_ know that @params in scalar context is the number of elements
in @params?

> }
>
> return \$total
> }
>
> I can't get the average of the array to even get to the other part of
> the code to divide the total.

Let's see if we can dig up something based upon your initial request:
"calculate the average of the numbers, the total of
all the numbers added together, and a new array of numbers which is the
other numbers divided by 2."

my \$sum = 0, \$avg = 0;
my @half;
foreach (@params) {
\$sum += \$_;
push @half, \$_ / 2;
}
\$avg = \$sum / @params;
#
# \$avg: the average of the numbers
# \$sum: the total of all the numbers added together
# @half: a new array of numbers which is the other numbers divided by 2
print "average: ", \$avg, "\n";
print "total: ", \$sum, "\n";
print "new array: ", join(" ", @half), "\n";

I can't, for the heck of it, find out what you mean by
"It will then return a new list with all of
the information, and a new array of number, the total."

--
Josef Möllers (Pinguinpfleger bei FSC)
If failure had no penalty success would not be a prize
-- T. Pratchett

Josef Moellers, Sep 30, 2005

[ Please provide context when replying to a message, as the posting
guidelines recommend! Those who may be able to help don't usually access

ed wrote:
>> ed wrote:
>>> I'm trying
>>> to write a script that takes an array of numbers (arbitrary in size).
>>> The function will calculate the average of the numbers, the total of
>>> all the numbers added together, and a new array of numbers which is the
>>> other numbers divided by 2.

>>
>> Where is the code you've written?

>
> Well here is what I have so far:

<code without strictures and warnings snipped>

>> Have you read the posting guidelines for this group?

Why didn't you make an attempt to follow the posting guidelines, when

Why did you choose to even ignore that question?

--
Email: http://www.gunnar.cc/cgi-bin/contact.pl

6. edGuest

The course I'm taking hasn't gotten to the use of warnings/strict but I
As far as the number go, my frustration level is way up there and I've
changed my script so many times that at this point not sure where
numbers came from.
The way I understand this "You _do_ know that @params in scalar context
is the number of elements in @params? " is that the the number is eight
(total elements), not the numbers 1,2,3,4,5,7,8,9? I was told to get a
list of numbers from STDIN or a list or arguments, is my array not the
correct way to go?

ed, Sep 30, 2005
7. A. Sinan UnurGuest

"ed" <> wrote in
news::

> The course I'm taking hasn't gotten to the use of warnings/strict but

Ask for your money back. The *first* step in writing any Perl program
must be to enable strictures and warnings.

Have you read the posting guidelines, yet. They explain how you can help

> As far as the number go, my frustration level is way up there and I've
> changed my script so many times that at this point not sure where
> numbers came from.

Well, you need to take five minutes, read the guidelines, and follow
them step by step. Then, go over your assignment, and come up with a
spec for your program which others can understand. Then, follow the spec
step by step to write it (with strictures and warnings enabled).

That way, when you his a snag, others will be able to help you.

> The way I understand this "You _do_ know that @params in scalar
> context is the number of elements in @params? " is that the the
> number is eight (total elements), not the numbers 1,2,3,4,5,7,8,9?

If @params consists of (1, 2, 3, 4, 5, 6, 7, 8, 9), then @params has 9
elements. \$params[0] is the first element. \$params[8] is the last
element.

You'll need to start quoting an appropriate amount of context, and
responding to specific questions others have asked to have any hope of
staying out of killfiles.

Sinan
--
A. Sinan Unur <>
(reverse each component and remove .invalid for email address)

comp.lang.perl.misc guidelines on the WWW:

A. Sinan Unur, Sep 30, 2005
8. Guest

Josef Moellers <> wrote:
> Let's see if we can dig up something based upon your initial request:
> "calculate the average of the numbers, the total of
> all the numbers added together, and a new array of numbers which is the
> other numbers divided by 2."

> my \$sum = 0, \$avg = 0;

This should be either:

my \$sum = 0, my \$avg =0;
or

my (\$sum, \$avg);

otherwise \$avg will fail 'use strict'.

Axel

, Sep 30, 2005
9. edGuest

ed, Sep 30, 2005
10. Josef MoellersGuest

ed wrote:

> The course I'm taking hasn't gotten to the use of warnings/strict but I
> As far as the number go, my frustration level is way up there and I've
> changed my script so many times that at this point not sure where
> numbers came from.
> The way I understand this "You _do_ know that @params in scalar context
> is the number of elements in @params? " is that the the number is eight
> (total elements), not the numbers 1,2,3,4,5,7,8,9? I was told to get a
> list of numbers from STDIN or a list or arguments, is my array not the
> correct way to go?
>

Well, the choice of data structure surely is an important part of the
analysis and design phase of a program, but in Perl, an array would be a
good thing to use.

What I was aiming at was your use of @params here:
> \$digits =+ @params;

@params has a different meaning depending on whether it is used in list
context (where it denotes the entire array) or in scalar context (where
it denotes the number of elements in @params).
Finding out the current context is not always easy, but trying to add
something to a scalar definitely is scalar context and from your code,
adding the size of the array for each element of the array is definitely
not what you wanted to do.
It may also have been a mis-spelling, as
> foreach \$params(@params){

assigns each arramy element of "@params" to a scalar variable "\$params",
so your code would have been better if you'd written
> \$digits += \$params;

(I'd have use "\$param" instead of "\$params", as the variable contains a
single parameter only in each loop iteration).

Josef
--
Josef Möllers (Pinguinpfleger bei FSC)
If failure had no penalty success would not be a prize
-- T. Pratchett

Josef Moellers, Oct 4, 2005

Josef Moellers <> wrote:

> Finding out the current context is not always easy,

Sure it is. You just replace the construct that you are wondering

context()

where

sub context {
warn wantarray() ? 'list context' : 'scalar context'
}

--
Perl programming
Fort Worth, Texas

12. John W. KrahnGuest

> Josef Moellers <> wrote:
>
>>Finding out the current context is not always easy,

>
>
> Sure it is. You just replace the construct that you are wondering
>
> context()
>
> where
>
> sub context {
> warn wantarray() ? 'list context' : 'scalar context'
> }

sub context {
warn defined wantarray() ? wantarray() ? 'list' : 'scalar' : 'void', '
context'
}

John
--
use Perl;
program
fulfillment

John W. Krahn, Oct 4, 2005