Do I really need to convert my project currency to NSDecimalNumber? - ios

Yes, I know I should use NSDecimalNumber to deal with currency, money, price... I've read this. The problem is, I adapted an existed project, which use NSString and NSNumber (float, double, CGFloat...) as currency. They deal with floating point by using NSNumberFormatter, as I can see it's not a big problem (yet?). Those currency is stored to coredata.
Now, if I want to convert all of those currency into NSDecimalNumber, I'll have to do a massive refactor in the code and migration in coredata. Here come the question:
If (I assume) double, CGFloat, NSNumber can hold the value as large as NSDecimalNumber, why should I use NSDecimalNumber since I can use
other with NSNumberFormatter? Is it because of performance?
In case of the necessary of the converting, can I do an auto migration with the help of MappingModel only, of course), or do I have
to adapt a custom migration policy?
Because the coredata use both NSString and NSNumber as currency, so please help me find a solution to migrate from both data type. I'm not used to work with NSDecimalNumber in coredata. Thanks in advance.
EDIT
Okay, I got it that NSDecimalNumber is necessary. Please help me answer the second question: Can I do auto migration, using mappingModel + the thing like FUNCTION($source.price, "decimalValue") (this is incorrect since decimalValue return NSDecimal, not NSDecimalNumber). Do I really have to write a custom migration policy?

Do I really have to write a custom migration policy?
Yes. Create a subclass of NSEntityMigrationPolicy for each entity that needs conversion and implement the menthod createDestinationInstancesForSourceInstance:entityMapping:manager:error:. Enter the name of this class in the Custom Policy of the entity in the mapping model.
- (BOOL)createDestinationInstancesForSourceInstance:(NSManagedObject *)sInstance
entityMapping:(NSEntityMapping *)mapping manager:(NSMigrationManager *)manager
error:(NSError **)error
{
BOOL aBool = [super createDestinationInstancesForSourceInstance:sInstance entityMapping:mapping manager:manager error:error];
if (aBool)
{
NSArray *aSourceArray = [[NSArray alloc] initWithObjects:&sInstance count:1];
NSArray *aDestArray = [manager destinationInstancesForEntityMappingNamed:[mapping name] sourceInstances:aSourceArray];
if (aDestArray && [aDestArray count] > 0)
{
NSManagedObject *dInstance = [aDestArray objectAtIndex:0];
// conversion
}
}
return aBool;
}

The issue is that double, CGFloat, and those value types stored in NSNumber cannot precisely handle decimal values. Thus, if you set 4.50, you may end up with some close approximation. This is generally not desirable for monetary values, as you want exact representations. You can perhaps get away with rounding using NSNumberFormatter, but if you start doing math with these values or need more accuracy than hundredths, you could end up with incorrect amounts.
I suspect you would have to do a heavy-weight migration to change the type of an attribute on a CoreData entity. Even if you didn't have to, you might want to, so that you could ensure the values that you convert to Decimal are rounded to the precision you choose (since they may or may not be what you want in the database right now).
It sounds like you have some technical debt in your project. Wether you choose to pay that debt down now or wait until it grows is up to you in the end.

You probably want to use NSDecimalNumber, not NSDecimal. NSDecimal is a struct, while NSDecimalNumber is an Objective-C class that inherits from NSNumber. Thus you can usually pass an NSDecimalNumber wherever an NSNumber is accepted. NSNumberFormatter formats NSDecimalNumber instances automatically, and if you set your formatter's generatesDecimalNumbers to YES, it will parse strings into NSDecimalNumber instances.
The range of NSDecimal and NSDecimalNumber is larger than double or CGFloat. The decimal types store more significant digits and have a larger exponent range.
I don't know the answer to the auto migration question, but I do know that Core Data supports NSDecimalNumber. You can set an attribute's type to decimal (NSDecimalAttributeType) in your model.

Related

iOS - Why Does It Work When I Compare Two NSNumbers With "=="?

