标签未在设置器中更新

Question

好吧，这个问题有点奇怪，因为我在应该打印文本的代码行前面的 NSLog 返回了正确的值。

代码如下：

-(void)setCurrentDate:(UILabel *)currentDate
{

NSInteger onDay = 1; //because if it's today, you are on day one, not zero... no such thing as a day zero

//get the nubmer of days left
if( [[NSUserDefaults standardUserDefaults] objectForKey:@"StartDate"] ){ //if there is something at the userdefaults
    onDay = [self daysToDate:[NSDate date]];
}//otherwise, onDay will just be one

self.theCurrentNumberOfDaysSinceStart = onDay;

NSLog(@"On day: %d", onDay); //this is returning the correct values....

//print it out on the label
[currentDate setText:[NSString stringWithFormat:@"On day: %d", onDay]];//echoes out the current day number 

}

因此，当应用程序首次启动时，一切都很好。标签更新等等。当我按下一个基本上获取新日期的按钮时，问题就出现了。在此过程中，它运行以下内容：

    //need to reload the "on day" label now
    [self setCurrentDate:self.currentDate];
    //and the "days left" label
    [self setDaysLeft:self.daysLeft];

再次，我认为这应该都是正确的，因为 NSLog 返回了正确的内容。我认为问题出在我展示的第一段代码中的最后一行......带有 setText 的行。

感谢您的帮助！

干杯， 马特

Answer 1

如果您使用笔尖

当笔尖加载并建立所有连接时...（来自 资源编程指南)

查找set形式的方法OutletName：，如果存在这样的方法则调用它

因此笔尖将加载并调用 setCurrentDate: 传递在未归档的 UILabel 中作为参数

在您的方法中，您使用传递到方法中的本地引用来配置 UILabel

[currentDate setText:[NSString stringWithFormat:@"On day: %d", onDay]];

您实际上不会存储参考这个ivar 中的 UILabel ，因此从技术上讲，您已经泄漏了标签，并且由于您没有设置 ivar currentDate ，它将被初始化为 nil 。这是用不正确的实现覆盖 setter 的危险。

在您的方法中的某个时刻，您应该将 ivar 设置为传入的变量。普通的设置器看起来像这样

- (void)setCurrentDate:(UILabel *)currentDate;
{
    if (_currentDate != currentDate) {
        [_currentDate release];
        _currentDate = [currentDate retain];
    }
}

但是

在你的例子中我根本不担心这个我会把它改为

//need to reload the "on day" label now
[self setCurrentDate:self.currentDate];

类似的

[self updateCurrentDate];

实现看起来像这样：

- (void)updateCurrentDate;
{
    NSInteger onDay = 1;

    if ([[NSUserDefaults standardUserDefaults] objectForKey:@"StartDate"]) {
        onDay = [self daysToDate:[NSDate date]];
    }

    self.theCurrentNumberOfDaysSinceStart = onDay;

    [self.currentDate setText:[NSString stringWithFormat:@"On day: %d", onDay]];
}