[user12345]

I have tried to get the sum of integer digits but i only get the sum of 12345 right. any other integer i enter, it gives the wrong sum. Please tell me where i am going wrong

my code is

#include <iostream>
#include <stdlib.h>
using namespace std;
void sumdigits(int );
void sumdigits(int )
{
	int a;
	int b[5];
	int sum=0;
	cout<<"Fill array with integer digits"<<endl;
	cin>>b[5];
	for ( a = 0 ; a < 6 ; a++)
{
  //Enter the integer values one by one to store in array
 sum = sum + b [5]; 
}
	cout<<"The sum of the digits is: "<<sum<<endl;
}
void main ()
{
int integer;
cout<<"Enter integer: ";
cin >> integer;
sumdigits(integer);

}

This post has been edited by JackOfAllTrades: 25 November 2012 - 03:54 AM
Reason for edit:: Fixed code tags

[snoopy11]

hi,

this line

sum = sum + b [5];
should be

sum = sum + b [a]; 

Please use code tags [ code] [ \code]

Snoopy

[AKMafia001]

Well! Lots of things to point out...
First, why you are passing an integer value to the sumdigits() which is never used in it...

Your array contains 5 elements, and you are counting 6, a should go till 4, so a < 5:

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

You are just adding some garbage value here, instead of the array index:

sum = sum + b [5];

The final index of the array is 4, 0-4, in order to add all the elements use the array index, b[a]....

Hope this Helps!

PS: And one thing real serious, cin>>b[5];...
You are thinking that this read 5 inputs in the array, but it doesn't...

You are accessing a location that is out of the array elements, a location out of bounds...

Even if you do this a[4], you will only read in the 5th location of the array, and not the whole...

So, just use a loop from 0-4 and take input on each index...

Edit: You have to read more about arrays here and here...

This post has been edited by AKMafia001: 25 November 2012 - 02:55 AM

[user12345]

Oh i got it now. Thanks a lot.