0

Popoverから戻ってきたときに感知するにはどうすればいいのでしょうか。
viewWillAppearなどは使えず、
swiftで画面遷移時の関数の実行
を参考にしてdismissした時に関数を実行させるようにしましたが、戻ってくる側のviewControllerの変数を使おうとするとnilが返ってきてしまいます。
ご存知の方がいらっしゃったらご教授ください。

Code

override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
    //セルの選択解除
    tableView.deselectRow(at: indexPath, animated: true)

    let back = storyboard!.instantiateViewController(withIdentifier: "View") as? ViewController
    back?.doAnything(text: text[indexPath.row])
    self.dismiss(animated: true, completion: {() in back?.received = self.text[indexPath.row]})
}

もう1つの方

var number:Int!

func doAnything(text: String) {
    print(text)
    switch text {
    case "1":
        let activityItem:[Int] = number //これがnilになる
        let activityVC = UIActivityViewController(activityItems: activityItem, applicationActivities: nil)
        self.present(activityVC, animated: true, completion: nil)

        print(url)

    case "2":
        break
    case "3":
        break
    case "4":
        break
    case "5":
        break
    default:
        break
    }
}
5
  • 1
    あなたがこれまで試したコードをお示しいただけませんか。現在の記述では漠然とした概要しかつかめず、何をしたから「戻ってくる側のviewControllerの変数を使おうとするとnilが返ってきて」しまうのかもわからないので、「どうすればいい」のかお示しすることが難しいです。
    – OOPer
    2019年4月12日 8:15
  • 質問が雑ですみませんでした。 popoverから遷移元の画面(popoverボタンを押す画面)に戻る時viewWillAppearなどは呼ばれないものなのですか? 2019年4月12日 9:01
  • 1
    viewWillAppearがどんな条件でいつ呼ばれることになるかはいろいろあって一概には言えません。このスレッドなんかも参考になるかと思います。ご自身の質問は「編集」から編集できますのでコードの提示をお願いします。
    – OOPer
    2019年4月12日 9:05
  • コードのご提示ありがとうございます。まだわかりにくいところがあるのですが、「Code」のすぐ下に書かれているのが、Popover 用のview controllerのコード、「もう1つの方」の下に書かれているのが、「そのPopoverを呼び出したview controller側のコードで、クラス名はViewController」と言うことでよろしいでしょうか?
    – OOPer
    2019年4月12日 9:40
  • 返事が遅くなってすみません。 Codeの方がPopoverで表示したtableViewのコードで、 もう1つの方がviewControllerです。 2019年4月12日 13:04

1 件の回答 1

1

確認ですが、あなたのコードは現在このようになっていることを想定しています。

呼び出し側のview controller:

class ViewController: UIViewController {

    //...

    //どこかのアクションメソッドからPopoverを`present`で表示している...ものと想定
    @IBAction func someAction(_ sender: Any) {
        //...
        let storyboard = UIStoryboard(name: "Main", bundle: nil)
        let popover = storyboard.instantiateViewController(withIdentifier: "Popover") as! PopoverViewController
        //...
        self.present(popover, animated: true, completion: nil)
    }

    var received: String? = nil

    var number: Int!

    func doAnything(text: String) {
        print(text)
        switch text {
        case "1":
            let activityItem: [Int] = [number]  //<- `[number]` ?
            let activityVC = UIActivityViewController(activityItems: activityItem, applicationActivities: nil)
            self.present(activityVC, animated: true, completion: nil)

            //print(url) //<- What is `url`?

        case "2":
            break
        case "3":
            break
        case "4":
            break
        case "5":
            break
        default:
            break
        }
    }
}

Popover用のview controller:

class PopoverViewController: UITableViewController {

    //...

    var text: [String] = []

    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        //セルの選択解除
        tableView.deselectRow(at: indexPath, animated: true)

        let back = storyboard!.instantiateViewController(withIdentifier: "View") as? ViewController
        back?.doAnything(text: text[indexPath.row])
        self.dismiss(animated: true, completion: {() in back?.received = self.text[indexPath.row]})
    }

    //...
}

あなたが示されたコード、一部コンパイルが通らない部分があったので修正してあります。質問用に少し書き換えられたのかもしれませんが、そのせいで根本的に違う結果になってしまう可能性もあるので注意してください。また、一部推定も含まれています。実際と大きく異なっている点があれば、ご質問に情報を追記するなり、コメントするなりしてお知らせください。


と言うわけで、あなたのコードで致命的に誤っているのはここだと思われます。

    let back = storyboard!.instantiateViewController(withIdentifier: "View") as? ViewController

Popoverから戻り先のview controllerにアクセスするのに、新しいインスタンスを作成してしまってはいけません。このコードは(同じViewControllerクラスではあるけれど)新たにインスタンスを作成するものであり、作成されたインスタンスは「戻り先」のview controllerとは別物です。

いろいろとやり方は考えられるでしょうが、一番確実なのは戻り先のview controllerを直接プロパティとしてPopover側に持たせてやることでしょう。

Popover側はこんな感じ。

protocol PopoverViewControllerDelegate: class {
    var received: String? {get set}
    func doAnything(text: String)
}
class PopoverViewController: UITableViewController {

    //...

    weak var delegate: PopoverViewControllerDelegate?

    var text: [String] = []

    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        //セルの選択解除
        tableView.deselectRow(at: indexPath, animated: true)

        delegate?.doAnything(text: text[indexPath.row])
        self.dismiss(animated: true) {
            self.delegate?.received = self.text[indexPath.row]
        }
    }

    //...
}

典型的なdelegateパターンということになるので、「戻り先」をdelegateとして、呼び出したい機能(メソッドやプロパティ)をprotocolにしてあります。

呼び出し側はこんな風になるでしょう。

//↓`PopoverViewControllerDelegate`への適合を追加
class ViewController: UIViewController, PopoverViewControllerDelegate {

    //...

    @IBAction func someAction(_ sender: Any) {
        //...
        let storyboard = UIStoryboard(name: "Main", bundle: nil)
        let popover = storyboard.instantiateViewController(withIdentifier: "Popover") as! PopoverViewController
        popover.delegate = self //<- Popoverの呼び出し時にdelegateを設定
        //...
        self.present(popover, animated: true, completion: nil)
    }

    var received: String? = nil

    var number: Int? //<-未初期化の可能性のある変数にImplicitlyUnwrappedOptionalは使わない

    func doAnything(text: String) {
        //`number`が非nilであることをguardで確認
        guard let number = number else {
            print("`number` is nil")
            return
        }
        print(text)
        //...
    }
}

numberのように呼び出しの順序によっては未初期化状態になる可能性のある変数にImplicitlyUnwrappedOptionalを使うべきではありませんので、普通のOptionalに変え、使う前にguardで非nilのチェックを入れています。


最初に書いたように一部推定を含んでいるので、全く使い物にならないものをお見せしてしまっている可能性もありますが、うまくあなたのコードに適用できそうでしたら、お試しください。

3
  • 申し訳ないですがうまくいきませんでした もう少し自分で頑張ってみます... 2019年4月13日 13:28
  • @AppleDeveloper さん、大変残念です。ご自身で頑張ってみてうまくいった場合には、自己回答をお願いします。または、「うまくいかない」について、追加の情報をいただければ(例えば、Popoverを表示するところのコードが全然違っている、なんてことも含めて)、お手伝い出来ることがあるかもしれません。
    – OOPer
    2019年4月13日 13:39
  • ありがとうございました。 まだ質問すると思いますのでその際はよろしくお願いします 2019年4月13日 13:56

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。