Delete from uitableview + write to plist not working

Question

i want to delete from uitableview and make it write to my plist. i'm pretty new to this objective-c iOS coding, so forgive me for mistakes

Right now with my code it crashed with this :

Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Invalid update: invalid number of rows in section 0.  The number of rows contained in an existing section after the update (11) must be equal to the number of rows contained in that section before the update (11), plus or minus the number of rows inserted or deleted from that section (0 inserted, 1 deleted) and plus or minus the number of rows moved into or out of that section (0 moved in, 0 moved out).'

Here's my code in my mainViewcontroller

- (void)viewWillAppear:(BOOL)animated{
// Property List.plist code
//Gets paths from root direcory.
NSArray *paths = NSSearchPathForDirectoriesInDomains (NSDocumentDirectory, NSUserDomainMask, YES);
//Get documents path.
NSString *documentsPath = [paths objectAtIndex:0];

//Get the path to our PList file.
plistPath = [documentsPath stringByAppendingPathComponent:@"Servers.plist"];

//Check to see if Property List.plist exists in documents.
if (![[NSFileManager defaultManager] fileExistsAtPath:plistPath]){
    [[NSFileManager defaultManager]copyItemAtPath:[[NSBundle mainBundle]pathForResource:@"Servers" ofType:@"plist"] toPath:plistPath error:nil];
    //If not in documents, get property list from main bundle.
}
arrayA = [[NSMutableArray alloc] initWithContentsOfFile:plistPath];
NSLog(@"%@\n%@", arrayA, [arrayA valueForKey:@"Hostname"]);

tableData = [arrayA valueForKey:@"Hostname"];
[super viewWillAppear:animated];
[self.tableView reloadData];
}


- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{

    return [tableData count];
}

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath

{
static NSString *cellIdentifier = @"showServerAction";

UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:cellIdentifier];

if (cell == nil) {

    cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifier];

}

cell.textLabel.text = [tableData objectAtIndex:indexPath.row];

return cell;

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

if (editingStyle == UITableViewCellEditingStyleDelete) {
    [tableView beginUpdates];

    [arrayA writeToFile:plistPath atomically: TRUE];
    [self.tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:UITableViewRowAnimationFade];
    [tableView endUpdates];
}
}

UPDATE here's how my plist looks. it's a NSMutableArray

Answer 1

Try this:

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

    [tableView beginUpdates];

    // You are going to modify the table view, but you also need
    // to modify the data source of your table. Since tableData
    // is derived from the content of arrayA, the actual data
    // source is arrayA, so we start with that (also, arrayA is
    // a mutable array, so we are allowed to remove objects from
    // it).
    [arrayA removeObjectAtIndex:indexPath.row];

    // Now we also need to modify tableData. If you don't do this,
    // tableData will report the wrong number of rows when
    // numberOfRowsInSection() is called the next time. This is the
    // direct source of the error that you mention in your question
    // (carefully read the error message, it's really meaningful!).
    // A small problem is that tableData is not a mutable array,
    // so we cannot use removeObjectAtIndex: to modify it (that's
    // probably the source of the error you got with Ilya's answer).
    // The solution is to simply recreate tableData from scratch
    // from the content of arrayA.
    tableData = [arrayA valueForKey:@"Hostname"];

    [arrayA writeToFile:plistPath atomically: TRUE];
    [self.tableView deleteRowsAtIndexPaths:[NSArray arrayWithObject:indexPath] withRowAnimation:UITableViewRowAnimationFade];

    [tableView endUpdates];
  }
}

Answer 2

You must change tableData after deleting, whereupon reload table view like this:

[self.tableView reloadSections:[NSIndexSet  indexSetWithIndex:<#(NSUInteger)#>] withRowAnimation:<#(UITableViewRowAnimation)#>];//With animation

or reload all table without animation:

[self.tableView reloadData];

instead of: [tableView beginUpdates];[tableView endUpdates];

Need code:

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

    if (editingStyle == UITableViewCellEditingStyleDelete) {

    [tableData removeObjectAtIndex:indexPath.row];
[arrayA setValue:tableData ForKey:@"Hostname"];
        [arrayA writeToFile:plistPath atomically: TRUE];
        [self.tableView reloadSections:[NSIndexSet  indexSetWithIndex:indexPath.section] withRowAnimation:UITableViewRowAnimationFade];

    }
    }