SwiftUI reorder CoreData Objects in List
Asked Answered
G

5

21

I want to change the order of the rows in a list that retrieves objects from the core data. Moving rows works, but the problem is that I can't save the changes. I don't know how to save the changed Index of the CoreData Object.

Here is my Code:

Core Data Class:

public class CoreItem: NSManagedObject, Identifiable{
    @NSManaged public var name: String

}

extension CoreItem{
    static func getAllCoreItems() -> NSFetchRequest <CoreItem> {
        let request: NSFetchRequest<CoreItem> = CoreItem.fetchRequest() as! NSFetchRequest<CoreItem>
        let sortDescriptor = NSSortDescriptor(key: "date", ascending: true)
        request.sortDescriptors = [sortDescriptor]
        return request
    }
}

extension Collection where Element == CoreItem, Index == Int {
    func move(set: IndexSet, to: Int,  from managedObjectContext: NSManagedObjectContext) {

        do {
            try managedObjectContext.save()
        } catch {
            let nserror = error as NSError
            fatalError("Unresolved error \(nserror), \(nserror.userInfo)")
        }
    }
} 

List:



struct CoreItemList: View {

    @Environment(\.managedObjectContext) var managedObjectContext

    @FetchRequest(fetchRequest: CoreItem.getAllCoreItems()) var CoreItems: FetchedResults<CoreItem>



var body: some View {
      NavigationView{
          List {
            ForEach(CoreItems, id: \.self){
                   coreItem in
                    CoreItemRow(coreItem: coreItem)
                  }.onDelete {
                  IndexSet in let deleteItem = self.CoreItems[IndexSet.first!]
                  self.managedObjectContext.delete(deleteItem)

                  do {
                      try self.managedObjectContext.save()
                  } catch {
                      print(error)
                     }
                  }
                .onMove {
                    self.CoreItems.move(set: $0, to: $1, from: self.managedObjectContext)
              }
            }
             .navigationBarItems(trailing: EditButton())
           }.navigationViewStyle(StackNavigationViewStyle())
        }
    }

Thank you for help.

Gallinaceous answered 14/1, 2020 at 21:44 Comment(5)
If you want to store the order then you need an attribute for it in your CoreItem entity and you need to use that attribute in the sort descriptor. But is this really a good idea, you are going to need to update this attribute for all objects that gets affected by the move?Minutiae
Maybe there is a better solution which I don’t know. The problem is that I want to let the user reorder the list and save the changes.Gallinaceous
Like I said, then you need to have an attribute that contains the orderMinutiae
A pattern I've seen that avoids the scaling problem alluded to by Joakim of having to update many items when a single item is moved, is have the sort criteria on the iterm be a decimal of some sort. Then when an item is moved, it's a matter of updating a single item by setting its order value to be between that of the items ahead and behind it in the list.Guardian
You could order based on a sequence of floats. When you add element at the end, you increment by 1. When you insert between two then you find their average. Eventually you will reach the resolution of float. This would require so many divisions to happen that you can use it to trigger pretty expensive - array reordering and spacing elements by 1.Kinch
M
50

Caveat: the answer below is untested, although I used parallel logic in a sample project and that project seems to be working.

There's a couple parts to the answer. As Joakim Danielson says, in order to persist the user's preferred order you will need to save the order in your CoreItem class. The revised class would look like:

public class CoreItem: NSManagedObject, Identifiable{
    @NSManaged public var name: String
    @NSManaged public var userOrder: Int16
}

The second part is to keep the items sorted based on the userOrder attribute. On initialization the userOrder would typically default to zero so it might be useful to also sort by name within userOrder. Assuming you want to do this, then in CoreItemList code:

@FetchRequest( entity: CoreItem.entity(),
                   sortDescriptors:
                   [
                       NSSortDescriptor(
                           keyPath: \CoreItem.userOrder,
                           ascending: true),
                       NSSortDescriptor(
                           keyPath:\CoreItem.name,
                           ascending: true )
                   ]
    ) var coreItems: FetchedResults<CoreItem>

The third part is that you need to tell swiftui to permit the user to revise the order of the list. As you show in your example, this is done with the onMove modifier. In that modifier you perform the actions needed to re-order the list in the user's preferred sequence. For example, you could call a convenience function called move so the modifier would read:

