NSFetchedResultsController 'didChange' с использованием нового CollectionDiffing вызывает странные операции изменения, которые приводят к сбою - PullRequest
0 голосов
/ 24 сентября 2019

Пример проекта, демонстрирующего сбой: https://github.com/d3mueller/TestProjectFetchedResultsController


У меня есть простая установка CoreData с двумя entities:

  • City, имеетотношение многие * incidentRoads к Road (устанавливается Cascade при удалении города)
  • Road, имеет отношение ко многим cities к City

В качестве примера я добавил два Cities и один Road, который "соединяет" оба города.В моем ViewController я настроил два NSFetchedResultsController, которые отвечают за поддержание двух массивов var cities: [City] и var roads: [Road] в указанном контроллере представления в актуальном состоянии.

Когда я иду вперед и удаляюодин из двух cities, в то время как NSFetchedResultsController наблюдает за всем, приложение зависло из-за Index out of range error при попытке обновить массивы в методе didChange делегата контроллера полученных результатов:

Неустранимая ошибка: индекс выходит за пределы диапазона

Я использую новый API делегата контроллера полученных результатов для отслеживания изменений с помощью CollectionDifference:

func controller(_ controller: NSFetchedResultsController<NSFetchRequestResult>, didChangeContentWith diff: CollectionDifference<NSManagedObjectID>)

И после удаления одного из cities этот метод вызывается несколько раз со странными вставками и удалениями (я не думаю, что было бы полезно размещать их здесь без контекста отладчика. Я добавилпример проекта ниже).

Метод вызывается с diff, который выглядит действительно странно:

