Protocolを用いてController層とView層でやり取りをしたい

Question

前提・実現したいこと

こんばんは。いつもお世話になってます。
Stoty board無しで開発をしています。ナビゲーションバーのOkボタンをタップした時にテキストフィールドの文字が空だったら、アラートを表示する。空じゃなかったら、特に何も起きない。という処理の実装をしたいのですが上手くいきません。

下記ソースコードの説明

ViewController.swfit と View.swift がやり取りをするために ViewController.swift でプロトコルを作っています。
Okボタンがタップされた時に、プロトコルのメソッドを実行して返ってきた値が true か false かでアラートを出すか出さないか決めています。

現状と推測

プロトコルのメソッドの返り値が true か false 以前に、 nil になってしまいます。 View.swift に移譲してあるプロトコルのメソッドが、ログを見る限り起動( print(return fasle), print(return true)の箇所 )していないので、 ViewContoller.swift の delegate? の部分でオプショナルチェイニング？がかかり、それ以降の .confirmTextFeild() が実行されず nil になってしまっているのだと僕は考えているのですが、解決できません。

プロトコルのメソッドに返り値を設定したのは今回が初めてですが、今までの返り値なしで自分で作成してきたプロトコルは何も問題無く動いていたので原因が分からず悩んでいます。

シュミレータとログ画面

該当のソースコード

import UIKit

protocol ViewControllerDelegate {
    func confirmTextFeild() -> Bool
}

class ViewController: UIViewController {
    
    var subView: UIView!
    var delegate: ViewControllerDelegate?

    override func viewDidLoad() {
        super.viewDidLoad()
        
        view.backgroundColor = .white
        configureNavItem()
        configureSubView()
    }
    
    override func viewDidLayoutSubviews() {
        subView.frame = CGRect(x: view.bounds.origin.x, y: view.bounds.origin.y, width: view.bounds.width, height: view.bounds.height)
    }
    
    func configureSubView() {
        subView = View()
        view.addSubview(subView)
    }
    
    func configureNavItem() {
        
        navigationItem.rightBarButtonItem = UIBarButtonItem(title: "OK", style: .plain, target: self, action: #selector(tappedOk))
        
    }

    @objc func tappedOk() {
        
        let flag = delegate?.confirmTextFeild()
        
        if flag == nil {
            print("nil")
            return
        }
        
        if !flag! {
            print("No")
            let alertController = UIAlertController(title: "メッセージ", message: "文字が記入されていません", preferredStyle: .alert)
            alertController.addAction(UIAlertAction(title: "ok", style: .default, handler: nil))
            present(alertController, animated: true, completion: nil)
        } else {
            print("ok")
        }
    }

}

import UIKit

class View: UIView {
    
    var textField: UITextField!
    var viewController: UIViewController!
    
    override init(frame: CGRect) {
        super.init(frame: frame)
        
        configureTextField()
        configureViewControllerDelegate()
    }
    
    func configureTextField() {
        textField = UITextField()
        textField.backgroundColor = .darkGray
        textField.textColor = .white
        addSubview(textField)
    }
    
    func configureViewControllerDelegate() {
        let localViewController = ViewController()
        localViewController.delegate = self
        viewController = localViewController
    }
    
    required init?(coder: NSCoder) {
        fatalError("init(coder:) has not been implemented")
    }
    
    override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
        self.endEditing(true)
    }
    
    override func layoutSubviews() {
        textField.frame = CGRect(x: Double(self.center.x / 2), y: Double(self.center.y), width: 200, height: 40)
    }
    
}

extension View: ViewControllerDelegate{
    
    func confirmTextFeild() -> Bool {
        if textField.text == "" {
            print("return false")
            return false
        } else {
            print("return true")
            return true
        }
    }
    
}

extension View: UITextFieldDelegate {
    
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        textField.resignFirstResponder()
    }
    
}

試したこと

View.swiftでViewController.swiftをインスタンス化して、delegate = self にする場所が悪いのかと思い、他の箇所に変えて実行したのですが、上手くいきませんでした。

引き続き自分でも原因を探りますが、どなたか原因が分かる方がいましたらご回答お願いします。

補足情報

xcodeバージョン11.3.1

Comments

[hoshi-takanori]

「View.swiftでViewController.swiftをインスタンス化」って普通は逆ですね。delegate の向きも逆で、ViewControllerDelegate ではなく ViewDelegate を定義して view.delegate = self (self は ViewController) とするのが普通です。

[Naoki_Pro]

お返事ありがとうございます。

View層側でデリゲートを記述し、処理はController層に移譲するというコードはよく見かけるのですが、今回僕が書いたような、ViewController層側でデリゲートを記述し、処理はView層に移譲するというパターンはないのでしょうか？

[hoshi-takanori]

はい。というか、もともとViewControllerはViewの持ち主なので、デリゲート機構を使わなくても目的のViewにアクセスできるはずです。また、Viewは表示に徹するべきで、入力内容が適切がどうかを判断するのはViewControllerの責務です。そして何より、Viewの中でViewControllerを作るのはありえないです。

[Naoki_Pro]

お返事ありがとうございます。

確かにViewController層はView層を管理していてView層にアクセスできるので、デリゲートの必要はなかったです。解説ありがとうございますm(__)m

Answer 1

直接 View の textfield を見れば良さそうに思えます。

(delegateの向きについては、hoshi-takanoriさんも指摘されていますが、コードをみてて同じような印象を受けました)。

View は UIView のサブクラスですから、subView は UIView ではなく View のクラスにできそうな気がしますが、いかがでしょうか。

こんな感じに変更すれば、おそらく期待通りに動くと思います（こちらでは動きました）。

• ViewController.swift

class ViewController: UIViewController {

    // UIView -> 自作の View に変更
    var subView: View!
    // 削除
//    var delegate: ViewControllerDelegate?

// 中略
    @objc func tappedOk() {
        // *** subView のテキストフィールドをチェック
        if subView.textField.text == "" {
            print("No")
            let alertController = UIAlertController(title: "メッセージ", message: "文字が記入されていません", preferredStyle: .alert)
            alertController.addAction(UIAlertAction(title: "ok", style: .default, handler: nil))
            present(alertController, animated: true, completion: nil)
        } else {
            print("ok")
        }
    }
}

Comments

[Naoki_Pro]

お返事ありがとうございます。
確かに仰る通りで、subView.textFeild.text == " " で空か否か確認すれば良いだけでした。

delegate の向きに関してなのですが、View層側で delegate を記述して、Controller層に処理を移譲するソースコードはよく見かけるのですが、僕が今回書いたようにController層で delegate を記述して、View層に処理を移譲するというパターンはないのでしょうか？

今回 subView.textFeild.text でテキストフィールドにアクセスできることを完全に忘れていたので、Controller層で delegate を記述し処理はView層に移譲して、テキストフィールドが空か否か判断できるのではないか考え、上記のようなデリゲートになってしまいました。

[TsukubaDepot]

一般的に言う「Controller層で delegate を記述して、View層に処理を移譲するというパターン」があるのか無いのかまではわかりませんが、現在のViewControllerクラスとViewクラスそれぞれの包含関係を整理（できれば図示して整理）してみると、なんだか変な関係になっているとわかるかと思います。

[Naoki_Pro]

お返事ありがとうございます。

ViewController層はView層を管理しているため、delegate を使わなくても今回のように subView.textField.text みたいにアクセスすれば良いだけなので、確かに変な関係になってしまいますね。。。

ベストアンサーにさせて頂きます。