Is there a better way to avoid 'Sign comparison' warning when comparing NSIndexPath's row and NSArray count?

StackOverflow https://stackoverflow.com/questions/21724824

I turned 'Signed Comparision' (aka -Wsign-compare) warnings for my iOS project in XCode (surprisingly, it was off by default). After that lots of warnings like this appeared:

/Users/michalciuba/projects/GlobeMobile/Classes/ACMailController.m:86:19: Comparison of integers of different signs: 'NSInteger' (aka 'long') and 'NSUInteger' (aka 'unsigned long')

They are usually caused by comparing row property of NSIndexPath which is NSInteger to the value returned by 'count' method of NSArray, like this:

if(indexPath.row < [self.myDataArray count]) 

The warning can be simply fixed by casting:

if(indexPath.row < (NSInteger)[self.myDataArray count]) 

However, it has to be done if every single place where such values are being compared. And they are being compared in dozens of places. I wonder if there is a better and more clever way to solve this problem? I don't want to turn off this warning, because it may help to prevent issues like unsigned integer underflow.

有帮助吗?

解决方案

You could implement a category that does it for you :

@implementation NSArray (SignedCount)

- (NSInteger) signedCount
{
    NSInteger count = (NSInteger)[self count];
    return count;
}

@end
许可以下: CC-BY-SA归因
不隶属于 StackOverflow
scroll top