There is some errors in the code that I can’t seem to spot.
The code is
# include <iostream>
using namespace std;
int main ()
{
int N; // Declaring and initializing variables
int n=0;
int i=0;
int fact=1;
cout << "Please Enter an Integer" << endl; // The prompt
cin >> N; // To read from the keyboard
// Part a
if (N==1 || N==0)
{ cout << "The Factorial of " << N << " = 1" << endl; } // Because it's known that the fact. of 0/1 = 1
while (N>1 && i>=0)
{ fact = fact * (N-i);
i = i+1;
}
cout << "The factorial of " << N << " = " << fact << endl;
// Part b
for ( int sigma=0; n=0 || n<=N ; n++)
{ sigma = sigma + (n*n);
}
return 0;
}
If you are debugging code (and that is what you are doing here), you need to step through the code and check your assumptions. That can be done with a traditional debugger that lets you see each line as it is run, or by putting some output statements in the code.
Your errors include
(a) the loop for the factorial never is run. since neither N nor i are changed in the while loop. This should be fairly obvious since you never reached the "factoraila == " statement
(b) you are using a loop to calculate [tex]\sum n^2[\tex] ?? There are simple formula to get the result, no loop.
If you are debugging code (and that is what you are doing here), you need to step through the code and check your assumptions. That can be done with a traditional debugger that lets you see each line as it is run, or by putting some output statements in the code.
Your errors include
(a) the loop for the factorial never is run. since neither N nor i are changed in the while loop. This should be fairly obvious since you never reached the "factoraila == " statement
(b) you are using a loop to calculate [tex]\sum n^2[\tex] ?? There are simple formula to get the result, no loop.
How come it is not changing in part (a) I did increament it
(b) I am using the baisc concept of it "Mathematically ... should it be altered somehow ?
while (N>1 && i>=0)
{ fact = fact * (N-i);
i = i+1;
}
Yes, i changes, but not in a way that will get you out of this loop. N doesn't change, so this is an infinite loop. Once inside, you'll never get out (well, eventually, after i overflows, you might possibly get out, but that'll take a few billion iterations). Change the conditions on your while loop so that it is no longer an infinite loop.
while (N>1 && i>=0)
{ fact = fact * (N-i);
i = i+1;
}
Yes, i changes, but not in a way that will get you out of this loop. N doesn't change, so this is an infinite loop. Once inside, you'll never get out (well, eventually, after i overflows, you might possibly get out, but that'll take a few billion iterations). Change the conditions on your while loop so that it is no longer an infinite loop.
aha ... so the problem is with the condition .. exactly in using (&&) ?
aha ... so the problem is with the condition .. exactly in using (&&) ?
No its not just the &&, its the whole statement. You need to rethink your condition. The one you have now is incorrect. Take a look at my earlier post for ideas.
// Part b
cout << "Please Enter an Integer" << endl; // The prompt
cin >> N; // To read from the keyboard
sum=0;
for ( n=0 ; n<=N ; n++)
{
sum = sum + (n*n);
}
return 0;
}
It did not use to appear so I thought I would add the prompt again
the prompt appeared and I entered the number but then it terminates
I think I don't have to put the prompt again .. maybe there's something wrong with the code at part b .. any hints?
It doesnt appear that you have sum declared anywhere, unless you included it in the part A of the code. Also, you are not telling the program to do anything after the loop takes place. Output the variable sum after the for loop to see if your program actually works. ;)
