I am trying to make some functions that take an array of n numbers and do certain things to them (like getting the sum, calculate statistical deviation, etc...)
Right now, I have:
But that isn't working. What's the problem?Code:int n; int main(void) { printf("How many numbers?:"); scanf("%f", n); float numbers[n]; ... ... ... sum=sum(numbers[n]); printf("Sum is:%f", sum); } float sum(float numbers[n]) { int x; float sum; for(x=0; x==n; ++x) { sum=numbers[x]+sum; } return sum; }
Make that your function accepts a float pointer. That'll make it work.
Teacher: "You connect with Internet Explorer, but what is your browser? You know, Yahoo, Webcrawler...?" It's great to see the educational system moving in the right direction
What do you mean?
Code:float sum(float* numbers) { int x; float sum; for(x = 0; x == n; ++x) { sum = numbers[x] + sum; } return sum; }
Variable length arrays are were only introducted in the C99 standard, as with mixed declarations.
Naturally you'll have to include the length of the array as an argument to the function, since otherwise you'll have no way of knowing when it ends.
[edit]
Ah, I see you are using globals. Disregard the above then, and continue on with your bad programming habits.
[/edit]
Quzah.
Last edited by quzah; 11-19-2006 at 08:26 PM.
Hope is the first step on the road to disappointment.
you need to initialize sum to zero aswell
better this way:
Code:float sum(float* numbers, int count) { int x; float sum = 0.0; for(x = 0; x < count; ++x) { sum += numbers[x]; } return sum; }
All problems in computer science can be solved by another level of indirection,
except for the problem of too many layers of indirection.
– David J. Wheeler
