What's the problem?

Question

#include <iostream>
using namespace std;

int main()
{

int a=0;
int b[3];
cin>> b[0]>>b[1]>>b[2];

cout<< "Sum of all digits:"<< endl;

for(int x=0 ; x>3 ; x++){

cout<<a+=b[x];
cout<<endl;
}

return 0;
}

Answer 1

what you have done will print the sum at each step, if you want to print just the total sum use this instead :

#include <iostream>
using namespace std;

int main()
{
int a=0;
int b[3];
cin>> b[0]>>b[1]>>b[2];

cout<< "Sum of all digits: ";
for(int x=0 ; x<3 ; x++){
a+=b[x];
}
cout <<a<<endl;
return 0;
}

I just moved the cout out of the loop.

Answer 2

the for loop have a problem, it should be:
for (int x=0; x<3;x++)

Answer 3

Super

Answer 4

The ">>" operator used with std::cout has precedence over the "+=" operator, thus the line

cout << a += b[ x ];

is interpreted as printing 'a' and then adding the value of "b[ x ]" to the output stream, which is not possible. Wrapping the assignment in parentheses should fix this problem, i.e.

cout << ( a += b[ x ] );

Answer 5

Sagar Thapaliya The code that is currently in the thread's description fails to compile due to what I said earlier, you can try it yourself.

Answer 6

The answer by Shadow was appropriate !

Answer by John Robotane was also correct but it didn't worked after it too.

Answer 7

John Robotane Thanks man!

Answer 8

x < 3