swift realm 「Object has been deleted or invalidated.」エラー

Question

swift初心者です。
swiftuiを使用してます。

データの削除をしようとしているのですが、
「Object has been deleted or invalidated.」が出てしまいます。
原因がわかりません。
教えていただきたいです。

また、
realmを見てもデータが登録されないのですが、
なぜでしょうか？

他に追記があった方がいい場合は追記いたします。

・View

import SwiftUI

// MARK: UserListView
struct UserView: View {
    @ObservedObject var viewModel = UserViewModel()

    @State private var isUserTextFieldPresented = false
    @State private var isDeleteAlertPresented = false
    @State private var isDeleteAllAlertPresented = false
    @State private var userTextField = ""

    var body: some View {
        NavigationView {
            VStack {
                if (isUserTextFieldPresented) {
                    TextField("メモを入力してください", text: $userTextField)
                        .textFieldStyle(DefaultTextFieldStyle())
                        .keyboardType(.asciiCapable)
                }
                List {
                    ForEach(viewModel.users.sorted {
                        $0.postedDate > $1.postedDate
                    }, id: \.self) { user in
                        HStack {
                            UserRowView(user: user)
                            Spacer()
                            // Buttonにすると行全体にタップ判定がついてしまったので、Text.onTapGestureを代わりに使っている
                            Text("削除").onTapGesture {
                                isDeleteAlertPresented.toggle()
                            }
                            .padding()
                            .foregroundColor(.white)
                            .background(Color.red)
                        }
                        .alert(isPresented: $isDeleteAlertPresented) {
                            Alert(title: Text("警告"),
                                  message: Text("メモを削除します。\nよろしいですか？"),
                                  primaryButton: .cancel(Text("いいえ")),
                                  secondaryButton: .destructive(Text("はい")) {
                                    viewModel.deleteUser = user
                                  }
                            )
                        }
                    }
                }
            }
            .navigationTitle("メモの一覧")
            .navigationBarTitleDisplayMode(.inline)
            .toolbar {
                ToolbarItem(placement: .navigationBarTrailing) {
                    Button("追加") {
                        if (isUserTextFieldPresented) {
                            viewModel.userTextField = userTextField
                            userTextField = ""
                        }
                        isUserTextFieldPresented.toggle()
                    }.disabled(isUserTextFieldPresented && userTextField.isEmpty)
                }
            }
        }
    }
}

// MARK: UserRowView
struct UserRowView: View {
    var user: User

    var body: some View {
        VStack(alignment: .leading) {
            Text(formatDate(user.postedDate))
                .font(.caption)
                .fontWeight(.bold)
            Text(user.userName)
                .font(.body)
        }
    }

    func formatDate(_ date : Date) -> String {
        let formatter = DateFormatter()
        formatter.dateStyle = .long
        formatter.timeStyle = .medium
        formatter.locale = Locale(identifier: "ja_JP")
        return formatter.string(from: date)
    }
}

・ViewModel

import Foundation
import Combine
import RealmSwift

class UserViewModel: ObservableObject {

    @Published private(set) var users: [User] = Array(User.findAll())
    @Published var userTextField = ""
    @Published var deleteUser: User?
    @Published var isDeleteAllTapped = false

    private var addUserTask: AnyCancellable?
    private var deleteUserTask: AnyCancellable?

    init() {
        addUserTask = self.$userTextField
            .sink() { username in
                guard !username.isEmpty else {
                    return
                }
                let user = User()
                user.userName = username
                self.users.append(user)
                User.add(user)
            }
        deleteUserTask = self.$deleteUser
            .sink() { user in
                guard let user = user else {
                    
                    return
                }
                if let index = self.users.firstIndex(of: user) {
                    do{

                        self.users.remove(at: index)
                        
                        User.delete(user)
                    }catch {
                      print("Error \(error)")
                    }
                }
            }
    }
}

・User

import Foundation
import RealmSwift

class User: Object {
    //ユーザID
    @objc  dynamic var id: Int = 0
    //ユーザ名
    @objc  dynamic var userName: String = ""
    
    @objc dynamic var postedDate = Date()
    
    //Primary Keyの設定
    override static func primaryKey() -> String? {
        return "id"
    }
}

extension User {
    private static var config = Realm.Configuration(schemaVersion: 1)
    private static var realm = try! Realm(configuration: config)

    static func findAll() -> Results<User> {
        realm.objects(self)
    }

    static func add(_ user: User) {
        try! realm.write {
            realm.add(user, update: .all)
        }
    }

    static func delete(_ user: User) {
        try! realm.write {
            realm.delete(user)
        }
    }
}

