26
double a;
a = 3669.0;
int b;
b = a;

I am getting 3668 in b, instead of 3669.

How do I fix This problem? And if have 3559.8 like that also I want like 3559 not 3560.

Donald Duck
  • 7,638
  • 19
  • 69
  • 90
ratty
  • 12,776
  • 29
  • 74
  • 107

4 Answers4

64

I suspect you don't actually have that problem - I suspect you've really got:

double a = callSomeFunction();
// Examine a in the debugger or via logging, and decide it's 3669.0

// Now cast
int b = (int) a;
// Now a is 3668

What makes me say that is that although it's true that many decimal values cannot be stored exactly in float or double, that doesn't hold for integers of this kind of magnitude. They can very easily be exactly represented in binary floating point form. (Very large integers can't always be exactly represented, but we're not dealing with a very large integer here.)

I strongly suspect that your double value is actually slightly less than 3669.0, but it's being displayed to you as 3669.0 by whatever diagnostic device you're using. The conversion to an integer value just performs truncation, not rounding - hence the issue.

Assuming your double type is an IEEE-754 64-bit type, the largest value which is less than 3669.0 is exactly

3668.99999999999954525264911353588104248046875

So if you're using any diagnostic approach where that value would be shown as 3669.0, then it's quite possible (probable, I'd say) that this is what's happening.

Jon Skeet
  • 1,335,956
  • 823
  • 8,931
  • 9,049
  • 1
    Try adding something like `printf("a = %.50f\n", a);` to see the actual value of `a`. – Keith Thompson Oct 05 '11 at 06:20
  • @KeithThompson: Personally I use a bit of code I've got in C# which handles the binary value directly :) – Jon Skeet Oct 05 '11 at 06:24
  • 5
    This is the answer. And an excellent answer at that! My question: What happens when @Jon Skeet's score gets too big for SO? Will our appreciation for Jon Skeet eventually kill SO? – Del Jan 13 '15 at 10:23
7
main() {
    double a;
    a=3669.0;
    int b;
    b=a;
    printf("b is %d",b);
  
}

output is :b is 3669

when you write b=a; then its automatically converted in int

see on-line compiler result : 

http://ideone.com/60T5b


This is called Implicit Type Conversion Read more here https://www.geeksforgeeks.org/implicit-type-conversion-in-c-with-examples/

Jeegar Patel
  • 24,980
  • 49
  • 151
  • 213
3

This is the notorious floating point rounding issue. Just add a very small number, to correct the issue.

double a;
a=3669.0;
int b;
b=a+ 1e-9;
Shamim Hafiz - MSFT
  • 20,466
  • 38
  • 110
  • 169
-2
int b;
double a;
a=3669.0;
b=a;
printf("b=%d",b);

this code gives the output as b=3669 only you check it clearly.

Gouse Shaik
  • 340
  • 1
  • 2
  • 14