if構文の中にデータを引き継ぐには

Question

Admobのインタースティシャル広告の設定の関係で、

    private func next(times:DefaultTime) {
        // 設定値を追加
        times.check = self.zeroOverCheck.isOn
         let vc = storyboard!.instantiateViewController(withIdentifier: "MainViewController") as! MainViewController
            vc.defaultTime = times
            self.present(vc, animated: true, completion: nil)
        }


```だったのを、


```ここに言語を入力

    private func next(times:DefaultTime) {
        // 設定値を追加
        times.check = self.zeroOverCheck.isOn

        
       // 画面遷移
        if interstitial.isReady {
            let vc = storyboard!.instantiateViewController(withIdentifier: "MainViewController") as! MainViewController
            vc.defaultTime = times
            interstitial.present(fromRootViewController: self)
            self.present(vc, animated: true, completion: nil)
        } else {
            print("Ad wasn't ready")
        }
    }

と修正しました。

すると、前は上手くいっていたデータの受け渡し（＝times）が、通らなくなってしまいました。
おそらく、if構文が壁となっているんだろうと思います。
この場合、どう処理したらいいのでしょうか。
よろしくお願いします。

Answer 1

インタースティシャル広告を使ったことはないのですが、

let vc = storyboard!.instantiateViewController(withIdentifier: "MainViewController") as! MainViewController
vc.defaultTime = times

の位置を調整してみてはいかがですか？

interstitial.present(fromRootViewController: self)の後に書いてみるとか
また、times.check = self.zeroOverCheck.isOnが離れているのも気になります。

ちなみに、インタースティシャル広告にはclosure的な関数の終わりや、
表示終了結果を得る方法は用意されていないのですか？
closure:
https://qiita.com/narukun/items/b1b6ec856aee42767694

関係のないことですが、引数で受け取ったものを利用して何かを行わず、
引数自体に他のメンバ変数を代入しているのは変な気がしますが、、、

googleのサンプルをみてみましたが、
https://github.com/googleads/googleads-mobile-ios-examples/blob/master/Swift/admob/InterstitialExample/InterstitialExample/ViewController.swift
closureなどで終了を取得する必要はないみたいですね。
単純に直列処理されているように見えますよ。

なので、以下のように修正すればいいのではないでしょうか？

private func next(times:DefaultTime) {
    if interstitial.isReady {
        interstitial.present(fromRootViewController: self)
    } else {
        print("Ad wasn't ready")
    }
    // 設定値を追加
    times.check = self.zeroOverCheck.isOn
    // 画面遷移
    let vc = storyboard!.instantiateViewController(withIdentifier: "MainViewController") as! MainViewController
    vc.defaultTime = times
    self.present(vc, animated: true, completion: nil)
}

広告が準備できていない時、画面遷移できないと、
ユーザーはその時点で止まってしまって、バグとなってしまいますし。

9/25 追記
うーん、難しいですね。
インタースティシャル広告を自分が使ってたら、実機で試してお答えできるのですが、、、

nextを呼び出す部分でどうなっているのかわかりませんが、
print()を代入し、実際に渡している値、順序を確認することですかね。

private func next(times:DefaultTime) {
    times.check = true  // <-これを適宜、各printの前に移動する。
    print("1:", times)
    if interstitial.isReady {
        print("2-1:", times)
        interstitial.present(fromRootViewController: self)
        print("2-2:", times)
    } else {
        print("3-1:" times)
        print("Ad wasn't ready")
    }
    print("4:", times)
    // 設定値を追加
    // times.check = self.zeroOverCheck.isOn  ここは一旦コメントアウト
    // 画面遷移
    let vc = storyboard!.instantiateViewController(withIdentifier: "MainViewController") as! MainViewController
    vc.defaultTime = times
    print("5:", times)
    self.present(vc, animated: true, completion: nil)
    print("6:", times)
}

まず、上記を試してみて、各printの前で値を変え、
期待の動きをする箇所を探すのがいいと思います。
もし、しないようなら、、、実装方法を変える必要があるということだと思います。
それこそ、closureを使うとか、GCDを使うかとか、、、

ちなみに、DefaultTimeの構造がわかったので、余計なことですが、
func next内ではDefaultTimeのcheck(Bool型)のみ変更しているので、
timesはメンバ変数で持っている？ことを前提にすると、

private func next(bool: Bool) {
    // ~~ 省略
    self.times.check = bool
    // ~~ 省略
}

viewを通して保持しておきたいと思いますので、メンバ変数にするのがいいと思います。
もし新たに作るとしたら、func next内で作ればいいと思いますし。

後、timesにすると、複数形なので [DefaultTime] のように複数あるイメージになりますよ。

Comments

[sansyou69]

心のこもったご返信、ありがとうございます。
職場でそれを読んで期待し、帰宅後、いただいた内容に修正して、試してみました。
結果は、残念ながら変わりませんでした。

ただ、私の方で、先に伝えておくべき内容のデータがありました。

DefaultTime.swift

import UIKit

class DefaultTime: NSObject {
var start:String = "09:00"
var end:String = "18:00"
var rest:String = "01:00"
var work:String = "08:00"
var check:Bool = false
}

こちらが、同アプリの違う階層にデータとしてあって、先のページで設定される数字（＝時刻）ではなく、こちらが呼び込まれてしまうという状態です。
説明が不十分で、申し訳ありません。

[sansyou69]

繰り返しのお返事、ありがとうございました。
なんとか、解決することができました。

協力者がいて、その方にお手伝いしていただき解決できたのですが、問題は一つだけではありませんでした。
何度か試行錯誤して、やっと解決できました。
言われるとおり、確かに部分的なコードだけ見せられても、分かりずらかったかもしれません。

近くにいたなら、ご飯とか一緒にして（もちろん私が払って）、お話したかったです。
また、何か問題が生じましたら、ご意見いただけると嬉しいです。

今回は、ありがとうございました。