In my app, I accidentally used "==" when comparing two NSNumber objects like so:
NSNumber *number1;
NSNumber *number2;
Later on, after these objects' int values were set, I accidentally did this:
if (number1 == number2) {
NSLog(#"THEY'RE EQUAL");
}
And, confusingly, it worked! I could have sworn I was taught to do it this way:
if (number1.intValue == number2.intValue) {
NSLog(#"THEY'RE EQUAL");
}
How did using "==" between the two NSNumber objects work, and why? Does that mean it's okay to compare them that way, or was it just a fluke and this is generally not guaranteed to work every time? It really confused me :(
It's not a fluke.
It's due to the tagged pointers feature of the Objective-C runtime while using an ARM64 CPU.
In Mac OS X 10.7, Apple introduced tagged pointers. Tagged pointers allow certain classes with small amounts of per-instance data to be stored entirely within the pointer. This can eliminate the need for memory allocations for many uses of classes like NSNumber, and can make for a good performance boost.[…] on ARM64, the Objective-C runtime includes tagged pointers, with all of the same benefits they've brought to the Mac
Source
That is possibly a fluke.
From NSHipster :
Two objects may be equal or equivalent to one another, if they share a common set of observable properties. Yet, those two objects may still be thought to be distinct, each with their own identity. In programming, an object’s identity is tied to its memory address.
Its possible that your statement evaluated to YES because number1 and number2 were pointing to the same object. This would not work if they had the same value but were two different objects.
The obvious reason NSNumber variables would point to the same would be that you explicitly assigned one to the other, like so:
number1 = number2;
But there's one other thing. From this answer :
This is likely either a compiler optimisation or an implementation detail: as NSNumber is immutable there's no need for them be separate instances. probably an implementation optimisation thinking about it. Likely numberWithInt returns a singleton when called subsequently with the same integer.
But anyways, its safest to use isEqualToNumber:, as there is no telling what other "things" are lurking in the depths of code that may or may not cause it to evaluate YES
From RyPress :
While it’s possible to directly compare NSNumber pointers, the isEqualToNumber: method is a much more robust way to check for equality. It guarantees that two values will compare equal, even if they are stored in different objects.
There two concepts of equality at work here:
Object identity: Comparing that two pointers point to the same objects
Value equality: That the contents of two objects are equal.
In this case, you want value equality. In your code you declare two pointers to NSNumber objects:
NSNumber *number1;
NSNumber *number2;
But at no point show assignment of a value to them. This means the contents of the pointers can be anything, and quite by chance you have two pointers pointing to the memory locations (not necessarily the same ones) where (number1.intValue == number2.intValue) happens to be true.
You can expect the behaviour to change in unstable ways - for instance as soon as you add any more code.
Of course you can compare two NSNumber* with ==. This will tell you whether the pointers are equal. Of course if the pointers are equal then the values must be the same. The values can be the same without the pointers being equal.
Now you need to be aware that MaxOS X and iOS do some significant optimisations to save storage, especially in 64 bit code. Many NSNumbers representing the same integer value will actually be the same pointer.
NSNumber* value1 = [[NSNumber alloc] initWithInteger:1];
NSNumber* value2 = [[NSNumber alloc] initWithInteger:1];
These will be the same pointers. In 64 bit, many others will be the same pointers. There are only ever two NSNumber objects with boolean values. There is only ever one empty NSArray object, and only one [NSNull null] object.
Don't let that lull you into any wrong assumptions. If you want to see if two NSNumbers have the same value, use isEqualToNumber: You may say "if (number1 == number2 || [number1 isEqualToNumber:number2])"; that's fine (didn't check if I got the names right).

Comparing NSNumber instances with isEqualToNumber

Yes, I've read the other posts on stackoverflow about comparing NSNumber and none of them seem to quite address this particular situation.
This solution was particularly bad ... NSNumber compare: returning different results
Because the suggested solution doesn't work at all.
Using abs(value1 - value2) < tolerance is flawed from the start because fractional values are stripped off, making the tolerance irrelevant.
And from Apple documentation... NSNumber explicitly doesn't guarantee that the returned type will match the method used to create it. In other words, if you're given an NSNumber, you have no way of determining whether it contains a float, double, int, bool, or whatever.
Also, as best I can tell, NSNumber isEqualToNumber is an untrustworthy method to compare two NSNumbers.
So given these definitions...
NSNumber *float1 = [NSNumber numberWithFloat:1.00001];
NSNumber *double1 = [NSNumber numberWithDouble:1.00001];
If you run the debugger and then do 2 comparisons of these identical numbers using ==, one fails, and the other does not.
p [double1 floatValue] == [float1 floatValue] **// returns true**
p [double1 doubleValue] == [float1 doubleValue] **// returns false**
If you compare them using isEqualToNumber
p [float1 isEqualToNumber:double1] **// returns false**
So if isEqualToNumber is going to return false, given that the creation of an NSNumber is a black box that may give you some other type on the way out, I'm not sure what good that method is.
So if you're going to make a test for dirty, because an existing value has been changed to a new value... what's the simplest way to do that that will handle all NSNumber comparisons.. not just float and double, but all NSNumbers?
It seems that converting to a string value, then compariing would be most useful, or perhaps a whole lot of extra code using NSNumberFormatter.
What are your thoughts?
It is not possible to reliably compare two IEEE floats or doubles. This has nothing to do with NSNumber. This is the nature of floating point. This is discussed in the context of simple C types at Strange problem comparing floats in objective-C. The only correct way to compare floating point numbers is by testing against a tolerance. I don't know what you mean by "fractional values are stripped off." Some digits are always lost in a floating point representation.
The particular test value you've provided demonstrates the problems quite nicely. 1.00001 cannot be expressed precisely in a finite number of binary digits. Wolfram Alpha is a nice way to explore this, but as a double, 1.00001 rounds to 1.0000100000000001. As a float, it rounds to 1.00001001. These numbers, obviously, are not equal. If you roundtrip them in different ways, it should not surprise you that isEqualToNumber: fails. This should make clear why your two floatValue calls do turn out to be equal. Rounded to the precision of float, they're "close enough."
If you want to compare floating point numbers, you must compare against an epsilon. Given recent advances in compiler optimization, even two identical pieces of floating point C code can generate slightly different values in their least-significant digits if you use -Ofast (we get big performance benefits by allowing that).
If you need some specific number of significant fractional digits, then it is usually better to work in fixed point notation. Just scale everything by the number of digits you need and work in integers. If you need floating point, but just want base-10 to work well (rather than base-2), then use NSDecimalNumber or NSDecimal. That will move your problems to things that round badly in base-10. But if you're working in floating point, you must deal with rounding errors.
For a much more extensive discussion, see "What Every Programmer Should Know About Floating-Point Arithmetic."

When would you use NSNumber literal to create encapsulated character values?

I'm just going through Apple's iOS development tutorial at the moment and reading the chapter on the Foundation framework and value objects.
Just on the NSNumber class, it says:
You can even use NSNumber literals to create encapsulated Boolean and
character values.
NSNumber *myBoolValue = #YES; NSNumber *myCharValue = #'V';
I'm just wondering, when, or why, or in what scenario, might you want to use NSNumber for a character value rather than using NSString, say?
An NSNumber is useful for encapsulating primitive values to be inserted into Objective-C collection classes such as NSArray, NSSet, NSDictionary, etc.
Image a scenario where you would want to iterate over each character in an ASCII string and extract a unique set of vowels used. You can evaluate each character and add it to an NSMutableSet. To do so, you would need to encapsulate each character in an NSNumber as NSMutableSet expects an Objective-C object. This is just one example, but the concept applies to many situations where primitives need to be added into a collection.
Well, one case is where you're using KVC to set a value for a key, and the property type is char:
[object setValue:#'a' forKey:someCharPropertyName];
You can use NSNumber with characters to return its ASCII Code, so V would return 86.
I don't think many people use it that much, but you could probably use it for character validation. I think it just one of those things where Apple went, yeah, lets put that in for the heck of it.
It's really not used for much else. The #YES and #NO is the same as YES and NO, so its kinda inelegant in some places.

Proper way of using NSDecimalNumber as properites of a custom class

Currently I am pulling data from a webservice and populating the data into a custom object.
I am storing decimals such as 4.56 etc.
I am slightly confused by NSDecimal and NSDecimalNumber. I have read that NSDecimalNumber should be used when dealing with money, which I am. So the question is whether or not my properties should be NSDecimal or NSDecimalNumber. I have read cases where your model would be NSDecimal and that you would use NSDecimalNumber for any arithmetic with the numbers.
I basically want to create behavior like such in ObjectiveC
private decimal thirtyYear;
public decimal getThirtyYear(){
return thirtyYear/100.0;
}
public void setThrityYear(decimal rate){
thirtyYear = rate;
}
So, should my thirtyYear property be NSDecimal or NSDecimalNumber. Also, when doing the dividing within the getThirtyYear() method should I use NSDecimalNumber for the arithmetic.
Question #1: Should my thirtyYear property be NSDecimal or NSDecimalNumber?
As you stated if you're dealing with any financial calculations always use NSDecimalNumber and NSDecimalNumber provides predictable rounding behavior and precision when performing base 10 calculations.
If you want more detailed information this is from the Apple Documentation directly:
NSDecimalNumber is an immutable subclass of NSNumber that provides an
object-oriented wrapper for doing base-10 arithmetic. An instance can
represent any number that can be expressed as mantissa x 10 exponent
where mantissa is a decimal integer up to 38 digits long, and exponent
is an integer between -128 and 127.
Now in regards to NSDecimal, this is a C struct and not as easy to work with in Objective C. As stated above that's where the NSDecimalNumber object comes in which is made to work with object oriented languages. In short NSDecimalNumber is the way to go. It is easier to work with.
Question #2: When dividing within the getThirtyYear() method should I use NSDecimalNumber for the arithmetic?
If precision calculations are crucial in your app then yes use NSDecimalNumber since they're guaranteed to be accurate.
Since you will be using NSDecimalNumber I suggest checking out the NSDecimalNumber Class Reference because the class has a number (no pun intended) of useful methods. Hope this helps.

NSNumber arithmetic

I want to perform some simple arithmetic on NSNumbers and preserve the type. Is this possible?
For example:
- (NSNumber *)add:(NSNumber *)firstNumber to:(NSNumber *)secondNumber;
Is my method definition, and firstNumber and secondNumber are integers then I would like to return an integer that is the integer addition of them both. Equally if both are doubles then to return the result as a double.
It looks like I can get the type (except for boolean) using [NSNumber objCType] as found in this question: get type of NSNumber but I can't seem to extract those types and do the calculation without lots of code to extract the values do the calculation and return the result for every possible type.
Is there a short and concise way of doing this?
If you want to perform arithmetic the best bet would be using an NSDecimalNumber.
NSDecimalNumber have methods to perform arithmetic operations like :
– decimalNumberByAdding:
– decimalNumberBySubtracting:
– decimalNumberByMultiplyingBy:
– decimalNumberByDividingBy:
– decimalNumberByRaisingToPower:
– decimalNumberByMultiplyingByPowerOf10:
– decimalNumberByAdding:withBehavior:
– decimalNumberBySubtracting:withBehavior:
– decimalNumberByMultiplyingBy:withBehavior:
– decimalNumberByDividingBy:withBehavior:
– decimalNumberByRaisingToPower:withBehavior:
– decimalNumberByMultiplyingByPowerOf10:withBehavior:
And since NSDecimalNumber extends NSNumber it also have all methods of an NSNumber, so i think that you could use it in your case without any problem.
For nearly all applications it will be fine to convert to double and back using -doubleValue and –initWithDouble:. This will let you use the standard C symbols (+, -, ...) and functions (exp(), sin()). The only way you would run into trouble is if you were using close to the full precision for 64-bit integer values.
If you want to stick with Objective-C class operations you can use NSDecimalNumber instead.
See also: How to add two NSNumber objects?
How about calculating the expression value as a double (with all the inputs as double values), and then checking if the result is an integer? Then you just use NSNumber numberWithInt: or NSNumber numberWithDouble: to return the result.
When you check if the result value is an integer, be sure to account for the rounding error (e.g. when 1 is expressed as 0.99999999, etc).
EDIT: Just noticed in the docs for NSNumber this phrase:
Note that number objects do not necessarily preserve the type they are
created with.
I think this means you can't reliably do what you're trying to do.

Resources