My CellForRowAtInexPath is sending wrong info?

Question

when I call the indexpath of cellForRowAtIndexPath: for deleting the cell it deletes de cell before the selected one... Heres how It works:

I add info to the tableview:

- (BOOL)textFieldShouldReturn:(UITextField *)textField{

    [self.tasks insertObject:textField.text atIndex:0];
    [self.userdefaults setObject:self.tasks forKey:@"tasks"];
    [self.userdefaults synchronize];

    NSIndexPath *newIndexPath = [NSIndexPath indexPathForRow:(0) inSection:0];
    [self.tableview beginUpdates];
    [self.tableview insertRowsAtIndexPaths:[NSArray arrayWithObject:newIndexPath] withRowAnimation:UITableViewRowAnimationLeft];
    [self.userdefaults synchronize];
    [self.tableview endUpdates];
}

And then I edit the info with some gestures from a CocoaControl (MCSwipeTableViewCell):

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
//This is for deleting a cell
[cell setSwipeGestureWithView:cross color:redColor mode:MCSwipeTableViewCellModeSwitch state:MCSwipeTableViewCellState2 completionBlock:^(MCSwipeTableViewCell *cell, MCSwipeTableViewCellState state, MCSwipeTableViewCellMode mode) {


        [tableView beginUpdates];

        I tried to use this index and didnt work-->//NSIndexPath *myindex = [self.tableview indexPathForSelectedRow];

        [self.tasks removeObjectAtIndex:[self keyForIndexPath:indexPath].row];
        [self.userdefaults setObject:self.tasks forKey:@"tasks"];
        [self.userdefaults synchronize];

        [tableView deleteRowsAtIndexPaths:@[indexPath] withRowAnimation:UITableViewRowAnimationRight];
        [tableView endUpdates];

        NSLog(@"Did swipe \"Cross\" cell");
    }];


}

- (NSIndexPath *)keyForIndexPath:(NSIndexPath *)indexPath
{
    if ([indexPath class] == [NSIndexPath class]) {
        return indexPath;
    }
    return [NSIndexPath indexPathForRow:indexPath.row inSection:indexPath.section];
}

So when I try to delete It deletes a cell before the one it was selected, I need someones help, thanks!

Pretty sure you need to capture `indexPath` as a local variable. — Hot Licks, Mar 17 '14 at 22:28
Unrelated but FYI - no need to call `synchronize`. And there is no need to use `beginUpdates/endUpdates` if you only call one table view insert/delete/reload. — rmaddy, Mar 17 '14 at 22:31
So I delete the begin and endUpdates? that can be the problem? — Karlo A. López, Mar 17 '14 at 22:39
@rmaddy - But the caller will change it. It needs to be copied. — Hot Licks, Mar 17 '14 at 22:39
@HotLicks `NSIndexPath` is immutable. Once the block keeps its own reference, how can it be changed? — rmaddy, Mar 17 '14 at 22:41
@user3290977 I simply pointed out that it isn't needed in this case but it isn't hurting. — rmaddy, Mar 17 '14 at 22:41
possible duplicate of [iOS 7 beginUpdates endUpdates inconsistent](http://stackoverflow.com/questions/18919459/ios-7-beginupdates-endupdates-inconsistent) — Hot Licks, Mar 17 '14 at 22:46
I Implement the answer of @HotLicks link and I got a crash: 'NSRangeException', reason: '*** -[__NSArrayM removeObjectAtIndex:]: index 3 beyond bounds [0 .. 2]' I updated my code so how it looks now — Karlo A. López, Mar 17 '14 at 23:45
It works sometimes but after 2 or three deletes It crashes or deletes the wrong selection — Karlo A. López, Mar 18 '14 at 00:21
Why not just use [tableView indexPathForCell:cell] in your completion block? — Paulw11, Mar 18 '14 at 01:43

score 0 · Answer 1 · answered Mar 18 '14 at 02:05

0

You can use [tableView indexPathForCell:cell] in your completion block to get the appropriate indexPath.

answered Mar 18 '14 at 02:05

Paulw11

108,386
14
159
186

score -2 · Answer 2 · answered Mar 17 '14 at 22:22

-2

Try using: [tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:UITableViewRowAnimationRight]; instead?

answered Mar 17 '14 at 22:22

Joey Clover

756
5
14

My CellForRowAtInexPath is sending wrong info?

2 Answers2