dismissを使ってモーダルビューを閉じる必要性

Question

質問

ある資料にこのようなコードがあったのですが，

import UIKit
import SafariServices

class ViewController: UIViewController, SFSafariViewControllerDelegate {
    @IBACtion func action(_ sender: Any) {
        let url = URL(string:"https://www.~~~.com")
        let vc = SFSafariViewController(URL: url)
        vc.delegate = self
        present(vc, animated: true, completion: nil)
    }

   func safariViewControllerDidFinish(_ controller: SFSafariViewController) {
        dismiss(animated: true)
    }
}

下のメソッドいらなくないですか？

func safariViewControllerDidFinish(_ controller: SFSafariViewController) {
        dismiss(animated: true)
    }

この部分がなくてもモーダルビューって閉じますよね．

このコードを書いた人がなぜ下のメソッドを書いたのかわかる方，いらっしゃらないですか．

環境
xcode 10
swift 4.2

Answer 1

ご提示されているdismissはViewControllerに対して行われているので、
モーダル表示しているSFSafariViewControllerが閉じたら
このViewControllerも閉じるという意味だと思います。

Comments

[vlb]

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

初歩的な質問で申し訳ないのですが、dismissでViewControllerを閉じるのとdismissを使わなかった時とで違いは出てくるのでしょうか。

私が実機で試した時はdismissがあるのとないのとでは画面に違いはないようにみえたので、、、

[nakasho_dev]

多分認識に誤りがあります。ご質問のコードでdismiss処理がなかったとしたらモーダルであるSFSafariViewControllerが閉じたときに、呼び出し元であるViewControllerが閉じません。

[TakeOne]

たぶん、今のViewControllerをアプリ起動時の初期表示画面にしていて、dismissしても戻り先がなくて何もしないので違いがわからないのだと思います。別のViewControllerを初期表示画面にし、そこから今のViewControllerに画面遷移し、そこからさらにSFSafariViewControllerに画面遷移するような画面構成にして検証すれば、dismissの有無の違いがわかると思います。

[vlb]

回答ありがとうございます．

TakeOneさんのおっしゃる通りでした．
納得できました．ありがとうございます．