remove(offset: 1, element: 0xcb6b486b0bd166fe <x-coredata://E2AA7DB9-7A1F-4130-9AE4-A9D0DC695159/City/p2>, associatedWith: Optional(0))

remove(offset: 0, element: 0xcb6b486b0bdd66fe <x-coredata://E2AA7DB9-7A1F-4130-9AE4-A9D0DC695159/City/p1>, associatedWith: nil)

insert(offset: 0, element: 0xcb6b486b0bd166fe <x-coredata://E2AA7DB9-7A1F-4130-9AE4-A9D0DC695159/City/p2>, associatedWith: Optional(1))

То, на что я ожидал, выглядит diff простоодно удаление первого элемента в арЛуч (первый city, который удаляется).Но это не имеет никакого смысла для меня.Я даже не понимаю, что они на самом деле пытаются мне сказать.Первое удаление связано с самим собой?И последняя вставка связана со вторым удалением.Я не понимаюЯ думаю, что проблема заключается здесь.Любые идеи?

Мой контроллер вида выглядит так (я прокомментировал, где происходит сбой):

class ViewController: UIViewController, NSFetchedResultsControllerDelegate {

    public var cities: [City] = []
    private lazy var citiesResultsController: NSFetchedResultsController<City> = {
        let request: NSFetchRequest<City> = City.fetchRequest()
        request.sortDescriptors = [NSSortDescriptor(key: "creationDate", ascending: true)]

        let controller = NSFetchedResultsController(fetchRequest: request, managedObjectContext: AppDelegate.managedObjectContext, sectionNameKeyPath: nil, cacheName: nil)

        controller.delegate = self
        return controller
    }()

    public var roads: [Road] = []
    private lazy var roadsResultsController: NSFetchedResultsController<Road> = {
        let request: NSFetchRequest<Road> = Road.fetchRequest()
        request.sortDescriptors = [NSSortDescriptor(key: "creationDate", ascending: true)]

        let controller = NSFetchedResultsController(fetchRequest: request, managedObjectContext: AppDelegate.managedObjectContext, sectionNameKeyPath: nil, cacheName: nil)

        controller.delegate = self
        return controller
    }()

    func controller(_ controller: NSFetchedResultsController<NSFetchRequestResult>, didChangeContentWith diff: CollectionDifference<NSManagedObjectID>) {
        if controller === citiesResultsController {
            for change in diff {
                switch change {
                case .insert(offset: let newPosition, element: _, associatedWith: let oldPosition):

                    if let oldPosition = oldPosition {
                        // was moved
// HERE IT CRASHES 
                        let city = cities.remove(at: oldPosition)
                        cities.insert(city, at: newPosition)
                    } else {
                        // was inserted
                        let city = citiesResultsController.object(at: IndexPath(item: newPosition, section: 0))
                        cities.insert(city, at: newPosition)
                    }
                case .remove(offset: let position, element: _, associatedWith: let associatedWith):
                    if associatedWith == nil {
                        _ = cities.remove(at: position)
                    }
                }
            }
        } else {
            for change in diff {
                switch change {
                case .insert(offset: let newPosition, element: _, associatedWith: let oldPosition):
                    if let oldPosition = oldPosition {
                        // was moved
                        let road = roads.remove(at: oldPosition)
                        roads.insert(road, at: newPosition)
                    } else {
                        // was inserted
                        let road = roadsResultsController.object(at: IndexPath(item: newPosition, section: 0))
                        roads.insert(road, at: newPosition)
                    }
                case .remove(offset: let position, element: _, associatedWith: let associatedWith):
                    if associatedWith == nil {
                        _ = roads.remove(at: position)
                    }
                }
            }
        }
    }

    override func viewDidLoad() {
        super.viewDidLoad()

        try? citiesResultsController.performFetch()
        cities = citiesResultsController.fetchedObjects ?? []

        try? roadsResultsController.performFetch()
        roads = roadsResultsController.fetchedObjects ?? []

        DispatchQueue.main.asyncAfter(deadline: .now() + .seconds(5)) {
            print("Now")
            AppDelegate.managedObjectContext.delete(AppDelegate.cityA)
            try! AppDelegate.managedObjectContext.save()
        }
    }
}

И мой appDelegate (где я настраивал объекты и вещи для проверки всего):

//
//  AppDelegate.swift
//  TestProject
//
//  Created by Dennis Müller on 23.09.19.
//  Copyright © 2019 Dennis Müller. All rights reserved.
//

import UIKit
import CoreData

@UIApplicationMain
class AppDelegate: UIResponder, UIApplicationDelegate {

    public static var managedObjectContext: NSManagedObjectContext {
        return (UIApplication.shared.delegate as! AppDelegate).persistentContainer.viewContext
    }

    public static var cityA: City!

    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
        // Override point for customization after application launch.

        if !UserDefaults.standard.bool(forKey: "setup") {
            UserDefaults.standard.set(true, forKey: "setup")

            let cityA = City(context: persistentContainer.viewContext)
            cityA.name = "cityA"
            cityA.creationDate = Date()

            let cityB = City(context: persistentContainer.viewContext)
            cityB.name = "cityB"
            cityB.creationDate = Date()

            let road = Road(context: persistentContainer.viewContext)
            road.creationDate = Date()
            road.addToCities(cityA)
            road.addToCities(cityB)

            saveContext()

        }

        let request: NSFetchRequest<City> = City.fetchRequest()
        request.predicate = NSPredicate(format: "name == %@", "cityA")
        AppDelegate.cityA = try! persistentContainer.viewContext.fetch(request).first!

        return true
    }

    // MARK: UISceneSession Lifecycle

    func application(_ application: UIApplication, configurationForConnecting connectingSceneSession: UISceneSession, options: UIScene.ConnectionOptions) -> UISceneConfiguration {
        // Called when a new scene session is being created.
        // Use this method to select a configuration to create the new scene with.
        return UISceneConfiguration(name: "Default Configuration", sessionRole: connectingSceneSession.role)
    }

    func application(_ application: UIApplication, didDiscardSceneSessions sceneSessions: Set<UISceneSession>) {
        // Called when the user discards a scene session.
        // If any sessions were discarded while the application was not running, this will be called shortly after application:didFinishLaunchingWithOptions.
        // Use this method to release any resources that were specific to the discarded scenes, as they will not return.
    }

    // MARK: - Core Data stack

    lazy var persistentContainer: NSPersistentContainer = {
        /*
         The persistent container for the application. This implementation
         creates and returns a container, having loaded the store for the
         application to it. This property is optional since there are legitimate
         error conditions that could cause the creation of the store to fail.
        */
        let container = NSPersistentContainer(name: "TestProject")
        container.loadPersistentStores(completionHandler: { (storeDescription, error) in
            if let error = error as NSError? {
                // Replace this implementation with code to handle the error appropriately.
                // fatalError() causes the application to generate a crash log and terminate. You should not use this function in a shipping application, although it may be useful during development.

                /*
                 Typical reasons for an error here include:
                 * The parent directory does not exist, cannot be created, or disallows writing.
                 * The persistent store is not accessible, due to permissions or data protection when the device is locked.
                 * The device is out of space.
                 * The store could not be migrated to the current model version.
                 Check the error message to determine what the actual problem was.
                 */
                fatalError("Unresolved error \(error), \(error.userInfo)")
            }
        })
        return container
    }()

    // MARK: - Core Data Saving support

    func saveContext () {
        let context = persistentContainer.viewContext
        if context.hasChanges {
            do {
                try context.save()
            } catch {
                // Replace this implementation with code to handle the error appropriately.
                // fatalError() causes the application to generate a crash log and terminate. You should not use this function in a shipping application, although it may be useful during development.
                let nserror = error as NSError
                fatalError("Unresolved error \(nserror), \(nserror.userInfo)")
            }
        }
    }

}

Извините, если это немного сбивает с толку, я не знаю, как правильно объяснить проблему (я даже не знаю, в чем проблема).Дайте мне знать, если вам нужна дополнительная информация.

Большое спасибо за вашу помощь.

1 Ответ

0 голосов
/ 24 сентября 2019

Хорошо, так получается, что я просто глупый.Перебирая изменения в diff, я ошибочно полагал, что associatedWith в случае .insert - это старый индекс элемента, который должен двигаться:

...
            case .insert(offset: let newPosition, element: _, associatedWith: let oldPosition):

                    // I called the associatedWith 'oldPosition'
                    if let oldPosition = oldPosition {
                        // was moved
// HERE IT CRASHES 
                        let city = cities.remove(at: oldPosition)
                        cities.insert(city, at: newPosition)
                    } else {
                        // was inserted
                        let city = citiesResultsController.object(at: IndexPath(item: newPosition, section: 0))
                        cities.insert(city, at: newPosition)
                    }
            ...

Это вызываетсбой, потому что associatedWith на самом деле является индексом изменения .remove, связанного с ним, чтобы указать, что это на самом деле движение вместо вставки.

Поэтому я заменил это:

case .insert(offset: let newPosition, element: _, associatedWith: let oldPosition):
...
   let city = cities.remove(at: oldPosition)
...

с этим:

case .insert(offset: let newPosition, element: let objectID, associatedWith: let associatedWith):
...
    let city = cities.first(where: {$0.objectID == objectID})!
....

Это не оптимально, оно добавляет линейную временную сложность методу, который мне не нравится.Но я не могу найти способ получить фактическую старую позицию элемента.Из-за удаления объекта city операция перемещения усложняется.Я не понимаю, почему происходит движение в первую очередь.Все, что нужно сделать, это удалить один город и все.Но я собираюсь открыть еще один вопрос для этого, поскольку это не по теме.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...