.onMove( perform: move )

Your move function will be passed an IndexSet and an Int. The index set contains all the items in the FetchRequestResult that are to be moved (typically that is just one item). The Int indicates the position to which they should be moved. The logic would be:

private func move( from source: IndexSet, to destination: Int) 
{
    // Make an array of items from fetched results
    var revisedItems: [ CoreItem ] = coreItems.map{ $0 }

    // change the order of the items in the array
    revisedItems.move(fromOffsets: source, toOffset: destination )

    // update the userOrder attribute in revisedItems to 
    // persist the new order. This is done in reverse order 
    // to minimize changes to the indices.
    for reverseIndex in stride( from: revisedItems.count - 1,
                                through: 0,
                                by: -1 )
    {
        revisedItems[ reverseIndex ].userOrder =
            Int16( reverseIndex )
    }
}

Technical reminder: the items stored in revisedItems are classes (i.e., by reference), so updating these items will necessarily update the items in the fetched results. The @FetchedResults wrapper will cause your user interface to reflect the new order.

Admittedly, I'm new to SwiftUI. There is likely to be a more elegant solution!

Paul Hudson (Hacking With Swift) has quite a bit more detail. Here is a link for info on moving data in a list. Here is a link for using core data with SwiftUI (it involves deleting items in a list, but is closely analogous to the onMove logic)

Mossy answered 7/6, 2020 at 1:44 Comment(7)
Worth mentioning, that for me in the simulator the moved row occasionally appears to jump back to its original position before ending up at the new location. While on an actual device it's less obvious but still present. It turns out that what is happening by default is the List view is being rebuilt by comparing each of its rows and updating. The fix is to add a .id(UUID()) modifier to the List which triggers replacement instead. HwS has an explanation of this for a similar case over here hackingwithswift.com/articles/210/…Guardian
This was a good solution and works in practice. I modified your answer to be more definitive. Well done and thank you!Aurthur
Hi Thanks for the help I implement this solution but move only gets called onceTrotta
I figured why onMove only called once SwiftUI Xcode 12 iOS 14 it was because the edit button needs to be in the toolbar modifierTrotta
This is a great answer. I love that we get to witness the magic of @FetchRequest where simply reordering the array automatically pushes the changes back to Core Data. 🙂Armorial
"This is done in reverse order to minimize changes to the indices." – How does it minimize changes if it still goes through the whole collection?Reticent
@KyryloZapylaiev I don't think the reverse iteration makes any difference at all. Clearly, it makes the exact same changes to the exact same entities. I use for index in 0..<revisedItems.count in my app. It works fine as is much clearer.Lianeliang
G
7

Below you can find a more generic approach to this problem. The algorithm minimises the number of CoreData entities that require an update, to the contrary of the accepted answer. My solution is inspired by the following article: https://www.appsdissected.com/order-core-data-entities-maximum-speed/

First I declare a protocol as follows to use with your model struct (or class):

protocol Sortable {
    var sortOrder: Int { get set }
}

As an example, assume we have a SortItem model which implements our Sortable protocol, defined as:

struct SortItem: Identifiable, Sortable {
    var id = UUID()
    var title = ""
    var sortOrder = 0
}

We also have a simple SwiftUI View with a related ViewModel defined as (stripped down version):

struct ItemsView: View {
    @ObservedObject private(set) var viewModel: ViewModel
    
    var body: some View {
        NavigationView {
            List {
                ForEach(viewModel.items) { item in
                    Text(item.title)
                }
                .onMove(perform: viewModel.move(from:to:))
            }
        }
        .navigationBarItems(trailing: EditButton())
    }
}

extension ItemsView {
    class ViewModel: ObservableObject {
        @Published var items = [SortItem]()
        
        func move(from source: IndexSet, to destination: Int) {
            items.move(fromOffsets: source, toOffset: destination)

            // Note: Code that updates CoreData goes here, see below
        }
    }
}

Before I continue to the algorithm, I want to note that the destination variable from the move function does not contain the new index when moving items down the list. Assuming that only a single item is moved, retrieving the new index (after the move is complete) can be achieved as follows:

