質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.91%

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,273

score 21

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

デリゲートををして、別クラスの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

    }

}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/10/23 13:44

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

    キャンセル

  • 2018/10/23 13:55

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

    キャンセル

  • 2018/10/23 14:20

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

    キャンセル

  • 2018/11/06 08:33

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

    キャンセル

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/10/23 09:53

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

    キャンセル

  • 2018/10/23 11:34

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

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

    キャンセル

  • 2018/10/23 13:41

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

    キャンセル

  • 2018/10/23 16:16

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.91%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る