Manejando NSError al leer desde el archivo?

Question

Tengo curiosidad si estoy haciendo esto bien.

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 (para reflejar los comentarios 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) 
.... 

Eso es incorrecto. Usted no debe asumir nada sobre el valor de retorno por referencia de fileError hasta que compruebe si fileContents era nulo. Jamas. Establecer fileError a cero antes de llamar al método pass-error-by-reference no sirve de nada.

Es decir, el código debe leer (fijo ahora que ya no estoy corriendo de un plano a otro y saltar sobre WiFi en las conexiones entre ...):

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];