将indexpath标记传递给tableview原型单元内的按钮

Question

I have created a tableview prototype cell in storyboard and I have added a button to cell and set its tag to indexpath.row. When I scroll my cells the scrolled cell on the top of tableview always set tag to zero instead of correct tag.

public func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
        let cell : UITableViewCell = tableView.dequeueReusableCellWithIdentifier("autoLoadReuseIndentifier", forIndexPath: indexPath)

        print("indexpath :\(indexPath.row)")


        cell.contentView.viewWithTag(100)?.layer.cornerRadius = 5
        tableView.separatorStyle = UITableViewCellSeparatorStyle.None
        let tempDict : NSDictionary = savedCardsArray.objectAtIndex(indexPath.row) as! NSDictionary


        let bankName = cell.contentView.viewWithTag(102) as! UILabel

        deleteButton = cell.contentView.viewWithTag(106) as? UIButton
        deleteButton?.tag = indexPath.row
        deleteButton?.addTarget(self, action: "deleteCard1:", forControlEvents: UIControlEvents.TouchUpInside)
        print("delete button:\(deleteButton)")
       // print("indexpath delete tag :\(deleteButton.tag)")
        if(self.isSetUpAutoloadSelected){
            deleteButton?.hidden = true
        }else{
            deleteButton?.hidden = false
        }
        return cell;
    }

Whenever I scroll the cells, delete button tag is always set to zero.

Answer 1

If you should go with other way so use follow code and get indexPath.

func deleteCard1(_ sender:deleteCard) {
     let buttonPosition:CGPoint = sender.convert(CGPointZero, to:self.tableView)
     let indexPath = self.tableView.indexPathForRow(at: buttonPosition)
 }

Answer 2

1. I think you don't need to follow this approach because firstly you set button tag statically in storyboard and again you are change it's tag in cellforrowatindexpath so when you scroll, cell will never find button with tag 106.If you want to follow this approach then you need to create customButton and add Variable of type NSInteger or whatever you want and store indexpath.row into that variable of customButton.

2. Another Approach is Create CustomTableViewCell Class and create button outlet in this custom Cell class and set indexpath.row into button tag like this

   CustomCellClassObject.buttonName.tag = indexPath.row

Answer 3

As Sumit said, it's better to use a custom cell and create outlet for the buttons and labels, as fetching sub views using tags will be tough to maintain the code in the future.

Additionally, you don't have to create the variable deleteButton, as I don't see a valid purpose.

Assign tag to the button and add target in cellForRowAtIndexPath, it should work fine.