Comments

[退会済みユーザー]

すみません、再現確認してなくて、コードを眺めただけなのですが、
Userのidは初期値の0以外に設定しているところはありますでしょうか？

[asedsa]

質問ありがとうございます。
Userのidは載せているコードの部分でしか設定していません。

[退会済みユーザー]

コメントありがとうございます。

> 「Object has been deleted or invalidated.」
> realmを見てもデータが登録されないのですが、

Realmについてあまり詳しくなくて、見当違いなことを書いてしまうかもしれませんが、
idが主キーになっているものの、
全てのデータが0なので、
正しく登録できなそうな気がしました。
登録されていないのであれば、削除することもできないのかな、と思いました。

＊一般的に主キーは一意という印象から書いています・・

[asedsa]

なるほどですね！
ありがとうございます！
確認してます！

Answer 1

まず、Realmの使用以前に削除ボタンをタップした時の処理がおかしいので、その点を指摘しておきます。

.alert(isPresented: $isDeleteAlertPresented) {
    Alert(title: Text("警告"),
          message: Text("メモを削除します。\nよろしいですか？"),
          primaryButton: .cancel(Text("いいえ")),
          secondaryButton: .destructive(Text("はい")) {
              viewModel.deleteUser = user
          }
    )
}

この中で viewModel.deleteUser = user で user を参照していますが、削除ボタンをタップした時のユーザは、この user には入っていません。

削除ボタンをタップした時のユーザは、onTapGestureの中でないと取得できませんので、削除ボタンをタップした時のユーザを覚えておくプロパティを次のように作成しておき、

@State private var deleteUser: User?

onTapGestureの中で次のように deleteUser に保管する処理を追加しておき、

self.deleteUser = user

Alertの中で、次のようにして削除ボタンをタップした時のユーザを渡す必要があります。

          secondaryButton: .destructive(Text("はい")) {
              viewModel.deleteUser = self.deleteUser
          }

次に、追記・修正依頼欄でxg63ex2bさんがコメントされている通り、UserをRealmに保存する時にidを正しく設定していないので、新しいユーザーを何個追加しても同じidのユーザーを上書きする処理になってしまい、1個しか保存されていません。
(画面上に表示されているのは、Realmに保存したデータそのものではなく、自分で追加・削除しているUserの配列を表示していますから、画面上には追加されたように見えていますが、Realmのデータベースの中には1個しか保存されていません。)

UserViewModel.swiftのaddUserTask処理の中で、次のようにして user.id を設定すると良いと思います。

let user = User()
user.id = (try! Realm().objects(User.self).max(ofProperty: "id") ?? 0) + 1
user.userName = username
self.users.append(user)
User.add(user)

これでRealmへの保存処理は正しくなりますが、これだけでは質問のエラーは解消されません。
質問のエラーが発生する原因は、Realmのデータ管理処理とSwiftUIの処理が競合してしまうためです。
そこで、今の実装をなるべく尊重するなら、RealmのモデルクラスであるUserをfreezeして処理すると
うまく行きます。
まず、先ほどのaddUserTaskの処理は、次のように先にRealmへの保存を実施し、その後freezeしたuserをusers配列に追加します。

let user = User()
user.id = (try! Realm().objects(User.self).max(ofProperty: "id") ?? 0) + 1
user.userName = username
User.add(user)
self.users.append(user.freeze())

次に、User.swiftのfindAllは、次のようにfreezeした一覧を返します。

static func findAll() -> Results<User> {
    realm.objects(self).freeze()
}

そして、User.swiftのdelete処理は、freezeされたuserのidを元にRealmに管理されているuserを取得し、それを使って削除します。

static func delete(_ user: User) {
    let actualUser = realm.object(ofType: User.self, forPrimaryKey: user.id)!
    try! realm.write {
        realm.delete(actualUser)
        
    }
}

これで正しく処理できるようになると思います。
(実際にやってみて正しく動作することを確認しました。)

なお、今の方法は、Realmデータベースに保存されているデータを一旦配列に全て読み込んで処理しており、データベースのメリットをあまり活かせていません。SwiftUIでRealmを使う時は、バージョン10.10以降からSwiftUIに対応した処理方法が用意されていますので、

https://tkgstrator.work/2021/09/27/observedresults/

で解説されているように、 @Persisted や @ObservedResults を使い、 append メソッド、 remove メソッドで追加/削除する方法で実装することをお勧めします。

Comments

[asedsa]

とても丁寧な解説ありがとうございます
ベストアンサーとさせていただきます