Неправильный объект добавлен в UITableView

Question

В моем приложении я получаю объект по NSNotificationCenter (образуют другой контроллер) и добавить объект UITableView:

-(void)viewWillAppear:(BOOL)animated 
{ 
    [[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(RosterSave:) name:@"RosterSave" object:nil]; 
} 

-(void)RosterSave:(NSNotification *)notification 
{ 
    NewRoster* newRoster = [[NewRoster alloc]init]; 
    newRoster = notification.object; 
    [myUser.rosterArray addObject:newRoster]; 
    [self.myRoster reloadData]; 
} 

Это метод Tableview:

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section 
{ 
    return myUser.rosterArray.count; 
} 

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath 
{  
    NSString *iden = @"MyTable"; 
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:iden]; 
    if (cell == nil) 
    { 
     cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:iden]; 
    } 
    NewRoster* myNewRoster = [myUser.rosterArray objectAtIndex:indexPath.row]; 
    cell.textLabel.text = myNewRoster.nameRoster; 
    return cell; 
} 

Когда пользователь добавляет первый объект, tableView получает собственную строку. Когда пользователь добавляет второй объект, он добавляет две строки второго объекта и таким образом.

Как исправить эту проблему?

Answer 1

У вас есть observer(notification) в viewWillAppear, которые вызывается каждый раз при появлении вида.

добавить уведомление в viewDidLoad вместо viewwillAppear.

Answer 2

Мне всегда нравится добавлять подписки NSNotification в init/и unsubscriptions в dealloc. Этот шаблон легко читается и отлаживается. Кроме того, он гарантирует, что вы никогда не сможете удвоить подписку или отказаться от подписки.

В вашем случае, вы склонны к созданию нескольких subscribtions в viewWillAppear

- (instancetype)init 
{ 
    ... 
    [[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(RosterSave:) name:@"RosterSave" object:nil]; 
    ... 
} 

- (void)dealloc 
{ 
    [[NSNotificationCenter defaultCenter] removeObserver:self]; 
} 

Answer 3

@Feroz прав о вас выделении нового объекта и заменить его notification.object. @Lion прав насчет viewDidLoad и viewDidAppear. Вы генерируете несколько уведомлений. Вам нужно генерировать только один объект. Поместите контрольную точку в свой код RosterSave и подсчитайте, сколько раз она вызывается для нового объекта. Также посмотрите на трассировку стека, чтобы узнать, кто генерирует эти вызовы. Речь идет о простом переходе, понимании вашего кода и понимании того, что происходит.