【SwiftUI】MVVMにおける値の連携方法（@Observed Objectなど）

Question

ViewModelでユーザー情報を取得するAPIを呼び、Modelに格納し、Viewに連携してあげたいです。
１つ目のやり方は成功したものの、コードを綺麗にするために２つ目のやり方を試したら失敗しました。
ご助言いただけますと幸いです。（MVVMアーキテクチャにおいて何かタブーなことをやっているのかもしれません。。）
何卒よろしくお願いいたします。

成功したコード

class AuthViewModel: ObservableObject {

@Published var userMe: User? //UserはユーザーIDや名前など諸々の情報が入るModelのStructです。

init() {//アプリが開かれた瞬間にinitされる
 fetchUserMe()
        }

 func fetchUserMe() {
   //API通信 中略
  self.userMe = try JSONDecoder().decode(User.self, from: data)
}
}

//このViewは、NavigationLinkで遷移する先となる子Viewです。親Viewは省略させていただきます。
struct ProfileHeaderView: View {
    @EnvironmentObject var viewModel: AuthViewModel

var body: some View {
     HStack {
     Text( viewModel.userMe?.usersName ?? "nil")
}
}

@main
struct FriendQA_SwiftUIApp: App {
    
    @UIApplicationDelegateAdaptor(AppDelegate.self) var appDelegate
    var body: some Scene {
        WindowGroup {
            ContentView().environmentObject(AuthViewModel.shared)
        }
    }
}

エラーになったコード

上記でうまくいくものの、AuthViewModelには既にたくさんの関数を書いてしまってごちゃごちゃしているので、
fetchUserMe関数を違うView Modelに移したいと思い、下記のコードに変えましたが、うまくいきませんでした。

class AuthViewModel: ObservableObject {

@ObservedObject var fetchUsersViewModel = FetchUsersViewModel()

init() { //アプリが開かれた瞬間にinitされる
 fetchUsersViewModel.fetchUserMe()
        }
}

class FetchUsersViewModel: ObservableObject {

@Published var userMe: User?

 func fetchUserMe() {
   //API通信 中略
  self.userMe = try JSONDecoder().decode(User.self, from: data)
  print(self.userMe) //ちゃんと中身も出力される
}
}

struct ProfileHeaderView: View {
     @ObservedObject var fetchUsersViewModel = FetchUsersViewModel()

var body: some View {
     HStack {
     Text(fetchUsersViewModel.userMe?.usersName ?? "nil") //nilになってしまう
     Text($fetchUsersViewModel.userMe.wrappedValue?.usersName ?? "nil")　//nilになってしまう
}
}

最後まで読んでいただきありがとうございます。お手数ですが、お力添えいただけますと幸いです。

Comments

[hoshi-takanori]

同じクラスの異なるインスタンスを作ってるからでは。

[ishiishiyay]

Hoshi-takanoriさん、いつもありがとうございます。

同じクラスの異なるインスタンスはここでしょうか？？:
@ObservedObject var fetchUsersViewModel = FetchUsersViewModel()

ちなみに、Profile Header Viewにinit()を記載しその中でfetchUsersViewModelのfetchUserMe()を呼び出したらうまく行ったのですが、Profile Header Viewを表示してからViewが更新されるまでタイムラグがあるので、出来ればアプリ起動と同時にAPI通信したく、

成功した一番最初の書き方しかないでしょうか...？

[退会済みユーザー]

横から書き込みしてしまってすみません・・

hoshi-takanori さんのおっしゃる通り、Viewから参照しているのは直接的な`FetchUsersViewModel`のようです。
`AuthViewModel`の`FetchUsersViewModel`を参照している訳ではなさそうです。

> 同じクラスの異なるインスタンスはここでしょうか？？:
> @ObservedObject var fetchUsersViewModel = FetchUsersViewModel()
そういうことだと思います。
`AuthViewModel`のfetchUsersViewModelを参照しないことには、思った通りにはならないように思います。

> 成功した一番最初の書き方しかないでしょうか...？
そんなことはないとは思いますが、
文字で説明するのが大変になってきそうです・・

クラスの構造や処理の流れを整理してご自身で一つずつ理解するところから始めた方が良さそうです。

---

> （MVVMアーキテクチャにおいて何かタブーなことをやっているのかもしれません。。）

私自身、MVVMの理解が浅いのですが、
かつ、SwiftでのMVVMは、業務レベルでは皆無ですが、
ModelでやるべきことをViewModelでやってしまっているのではないか、
という気もします。

> ViewとViewModelでプレゼンテーション・ロジックを分担します。ViewModelはUI要素と結合しないプレゼンテーション・ロジックを担当し、ViewはUI要素に結合したプレゼンテーション・ロジックを担当します（具体的にはSystem.Windows名前空間ではじまる列挙体・構造体以外の型のプロパティはViewModelでは所持しないなどの切り分けが可能です）。
>
> https://atmarkit.itmedia.co.jp/fdotnet/chushin/greatblogentry_02/greatblogentry_02_03.html

.

> MVVMにおいてはViewModel同士がお互いを操作する必要なんて基本的にはありません。所有関係はあることはありますがそもそもViewModelはModelの影なのです。そしてまたViewはViewModelの影でもあります。
>
> https://ugaya40.hateblo.jp/entry/model-mistake

(MVVMの発祥がC#な気がしておりますので、Swiftに関する質問ですが、あえてC#の記事を引用させていただきました)

[ishiishiyay]

xg63ex2bさん、ご詳細に誠にありがとうございます！ViewModel同士がお互いを操作する必要ないんですね。関数をコールするだけならいいかなと思ってしまいました。設計の方法から勉強し直します！ありがとうございます。

Answer 1

onAppear()でうまくいくかもしれません。

class FetchUsersViewModel: ObservableObject {
  @Published var userMe: User?

  func fetchUserMe() {
    self.userMe = try JSONDecoder().decode(User.self, from: data)
    print(self.userMe) //ちゃんと中身も出力される
  }
}

struct ProfileHeaderView: View {
  @ObservedObject var fetchUsersViewModel = FetchUsersViewModel()

  var body: some View {
     HStack {
       Text(fetchUsersViewModel.userMe?.usersName ?? "nil")
       Text($fetchUsersViewModel.userMe.wrappedValue?.usersName ?? "nil")
    }
    .onAppear {
      fetchUsersViewModel.fetchUserMe()
    }
  }
}