Swiftでカスタムセルのテキストフィールドの値を渡したい

Question

タスクリストを作っています。

tasksをカスタムセルのtaskCellでTaskTableに表示しています。
taskCellの中のTextFieldの値を変更したあと、TableViewを更新すると、TextFieldの変更が適用されていません。
TextFieldの値をどうにかしてtasksに代入させたいのですが、やり方がわかりません。

import UIKit

class Task: Identifiable {
    var id: UUID
    var taskName: String
    var isCompleted: Bool
    
    init(id: UUID, taskName: String, idCompleted: Bool) {
        self.taskName = taskName
        self.id = id
        self.isCompleted = idCompleted
    }
}

import UIKit
import SwiftUI

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource, UITextFieldDelegate {
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return tasks.count
    }
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell: TaskCell = tableView.dequeueReusableCell(withIdentifier: "TaskCell", for: indexPath) as! TaskCell
        cell.taskTextField.delegate = self
        cell.setItem(task: tasks[indexPath.row])
        return cell
    }
    
    
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
    }

    
    
    
    
    var tasks = [Task]()
    
    @IBOutlet weak var taskTable: UITableView!
}

import Foundation
import UIKit

class TaskCell: UITableViewCell, UITextFieldDelegate {
    var cellTask = Task(id: UUID(), taskName: "", isCompleted: false)
    @IBOutlet weak var completeButton: UIButton!
    @IBOutlet weak var taskTextField: UITextField!
    
    func setItem(task: Task) {
        cellTask = task
//TextFieldとButtonにcellTaskの要素を代入する処理
    }
    
    @IBAction func completeButon(_ sender: UIButton) {
//completeButtonの切り替え処理
    }
    
}

Answer 1

どのように作るか次第ですが、たぶん UITextField の処理で混乱されているように思えます。

class ViewController: UIViewController, UITableViewDelegate, UITableViewDataSource, UITextFieldDelegate {
    // 中略
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell: TaskCell = tableView.dequeueReusableCell(withIdentifier: "TaskCell", for: indexPath) as! TaskCell
        cell.taskTextField.delegate = self
    }
}

上記のコードでは、

cell.taskTextField.delegate = self

と、Delegate に ViewController のインスタンスを入れていますが、そうではなくて、UITableViewCell のインスタンスを代入し、セルの中で処理を行ったほうが簡潔に書けそうな気がします。

つまり、

class TaskCell: UITableViewCell, UITextFieldDelegate {

カスタムセルに UITextFieldDelegate を準拠させ、

    private lazy var textField: UITextField = {
        let tf = UITextField()
        tf.borderStyle = .line
        
        // ここの self は UITableViewCell 自身
        tf.delegate = self
        
        return tf
    }()

UITableViewCell 内部の UITextField の delegate はセル自身にした上で

    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        label.text = textField.text
        textField.text = ""
        task?.taskName = textField.text ?? ""
        textField.resignFirstResponder()
        
        return true
    }

セルの中で UITextFiled の処理を行うといった感じです。

---

たとえば、こんな感じです（StoryBoard は使っていなので、このままコピーてして実行すると動くと思います）。

import UIKit

class Task {
    var taskName: String

    init(taskName: String) {
        self.taskName = taskName
    }
}

class TaskCell: UITableViewCell, UITextFieldDelegate {
    private var task: Task?
    
    override init(style: UITableViewCell.CellStyle, reuseIdentifier: String?) {
        super.init(style: style, reuseIdentifier: reuseIdentifier)
        
        setUp()
    }
    
    required init?(coder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
    
    private let label = UILabel()
    
    private lazy var textField: UITextField = {
        let tf = UITextField()
        tf.borderStyle = .line
        
        // ここの self は UITableViewCell 自身
        tf.delegate = self
        
        return tf
    }()
   
    private lazy var button: UIButton = {
        let b = UIButton(type: .system)
        b.setTitle("set", for: .normal)
        b.backgroundColor = .yellow
        b.addTarget(self, action: #selector(buttonTapped), for: .touchUpInside)
        
        return b
    }()

    // ボタンが押されたら、ラベルに UITextField の内容を表示し、UITextFiled は空にし、キーボードを閉じる
    @objc private func buttonTapped(_ sender: UIButton) {
        textField.resignFirstResponder()
        label.text = textField.text
        task?.taskName = textField.text ?? ""
        textField.text = ""
    }
    
    // コードでカスタムセルを作成しているだけ。
    private func setUp() {
        let sv = UIStackView(arrangedSubviews: [label, textField, button])
        contentView.addSubview(sv)
        sv.axis = .vertical
        sv.spacing = 6
       
        sv.translatesAutoresizingMaskIntoConstraints = false
        sv.topAnchor.constraint(equalTo: self.contentView.layoutMarginsGuide.topAnchor).isActive = true
        sv.leadingAnchor.constraint(equalTo: self.contentView.layoutMarginsGuide.leadingAnchor).isActive = true
        sv.trailingAnchor.constraint(equalTo: self.contentView.layoutMarginsGuide.trailingAnchor).isActive = true
        sv.bottomAnchor.constraint(equalTo: self.contentView.layoutMarginsGuide.bottomAnchor).isActive = true
    }
    
    // セルに各要素をセット
    func setItem(_ task: Task) {
        self.task = task
        label.text = task.taskName
        textField.text = ""
    }
    
    // セルの中で UITextFiled の処理を行う
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        label.text = textField.text
        textField.text = ""
        task?.taskName = textField.text ?? ""
        textField.resignFirstResponder()
        
        return true
    }
}

class ViewController: UIViewController {
    var data = (1...10).map { _ in Task(taskName: "not set") }
    
    lazy var tableView: UITableView = {
        let tv = UITableView(frame: CGRect(x: 0, y: 0, width: view.frame.size.width, height: view.frame.size.height))
        tv.dataSource = self
        tv.delegate = self
        
        tv.register(TaskCell.self, forCellReuseIdentifier: "cell")
        
        return tv
    }()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
    
        view.addSubview(tableView)
    }
}

extension ViewController: UITableViewDelegate, UITableViewDataSource {
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return data.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! TaskCell
        
        cell.setItem(data[indexPath.row])
        
        return cell
    }
}

Comments

[Yuto409]

これを参考にコードを書いたところ、自分の思うように動作しました！
本当にありがとうございました。

[TsukubaDepot]

ViewControllerをdelegate にすることも可能ですが、そうなるとどのセルが操作されたのか確認するのが面倒になるので、ここは単純にセル（も独立したインスタンス）に全ても任せる（責務の分離、というのでしょうか）のがわかりやすいかと思います。

Answer 2

TableViewを更新すると、TextFieldの変更が適用されていません。

更新したテキストをtasksで変更していないからです。