【SwiftUI】VideoURLが保存されない or 表示されない

Question

実現したいこと

献立アプリを作っています。
Core Dataを使用してユーザーがデータベースに料理のメニューを追加でき、またデータベースから料理の情報を取り出してリストに表示できるようにしたいです。
保存、表示したい情報としてはname/sort/time/calorie/image/videoがあります。

発生している問題・分からないこと

AddRecipeViewで、videoの選択はできるのですが、再度開いた時に保存されていない、もしくは保存はされているが表示されないという状態です。

該当のソースコード

import SwiftUI

struct AddRecipeView: View {
    @Environment(\.presentationMode) var presentationMode: Binding<PresentationMode>
    @Environment(\.managedObjectContext) private var viewContext
    @EnvironmentObject var dateHolder: DateHolder
    
    @State var selectedRecipe: Menu?
    @State var name: String
    let sorts = ["", "PASTA & RICE", "MEAT & FISH", "SALAD", "SOUP"]
    @State private var sortNo = 1
    @State private var time = 0
    @State private var calorie = ""
    @State private var selectedImage: UIImage?
    @State private var isShowingImagePicker = false
    @State private var selectedVideoURL: URL?
    @State private var isShowingVideoPicker = false
    
    init(alreadyMadeMenu: Menu?) {
        if let menu = alreadyMadeMenu {
            _selectedRecipe = State(initialValue: menu)
            _name = State(initialValue: menu.name ?? "")
            _sortNo = State(initialValue: Int(menu.sortNo?.sortNo ?? 0))
            _time = State(initialValue: Int(menu.time))
            _calorie = State(initialValue: menu.calorie ?? "")
            _selectedImage = State(initialValue: menu.selectedImage.flatMap { UIImage(data: $0) })
            if let videoURL = menu.selectedVideoURL as? URL {
                _selectedVideoURL = State(initialValue: videoURL)
            } else {
                _selectedVideoURL = State(initialValue: nil)
            }
        } else {
            _name = State(initialValue: "")
            _sortNo = State(initialValue: 0)
            _time = State(initialValue: 0)
            _calorie = State(initialValue: "")
            _selectedImage = State(initialValue: UIImage())
            _selectedVideoURL = State(initialValue: URL(string: ""))
        }
    }
    var body: some View {
        Form {
            Section(header: Text("Menu")){
                TextField("料理名", text: $name)
                Picker("種類を選択", selection: $sortNo) {
                    ForEach(0 ..< sorts.count, id: \.self) { num in
                        Text(self.sorts[num])
                    }
                }
                HStack {
                    Text("調理時間")
                    Spacer()
                    Picker("", selection: $time) {
                        ForEach(1 ..< 91, id: \.self) { num in
                            Text("\(num)分")
                        }
                    }
                }
                TextField("カロリー", text: $calorie)
                VStack {
                    HStack {
                        Text("Photo")
                            .font(.title3)
                        Spacer()
                    }
                    Spacer()
                    HStack {
                        if let image = selectedImage {
                            Image(uiImage: image)
                                .resizable()
                                .aspectRatio(contentMode: .fit)
                                .frame(width: 200, height: 200)
                        } else {
                            Text("写真が選択されていません")
                        }
                        Spacer()
                        Button("写真を選択") {
                            // ボタンがタップされたときに画像選択UIを表示する
                            isShowingImagePicker.toggle()
                        }
                    }
                    .sheet(isPresented: $isShowingImagePicker) {
                        // 画像選択UIを表示する
                        ImagePicker(image: $selectedImage)
                    }
                }
                VStack {
                    HStack {
                        Text("Video")
                            .font(.title3)
                        Spacer()
                    }
                    Spacer()
                    HStack {
                        if let videoURL = selectedVideoURL {
                            Text("Selected Video: \(videoURL.lastPathComponent)")
                        } else {
                            Text("ビデオが選択されていません")
                        }
                        Spacer()
                        Button("ビデオを選択") {
                            // ボタンがタップされたときに動画選択UIを表示する
                            isShowingVideoPicker.toggle()
                        }
                    }
                    .sheet(isPresented: $isShowingVideoPicker) {
                        // 動画選択UIを表示する
                        DocumentPicker(url: $selectedVideoURL)
                    }
                }
            }
            Section() {
                Button("Save", action: saveAction)
                    .font(.headline)
                    .frame(maxWidth: .infinity, alignment: .center)
            }
        }
    }
    func saveAction() {
        withAnimation {
            if selectedRecipe == nil {
                selectedRecipe = Menu(context: viewContext)
            }
            selectedRecipe?.created = Date()
            selectedRecipe?.name = name
            selectedRecipe?.time = Double(time)
            selectedRecipe?.calorie = calorie
            if let selectedImage = selectedImage {
                if let imageData = selectedImage.jpegData(compressionQuality: 1.0) {
                    selectedRecipe?.selectedImage = imageData
                }
            }
            selectedRecipe?.selectedVideoURL = selectedVideoURL?.absoluteString
            // Sortエンティティのインスタンスを取得する
            let sort = Sort(context: viewContext)
            sort.sortNo = Int64(sortNo)
            // selectedRecipeのsortNoプロパティにSortエンティティを代入する
            selectedRecipe?.sortNo = sort
            
            dateHolder.saveContext(viewContext)
            self.presentationMode.wrappedValue.dismiss()
        }
    }
}

