CGFloat floor to NSInteger

Question

In Xcode, the compiler is complaining about the following cast:

CGFloat width = 5.6f; NSInteger num = (NSInteger)floor(width);

Saying "cast from function call of type 'double' to non-matching type 'NSInteger' (aka 'int')"

One workaround would be to simply cast the CGFloat to NSInteger which truncates but I want to make the code clear/easy to read by explicitly flooring. Is there a function for flooring that returns an int? Or some other (clean) way of doing this?

My compiler settings under "Apple LLVM 6.0 - Compiler Flags", in "Other C Flags", I have -O0 -DOS_IOS -DDEBUG=1 -Wall -Wextra -Werror -Wnewline-eof -Wconversion -Wendif-labels -Wshadow -Wbad-function-cast -Wenum-compare -Wno-unused-parameter -Wno-error=deprecated

Thanks!

I tried the code and it works perfectly fine without any errors. Did you try compiling it again? This can sometimes help. — Palle
I think this may have to do with compiler settings--mine are on a fairly strict setting (and must stay that way). — Dave
i tried the code you specified and there were no errors. What settings do you have? — jacob bullock
In "Apple LLVM 6.0 - Custom Compiler Flags" in my Build Settings I have: -O0 -DOS_IOS -DDEBUG=1 -Wall -Wextra -Werror -Wnewline-eof -Wconversion -Wendif-labels -Wshadow -Wbad-function-cast -Wenum-compare -Wno-unused-parameter -Wno-error=deprecated — Dave

Palle Palle · Accepted Answer · 2015-07-29T01:15:11

Okay, as you mentioned strict compiler settings, I tried again and found the solution. The compiler warning is because you are trying to cast the floor function to a NSInteger value and not the returned value.

To solve this, all you have to do, is to put floor(width) in parentheses like this:

NSInteger num = (NSInteger) (floor(width));

or save the result of the floor operation to another CGFloat and cast the new variable to NSInteger

CGFloat floored = floor(width);
NSInteger num = (NSInteger) floored;

CGFloat floor to NSInteger

2 Answers