modelの値を更新しても、親Viewのテキストが更新されない。 子ビューのみ更新される。

Question

わからないこと

modelの値を更新しても、親Viewのテキストが更新されない。
親Viewから更新しても、子Viewから更新されても、子Viewのテキストが更新されるだけで親Viewのテキストは更新されない

下記ソースコード内のコメントにあるように、

1. 子View内から値を更新すると、子Viewのテキストは更新されるが親Viewのテキストは更新されない
2. 親View内から値を更新しても、上記と同様

親Viewのテキストを自動的に更新するようにしたいです.
＠Published したものは、更新されると自動的に通知をするようですが、何故かされない状況です.
よろしくお願いします.

環境

• Xcode: 11.3
• Swift: 5.1.3

ソースコード

import Foundation
import SwiftUI

//親View
struct ContentView: View {
    @ObservedObject var viewModel: ContentViewModel
    
    var body: some View {
        VStack {
            List(viewModel.songs) { song in
                SongCell(viewModel: self.viewModel, song: song)
            }
            
            //3. ここの値が更新されない（起動時のデフォルト値は"title1")
            Text("I am ContetView : (viewModel.songs[0].title)")
            
            //2. セルをタップ後(viewModel.songs[0].titleを"changed from ContentView"に更新)
            //上にある3.のテキストは更新されない
            Button(action: {
                self.viewModel.songs[0].title = "changed from ContentView"
            }, label: {Text("contentView add song")})
        }
    }
}

//楽曲ごとのセル、子ビュー
struct SongCell: View {
    @ObservedObject var viewModel: ContentViewModel
    @ObservedObject var song: Song
    
    
    var body: some View {
        HStack {
            Text("(song.title) / (song.artist)")
            Button(action: {
                
                //1. セル内の表記のみ変化する(viewModel.songs[0].titleを更新）
                // 親ビューの3.のテキストは更新されない
                self.song.title = "changed from SongCell"
                
            }, label: {
                Text("button")
            })
        }
    }
}

//ViewModel
class ContentViewModel: ObservableObject {
    @Published var songs: [Song] = [
        Song.init(title: "title1", artist: "artist1", url: "url1"),
    ]
    
    func addSong(title: String, artist: String, url: String) -> Void {
        let song = Song(title: title, artist: artist, url: url)
        songs.append(song)
    }
    
    func titleChange(song: Song, newTitle: String) {
        song.title = newTitle
    }
}


//各曲のデータ
class Song: Identifiable, ObservableObject{
    var id = UUID()
    @Published var title: String
    @Published var artist: String
    @Published var url: String
    
    init(title: String, artist: String, url: String) {
        self.title = title
        self.artist = artist
        self.url = url
    }
}

Answer 1

現象再現しました。
原因は、Songの変更が、ContentViewModelの変更とはみなされない為だと思います。
親viewはContentViewModelをみていて、子viewはSongsをみています。
あるSong.titleを変えたとき、Songは@Publishしているtitleの変化を通知するので子viewは変化を反映します。
しかし、親viewのみているContentViewModel.songsには変化が生じないため、ContentViewModelが変化したとはみなされず通知されていないのだと思います。
これはSongがクラスで参照型なので生じる現象です。
以下にSongを値型であるstructに変えた場合にどうなるかを例示します。
この実装なら、親viewも子viewも両方とも変化します。

import SwiftUI

//親View
struct ContentView1: View {
    @ObservedObject var viewModel: ContentViewModel

    var body: some View {
        VStack {
            List(viewModel.songs) { song in
                SongCell(viewModel: self.viewModel, song: song)
            }

            Text("I am ContetView : (viewModel.songs[0].title)")

            Button(action: {
                self.viewModel.changeTitle(of: self.viewModel.songs[0], to: "changed from ContentView")
            }, label: { Text("contentView add song") })
        }
    }
}

//楽曲ごとのセル、子ビュー
struct SongCell: View {
    @ObservedObject var viewModel: ContentViewModel
    var song: Song

    var body: some View {
        HStack {
            Text("(song.title) / (song.artist)")

            Button(action: {
                self.viewModel.changeTitle(of: self.song, to: "changed from SongCell")

            }, label: {
                Text("button")
            })
        }
    }
}

//ViewModel
class ContentViewModel: ObservableObject {
    @Published var songs: [Song] = [
        Song.init(title: "title1", artist: "artist1", url: "url1"),
    ]

    func addSong(title: String, artist: String, url: String) -> Void {
        let song = Song(title: title, artist: artist, url: url)
        songs.append(song)
    }
    func changeTitle(of song: Song, to newTitle: String) {
        guard let index = songs.firstIndex(where: { $0.id == song.id }) else { return }
        let updatedSong = song.changingTitle(to: newTitle)
        songs[index] = updatedSong
    }
}


//各曲のデータ
struct Song: Identifiable {
    var id: UUID
    var title: String
    var artist: String
    var url: String

    init(title: String, artist: String, url: String) {
        id = UUID()
        self.title = title
        self.artist = artist
        self.url = url
    }
    
    init(id: UUID, title: String, artist: String, url: String) {
        self.id = id
        self.title = title
        self.artist = artist
        self.url = url
    }
    
    func changingTitle(to newTitle: String) -> Song {
        return Song(id: id, title: newTitle, artist: artist, url: url)
    }

    func changingArtist(to newArtist: String) -> Song {
        return Song(id: id, title: title, artist: newArtist, url: url)
    }

    func changingUrl(to newUrl: String) -> Song {
        return Song(id: id, title: title, artist: artist, url: newUrl)
    }
}

既存のコードに対しては大きな変更になるでしょうし、変更がある度に作り直すのはパフォーマンス上も気になってしまうかもしれませんが、appleは値型によるコーディングを推奨していますし、このほうがSwiftUIとも相性がいいと思います。

もっとよい実装もありそうな気もしますが・・・他の方の回答に期待します。

Comments

[tabe_unity]

ありがとうございます。無事動作を確認することができました。
やはり親Viewが検知できていなかったんですね。確かにsongsそのものに変化はないですね。次から値型のプログラミングを意識していきます！

詰まって進まなかったので助かりました！　これで開発を再開することができます！　ありがとうございました！

[tabe_unity]

すいません、一つ質問があります。
Song struct内にて定義されている関数で、返されるSongのアップデートされないプロパティは元のプロパティがそのまま入るという解釈で大丈夫でしょうか？

[eytyet]

はい。idも含めて元の値がそのまま維持されます。

[tabe_unity]

ありがとうございます。