I think the problem isfor( int i=1;i0;j--){ i*=j; //You are changing the value of i here } e+=1.0/i; }