SwiftUIでモーダルAからモーダルBへ連続して画面遷移させたい

Question

前提・実現したいこと

ImagePickerで画像を選択したあと、選択した画像をクロッピング処理への遷移を想定しています。

画像云々の部分はあまり関係ない話ですので、ここでは便宜上画面A・画面Bと表現します。

実現したいこととしては、下記の通りです。

1. 画面Aをモーダル表示する
2. 画面Aを閉じられると、画面Bをモーダル表示する
3. その後、画面Bが閉じられること

画面Aを開いて閉じて、画面Bを開いて閉じて、の動作をSwiftUIでスムーズにできるようになりたいです。

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

現在発生している問題は、

• 画面Aを閉じたあと、すぐに画面Bを表示した場合、画面Bは閉じられない (dismissが効かなくなる)
• 画面Aを閉じたあと、1秒待ってから画面Bを表示した場合、画面Bは閉じられる

現在、画面A -> (1秒待ち) -> 画面B となっているので、この1秒待ちをなくしたいと考えています。

該当のソースコード

import SwiftUI

struct SampleTwoModalView : View {
    
    enum SheetType {
        case imagePick
        case imageCrop
    }
    
    @State private var currentSheet: SheetType = .imagePick
    @State private var sheetIsPresented = false
    @State private var image: UIImage? = nil
    
    var body: some View {
        VStack {
            Button(action: {
                self.currentSheet = .imagePick
                self.sheetIsPresented = true
            }, label: { Text("select images & crop images") })
        }
        .sheet(isPresented: $sheetIsPresented) {
            if (self.currentSheet == .imagePick) {
                ImagePickerView(sourceType: .photoLibrary, onCanceled: {
                    // on cancel
                }) { (image) in
                    self.image = image
                    
                    //画像を選択するとクロッピング画面へ遷移させる (X)
                    DispatchQueue.main.asyncAfter(wallDeadline: .now() + .milliseconds(1000)) {
                        self.currentSheet = .imageCrop
                        self.sheetIsPresented = true
                    }
                }
            } else if (self.currentSheet == .imageCrop) {
                ImageCropView(originalImage: self.image!, onCanceled: {
                    // on cancel
                }) { (image) in
                    // on success
                }
            }
        }
    }
}

試したこと

(X)の処理を下記のようにすると、画面Bへの遷移はできるが、画面Bをdismissできない状態になります。

//画像を選択するとクロッピング画面へ遷移させる (X)
DispatchQueue.main.async {
    self.currentSheet = .imageCrop
    self.sheetIsPresented = true
}

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

• Xcode 11.6
• iOS 13.3, iOS 13.6

Comments

[fuzzball]

一つ前のブランチ https://github.com/CH3COOH/TOCropViewController/tree/43c05800039a72bb26b999420935118b1ecec6b1 を落としてきて、

DispatchQueue.main.async { //After(wallDeadline: .now() + .milliseconds(1000)) {

に変更して実行してみたのですが、画面B（クロッピング画面）は正常に閉じました。
検証方法はこれで合っているでしょうか？
Xcode 11.5 + iOS 13.5 (Simulator)

[ch3cooh]

環境自体は Xcode 11.6 + iOS 13.6 (Sim) でしたが、検証方法は合ってます！

dismissアニメーション中に `sheetIsPresented`を`true`にしてシートを表示させることで内部的なフラグ(？）がおかしくなっているのではないかと推測しているのですが、 Xcode 11.5と11.6 (もしくは iOS 13.5と13.6)で違いがあるのでしょうか。

[fuzzball]

SwiftUIをチビチビ勉強中で、勉強がてら覗いてみただけなんです。
なので詳しいことはよく分かりません。
あとでXcode落としてみます。

[fuzzball]

11.6でも正しく動いたのでおかしいな‥と思ったら、SchemeがSwiftUIじゃないやつになってました。失礼しました‥。

Answer 1

一つ前のブランチ 43c0580 からの修正です。
偶然見つけただけで、どれほど正しいのか分かりませんが、合っているような雰囲気はあります。

ImagePickerView(croppingStyle: self.croppingStyle, sourceType: .photoLibrary, onCanceled: {
    // on cancel
}) { (image) in
    self.originalImage = image
}.onDisappear() {
    self.currentSheet = .imageCrop
    self.sheetIsPresented = true
}

Comments

[ch3cooh]

シートに対してonDisappear Modifierで閉じたタイミングが取得できるんですね。これはアリです！！ありがとうございます。

ただ使ってるのが UIViewControllerRepresentable でラップしたUIKitなので、UIKit側でdismissした方が良いのか悩ましいですね。

Answer 2

dismissアニメーション中に次の画面をモーダル表示させることで、SwiftUI内部のステータス管理が不正な状態になってしまうのが原因ではないかと考えています。今後のSwiftUIの修正に期待したい部分です。

またSwiftUI側ではdismissが完了したタイミングを取得できないため、dismissが終了してすぐに次の画面へ遷移できません。

私の結論としては、Xcode11.6時点ではUIImagePickerControllerなどのUIKitを使っている以上、SwiftUIで連続してスムーズにモーダル画面を表示するのは難しいので、UIKit側でdismissのタイミングを取る、でした。

本質部分ではないので、ここでは該当部分のみを転記したいと思います。イメージとしては以下の通りです。

public func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey: Any]) {
	guard let image = info[.originalImage] as? UIImage else {
		picker.dismiss(animated: true) {
			self.onImagePicked(nil)
		}
		return
	}
	
	 // UIKit側で閉じるのを待ってからSwiftUI側に画像を渡す
	picker.dismiss(animated: true) {
		self.onImagePicked(image)
	}
}

ソースコードは下記のリポジトリで参照可能です。

• https://github.com/CH3COOH/TOCropViewController/tree/issue/421