Obj-C, Objekt mit Referenzzählung wird verwendet, nachdem es freigegeben wurde?

Question

Ich erhalte die folgende Warnung.

Referenzzählung Objekt verwendet wird, nachdem es

ARC freigegeben wird, wird für diese Datei aktiviert.

Ich habe meine App nicht zur Verwendung von ARC konvertiert und habe derzeit kein richtiges Verständnis davon. Aber auf einen Blick bin ich verwirrt, warum dieser Code ein Problem haben sollte (aus einer Bibliothek), ich meine, ich weiß, Autorelease sollte hier nicht verwendet werden?

NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults]; 
    id uuid = [defaults objectForKey:@"uniqueID"]; 
    if (uuid) 
     uniqueID = (NSString *)uuid; 
    else { 
     CFStringRef cfUuid = CFUUIDCreateString(NULL, CFUUIDCreate(NULL)); 
     uniqueID = (__bridge NSString *)cfUuid; 
     CFRelease(cfUuid); 
     [defaults setObject:uniqueID forKey:@"uniqueID"]; // warning here 
    } 

EDIT @JohnCalsbeek, @epatel, @Chuck Danke, habe ich versucht, nur, dass ich eine neue Warnung auf CFStringRef cfUuid Linie bekommen, potenziellen Leck sagen.

Heres die vollständige Datei https://github.com/MugunthKumar/MKStoreKit/blob/master/MKSKProduct.m

Answer 1

Versuchen Sie dieses, srry für die Miss Verständnis haha ​​

NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults]; 
id uuid = [defaults objectForKey:@"uniqueID"]; 

if (uuid) 
    uniqueID = [NSString stringWithString:(NSString *)uuid]; 
else { 
    CFUUIDRef temp = CFUUIDCreate(NULL); 
    CFStringRef cfUuid = CFUUIDCreateString(NULL, temp); 
    uniqueID = [NSString stringWithString:(__bridge NSString*) cfUuid]; 
    CFRelease(cfUuid); 
    CFRelease(temp); 

    [defaults setObject:uniqueID forKey:@"uniqueID"]; // warning here 
} 

Answer 2

ich glaube, dass diese Linie sein kann, das nicht das, was Sie

uniqueID = (__bridge NSString *)cfUuid; 

erwarten tut Es ist eine einfache Besetzung und die nächste Zeile wird Release tatsächlich sowohl cfUuid und uniqueID (das ist eigentlich das gleiche Objekt).

Also würde ich die Reihenfolge der folgenden Zeilen ändern

[defaults setObject:uniqueID forKey:@"uniqueID"]; // warning here 
CFRelease(cfUuid); 

Answer 3

Sie mit CFUUIDCreateString(NULL, CFUUIDCreate(NULL) einen String erstellen. Dann weisen Sie uniqueID zu, um auch auf diesen String zu verweisen. Dann geben Sie die Zeichenfolge mit CFRelease(cfUuid) frei. Dann verwenden Sie die Zeichenfolge (wie uniqueID) in der nächsten Zeile. Das ist ungültig, weil Sie es gerade veröffentlicht haben.

Answer 4

Eine __bridge Umwandlung ist keine Eigentumsübertragung. uniqueID wird keine Eigentümerreferenz und wird daher an der Zeile mit der Warnung ungültig. Sie können entweder __bridge_transfer (das die Eigentumsrechte an die Objective-C-Referenz überträgt) verwenden und Ihren CFRelease-Anruf entfernen oder den Anruf zu CFRelease ändern, um nach dem Anruf zu -setObject:forKey: zu sein.