swift デリゲートでlabel.textにアクセスできない

Question

前回の質問と少し被ってしまうのですが、改善ができなかったので改めて投稿いたします。

デリゲートををして、別クラスのlabel.textにアクセスして値を送りたいのですがoptional nilだと言われ代入できません。
プロトコルを作成しメソッドにはアクセスできましたが、labelにアクセスできないのはなぜでしょうか。
改善策をよろしくお願いいたします。

アクセスされる側

import UIKit


class ViewController: UIViewController, UIImagePickerControllerDelegate, UINavigationControllerDelegate, VCDelegate {
    
    
    @IBOutlet weak var layerView: UIView!
    @IBOutlet weak var timerLabel: UILabel!　// これに代入したい。最下部にメソッドあり
    @IBOutlet weak var startBtn: CustomButton!
    @IBOutlet weak var stopBtn: CustomButton!
    @IBOutlet weak var finishBtn: CustomButton!
    
    var totalTime = 0
    
    var timer = Timer()
    var isTimerRunning = false
    var resumeTapped = false
    var addLap: Bool = false

    @IBAction func startBtn(_ sender: Any) {
        
        if isTimerRunning == false {
            runTimer()
            self.startBtn.isEnabled = false
            stopBtn.isEnabled = true
        }
        
    }
    
    @IBAction func stopBtn(_ sender: Any) {
        
        if resumeTapped == false {
            timer.invalidate()
            self.resumeTapped = true
            self.stopBtn.setTitle("Resume", for: .normal)
        }
        
    }
    
    @IBAction func finishBtn(_ sender: Any) {
        
        alertShow()
        
        
    }
    
    
    
    @IBAction func settingBtn(_ sender: Any) {
        
        let storyboard: UIStoryboard = self.storyboard!
        let set = storyboard.instantiateViewController(withIdentifier: "settingVC")

　　　　　　// これが足りないのかと思い足してみましたが、delegeteがないと言われます
//        set.delegate = self
        self.present(set, animated: true, completion: nil)
        
    }
    

    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        layerView.backgroundColor = UIColor(red: 0, green: 0, blue: 0, alpha: 0.5)
        
    }

}



extension ViewController {
    
    
    
    func timeString(time: TimeInterval) -> String{
        let hour = Int(time) / 3600
        let minute = Int(time) / 60 % 60
        let second = Int(time) % 60
        print("(hour) a")
        return String(format:"%02i:%02i:%02i", hour, minute, second)
    }
    
    
    
    @objc func updateTimer() {
        if totalTime < 1 {
            timer.invalidate()
        }else{
            totalTime -= 1
            timerLabel.text = timeString(time: TimeInterval(totalTime))
        }
    }
    
    
    
    func runTimer() {
        timer = Timer.scheduledTimer(timeInterval: 1, target: self, selector: #selector(updateTimer), userInfo: nil, repeats: true)
        isTimerRunning = true
        stopBtn.isEnabled = true
    }
    
    
    
    func alertShow(){
        
        let alert: UIAlertController = UIAlertController(title: "Confirmation", message: "Have you finished?", preferredStyle:  UIAlertController.Style.alert)
        
        let saveAction: UIAlertAction = UIAlertAction(title: "Save Record", style: UIAlertAction.Style.default) { (UIAlertAction) in
            
            // save process
            
        }
        
        let unsaveAction: UIAlertAction = UIAlertAction(title: "Unsave Record", style: UIAlertAction.Style.destructive) { (UIAlertAction) in
            
            // unsave process
            
        }
        
        let cancelAction: UIAlertAction = UIAlertAction(title: "Cancel", style: UIAlertAction.Style.cancel) { (UIAlertAction) in
            
            // cancel process
            
        }
        
        alert.addAction(saveAction)
        alert.addAction(unsaveAction)
        alert.addAction(cancelAction)
        
        self.present(alert, animated: true, completion: nil)
        
    }
    
    
    func setTime(_ time: String){
        print("success (time)")
　　　　　// ここに代入したいがnilだと言われる
        timerLabel.text = time
        
    }
}

。
。
。
アクセスする側

import UIKit

var records = ["test", "tst"]

// プロトコル
protocol VCDelegate {
    func timeString(time: TimeInterval) -> String
    func setTime(_ time: String)
}


class settingVC: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    var delegate: VCDelegate?
    
    @IBOutlet weak var settingView: DesignableView!
    @IBOutlet weak var tableView: UITableView!
    @IBOutlet weak var hField: UITextField!
    @IBOutlet weak var mField: UITextField!
    @IBOutlet weak var sField: UITextField!
    
