attempt to insert nil object from objects[0]?

Question

In my application I have a UITableView and some buttons that the user can click to sort the array to the order based upon some NSDate's or ints. So this is my method to try to sort my UITableView:

- (void)sortStats:(id)sender reloadData:(BOOL)dataBool {
    NSSortDescriptor *descriptor = nil;
    UIButton *clickedButton = (UIButton*)sender;
    if (clickedButton.tag == 1) {
            descriptor = [NSSortDescriptor sortDescriptorWithKey:@"Date" ascending:NO];
    }
    [self.cellArray sortUsingDescriptors:[NSArray arrayWithObject:descriptor]];
    [[NSUserDefaults standardUserDefaults] setObject:self.cellArray forKey:@"cellArray"];
    if (dataBool) {
        [ivstatstableView reloadData];
    }
    [ivstatstableView scrollRectToVisible:CGRectMake(0, 0, 1, 1) animated:YES];
}

So originally I just had this method without the reloadData parameter because it seemed that reloadData on the UITableView was causing the crash.

This is what the console crash log is:

Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[__NSPlaceholderArray initWithObjects:count:]: attempt to insert nil object from objects[0]'

Anyway is there any code here that would be causing this crash? I really want to get this functionality working in my app and I know I am close to fixing it but I am just not sure whats causing this issue.

Any ideas?

score 14 · Accepted Answer · answered Aug 13 '12 at 13:48

There is at least one place where this could be happening, and perhaps others. In the code where you get the contents of the user defaults, you don't check for nil. So instead of this:

[self.cellArray addObjectsFromArray:[[NSUserDefaults standardUserDefaults] objectForKey:@"array"]];

you might want to try this:

NSArray *defaultCellArray = [[NSUserDefaults standardUserDefaults] objectForKey:@"array"];
if (defaultCellArray) {
    [self.cellArray addObjectsFromArray:defaultCellArray];
}

It is also possible that the initWithObjects call that is failing (according to your error message) is nested within another call, perhaps in this call:

[self.cellArray sortUsingDescriptors:[NSArray arrayWithObject:descriptor]];

If you look at the full call stack it would tell you if this call is making subsequent calls to initWithObjects and possibly passing it a nil. In your case, for example, you would pass in a nil yo the array you are creating if clickedButton.tag has not yet been set to a value of 1 or 2. You might want to add an additional else clause to help diagnose the problem:

if (clickedButton.tag == 1) {
        descriptor = [NSSortDescriptor sortDescriptorWithKey:@"Date" ascending:NO];
} else if (clickedButton.tag == 2) {
        descriptor = [NSSortDescriptor sortDescriptorWithKey:@"Score" ascending:NO];
} else {
    NSLog(@"Unexpected error: Can't determine sort descriptor");
}

Whenever I call the sortUsingDesriptors line, it only gets executed if the self.cellArray.count is not = 0. I also fixed the tag issue so the descriptor would properly be initialized but I still get the same crash. The self.cellArray is not nil and it has a count of 2 or more (1 or less it doesn't get called by my conditionals). What else could the issue be? — SimplyKiwi, Aug 14 '12 at 03:09
Hard to say - you should show us more of the full stack trace to identify where in your code the problem is stemming from. Also, did you put in the check I suggested to make sure you are getting a non-nil value back from NSUserDefaults? — Tim Dean, Aug 14 '12 at 12:52

score 2 · Answer 2 · edited Jun 19 '15 at 07:04

2

This is all pretty complicated, but one thing sticks out:

[[NSUserDefaults standardUserDefaults] setObject:self.cellArray 
                                          forKey:@"cellArray"];

I think you should change that to this:

NSArray *array = [NSArray arrayWithArray:self.cellArray];
[[NSUserDefaults standardUserDefaults] setObject:array forKey:@"cellArray"];

Secondly, add asserts() after every place where you think you should have an object, i.e.:

UIButton *theButton = ...
assert(theButton);

You also say that the number of rows is double the cellArray count, guess thats intentional.

edited Jun 19 '15 at 07:04

Alex Cio

6,014
5
44
74

answered Aug 13 '12 at 13:25

David H

40,852
12
92
138

Why should I change the setObject line to yours? What is the reasoning just so I understand the logic. Also it is intentional that I multiply the cellArray count by 2. – SimplyKiwi Aug 14 '12 at 02:30
Because cellArray is mutable, and if the userDefaults just retains it and does not copy it (and I don't know for sure what it does), then if you change cellArray, it will change in the copy user defaults is holding for you. – David H Aug 14 '12 at 11:13

attempt to insert nil object from objects[0]?

2 Answers2