Question

In my prepareForSegue method, I pass an immutable array retrieved from NSUserDefaults to a DetailViewController mutable dictionary property. Do I need to create a mutable copy of the array before I modify it or does that happen automatically in the NSMutableDictionary class setter method?

My code…

ViewController.m

- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender
{
    if ([segue.identifier isEqualToString:@"EditReminder"])
    {
        UINavigationController *navigationController = segue.destinationViewController;
        DetailViewController *detailViewController = [[navigationController viewControllers] objectAtIndex:0];
        detailViewController.delegate = self;
        [detailViewController setTitle:@"Edit Reminder"];
        // Pass ReminderData to detailVC if editing
        NSIndexPath *selectedRowIndex = [self.tableView indexPathForSelectedRow];
        NSArray *remindersArray = [[NSUserDefaults standardUserDefaults] objectForKey:@"reminders"];
        detailViewController.reminderData = [remindersArray objectAtIndex: selectedRowIndex.row];
    }
}

DetailViewController.h

@property (strong, nonatomic) NSMutableDictionary *reminderData;

DetailViewController.m

- (void)viewDidLoad
{
    [super viewDidLoad];

    if (self.reminderData) {
        // Reminder data from user defaults is immutable. Create mutable copy.
        // Is this necessary?
        self.reminderData = [self.reminderData mutableCopy];
    }
    else {
        self.reminderData = [[NSMutableDictionary alloc] init];
    }
}

Answer 1

You need to create the mutable copy, but your implementation is wrong.

In prepareForSegue, you need to do the mutableCopy, or you already have a wrong object stored in your property. There is no reason to do that in viewDidLoad, and it can be considered a bug.