    var hours = 0
    var minutes = 0
    var seconds = 0
    var totalTime = 0
    
    
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return records.count
    }
    
    
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = UITableViewCell(style: UITableViewCell.CellStyle.default, reuseIdentifier: "cell")
        cell.textLabel?.text = records[indexPath.row]
        cell.backgroundColor = UIColor.clear
        cell.textLabel?.textColor = UIColor.white

        return cell
    }

    
    
    @IBAction func cancelBtn(_ sender: Any) {
        self.dismiss(animated: true, completion: nil)
    }
    
    
    // ここで値を送り込みたい
    @IBAction func setBtn(_ sender: Any) {
        
        if hField.text != "" && mField.text != "" && sField.text != "" {
            
            hours = Int(hField.text!)! * 3600
            minutes = Int(mField.text!)! * 60
            seconds = Int(sField.text!)!
            totalTime = hours + minutes + seconds
            
            var delTime = delegate?.timeString(time: TimeInterval(totalTime))
            delegate?.setTime(delTime!)
            
         
            self.dismiss(animated: true, completion: nil)
        }
        
    }
    
    
    
    override func viewDidLoad() {
        
        super.viewDidLoad()
        tableView.backgroundColor = UIColor.clear
        settingView.backgroundColor = UIColor(white: 0.7, alpha: 0.8)
        
        // デリゲートします宣言？？
        let vc = ViewController()
        self.delegate = vc
        
    }

}

Answer 1

UI部品を直接アクセスするのではなく、設定された値を介して受け渡しをします。
画面遷移でsegueを使う場合には大抵prepareメソッドで受け渡します。
その他、UserDefaultsやAppDelegate経由などにする場合もあります。

delegateを使う場合には、なんらかのタイミングでdelegateに指定するインスタンスを
設定する必要がありますし、その様に実装する必要があります。

Comments

[mogiruri]

回答ありがとうございます。AppDelegateを使用するやり方はどこかがでみたような気がします。デリゲートの理解がまだ深くないので理解しないといけませんね。問題自体は解決していませんが、まだ調べたりななさそうなところをあさってみます。

[t_obara]

UserDefaultsやAppDelegate経由は、ちょっとしたグローバル変数に近い概念なので、利用するには注意した方が良いです。
画面遷移で値を受け渡すサンプルはそこいらに転がっているので、ググれば結構出てきます。

[mogiruri]

ありがとうございます。後ほどその方向で探ってみます。

[mogiruri]

大変遅くなりましたがいろいろ試してみた結果、userdefaultsに保存し、dismissで親ファイルに戻ると同時にset func を叩くようにしてセットできました。。
はじめ、userdefaltsでもうまくいかなかったのですが、原因はインスタンスを新規に作成し　vc.setValue() としていたのが問題でした。この時点では、別クラスにアクセスするためにインスタンス作成→UIと繋がっているクラスとは別物のインスタンス　という概念がなかったの　なぜtimerLable.text がnilなんだ？という疑問がずっと解決できませんでした。
UIと接続されている”そのまま呼び出し元のファイル”を参照するためには　presentingViewController を使うというのを見つけまして、それで繋ぐことで解決できました。

Answer 2

Swiftはしばらく書いていないので勘違いだったらすみません。
以下のコードは新しいViewControllerオブジェクトを作成して、デリゲートに渡していますがこの部分が意図した動作にならない原因だと思います。
新しく作成したオブジェクトのtimerLabelは、UI側と関連付けされていないからnilですよね?

// デリゲートします宣言？？
let vc = ViewController()
self.delegate = vc

Comments

[mogiruri]

なるほど、また新たにvcが出来てしまったから中身はカラということですね。
これはどうしたらアクセスできるのでしょうか？

[YasuhiroMiyake]

どのオブジェクトにデリゲート経由でアクセスさせたいかが、自身でイメージ出来ていますでしょうか。
まずはビューの部分、ソース部分を図に書いて、イメージをはっきりさせたほうが良いような気がします。
現状のコードでは新しいオブジェクトに対して、デリゲート経由でアクセスしているので、それを図に書いてイメージと異なっている部分を確認して下さい。

分かってしまえば、それほど難しいことではありませんがココが最初に引っかかりやすいところなので、もう少しだけデリゲートやビューとソースのイメージをはっきりさせたほうが今後のためにも良いと思います。

[mogiruri]

ご回答ありがとうございます。そうですね、基本的な部分がまだ完全に理解できていないと思います。
プロトコルなどを用いると、それが橋になって向こうのクラスに渡っていけるというような理解でしたがまだ情報が足りていないよです。もう少し情報を集めてみます。

[YasuhiroMiyake]

そのイメージで問題ないのですが、橋をかける先が間違っているのだと思います。