TableViewCellごとにその中にあるボタンを押した回数を数える方法

Question

###前提・実現したいこと
Xcode 8.3.2 Swift 3 です。

現在、TableViewCellにButtonとLabelを配置して(Cellの数は複数です)、Buttonが押されたら、そのセルのLabelの数がどんどん増えていく、というものを作ろうとしているのですが、現在のコードだと、どのButtonを押しても、1つだけのLabelだけ、どんどん増えていってしまいます。

どのようにコードを変えればいいか教えてくださいますか？

###ソースコード

import UIKit

class ViewController:  UIViewController, UITableViewDataSource, UITableViewDelegate {

  //TableViewの登録
  @IBOutlet weak var TableView: UITableView!

  //最初からあるメソッド
  override func viewDidLoad() {
    super.viewDidLoad()
  }

  //押す回数を宣言
  var counter = 0

  //ボタンが押されたときの処理
  @IBAction func buttonTapped(_ sender: Any) {

    //押した回数を1回増やす
    counter = counter + 1

    //TableViewからTagでLabelを持ってくる
    let label = TableView.viewWithTag(1) as! UILabel

    //Laelのテキスト
    label.text = "\(counter)"
  }

  //最初からあるメソッド
  override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
  }

import UIKit

class TableViewCell: UITableViewCell {

  //セルの登録
  static let reuseIdentifier = "cell"

  //Labelの登録
  @IBOutlet weak var Label: UILabel

  //Buttonの登録
  @IBOutlet weak var Button: Button!

  //最初からあるメソッド
  override func awakeFromNib() {
    super.awakeFromNib()
  }

  //最初からあるメソッド
  override func setSelected(_ selected: Bool, animated: Bool) {
  super.setSelected(selected, animated: animated)

Answer 1

    //TableViewからTagでLabelを持ってくる
    let label = TableView.viewWithTag(1) as! UILabel

ここがおそらく意図通りの実装になっていないのだと思います。
TableView.viewWithTag(1) だと、どのセルのボタンだろうとTableView全体をサーチしてtag == 1 のviewを取ってきてしまいます。

タップしたボタンと同じセルにあるラベルにアクセスしたいのなら、今回のコードでは
buttonTapped(_ sender: Any)のsender引数にタップしたボタンが入ってくるので、これを利用します。superviewプロパティでボタンの親のセルを取得してからviewWithTag(1)でラベルを取得することができます。

    @IBAction func tappedButton(_ sender: Any) {
        counter = counter + 1
        
        // senderをUIButtonでキャストしてsuperviewで親のUICellを取得し、さらにviewWithTagでUILabelを取得する
        if let label: UILabel = (sender as? UIButton)?.superview?.viewWithTag(1) as? UILabel {
            label.text = "\(counter)"
        }
    }

例えば上記のようにすることで、ボタンと同じセルのラベルのテキストを変更できます。

余談ですが、この実装ではカウンターが画面で一つしかないので、enohpiさんの意図する挙動とは違うかもしれません。

ラベルの初期値が0だとします。現在上から一番目のセルをタップすると0 → 1 → 2 ...と加算されていきますが、ここで上から二番目のセルをタップすると0 → 3に一足飛びに数値が変わってしまいます。応急処置としては以下のような実装が考えられます。

    @IBAction func tappedButton(_ sender: Any) {
        counter = counter + 1
        
        if let label: UILabel = (sender as? UIButton)?.superview?.viewWithTag(1) as? UILabel {
            label.text = counterStringIncrement(string: label.text) ?? label.text
        }
    }

    // 数字文字列に１を加えてから文字列に変換し直して返すメソッド
    private func counterStringIncrement(string: String?) -> String? {
        guard let string = string, let i: Int = Int(string) else { return nil }
        return String(i + 1)
    }

単に表示を変えるだけなら上のコードで大丈夫だと思いますが、もしカウンターをセルごとに管理したいのでしたらcounterを配列で持つ必要が出てくるので、実装を大幅に見直す必要が出てくるのではないかと思います。

Comments

[enohpi]

返信が遅くなってしまい申し訳ありません。

下のコードで出来ました。
本当にありがとうございます。

カウンターをセルごとに管理したいとき、counterを配列で持つ必要があるなどということを考えたことがなかったので、参考になりました。

[enohpi]

何度も申し訳ありません。

このコードを実装したところ、
conditional cast from 'UIButton' to 'UIButton' always suceeds

というエラーが出たのですが、これについてはどう対処すれば良いのでしょうか。
お手数ですがよろしくお願いします。

[enohpi]

(このコメントは削除しました。)

Answer 2

間違っていたらごめんなさい。実際に試していないので。
最初にボタンですがcell作成の際にtagづけしてsenderの中にあるボタンのtagで分けるのと。現在tagが1のものに処理を行っているみたいなのでラベルにもtagをつけてこのtagだったらみたいな処理はどうでしょうか？とりあえず、今自分がパッと考えたものなので一つの参考程度にしていただけたら嬉しいです。(｡･ω･｡)

Comments

[enohpi]

素早い回答ありがとうございます。
LabelにはTag(1)を既に付けています。

Buttonをcell作成の際にTag付けしてSenderの中にあるButtonのTagで分ける、とはどのように行うのでしょうか？

よろしくお願いします。

[sachiko-kame]

tableViewのデリゲートメソッドを定義して作成したセルを並べているかと思いますが、セルを作成しているデリゲートメソッドはわかりますか？
そこで、cellのボタンなどの設定ができるといいのですが……
ちなみに現在表示しているセルの値を変えているのならその容量で行ってみてください。

参考になりそうなurl貼っときます。
https://i-app-tec.com/ios/tableview.html

[enohpi]

返信が遅くなってしまい、申し訳ありません。

2回も返事していただきありがとうございました。
無事完成することが出来たので、お二方ともベストアンサーにしたかったのですが、コードを詳しく書いてくださった、idonotknowさんを選ばせていただきました。

今後ともお世話になります。よろしくお願いします。

[sachiko-kame]

無事完成おめでとうございます(✿´ ꒳ ` )
これからもどんどん色々なことに挑戦していってください⭐️