Подтвердить что ты не робот

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

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

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 (чтобы отразить комментарии к альбому)

.

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;

Гэри

4b9b3361

Ответ 1

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

Это неверно. Вы не должны предполагать ничего о возвращаемом ссылочном значении fileError, пока не проверите, было ли fileContents нулевым. Никогда не. Установка fileError на nil до вызова метода 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];