变量是否应该被释放？ iphone-sdk

Question

我有一本书的以下代码。

有一个函数 loadPrefs，其中 NSString *userTimeZone 在函数结束之前被释放。

为什么？该字符串不是用 alloc 创建的，我假设 stringForKey 函数返回一个自动释放的 NSString。这是一个错误还是我错过了什么？难道是书上的错误？ （我刚接触 Objective-C）

在 stringForKey 的文档中，它唯一提到的是：

特别注意事项

返回的 字符串是不可变的，即使值 你最初设置的是一个可变的 字符串。

代码：

- (void) loadPrefs {
    timeZoneName = DefaultTimeZonePref;

    NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
    NSString *userTimeZone = [defaults stringForKey: TimeZonePrefKey];
    if (userTimeZone != NULL)
        timeZoneName = userTimeZone;
    [userTimeZone release];
    show24Hour = [defaults boolForKey:TwentyFourHourPrefKey];
}

谢谢！！！！

Answer 1

你是对的。这段代码有两个问题：它不正确地从stringForKey: 释放字符串，并且它不在将值分配给实例变量时保留userTimeZone。

这是一个更好的尝试：

- (void) loadPrefs {
    [timeZoneName release]; // In case there was a previous value

    timeZoneName = DefaultTimeZonePref; // ASSUMPTION: DefaultTimeZonePref is a constant
    // And thus doesn't need retain/release.

    NSUserDefaults *defaults = [NSUserDefaults standardUserDefaults];
    NSString *userTimeZone = [defaults stringForKey: TimeZonePrefKey];
    if (userTimeZone != NULL) {
        timeZoneName = [userTimeZone retain];
    }

    show24Hour = [defaults boolForKey:TwentyFourHourPrefKey];
}

别忘了在dealloc 中释放timeZoneName。