Это лучший способ загрузить существующие, иначе создайте новые NSManagedObjects?

Question

Мне нужно загружать много данных через API и использовать это для создания новых или обновления существующих объектов и их отношений по мере продвижения. Правильно ли это? Кажется, это так долго, я чувствую, что здесь что-то не хватает.

// Check for existing Object 
NSFetchRequest *fetchRequest = [[NSFetchRequest alloc] init]; 
[fetchRequest setEntity:[NSEntityDescription entityForName:@"Object" inManagedObjectContext:managedObjectContext]]; 
[fetchRequest setPredicate:[NSPredicate predicateWithFormat:@"name == %@", objectName]]; 
[fetchRequest setFetchLimit:1]; 

NSError *error = nil; 
NSArray *results = [managedObjectContext executeFetchRequest:fetchRequest error:&error]; 
if (results == nil) { 
    NSAssert(NO, @"Error executing fetch: %@\n%@", [error localizedDescription], [error userInfo]); 
} 

Object *object = [results lastObject]; 

if (object == nil) { 
    // Create new object 
    object = [NSEntityDescription insertNewObjectForEntityForName:@"Object" inManagedObjectContext:managedObjectContext]; 
    object.name = objectName; 
} 


// Check for existing Other 
fetchRequest = [[NSFetchRequest alloc] init]; 
[fetchRequest setEntity:[NSEntityDescription entityForName:@"Other" inManagedObjectContext:managedObjectContext]]; 
[fetchRequest setPredicate:[NSPredicate predicateWithFormat:@"name == %@", otherName]]; 
[fetchRequest setFetchLimit:1]; 

results = [managedObjectContext executeFetchRequest:fetchRequest error:&error]; 
if (results == nil) { 
    NSAssert(NO, @"Error executing fetch: %@\n%@", [error localizedDescription], [error userInfo]); 
} 

Other *other = [results lastObject]; 

if (other == nil) { 
    // Create new Other 
    other = [NSEntityDescription insertNewObjectForEntityForName:@"Other" inManagedObjectContext:managedObjectContext]; 
    other.name = otherName; 
} 


// Finally 
[object addOtherObject:other]; // A many-to-many relationship 

if (![managedObjectContext save:&error]) { 
    NSAssert(NO, @"Error saving context: %@\n%@", [error localizedDescription], [error userInfo]); 
} 

Все это, конечно, внутри петли, оно кажется ужасно неэффективным.

Answer 1

Существует множество способов свести к минимуму шаблон. Я создал подкласс NSManagedObject, который я использую в качестве базы для всех объектов модели. Он содержит такие методы, как в следующем:

+ (NSString *)entityName; 

+ (instancetype)newEntityInContext:(ManagedObjectContext *)context; 
+ (instancetype)newEntityInContext:(ManagedObjectContext *)context properties:(NSDictionary *)properties; 

+ (NSArray *)entitiesWithPredicate:(NSPredicate *)predicate inContext:(RoundsManagedObjectContext *)context; 
+ (NSArray *)entitiesWithPredicate:(NSPredicate *)predicate 
         sortedBy:(NSArray *)sortDescriptors 
        inContext:(ManagedObjectContext *)context; 

+ (NSArray *)entitiesWithProperties:(NSDictionary *)properties inContext:(ManagedObjectContext *)context; 

+ (NSInteger)countOfEntitiesWithPredicate:(NSPredicate *)predicate inContext:(ManagedObjectContext *)context; 

Я оставляю вам, чтобы увидеть, как они заботятся о шаблонном создании запросов и т.п. выборки. Это действительно делает ваш код, который на самом деле использует данные ядра, чтобы быть более читаемыми.

Теперь, чтобы говорить об эффективности ведения тех неправдоподобных в цикле:

не делать. То есть, не делайте отдельные выборки. Вместо этого, соберите objectName s в массив и выберете сразу все связанные объекты. Предполагая, что вы реализуете методы, аналогичные тем, что я написал выше, вы можете иметь псевдокод, который выглядит примерно так:

NSArray *objectNames = ...; 
NSArray *otherNames = ...; 

NSManagedObjectContext *context = ...; 

NSArray *existingObjects = [Object entitiesWithPredicate:[NSPredicate predicateWithFormat:@"name IN %@", objectNames]; 
NSArray *existingOthers = [Other entitiesWithPredicate:[NSPredicate predicateWithFormat:@"name IN %@", otherNames]; 

NSArray *missingObjectNames = [objectNames removeObjects:[existingObjects valueForKeyPath:@"name"]]; 

for (NSString *name in missingObjectNames) { 
    Object *object = [Object newEntityInContext:context]; 
    object.name = name; 

    // Do stuff with Other objects here. 
}