the question is
Write a program that estimates then prints the value of the mathematical constant e by using the formula:
e=1+ 1/1!+ 2/2! + 3/3!+ ...
(calculate till 5/5!)
my answer
can any one tell what is wrong here?Code:#include <stdio.h> int main() { int factorial=1; float formula; int counter=0; float e=0.0; while(counter<=5) { if(counter==0){factorial=1; counter=counter+1;} factorial*=counter; formula=(float)counter/factorial; e=e+formula; counter++; } printf("%f",e); return 0; }
You don't indent your code.Originally Posted by hula2007
You need to tell us what's wrong. Do you get an error, a wrong result, ...?
Bye, Andreas
you are incrementing counter twice in the first iteration of the loop but as you are checking <= you get away with that, you dont need to assigin 1 to factorial in the IF statement as it was assigned 1 to start with and the WHILE control is not repeated (which would mean reassignment was neccesary) Your program is a bit clunky and could be cleaner, certainly better formatted, but apart from that it should work ok
Sorry but i dont really understand maths notation, wish i did..dont know what the ! means so i cant figure out your description of the formula relative to the code and tell you if there is a problem, but it could certainly be expressed with one line using appropriate parentheses to get the order right.
Last edited by rogster001; 03-17-2013 at 03:16 AM.
Thought for the day:FLTK: "The most fun you can have with your clothes on.""Are you sure your sanity chip is fully screwed in sir?" (Kryten)
Stroustrup:
"If I had thought of it and had some marketing sense every computer and just about any gadget would have had a little 'C++ Inside' sticker on it'"
It probably doesn't help that your mathematical description is wrong. I haven't even checked to see if the code matches the formula.
e = 1 + 1/1! + 1/2! + 1/3! + ..... not 1 + 1/1! + 2/2! + 3/3! + .....
