need help with passing 3 arrays into a function

tommy69 · #1 04-Apr-2004, 22:59

I have another program and boy, this is a tough one as well. I think I am slowly getting better and this is slow, but surely making sense the more I program. Sorry to keep bugging everyone.

Here is what I need to do:

I need to write a program that declares 3 single-dimensional arrays named bookPrice, numBooks, and totAmt. Each array should be declared in main90 and capable of storing 6 values. Make up numbers for price and quantity. Your program should pass these 3 arrays to a function named calc(), which should calculate the elements in the totAmt array as the product of the corresponding elements in bookPrice and numBooks. After calc() has put values into the totAmt array, the values in the array should be displayed from within main().

CPP / C++ / C Code:

#include <stdio.h>
#include <conio.h>

float calc( float bookPrice[], int numBooks [], float totAmt[],int numEls);  /*function prototype*/
                                     
int main()
{
     int a;
     
     float bookPrice[6]={4.95,6.50,8.75,12.25,16.60,20.50};
     int numBooks[6]={4,6,9,12,16,19};
     float totAmt[6];
	
	float calc();

	for (a=0;a<=6;++a)
		printf("\nThe total prices for each book is:  %.2f\n",  totAmt);

     getch();

     return 0;

}
float calc(float bookPrice[], int numBooks[], float totAmt[], int numEls) /*function declaration*/
{
     int a;

	
	 for (a=1;a<numEls;++a)
	
	   totAmt[a]=numBooks[a]*bookPrice[a];
	 
	 return (totAmt[a]);

}

WaltP · #2 05-Apr-2004, 01:25

Well, you told us what you need to do
You showed us what you wrote
You did not tell us what is wrong.

Please give us all the information we need to understand the problem you're having.

tommy69 · #3 05-Apr-2004, 05:19

My output is:

CPP / C++ / C Code:


The total prices for each book is:  0.00

The total prices for each book is:  0.00

The total prices for each book is:  0.00

The total prices for each book is:  0.00

The total prices for each book is:  0.00

The total prices for each book is:  0.00

The total prices for each book is:  0.00

The output should be the product of multiplying each element of the numBooks array by each element of the bookPrice array to get values into the totAmt array and then display these new values of the totAmt array.

tommy69 · #4 05-Apr-2004, 06:09

CPP / C++ / C Code:


#include <stdio.h>
#include <conio.h>

float calc( float bookPrice[], int numBooks [], float totAmt[],int);  /*function prototype*/
                                     
int main()
{
     int a;
     
     float bookPrice[6]={4.95, 6.50, 8.75, 12.25, 16.60, 20.50};
     int numBooks[6]={4,6,9,12,16,19};
     float totAmt[6];
	
	 calc(bookPrice,numBooks,totAmt,6); 

	for (a=0;a<=6;++a)
		printf("\nThe total prices for each book is:  %.2f\n",  totAmt);

     getch();

     return 0;

}
float calc (bookPrice[],numBooks[],totAmt[],int numEls) /*function declaration*/
{
     int a;

 
	 for (a=1;a<numEls;++a)
	
	 totAmt[a] = bookPrice[a]*numBooks[a]; 
	   
	 
	 return (totAmt[a]);

}

Output is:

The total prices for each book is: -107374176.00

The total prices for each book is: 39.00

The total prices for each book is: 78.75

The total prices for each book is: 147.00

The total prices for each book is: 265.60

The total prices for each book is: 389.50

Now all I am trying to figure out is why the first print otu is wrong.

tommy69 · #5 05-Apr-2004, 06:26

Never mind!! I changed this code:

Code:

 for (a=1;a<numEls;++a)

to

Code:

 for (a=0;a<numEls;++a)

Thanks for everyone's help, esp. to Aaron and Waltp. You guys definitely know your programming and should be promoted

JdS · #6 05-Apr-2004, 07:25

Quote:

