mirror of
https://github.com/JohnEstropia/CoreStore.git
synced 2026-01-11 20:00:30 +01:00
SwiftUI @Published ViewModel example #411
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @elprl on GitHub (May 3, 2023).
I'm a little confused as to how to use this lib in a ViewModel SwiftUI context. I'd rather not have the View have a dependency on the CoreStore lib.
I'm not sure what to do with the snapshot and how to convert it to the @Published array. Will this array then be diffable?
I'm more likely to map the objects to a detailedViewModel object. Something like:
I'm fundamentally not understanding how to get access to the Core Data objects from the listSnapshot.
@elprl commented on GitHub (May 4, 2023):
Update:
I managed to get access to the Core Data objects in the snapshot via:
Not mentioned in the docs, so not sure if this is a good tactic.
@JohnEstropia commented on GitHub (May 9, 2023):
Hi, have you checked the CoreStoreDemo project? There are examples on how to use
ListPublishers in tandem withListReaders andObjectReaders depending on whether you needObjectPublishers orObjectSnapshots in your SwiftUIViews@elprl commented on GitHub (May 9, 2023):
Yes, I did thank you. I also looked at the unit tests. Like I said in my intro, I'd rather not have the View have a dependency on the CoreStore library, but the view model is fine.
In my experience, in more complex enterprise use cases, one rarely goes from a Core Data object straight to View. A ViewModel or Interactor grooms, processes, filters, combines, splices the core data before forming a viewModel object.
Additionally, the
.objectvariable isn't mentioned in the readme. I think your Readme section on Combine needs a better example that accounts for this grooming & processing.I went straight to this section and thus was confused as to what the
datasourceobject was. It doesn't help situations that don't need to be coupled with the View.@JohnEstropia commented on GitHub (May 10, 2023):
The API provides the necessary endpoints for your app. If you prefer not to depend on CoreStore, then you would have to write that layer yourself.
.objectis not documented because it's not the recommended way to access values from anObjectPublisher, especially in SwiftUI where theViewworks with value types. The framework ensures yourViews properly receives notifications because.objectinstance on its own will not tell yourViewto refresh if that object gets updated. I'm not sure how you'd even use.objectwithout depending on CoreStore, because that object is ofCoreStoreObjecttype anyway, in which case it is still better to use eitherObjectPublisherdirectly, orObjectSnapshotwhich is a value type.@JohnEstropia commented on GitHub (May 10, 2023):
Ah, I guess you are using
NSManagedObjects directly instead ofCoreStoreObjects. Nevertheless, you will still be better off using the right wrappers (ObjectPublisherorObjectSnapshot) to properly sync your views.@elprl commented on GitHub (May 10, 2023):
Appreciate the comments John, I do like what CoreStore has achieved. I think the library is not addressing a common architectural pattern, where a
ViewModelclass is doing the syncing as you mentioned. If I wasn't going to use CoreStore, I would be doing something like this:https://www.donnywals.com/observing-changes-to-managed-objects-across-contexts-with-combine/
To give a crude example, let's say you have a Whatsapp style app. Core Data would store the encrypted
Message, theViewModelclass would create a listener for new & updatedMessageobjects, and when they arrive would process eachMessage, decrypt them, convert dates into strings, add colours, etc, and finally create aMessageViewModelobject for theViewto consume. In this scenario, theViewis decoupled from both Core Data and Core Store. We all know mocking Core Data for SwiftUI previews and unit tests is overly complex and hard work.So how does one achieve this? Something like:
@JohnEstropia commented on GitHub (May 11, 2023):
I understand that there are architectures where you have to provide the per-object ViewModels directly, and in fact we do use similar cases in our projects. The problem is that SwiftUI kind of forces us to wrap our ViewModels in some sort of
@Stateor@ObservableObjector something similar forViews to get updated properly. A large part of the observation logic similar to the one in Donny Wal's article you have linked is already provided in CoreStore's@ListStateand@ObjectStateand are ready to be used in SwiftUI projects.(see Modern.ColorsDemo.SwiftUI.ListView.swift and Modern.ColorsDemo.SwiftUI.DetailView.swift in the Demo app)
Now of course you are free to limit the dependency to CoreStore and implement this yourself. In that case, I would still recommend you check how @ListState and @ObjectState does this internally and base your implementation on that. (Or alternatively, ListReader and ObjectReader depending on your requirements)