[sf18]

I am passing 3 int arguments into a function. And in the function I am calculating the average. How would do I return the output (average) as a double into the main? Hope this question makes sense. My output now comes out as 0.0000. What am I doing wrong?

Thanks for the helpful replies

#include <stdio.h>
#include <stdlib.h>

int funcAverage( int x, int y, int z ){
	double sumTotal, average;

	sumTotal = x + y + z;
	average = sumTotal / 3;


	return average;
}


main(){
int value1 = 9, value2 = 5 , value3 = 12, average1;


average1 = funcAverage( value1, value2, value3 );
printf("\nThe first average is: %.4lf\n\n", average1);


	system("pause");

}

This post has been edited by sf18: 23 February 2010 - 02:24 PM

[KYA]

You're intermixing ints and doubles.


You're using variables that aren't declared and then printing out values that have no meaningful value. [assign the return of the function to average1]

[sf18]

@KYA

Not quite sure if I understand your answer, but I made a few typos in my code and just edited it. I'm still getting 0.0000. Any other suggestions.

[Aphex19]

You return a double simply by specifying the return type of the function as double.

double funcAverage( int x, int y, int z ){
        double sumTotal, average;

        sumTotal = x + y + z;
        average = sumTotal / 3;


        return average;
}

Keep in mind that you can return just about any data type, like strings, char's, pointers etc...

Another thing to keep in mind is that typecasting will allow you to force the variable to a double if you wish. Here is an example

int num = 5;
printf("%f", (double)num / 2);
// Or you can typecast this way...
printf("%f", double(num) / 2);

This post has been edited by Aphex19: 23 February 2010 - 02:27 PM

[sf18]

@Aphex19

Here is my printf:

printf("\nThe first average is: %.4lf\n\n", (double)average1);

But now it outputs as: 8.0000 and it should be 8.6667.

??

[Aphex19]

For two main reasons.

1) you're averaging function is returning in integer when it needs to return a double.

2) The variable you are declaring to hold the return value of you're averga function is an integer. It should be a double as it needs to hold a double.

One more thing, it is good practice to return 0 from the main.

#include <stdio.h>
#include <stdlib.h>

// This function needs to return a double
double funcAverage( int x, int y, int z )
{
    double sumTotal, average;

    sumTotal = x + y + z;
    average = sumTotal / 3;

    return average;
}


int main()
{
    int value1 = 9, value2 = 5 , value3 = 12;
    double average1;    // Averge1 must be a double as it takes the return value of funcAverage

    average1 = funcAverage( value1, value2, value3 );
    printf("\nThe first average is: %.4lf\n\n", average1);

    system("pause");
    return 0;
}

EDIT: changed printf, doesnt need to typecast to double anymore.

This post has been edited by Aphex19: 23 February 2010 - 02:45 PM

[jjl]

really your function can just be simplified to

// This function needs to return a double
double funcAverage( int x, int y, int z )
{
	return (double)(x+y+z)/3;
}

but either way works

[sf18]

@Aphex19

OoOoOooo okay thanks for the break down. I very much appreciate it!

[Aphex19]

sf18, on 23 February 2010 - 01:47 PM, said:

@Aphex19

OoOoOooo okay thanks for the break down. I very much appreciate it!

you're welcome