Originally Posted by tommy69

...You guys definitely know your programming and should be promoted

Members get promoted automatically by their REPUTATION, so if they helped you, please use the REPUTATION feature to pass them some well deserved points.

aaroncohn · #7 05-Apr-2004, 12:06

Okay, tommy... well here's one thing I noticed. At the end of your calc function, your return value is totAmt[a]. The problem with this is that you're only returning the value stored in whatever element is in totAmt[a]. There's another problem. You don't want to return just one piece of the array, you want to return the whole thing. The deal with that is that you don't need to! Arrays are always passed byreference. This means that you do not have to have a return value in your function when you want to return an array. You can simply make your changes to the array within the function, and those changes will appear when you get back into main(). So, again, you can just let the function return void. Whatever changes you make to the array inside the function will also happen outside the function, so you do NOT need to return it.

For the output, your printf statement is trying to display the entire array at once, whereas you only want to display one element at a time. You have it in a loop that uses "a" as a counter, but that's all "a" is doing. Put "a" in the index for totAmt in your printf, and you'll be in business. Also, as a rule of thumb, usually you want to use a++ rather than ++a when you do increment operations, unless you specifically want ++a. You know what ++a does, right?

*edit* I have 200 posts and a spectacular aura about me

tommy69 · #8 05-Apr-2004, 12:17

This is the code that I handed in to the teacher. It's not much different thatn the one I posted earlier. For the printf statement, I did sue the totAmt[a]. I found this problem to be easier and maybe it's because I am slowly learning. Plus this problem is easy, esp for you guys.

CPP / C++ / C Code:

#include <stdio.h>
#include <conio.h>

float calc( float bookPrice[], int numBooks [], float totAmt[], int);  /*function prototype*/
                                     
int main()
{
     int a=0;
     

     float bookPrice[6]={4.95, 6.50, 8.75, 12.25, 16.60, 20.50};
     int numBooks[6]={4,6,9,12,16,19};
     float totAmt[6];
	
	 calc(bookPrice,numBooks,totAmt,6); 

	for (a=0;a<=5;a++)
		printf("\nThe total prices for each book is:  %.2f\n",  totAmt[a]);

     getch();

     return 0;

}
float calc (float bookPrice[],int numBooks[], float totAmt[],int numEls) /*function declaration*/
{
     int a;

	 for (a=0;a<numEls;a++)
	
	 totAmt[a] = bookPrice[a]*numBooks[a]; 
	  	 
	 return (totAmt[a]);

}

aaroncohn · #9 05-Apr-2004, 12:19

Were you forced to hand it it? I wouldn't hand it in unless it worked, even if that meant turning it in late.

tommy69 · #10 05-Apr-2004, 12:27

Quote:

Originally Posted by aaroncohn

Were you forced to hand it it? I wouldn't hand it in unless it worked, even if that meant turning it in late.

Pretty much so because this is the only teacher that I have ever known that we CAN"T turn it in late. It's due on the due date and if you try to turn it in late, you get a zero. Not sure why she does this, but there isnt anything I can do about it. I hope to get a decent grade. I didnt get a very good grade on one of my programs because I was stuck on a problem and I just turned it in anyways. I hate to keep asking people here about every problem I have.

Thread Tools	Search this Thread
Show Printable Version Email this Page	Search this Thread: Advanced Search
Rate This Thread
Rate This Thread:

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Help on passing in arrays in functions?	nusstu	C Programming Language	10	02-Apr-2004 10:42
Another problem dealing with main() and driver function	tommy69	C Programming Language	4	20-Mar-2004 19:46
Passing Pointers To Pointers in Functions	elumira	C Programming Language	8	05-Mar-2004 21:23
Calling functions within a function	spudtheimpaler	C Programming Language	5	02-Mar-2004 08:02
i/o: set value x (long) before it is used by a function	gmn	C Programming Language	1	18-Nov-2003 01:12