NSMutableArray removeObjectAtIndex:indexPath.row crashes

3
votes

I have a UITableViewController with an NSMutableArray in the header like this:

NSMutableArray *someArray;

Also I have the property declared:

@property (nonatomic, retain) NSMutableArray *someArray;

In the .m file I load the array in the method:

- (void)viewDidAppear:(BOOL)animated {
    [super viewDidAppear:animated];

    someArray = [[NSMutableArray alloc] init];

    [self loadArrayData];

    [tableview reloadData];
}

The table populates great, adding is no problem, but when I try to delete a row, the app crashes on the following line:

- (void)tableView:(UITableView *)tableView commitEditingStyle:(UITableViewCellEditingStyle)editingStyle forRowAtIndexPath:(NSIndexPath *)indexPath {

    if (editingStyle == UITableViewCellEditingStyleDelete) {

    [someArray removeObjectAtIndex:indexPath.row];

    //update table etc..

If I use the debugger, I can see the array has some objects, and when I Log indexPath.row I get a value which is inside the array size.

I don't understand why it is crashing on this line... Who can help me?

The console outputs:

* Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[__NSArrayI removeObjectAtIndex:]: unrecognized selector sent to instance 0x6193890'

3
iphonexcodeios4ipad
A little more information on what the crash is may help. What kind of crash is occurring? - Brenton Morse
Sorry, I've added the console error output.. - CyberK
Hmmm, I honestly don't know. It's obvious an invalid argument is being passed to removeObjectAtIndex:. However indexPath.row should return an NSUInteger which is a valid argument. - Brenton Morse
That's the exact same thing I was thinking... Also passing just value 0 into the method causes it to crash... - CyberK
and you are 100% sure that the array doesn't get replaced by an immutable array at some place? Check again. - Matthias Bauch

3 Answers

2
votes

I had a very similar problem and I was initiating my array with CoreData like this:

self.allContacts = (NSMutableArray*)[cdc.managedObjectContext executeFetchRequest:fetchRequest error:&error];

trying to cast the result from CoreDataController fetch request to NSMutableArray.

Turns out I need the function called "mutableCopy" at the end of fetch request.

self.allContacts = (NSMutableArray*)[[cdc.managedObjectContext executeFetchRequest:fetchRequest error:&error] mutableCopy];

I don't think cast to (MutableArray *) is required but I have it just in case.

Hope this helps.

1
votes

I had the same issue a week back. What I did was this:

Declared a NSMutableArray* globalArray as a global within the same tableViewController.m implementation file. Then inside 

if (editingStyle == UITableViewCellEditingStyleDelete) {
   globalArray = [[NSMutableArray alloc] initWithArray:someArray];
   [globalArray removeObjectAtIndex:[indexPath row]]; 
}

And then I copied back this globalArray into someArray before the reloadData call. Works.

0
votes
  1. not sure if that cause one of your issues, but if you have property for someArray then use self.someArray instead of just someArray . Of course that after using self, you should auto release it like this :

self.someArray = [[[NSMutableArray alloc] init] autorelease];

(call this inside your viewController init method)

then try to :

[self.someArray removeObjectAtIndex:indexPath.row];

2. try to call [self loadArrayData]; inside viewDidLoad. It should be called prior to cellForRowAtIndexPath, and that way you won't need to call reloadData

if that still won't help, then post the code of loadArrayData