func move(from source: IndexSet, to destination: Int) {
    items.move(fromOffsets: source, toOffset: destination)
    
    if let oldIndex = source.first, oldIndex != destination {
        let newIndex = oldIndex < destination ? destination - 1 : destination
        
        // Note: Code that updates CoreData goes here, see below
    }
}

The algorithm itself is implemented as an extension to Array for the case that the Element is of the Sortable type. It consists of a recursive updateSortOrder function as well as a private helper function enclosingIndices which retrieves the indices that enclose around a certain index of the array, whilst remaining within the array bounds. The complete algorithm is as follows (explained below):

extension Array where Element: Sortable {
    func updateSortOrder(around index: Int, for keyPath: WritableKeyPath<Element, Int> = \.sortOrder, spacing: Int = 32, offset: Int = 1, _ operation: @escaping (Int, Int) -> Void) {
        if let enclosingIndices = enclosingIndices(around: index, offset: offset) {
            if let leftIndex = enclosingIndices.first(where: { $0 != index }),
               let rightIndex = enclosingIndices.last(where: { $0 != index }) {
                let left = self[leftIndex][keyPath: keyPath]
                let right = self[rightIndex][keyPath: keyPath]
                
                if left != right && (right - left) % (offset * 2) == 0 {
                    let spacing = (right - left) / (offset * 2)
                    var sortOrder = left
                    for index in enclosingIndices.indices {
                        if self[index][keyPath: keyPath] != sortOrder {
                            operation(index, sortOrder)
                        }
                        sortOrder += spacing
                    }
                } else {
                    updateSortOrder(around: index, for: keyPath, spacing: spacing, offset: offset + 1, operation)
                }
            }
        } else {
            for index in self.indices {
                let sortOrder = index * spacing
                if self[index][keyPath: keyPath] != sortOrder {
                    operation(index, sortOrder)
                }
            }
        }
    }
    
    private func enclosingIndices(around index: Int, offset: Int) -> Range<Int>? {
        guard self.count - 1 >= offset * 2 else { return nil }
        var leftIndex = index - offset
        var rightIndex = index + offset
        
        while leftIndex < startIndex {
            leftIndex += 1
            rightIndex += 1
        }
        while rightIndex > endIndex - 1 {
            leftIndex -= 1
            rightIndex -= 1
        }
        
        return Range(leftIndex...rightIndex)
    }
}

First, the enclosingIndices function. It returns an optional Range<Int>. The offset argument defines the distance for the enclosing indices left and right of the index argument. The guard ensures that the complete enclosing indices are contained within the array. Further, in case the offset goes beyond the startIndex or endIndex of the array, the enclosing indices will be shifted to the right or left, respectively. Hence, at the boundaries of the array, the index is not necessarily located in the middle of the enclosing indices.

Second, the updateSortOrder function. It requires at least the index around which the update of the sorting order should be started. This is the new index from the move function in the ViewModel. Further, the updateSortOrder expects an @escaping closure providing two integers, which will be explained below. All other arguments are optional. The keyPath is defaulted to \.sortOrder in conformance with the expectations from the protocol. However, it can be specified if the model parameter for sorting differs. The spacing argument defines the sort order spacing that is typically used. The larger this value, the more sort operations can be performed without requiring any other CoreData update except for the moved item. The offset argument should not really be touched and is used in the recursion of the function.

The function first requests the enclosingIndices. In case these are not found, which happens immediately when the array is smaller than three items or either inside one of the recursions of the updateSortOrder function when the offset is such that it would go beyond the boundaries of the array; then the sort order of all items in the array are reset in the else case. In that case, if the sortOrder differs from the items existing value, the @escaping closure is called. It's implementation will be discussed further below.

When the enclosingIndices are found, both the left and right index of the enclosing indices not being the index of the moved item are determined. With these indices known, the existing 'sort order' values for these indices are obtained through the keyPath. It is then verified if these values are not equal (which could occur if the items were added with equal sort orders in the array) as well as if a division of the difference between the sort orders and the number of enclosing indices minus the moved item would result in a non-integer value. This basically checks whether there is a place left for the moved item's potentially new sort order value within the minimum spacing of 1. If this is not the case, the enclosing indices should be expanded to the next higher offset and the algorithm run again, hence the recursive call to updateSortOrder in that case.

