Why do I get the wrong answer?

[mtbrooks1993]

When I run this code I get a sum of 473 instead of a correct sum of 495. Why am I not getting the right answer?

#include<iostream>

using namespace std;

int arysum(int *, int);

int main()
{
	int const size = 9;
	int ary[size] = {33,55,77,99,88,66,44,22,11};
	int sum;

	sum = arysum(ary, size);

	cout<<sum<<endl;

	return 0;
}

int arysum(int *p, int size)
{
	int sum = 0;
	if(size == 1)
	{	
		cout<<*p<<endl;
		return *p;
	}
	cout<<*p<<endl;
	sum = *p + arysum(++p, --size);

	return sum;

}

[ne555]

In function ‘int arysum(int*, int)’:
29:32: warning: operation on ‘p’ may be undefined [-Wsequence-point]

[Pindrought]

I am not sure why your code doesn't work, but I rewrote it as so and it works this way.

#include<iostream>

using namespace std;

int arysum(int *, int);

int main()
{
	int const size = 9;
	int ary[size] = { 33, 55, 77, 99, 88, 66, 44, 22, 11 };
	int sum;

	sum = arysum(ary, size);

	cout << sum << endl;
	return 0;
}

int arysum(int *p, int size)
{
	int sum = 0;
	if (size == 1)
	{
		cout << *p << endl;
		return *p;
	}
	cout << *p << endl;
	sum = *p;
	++p;
	--size;
	sum += arysum(p, size);
	return sum;
}

The reason it wasn't working has something to do with your arguments in sum = *p + arysum(++p, --size);

[mtbrooks1993]

Thanks for the help! I will rewrite it like that.

[pheininger]

sum = *p + arysum(++p, --size);
I suspect undefined behavior (but I have not traced it to prove it).
I doubt there is a required order for the evaluation among *p, ++p, and the call to arysum. So which value of p is used for *p.

The rewrite eliminates any ambiguity.