Firestoreのデータの取得で'self' used in property access 'model' before all stored properties are initialized

Question

前提

XcodeでFirestoreのデータを取得しようとしています。
ですが、実行するとLocationsViewModel.swiftのlet locations = model.storesに
'self' used in property access 'model' before all stored properties are initialized
とエラーが現れます。
プロジェクトとFirebaseは連携できていることは確認できています。
回答していただけると大変ありがたいです。
どうぞ、よろしくお願いいたします。

実現したいこと

エラーを解決し、Firestoreのデータをアプリに反映させる。

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

LocationsViewModel.swift1行目のlet locations = model.stores

'self' used in property access 'model' before all stored properties are initialized

該当のソースコード

class LocationsViewModel: ObservableObject {
        @ObservedObject var model = FirebaseViewModel()

        init() {
        let locations = model.stores
        self.locations = locations
        self.mapLocation = locations.first!
        self.updateMapRegion(location: locations.first!)
        }
}

Firestoreからデータを取得するコード

import SwiftUI
import FirebaseFirestore
import MapKit

class FirebaseViewModel: ObservableObject {
    @Published var stores = [Location]()
    
    init() {
        self.stores = stores
        getData()
    }


    func getData() {

        let db = Firestore.firestore()

        db.collection("stores").getDocuments { snapshot, error in
            if error == nil {
                // No errors
                if let snapshot = snapshot {
                    DispatchQueue.main.async {
                        self.stores = snapshot.documents.map { d in
                            return Location(name: d["name"] as? String ?? "",
                                        category: d["category"] as? String ?? "",
                                        link: d["url"] as? String ?? "",
                                        description: d["description"] as? String ?? "",
                                        coordinates: d["coordinates"] as! GeoPoint,
                                        id: d.documentID)
                        }
                    }
                }
            }
            else {
            }
        }
       
    }
}

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

macOS バージョン12.4 1.1GHz 4コアIntel Core i5
Xcodeバージョン13.4.1

Answer 1

エラーメッセージの通りですが、
エラーの意味は、
LocationsViewModelのインスタンスが完全に生成されるまでは
そのインスタンスのプロパティを参照することはできません
ということだと思います。

イニシャライザ(init())でインスタンスを生成することになりますので、
(イニシャライザが終わるまではインスタンスはまだ完全に生成されていないことになります)
この処理の中でインスタンスのプロパティ(model)を参照していてエラーになっていると思います。

影響がわからないですが、次のような感じはいかがでしょうか。

    @ObservedObject var model: FirebaseViewModel // ← プロパティの初期値は設定しないでイニシャライザで設定します。
    init() {
        // イニシャライザでプロパティを設定します(modelプロパティも含めて)。
        let initmodel = FirebaseViewModel()
        self.model = initmodel
        let locations = initmodel.stores
        self.locations = locations
        self.mapLocation = locations.first!
        self.updateMapRegion(location: locations.first!)
    }

イニシャライザは奥が深いと感じますので、ドキュメントも読み込むと良いかもしれませんね。

Initialization — The Swift Programming Language (Swift 5.7)

---

それから気になったのはObservableObjectについてです。

View -> LocationsViewModel -> FirebaseViewModel
のようにクラスの参照があるのかなと思いますが、
FirebaseViewModel.storesのプロパティが変わったとしても
Viewはちゃんと変更を受け取れないように思いました。
＊内部の値が変わったとしても画面にその内容が表示されないと思います

Viewが変更を受け取れるのは
LocationsViewModelの中に@Publishedプロパティがあったとして
そのプロパティ自体が変わった場合のみだと思います。

ちょっと今回の質問とは別の問題になると思いますので、
必要であればまた別に質問していただければと思います。

追記です。

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

不明な点がありましたら
解決済みにせずに回答者に確認しても良いかもしれませんね。

エラーは解消されましたが、getData()が呼び出されていないみたいです、、、

おそらく getData() は呼び出されているのではないかと思っています。
問題を正しく認識しないと対処もできないと思いますので、
問題になっていそうなポイントをサンプルのコードとして書いてみました。
タップするとindex(カウンター)が1ずつ増えていくだけのものです。

でもこれは動きません。
View -> LocationsViewModel -> FirebaseViewModel
という参照があって、
FirebaseViewModelの内容を変更しているためです。
Viewが変更を受け取れるのはLocationsViewModelの中のプロパティ”自身”だけです。
LocationsViewModelのmodelプロパティの”中身”(FirebaseViewModelの中のプロパティ)が変わってもViewは変更を受け取れないのです・・

どう対処するのが良いのかは私自身まだ理解できていませんので、
勉強しておきたいと思います・・

import SwiftUI

struct ContentView: View {
    @ObservedObject var vm = LocationsViewModel()
    var body: some View {
        Text("Hello, world!")
            .padding()
        Text("index: \(vm.model.index)")
        Button("tap", action: action)
    }
    func action() {
        // indexを変更してもViewには反映されません。
        vm.model.index += 1
        print("index: \(vm.model.index)")
        // model自体が変わるのであればViewは変更を受け取れます。
        // 例えば強引ですが...modelをmodelに再代入するとViewは変更を受け取れます...
        // vm.model = vm.model
    }
}

class LocationsViewModel: ObservableObject {
    @Published var model: FirebaseViewModel
    init() {
        let initmodel = FirebaseViewModel()
        self.model = initmodel
    }
}

class FirebaseViewModel: ObservableObject {
    @Published var index = 0
    init() {
        getData()
    }
    func getData() {
        print("getData")
        DispatchQueue.main.async {
            // indexを変更してもViewには反映されません。
            self.index = 10
            print("index(getData): \(self.index)")
        }
    }
}

Comments

[avocadoningen]

丁寧にありがとうございます。
エラーは解消されましたが、getData()が呼び出されていないみたいです、、、
頑張って呼び出します。

[avocadoningen]

ですがprint文をgetData()の中に書くと、表示されませんでした。
心当たりありませんか？

[退会済みユーザー]

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

> ですがprint文をgetData()の中に書くと、表示されませんでした。
そうだったのですね。
FirebaseViewModelのinit()の中にprintを記述すると出力されますでしょうか？
init()の中でgetDataを呼び出していますよね？
手元の環境ではgetData()自体は呼び出されているのですけどね。

[avocadoningen]

View同士の関係がよくわからなかったので、すべてLocationsViewModelに入れるとうまくいくようになりました！親切にありがとうございました！！

[退会済みユーザー]

コメントありがとうございます。
すべてLocationsViewModelに入れることにしたのですね。
先に進めたようで何よりです。

Answer 2

すべてをFirebaseViewModelの中身をLocationsViewModelに入れると解決しました

import Foundation
import MapKit
import SwiftUI
import FirebaseFirestore

class LocationsViewModel: ObservableObject {
    
    @Published var locations = [Location]()
    
    init() {
        getData()
        self.locations = locations  
    }
    
    func getData() {

        let db = Firestore.firestore()

        db.collection("stores").getDocuments { snapshot, error in
            if error == nil {
                // No errors
                if let snapshot = snapshot {
                    DispatchQueue.main.async {
                        self.locations = snapshot.documents.map { d in
                            return Location(name: d["name"] as? String ?? "",
                                        category: d["category"] as? String ?? "",
                                        link: d["url"] as? String ?? "",
                                        description: d["description"] as? String ?? "",
                                        latitude: d["latitude"] as! Double,
                                        longitude: d["longitude"] as! Double,
                                        id: d.documentID)
                        }
                    }
                }
            }
            else {
                //Handle the error
                print("there is an error with gettiing codes from Firebase")
            }
        }
    }
}