送信者が間違った値を送信することがあります

Question

UICollectionViewにはカスタムフローレイアウトがあり、その目的はアイテムを削除するための空き状況です。インデックスを取得するために関数を使用します。

-(void)aMethod:(UIButton*)sender{ 
    [self.viewModel deleteAt:[sender tag]]; 
    [self.myCollectionView reloadData]; 
} 

時には（まれに）クラッシュしました。私はそれを掘るとき、私は時々[sender tag]が間違っていた、実際には、アイテムの数の高さの配列が見つかりました。なぜそれが起こるのですか？私は配列が5つの項目しか持っていなかったときに8を送信することが分かった。

ボタンこのようUITableViewCellの上方に配置され、単純な40×40の幅/高さ画像である：

UIButton *button = [UIButton new]; 
     [button setImage:[UIImage imageNamed:@"m_delete"] forState:UIControlStateNormal]; 
     if (self.shouldEdit){ 

      self.layout.longPressGestureRecognizer.minimumPressDuration = 0.3f; 
      NSLog(@"1 blk called"); 
     [button addTarget:self 
        action:@selector(aMethod:) 
     forControlEvents:UIControlEventTouchUpInside]; 
     [button setTag:indexPath.row]; 
     [cell addSubview:button]; 

     [button mas_makeConstraints:^(MASConstraintMaker *make) { 

      make.left.equalTo(cell.mas_left).with.offset(2); 
      make.top.equalTo(cell.mas_top).with.offset(2); 

     }]; 

     } 

Answer 1

ボタンのタグを更新せずに古いセルを再利用していると思われます。 [button setTag:indexPath.row];を意味するif (self.shouldEdit)という条件があるので、常に呼び出されるわけではありません。ボタンの目的はアイテムを削除することで、5つのセルの1つとして古い/削除されたセル（タグ8を持つもの）を再利用し、setTagを呼び出さないことがあると述べました。

セルの作成/再利用のたびに常にボタンのタグを更新するように、条件外でsetTag行を使用することができます。

また、UITableViewCellのprepareForReuseまたはdequeueReusableCellWithReuseIdentifier(...)の直後のcellForRowAtIndexPathで再利用する前に、古いボタンがセルから削除されていることを確認することもできます。これを行うには、古いものがセルから削除されているため、常に新しいボタンを追加する必要があります。

Answer 2

ボタンタグがcellForRowAtIndexPathそうcallledそのメソッドに設定タグをいけないたびに上書きされ この行を削除

[button setTag:indexPath.row]; 

使用この1

-(void)aMethod:(UIButton*)sender 
{ 
    CGPoint point=[sender convertPoint:CGPointZero toView:collectiewname]; 
    NSIndexPath *indexPath=[collectiewname indexPathForItemAtPoint:point]; 
    NSLog(@"row :%ld",(long)indexPath.row) 


}