Fixing this code to avoid the rounding warning when compiling

Question

Hi

Take this code:

rectDraw.SetRect(0, 0, rectDraw.right, rectDraw.right / nRatioImage);

nRatioImage is of tale float. And the others are of type int.

At the moment I get a compile warning:

warning C4244: 'argument': conversion from 'float' to 'int', possible loss of data

I tried to adjust the line like this:

rectDraw.SetRect(0, 0, rectDraw.right, rectDraw.right / (int)nRatioImage);

But it yields the wrong results. What is the right way to use a cast with this line of code? I have a similar line here:

rectDraw.SetRect(0, 0, rectDraw.bottom * nRatioImage, rectDraw.bottom);

Thank you.

Answer 1

Hello,
be aware that a division/multiplication with a float will result in a float. If you cast the result to int, the value will be truncated. Use rounding functions like floor, ceil or so.

But if you only want to get rid of the warning, use this for example:

rectDraw.SetRect(0, 0, rectDraw.right, (int)(rectDraw.right / nRatioImage));
rectDraw.SetRect(0, 0, (int)(rectDraw.bottom * nRatioImage), rectDraw.bottom);

It will truncate the float result to int according to the compiler settings (normally without rounding).
Regards, Guido