NavigationControllerを利用した状態でモーダルを閉じた時に同時に更新をかけたい

Question

前提・実現したいこと

当方swiftに関しては初心者です。

UINavigationViewを用いたプロジェクトで、Twitterクライアントを開発しています。
投稿用画面をモーダルで表示し、投稿が完了したら同時に前画面のタイムラインを更新させたいと思っています。
しかし、同じようなことをここで試そうとしている方がいて、NavigationControllerを利用しているとうまくいかない、という旨が記述された質問を見つけました。

• Swiftにて、modalをdismissしたタイミングで親ViewのtableviewをreloadDataしたい。 | teratail

私も上記記事を参考に下記コードを試したのですが、やっぱりNavigationControllerを介しているため、更新処理がうまくできません。
でもやっぱり、NavigationControllerを利用した状態で、実現したいことができないか？とあれこれ試してみたのですが、どうもうまくいきません。

発生している問題・エラーメッセージ

NavigationControllerを利用した状態でモーダルを閉じた時に同時に更新をかけたいが、うまくいかない

該当のソースコード

    private func postTweet(userID: String, postContent: String?) {
        // FIXME : こいつから成功か失敗かを返した方がいい
        TwitterAPI.postTweet(userId: userID, postContent: postContent!)
        
        guard let parent = storyboard?.instantiateViewController(withIdentifier: "TimelineViewController") as? TimelineViewController else {
            fatalError()
        }
        print(parent)
        
        // 呟いたら消す
        navigationController?.popViewController(animated: true)
        self.dismiss(animated: true, completion: nil)
    }

この他、下記も試しました。（質問に載っている方法です）

• Swiftにて、modalをdismissしたタイミングで親ViewのtableviewをreloadDataしたい。 | teratail

    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
        if let presented = self.presentedViewController {
            if type(of: presented) == TwitterPostViewController.self {
                //PopupViewControllerから戻ってきたときはrefresh()
                self.loadTweets()
            }
        }
    }

試したこと

上記の通り

補足情報（FW/ツールのバージョンなど）

xcode version :11.1
target:iOS13.0
swift version:4
テスト端末:iPhone8(実機)

Answer 1

iOS13のModal遷移はデフォルトだとDismiss時に遷移元のviewWillAppear等が呼ばれない
この記事が参考になると思います。

これとは別に、モーダルを閉じきったとき（viewDidDisappear）に更新処理を呼び出す方法を書いておきます。

var timelineViewController: TimelineViewController?

override func viewDidLoad() {
    super.viewDidLoad()
    //TimelineViewController取得
    timelineViewController = (self.presentingViewController as? UINavigationController)?.viewControllers.last as? TimelineViewController
}

override func viewDidDisappear(_ animated: Bool) {
    super.viewDidDisappear(animated)
    //更新処理
    timelineViewController?.loadTweets()
}

期待している動作になっているかどうか分かりませんが、都合のよい方でどうぞ。

Answer 2

そもそもの設計の問題になると思いますが、
最初にviewを表示した時、データの更新は行なっていますか？

例えば、viewDidLoad()でデータのダウンロードをし、
tableViewのdatasourceの各所で表示をしているとしましょう。
この時、一旦アプリをバックグラウンド(電話がきた、メールを見に行くetc)
で中断がかかった場合どうなるでしょう？
10minとか経過した後、ホームボタン2重クリックから、
アプリを再度フォアグラウンド(表示)した場合、
viewDidLoad()は実行されないので、データは更新されず、
古い状態のまま表示されることとなります。

上記はきっとshtr_tさんの望む動きではないと思います。

なので、そもそもmodalをdismissし戻ったタイミング「だけ」じゃなく、
いつでも表示される時はデータを最新にし、更新した方が合理的だと思います。

そう考えると、viewWillAppearかviewDidAppearに
データを読み込むコードを書き、reloadData()をしなければなりません。

とすると、modalをdismissした時にという限定条件はなく、そういうコードをする必要もなく、
単純に downloadData() -> reloadData()とすればいいのではないでしょうか？

Comments

[shotakeu]

hamejiさんの仰る通り、常に更新された情報を表示したいのはヤマヤマなのですが、「バックグラウンドからフォアグラウンド」といったタイミングで更新されるのは現状のTwitter公式アプリではない仕様（間違っていたらすみません）なのと、なるべくユーザーが更新したいタイミングに更新をおまかせしたいという意図があります。というのもTwitterAPIのhometimelineAPIの上限が厳しいからです。

> そう考えると、viewWillAppearかviewDidAppearに
> データを読み込むコードを書き、reloadData()をしなければなりません。

とはいえ、設計的にはhamejiさんのいう通りが理想系だと思います。

viewWillAppearは試したのですが、モーダルを閉じた時にはリロードは走りませんでした。
tableViewの更新処理に問題がある気がしていますが、詳細は調査中です。

ご回答いただきありがとうございます。

[hameji]

よく仕様書を読んでませんが、hometimelineAPIの上限が厳しいのなら、、、
ユーザーにボタンを押させるかスクロールさせる等のactionを求めるのが
一番の節約になるのではないでしょうか？
とはいえ、不便なアプリになってしまうでしょうね。。。