Обработка NSError при чтении из файла?

Question

Мне просто любопытно, правильно ли я это делаю.

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

.

РЕДАКТИРОВАТЬ (чтобы отразить комментарии 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;

1011 * Гэри *

Answer 1

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

Это неверно. Вы не должны предполагать что-либо о значении возврата по ссылке fileError, пока не проверите, имеет ли fileContents nil . Никогда не. Установка fileError на ноль перед вызовом метода pass-error-by-reference ничего не даст.

То есть ваш код должен читаться (исправлено теперь, когда я больше не бегаю из самолета в самолет и переключаюсь по WiFi между соединениями ...):

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