struct AddRecipeView_Previews: PreviewProvider {
    static var previews: some View {
        AddRecipeView(alreadyMadeMenu: Menu())
    }
}

import Foundation
import CoreData

class DateHolder: ObservableObject {
    init(_ context: NSManagedObjectContext) {
        
    }
    
    func saveContext(_ context: NSManagedObjectContext) {
        do {
            try context.save()
        } catch {
            let nsError = error as NSError
            fatalError("Unresolved error \(nsError), \(nsError.userInfo)")
        }
    }
}

import SwiftUI
import MobileCoreServices

struct DocumentPicker: UIViewControllerRepresentable {
    @Binding var url: URL?
    @Environment(\.presentationMode) var presentationMode

    class Coordinator: NSObject, UIDocumentPickerDelegate {
        let parent: DocumentPicker

        init(parent: DocumentPicker) {
            self.parent = parent
        }

        func documentPicker(_ controller: UIDocumentPickerViewController, didPickDocumentsAt urls: [URL]) {
            guard let url = urls.first else { return }
            parent.url = url
            parent.presentationMode.wrappedValue.dismiss()
        }

        func documentPickerWasCancelled(_ controller: UIDocumentPickerViewController) {
            parent.presentationMode.wrappedValue.dismiss()
        }
    }

    func makeCoordinator() -> Coordinator {
        Coordinator(parent: self)
    }

    func makeUIViewController(context: Context) -> UIDocumentPickerViewController {
        let picker = UIDocumentPickerViewController(documentTypes: [String(kUTTypeMovie)], in: .open)
        picker.delegate = context.coordinator
        return picker
    }

    func updateUIViewController(_ uiViewController: UIDocumentPickerViewController, context: Context) {}
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

以下のようなデバック出力を追加してみましたが、 selectedRecipe?.selectedVideoURL は正しく設定されているようでした。

print("selectedVideoURL: \(selectedVideoURL)")
print("selectedRecipe?.selectedVideoURL: \(selectedRecipe?.selectedVideoURL)")
selectedRecipe?.selectedVideoURL = selectedVideoURL?.absoluteString
print("After assignment:")
print("selectedRecipe?.selectedVideoURL: \(selectedRecipe?.selectedVideoURL)")

selectedRecipe?.selectedVideoURL = selectedVideoURL?.absoluteString
print("selectedVideoURL: \(selectedVideoURL?.absoluteString ?? "nil")")
print("selectedRecipe?.selectedVideoURL: \(selectedRecipe?.selectedVideoURL ?? "nil")")

補足

Xcode version14.2
回答にあたり必要な情報があれば伝えていただきたいです。
お分かりになる方、回答よろしくお願いします。

Answer 1

実際に動かして確認などできていないのですが、ソースコードを見て気になった点を書きますね。

16行目のselectedVideoURLはURL型ですよね。
133行目で selectedRecipe?.selectedVideoURL = selectedVideoURL?.absoluteString のようにString型を保存しているように見えます。

27行目では if let videoURL = menu.selectedVideoURL as? URL { のようにURL型にキャストしていますね。
Menu型(エンティティ？)のselectedVideoURLはString型だと思いますので、ここは必ずelseに入りそうに思いましたが、いかがでしょうか？

＊今回はビデオに関する項目が問題になっていると思いますので、
そのほかの項目(料理名、調理時間など)の項目は削除した状態で
該当のソースコードをご記載いただけますと読みやすいかなと思いました。
＊Menuエンティティの定義もご記載いただけますとより問題が把握しやすいかなと思いました。

追記です。

コメントありがとうございます。
原因がわかったようでよかったです。

・attributeでURL型が選択できない

URLはURIの部分集合らしいですので、
AttributeのTypeにURIを指定すると
SwiftのソースコードからはURL?型として使うことができるみたいですね。

・一方108行目ではBinding<URL?>型を代入しなくてはならない

1点目の対処でもう不要になるかもしれませんが、
URL型が使いづらい場合は、
extensionでString型をURL型に変換してくれるような
Computed Properties を作成するのも良いかもしれませんね。

extension Menu {
    var selectedVideoURLString: String {
        get {
            selectedVideoURL?.absoluteString ?? ""
        }
        set {
            selectedVideoURL = URL(string: newValue)
        }
    }
}

// getの使い方
Text(selectedRecipe.selectedVideoURLString)
// setの使い方
selectedRecipe.selectedVideoURLString = "https://tenki.jp/forecast/3/"

＊後から気づきましたが、例が逆かもしれませんね・・
エンティティはString型でComputed PropertiesはURL型
という記述の方が質問に対する回答として適切でしたね・・
(selectedVideoURLはURL型の前提で記述してしまいました・・)

Comments

[runrun]

なるほど！本当にその通りだと思います。（記載の件も参考になります！）
Menuというエンティティの中にselectedVideoURLというattributeを作っているのですが、
このattributeで選択できる型にURL型が無いため、仕方なくString型にして変換をしていました。
以下の2点に悩まされているのですが、どのようにコードを書いて解決すれば良いでしょうか？
・attributeでURL型が選択できない
・一方108行目ではBinding<URL?>型を代入しなくてはならない

[runrun]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。