CODEWITHSUNDEEP
iphoneiosnsrangeexception
user1766496
user1766496

Reputation:

'NSRangeException', reason: '* -[__NSArrayM objectAtIndex:]: index 2 beyond bounds [0 .. 1]'

I'm trying to delete some items but i'm receiving this NSException:

'NSRangeException', reason: '* -[__NSArrayM objectAtIndex:]: index 2 beyond bounds [0 .. 1]'

Here is my code:

-(void)deletePressed:(id)sender {

if (data.count > 0) {

    NSString *path = [NSHomeDirectory() stringByAppendingString:@"/Documents/Galeria/"];

    NSFileManager *manager = [NSFileManager defaultManager];

    for (NSIndexPath *indexPath in itensSelecionados) {

        NSString *result = [path stringByAppendingFormat:@"%@", [[manager contentsOfDirectoryAtPath:path error:nil] objectAtIndex:indexPath.row]];

        [manager removeItemAtPath:result error:nil];

    }

    [self viewWillAppear:YES];

}}

Anyone could help?

Upvotes: 1

Views: 5772

Answers (2)

Michael Kessler
Michael Kessler

Reputation: 14235

You can't remove objects from an array that you are iterating through.
There may be few solutions for you.

One is to use an additional mutable array that will hold all the objects that should be deleted and then iterate through it and remove the objects from the original array:

-(void)deletePressed:(id)sender {
    if (data.count > 0) {
        NSString *path = [NSHomeDirectory() stringByAppendingString:@"/Documents/Galeria/"];
        NSFileManager *manager = [NSFileManager defaultManager];
        NSMutableArray *filesToDelete = [NSMutableArray array];

        // Build a list of files to delete
        for (NSIndexPath *indexPath in itensSelecionados) {
            NSString *result = [path stringByAppendingFormat:@"%@", [[manager contentsOfDirectoryAtPath:path error:nil] objectAtIndex:indexPath.row]];
            [filesToDelete addObject:result];
        }

        // Actually delete the files
        for (NSString *indexPathString in filesToDelete) {
            [manager removeItemAtPath:indexPathString error:nil];
        }

        // Why do you call viewWillAppear directly ??
        [self viewWillAppear:YES];
    }
}

EDIT
Fixed the NSIndexPath to NSString in the second iteration due to Thiago's advise.

Upvotes: 2

rmaddy
rmaddy

Reputation: 318794

You need to do the deletion in reverse row order. Lets say you have 3 rows and you want to delete the rows at index 0 and 2.

If you delete the row at index 0 first, then when you try to delete the row at index 2, it crashes because now there are only 2 rows left.

If you delete the row at index 2 first, then index 0, everything will be OK.

Upvotes: 0

Related Questions