Gibt es eine speichereffizientere Möglichkeit, eine Core Data-Datenbank zu durchsuchen?

Question

Ich muß sehen, ob ein Objekt, das ich aus einer CSV-Datei mit einer eindeutigen Kennung in meiner Core Data-Datenbank vorhanden ist, erhalten habe, und dies ist der Code, den ich für diese Aufgabe geeignet erachtet:

NSFetchRequest *fetchRequest = [[NSFetchRequest alloc] init]; 
NSEntityDescription *entity; 
entity = 
[NSEntityDescription entityForName:@"ICD9" 
      inManagedObjectContext:passedContext]; 
[fetchRequest setEntity:entity]; 
NSPredicate *pred = [NSPredicate predicateWithFormat:@"uniqueID like %@", uniqueIdentifier]; 
[fetchRequest setPredicate:pred]; 
NSError *err; 
NSArray* icd9s = [passedContext executeFetchRequest:fetchRequest error:&err]; 
[fetchRequest release]; 
if ([icd9s count] > 0) { 
    for (int i = 0; i < [icd9s count]; i++) { 
     NSAutoreleasePool *pool = [[NSAutoreleasePool alloc]init]; 
     NSString *name = [[icd9s objectAtIndex:i] valueForKey:@"uniqueID"]; 
     if ([name caseInsensitiveCompare:uniqueIdentifier] == NSOrderedSame && name != nil) 
     { 
      [pool release]; 
      return [icd9s objectAtIndex:i]; 
     } 
     [pool release]; 
    } 
} 
return nil; 

Nach gründlichem Testen scheint es, dass dieser Code für eine große Menge an Lecks in der App verantwortlich ist, die ich schreibe (es stürzt auf einem 3GS, bevor es 20 Prozent durch die 1459 Elemente macht). Ich denke, das ist nicht der effizienteste Weg, dies zu tun, irgendwelche Vorschläge für einen mehr speichereffizienten Weg? Danke im Voraus!

Answer 1

Verwenden Sie die Leaks-Schablone in Instruments, um das oder die Lecks zu suchen. Ihr aktueller Code ist möglicherweise in Ordnung, sobald Sie sie behoben haben. Das Leck (die Lecks) können sogar irgendwo anders als Code sein.

Andere Probleme:

• fast enumeration Verwendung wird die Schleife über das Array (1) schneller und (2) viel leichter lesbar machen.
• Senden Sie release nicht an einen Autorelease-Pool. Wenn Sie den Code jemals in auf Garbage Collection gesammelten Cocoa portieren, wird der Pool nichts tun. Senden Sie es stattdessen drain; in Retain-Release Cocoa und in Cocoa Touch funktioniert dies genauso wie release, und in Müll gesammelt Kakao, stößt es den Müllsammler, der das nächstgelegene Äquivalent in GC-Land ist, um den Pool zu entwässern.
• Don't repeat yourself. Sie haben derzeit zwei [pool release]; Zeilen für einen Pool, was jeden erfahrenen Cocoa und Cocoa Touch Programmierer wirklich beunruhigt. Speichern Sie das Ergebnis Ihrer Tests auf den Namen in einer Booleschen Variablen, leeren Sie dann den Pool vor der Bedingung und geben Sie das Objekt dann bedingt zurück.
• Seien Sie vorsichtig mit Variablentypen. -[NSArray count] zurückkehrt und -[NSArray objectAtIndex:] dauert ein NSUInteger, kein int. Versuchen Sie, alle Ihre Typen immer passend zu halten. (Die Umschaltung auf schnelle Aufzählung löst diese Instanz dieses Problems natürlich auf andere Weise.)
• Verstecken Sie keine Freigaben. Ich habe dich beinahe beschuldigt, die Abrufanforderung zu verraten, und dann bemerkt, dass du sie in der Mitte des Codes vergraben hast. Machen Sie Ihre Releases prominent, so dass es weniger wahrscheinlich ist, dass Sie versehentlich redundante (d. H. Absturz verursachende) hinzufügen.

Answer 2

• Verwenden Sie den like Operator in Ihrer Anfrage Prädikat nicht. Verwenden Sie =. Das sollte viel schneller sein.
• Sie können die Groß-/Kleinschreibung der Suche über das Vergleichselement mit dem Modifikator [c] angeben.
• Es ist nicht notwendig, eine NSAutoreleasePool bei jeder Iteration Ihrer Schleife zu erstellen und zu zerstören. In der Tat wird es wahrscheinlich überhaupt nicht benötigt.
• Sie müssen keine Überprüfung innerhalb der for() Schleife durchführen. Du verdoppelst die Arbeit deines Prädikats.

So würde ich den Code ändern zu sein:

NSFetchRequest *fetchRequest = [[NSFetchRequest alloc] init]; 
[fetchRequest setEntity:...]; 
[fetchRequest setPredicate:[NSPredicate predicateWithFormat:@"uniqueID =[c] %@", uniqueIdentifier]]; 
NSError *err = nil; 
NSArray *icd9s = [passedContext executeFetchRequest:fetchRequest error:&err]; 
[fetchRequest release]; 
if (error == nil && [icd9s count] > 0) { 
    return [icd9s objectAtIndex:0]; //we know the uniqueID matches, because of the predicate 
} 
return nil;