Gestion de NSError lors de la lecture d'un fichier?

Question

Je suis juste curieux de savoir si je fais cela correctement.

NSString *fileContents;  
NSError *fileError = nil; 

fileContents = [[NSString stringWithContentsOfFile:fileOnDisk 
          encoding:NSMacOSRomanStringEncoding 
          error:&fileError] retain]; 

if(fileError != nil) { 
    NSLog(@"Error : %@", [fileError localizedDescription]); 
} 

// Other Code ... 
[fileContents release]; 

.

EDIT (pour refléter les commentaires de bbums)

.

NSString *fileOnDisk = @"/Users/Gary/Documents/Xcode/RnD/Maya.MEL"; 
NSError *fileError; // Should this be *fileError = nil; 
NSString *fileContents; 
int  status = 0; 

fileContents = [[NSString stringWithContentsOfFile:fileOnDisk 
          encoding:NSMacOSRomanStringEncoding 
          error:&fileError] retain]; 

if(fileContents == nil) { 
    NSLog(@"FileError: %@", [fileError localizedDescription]); 
    status = 1; 
} else { 
    NSLog(@"Success : %@", fileContents); 
} 

// Clean up 
[fileContents release]; 
[pool drain]; 
return status; 

gary

Answer 1

NSError *fileError = nil; 
.... 
if(fileError != nil) 
.... 

qui est incorrect. Vous ne doit pas supposer quoi que ce soit à propos de la valeur return-by-reference de fileError jusqu'à ce que vous vérifiiez si fileContents était nul. Jamais. Mettre fileError à zéro avant d'appeler la méthode pass-error-by-reference ne sert à rien.

C'est, votre code devrait READ (fixe maintenant que je ne suis plus en cours d'exécution d'un plan à sauter sur et connexion Wi-Fi entre les connexions ...):

NSString *fileContents;  
NSError *fileError; 

fileContents = [[NSString stringWithContentsOfFile:fileOnDisk 
          encoding:NSMacOSRomanStringEncoding 
          error:&fileError] retain]; 

if(fileContents == nil) { 
    NSLog(@"Error : %@", [fileError localizedDescription]); 
    // ... i.e. handle the error here more 
    return ...; // often returning after handling the errors, sometimes you might continue 
} 

// Other Code ... 
[fileContents release];