[graFight]

I feel like i'm doing the actual function right but i'm not sure how to actually call it to see if it is right

I keep getting an error on the call line.

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

double arrayAvg(double D[], int len);

int main()
{
        double arr[4] = {1.0, 2.0, 3.0, 4.0};
        
        double value = arrayAvg(arr[], 4);

        printf("%f\n", value);
        
        system("PAUSE");
        return 0;
        
}


double arrayAvg(double D[], int len)
{
       double sum;
       double average;
       
       for(int i=0; i <= len; i++)
       {
              sum += [i];     
       }
       
       average = sum / (double) len;
       return average;
     }

This post has been edited by graFight: 29 January 2013 - 05:22 PM

[macosxnerd101]

Call arrayAvg(arr, 4);, removing the brackets around arr when invoking the function.

[Adak]

graFight, on 29 January 2013 - 05:21 PM, said:

I feel like i'm doing the actual function right but i'm not sure how to actually call it to see if it is right

I keep getting an error on the call line.

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

double arrayAvg(double D[], int len);

int main()
{
        double arr[4] = {1.0, 2.0, 3.0, 4.0};
        
        double value = arrayAvg(arr, 4);

        printf("%f\n", value);
        
        system("PAUSE");
        return 0;
        
}


double arrayAvg(double D[], int len)
{
       double sum=0.0;
       double average;
       
       for(int i=0; i <= len; i++)
       {
              //sum += [i];     
              sum+=i;
       }
       
       average = sum / (double) len;
       return average;
     }

[graFight]

ok so now that i switched it to just arr that part is working but now i get an error with sum += [i] and then i tried sum += i but that will not give the correct results...

[CTphpnwb]

Just what do you think [i] is??? Why aren't you using the array?

[Adak]

Let me guess - you forgot:

double sum=0.0;

If you want to average the array, then don't just average the index!

This post has been edited by Adak: 29 January 2013 - 05:48 PM

[graFight]

CTphpnwb, on 29 January 2013 - 05:45 PM, said:

Just what do you think [i] is??? Why aren't you using the array?

i'm sorry i also tried D[i] and still gives me a wrong answer.

Adak, on 29 January 2013 - 05:46 PM, said:

Let me guess - you forgot:

double sum=0.0;

actually yes but still does not fix the new issue lol

[Adak]

It works fine for me. Did you change sum+=i to sum+=D[i]?

[CTphpnwb]

I get 2.500000 and that seems right to me.

Using D[i] by the way.

[graFight]

Adak, on 29 January 2013 - 05:53 PM, said:

It works fine for me. Did you change sum+=i to sum+=D[i]?

ya, thats when it gave me a garbage number.

CTphpnwb, on 29 January 2013 - 05:55 PM, said:

I get 2.500000 and that seems right to me.

Using D[i] by the way.

Hmm...idk why mine is showing some weird number then

[Adak]

You either have a copy error, or you made some changes manually to your program, but didn't make all the changes that you should have made.

Here's your program fully working - copy and paste all of it, and run it. Note that I have extended the array size to FIVE, instead of four, to make the average more obvious to me.

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

double arrayAvg(double D[], int len);

int main()
{
        double arr[5] = {10.0, 9.0, 8.00, 11.0, 12.0};
        
        double value = arrayAvg(arr,5); 

        printf("%f\n", value);
        
        system("PAUSE");
        return 0;
        
}

double arrayAvg(double D[], int len)
{
       double sum=0.0;
       double average;
       
       for(int i=0; i < len; i++)  //correction! (thanks jjl! )
       {
              sum += D[i];     
       }
       
       average = sum / (double) len;
       return average;
}

I suggest you change your indentation value from 7 chars wide, to either 3,4,or 5 chars wide.

This post has been edited by Adak: 30 January 2013 - 02:07 AM

[jjl]

Quote

Here's your program fully working - copy and paste all of it, and run it.

Your program looks outsides the bounds of the array in your for loop.

This post has been edited by jjl: 30 January 2013 - 01:03 AM

[Adak]

Thanks, jjl! Correction made.

[raghav.naganathan]

Well, don't you think that 'i' should be declared outside the for loop to be in accordance with C99 mode.

something like this.

 int i;
 for( i=0; i < len; i++)  //correction! (thanks jjl! )
 {
     sum += D[i];    
 }

regards,
Raghav

This post has been edited by raghav.naganathan: 30 January 2013 - 02:32 AM

[Adak]

No. It is working just fine.

I prefer the variable be declared once at the top of the function, but this works for me, as well.

If your compiler is giving out errors because of it, then you have a problem, for sure, and should change where i is declared.

This post has been edited by Adak: 30 January 2013 - 07:22 AM