When all was successful, the new spacing should be determined for the items between the enclosing indices. Then all enclosing indices are looped through and each item's sorting order is compared to the potentially new sorting order. In case it changed, the @escaping closure is called. For the next item in the loop the sort order value is updated again.

This algorithm results in the minimum amount of callbacks to the @escaping closure. Since this only happens when an item's sort order really needs to be updated.

Finally, as you perhaps guessed, the actual callbacks to CoreData will be handled in the closure. With the algorithm defined, the ViewModel move function is then updated as follows:

func move(from source: IndexSet, to destination: Int) {
    items.move(fromOffsets: source, toOffset: destination)
    
    if let oldIndex = source.first, oldIndex != destination {
        let newIndex = oldIndex < destination ? destination - 1 : destination
        items.updateSortOrder(around: newIndex) { [weak self] (index, sortOrder) in
            guard let self = self else { return }
            var item = self.items[index]
            item.sortOrder = sortOrder
            
            // Note: Callback to interactor / service that updates CoreData goes here
        }
    }
}

Please let me know if you have any questions regarding this approach. I hope you like it.

Georgiannageorgianne answered 30/12, 2020 at 2:2 Comment(0)
S
1

Had a problem with Int16 and solved it by changing it to @NSManaged public var userOrder: NSNumber? and in the func: NSNumber(value: Int16( reverseIndex ))

As well I needed to add try? managedObjectContext.save() in the func to actually save the new order.

Now its working fine - thanks!

Stefansson answered 5/8, 2020 at 10:43 Comment(0)
A
1

I'm not sure using a CoreData NSManagedObject for a view model object is the best approach, but if you do below is a sample for moving items in a SwiftUI List and persisting an object value based sort order.

An UndoManager is used in the event an error occurs during the move to rollback any changes.

class Note: NSManagedObject {

    @nonobjc public class func fetchRequest() -> NSFetchRequest<Note> {
        return NSFetchRequest<Note>(entityName: "Note")
    }

    @NSManaged public var id: UUID?
    @NSManaged public var orderIndex: Int64
    @NSManaged public var text: String?

} 

struct ContentView: View {

    @Environment(\.editMode) var editMode
    @Environment(\.managedObjectContext) var viewContext

    @FetchRequest(sortDescriptors: 
        [NSSortDescriptor(key: "orderIndex", ascending: true)],
        animation: .default)
    private var notes: FetchedResults<Note>

    var body: some View {
        NavigationView {
            List {
                ForEach (notes) { note in
                    Text(note.text ?? "")
                }
            }
            .onMove(perform: moveNotes)
        }
        .navigationTitle("Notes")
        .toolbar {
            ToolbarItem(placement: .navigationBarTrailing) {
                EditButton()
            }
        }
    }

    func moveNotes(_ indexes: IndexSet, _ i: Int) {
        
        guard
            1 == indexes.count,
            let from = indexes.first,
            from != i
        else { return }
        
        var undo = viewContext.undoManager
        var resetUndo = false
        
        if undo == nil {
            viewContext.undoManager = .init()
            undo = viewContext.undoManager
            resetUndo = true
        }
        
        defer {
            if resetUndo {
                viewContext.undoManager = nil
            }
        }
    
        do {

            try viewContext.performAndWait {
                undo?.beginUndoGrouping()
                let moving = notes[from]
                
                if from > i { // moving up
                    notes[i..<from].forEach {
                        $0.orderIndex = $0.orderIndex + 1
                    }
                    moving.orderIndex = Int64(i)
                }
    
                if from < i { // moving down
                    notes[(from+1)..<i].forEach {
                        $0.orderIndex = $0.orderIndex - 1
                    }
                    moving.orderIndex = Int64(i)
                }
                
                undo?.endUndoGrouping()
                try viewContext.save()
                
            }
            
        } catch {
            
            undo?.endUndoGrouping()
            viewContext.undo()
            
            // TODO: something with the error
            // set a state variable to display the error condition
            fatalError(error.localizedDescription)
        }
        
    }
    
}
Absolutism answered 9/10, 2021 at 16:2 Comment(0)
R
1

if do like this

.onMove {
                    self.CoreItems.move(set: $0, to: $1, from: self.managedObjectContext)
try? managedObjectContext.save()
Rima answered 23/12, 2022 at 20:23 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.