TableView delete after I click button delete - iphone

Question

So I am trying to delete rows in table view.
Here is my code:
- (IBAction)done:(UIStoryboardSegue *)segue
{
DetailGodsViewController *detailController = [segue sourceViewController];
NSIndexPath *path = [NSIndexPath indexPathForRow:detailController.row inSection:detailController.section];
[self.tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:path]
withRowAnimation:NO];
[self.listOfGoods deleteGood: detailController.row];
[[self tableView] reloadData];
[self dismissViewControllerAnimated:YES completion:NULL];
}
I have ControlViewTable in storyBoard, after I click on a row in ControlViewTable it jumps to Detail view And there is other info, also I store info about the row and section in this function:
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender
{
if ([[segue identifier] isEqualToString:#"ShowGoodDetails"]) {
DetailGodsViewController *detailViewController = [segue destinationViewController];
detailViewController.row = [self.tableView indexPathForSelectedRow].row;
detailViewController.section = [self.tableView indexPathForSelectedRow].section;
detailViewController.good = [self.listOfGoods getGoodAtIndex:detailViewController.row ];
}
and there is also a button in detail view for delete, after I click on it, it jumps to the function:
- (IBAction)done:(UIStoryboardSegue *)segue.
But it always crashes in deleteRows. Could someone please help?

Answer 1

One problem might be that you're still responding to the button while you're trying to get rid of the cell containing the button. You need to let that action method end, and then call deleteRows. You should probably do the sort of thing I recommend here:
https://stackoverflow.com/a/13907375/341994
However, the biggest problem is probably that you must update the model data before deleting a row of the table.

Answer 2

Your code in the done: method is doing a few things out of order plus some extra things you don't need. It should be:
- (IBAction)done:(UIStoryboardSegue *)segue
{
DetailGodsViewController *detailController = [segue sourceViewController];
NSIndexPath *path = [NSIndexPath indexPathForRow:detailController.row inSection:detailController.section];
[self.listOfGoods deleteGood: detailController.row];
[self.tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:path]
withRowAnimation:NO];
[self dismissViewControllerAnimated:YES completion:NULL];
}
Basically, you need to update your data before you update the table. Also, don't call reloadData on the table after calling deleteRowsAtIndexPaths:. Do one or the other, not both.