使用标签向 tableViewCell 中的按钮添加操作是否错误？

Question

我有一个 UItableViewCell，里面有一个按钮，我设置了按钮的标签，并使用标签在我的 ViewController 中添加了按钮的操作。

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "BillHistoryTableViewCell", for: indexPath) as! BillHistoryTableViewCell
        let cellData = billHistories[indexPath.row]
        cell.setup(with: cellData)
        cell.retryButton.tag = indexPath.row
        return cell
}

@IBAction func billHistoryRetryButtonDidTap(_ sender: UIButton) {
        let index = sender.tag
        if let id = billHistories[index].transactionInfo?.billUniqueID {
            hidePayIdGeneralTextField()
            billIdTextField.text = id.toNormalNumber()
            inquiryGeneralBillRequest()
        }
}

我想知道出于某种原因这是错误的吗？ 有人告诉我这不好，因为它使用大量 memory 来使用标签。

Answer 1

它会起作用吗？ 是的，但如上所述，这不是最好的方法，我会避免使用标签，除非这只是为了一些 POC。 有更好的方法来处理它。 我建议的第一个是使用委托来通知 controller，这是一个示例：

class BillHistoryTableViewController {
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "BillHistoryTableViewCell", for: indexPath) as! BillHistoryTableViewCell
        let cellData = billHistories[indexPath.row]
        cell.setup(with: cellData)
        cell.index = indexPath.row
        
        cell.delegate = self
        return cell
    }
}

extension BillHistoryTableViewController: BillHistoryTableViewCellDelegate {
    func didTapButton(index: Int) {
        print("tapped cell with index:\(index)")

        if let id = billHistories[index].transactionInfo?.billUniqueID {
            hidePayIdGeneralTextField()
            billIdTextField.text = id.toNormalNumber()
            inquiryGeneralBillRequest()
        }
    }
}

protocol BillHistoryTableViewCellDelegate: AnyObject {
    func didTapButton(index: Int)
}

class BillHistoryTableViewCell: UITableViewCell {
    weak var delegate: BillHistoryTableViewCellDelegate?
    var cellData: CellData?
    var index: Int?
    
    func setup(with cellData: CellData) {
        self.cellData = cellData
    }
    
    @IBAction func buttonPressed(_ sender: UIButton) {
        guard let index = index else {
            return
        }
        
        delegate?.didTapButton(index: index)
    }
}

我最近更喜欢的另一种方法是使用 Combine 的 PassThroughSubject，它需要更少的连线和委托定义。

import Combine

class BillHistoryTableViewController {
    var cancellable: AnyCancellable?
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "BillHistoryTableViewCell", for: indexPath) as! BillHistoryTableViewCell
        let cellData = billHistories[indexPath.row]
        cell.setup(with: cellData)
        cell.index = indexPath.row
        
        cancellable = cell.tappedButtonSubject.sink { [weak self] index in
            guard let self = self else { return }
            print("tapped cell with index:\(index)")

            if let id = self.billHistories[index].transactionInfo?.billUniqueID {
                self.hidePayIdGeneralTextField()
                self.billIdTextField.text = id.toNormalNumber()
                self.inquiryGeneralBillRequest()
            }
        }
        
        return cell
    }
}

class BillHistoryTableViewCell: UITableViewCell {
    var tappedButtonSubject = PassthroughSubject<Int, Never>()
    
    var cellData: CellData?
    var index: Int?
    
    func setup(with cellData: CellData) {
        self.cellData = cellData
    }
    
    @IBAction func buttonPressed(_ sender: UIButton) {
        guard let index = index else {
            return
        }
        
        tappedButtonSubject.send(index)
    }
}

您可以通过使用 cellData 注入索引来使其更短，例如：

func setup(with cellData: CellData, index: Int) {
    self.cellData = cellData
    self.index = index
}

但从我在您的示例中看到的情况来看，您甚至不需要索引，您只需要 CellData，所以如果我们采用组合示例，这些是您必须进行的主要小改动：

var tappedButtonSubject = PassthroughSubject<CellData, Never>()

tappedButtonSubject.send(cellData)

并通过以下方式观察它：

cancellable = cell.tappedButtonSubject.sink { [weak self] cellData in
    if let id = cellData.transactionInfo?.billUniqueID {
        //
    }
}