不确定如何修复此EXC_BAD_ACCESS错误

Question

所以我有一个Task实体类（Core Data），我试图覆盖其中一个字符串的setter（timeIntervalString），因此它可以显示在表视图单元格的详细文本标签中。出于某种原因，我得到一个像这样的EXC_BAD_ACCESS错误：

[Tasks timeIntervalString]一直持续到37355 ......

这是我的代码：

-(NSString *)timeIntervalString{

    NSUInteger seconds = (NSUInteger)round(self.timeInterval);
if ((seconds/3600) == 0){
    if (((seconds/60) % 60) == 1) {
        self.timeIntervalString = [NSString stringWithFormat:@"%u MIN", ((seconds/60) % 60)];
    } else {
        self.timeIntervalString = [NSString stringWithFormat:@"%u MINS", ((seconds/60) % 60)];
    }
} else if ([self.conversionInfo hour] == 1) {
    if (((seconds/60) % 60) == 0){
        self.timeIntervalString = [NSString stringWithFormat:@"%u HR", (seconds/3600)];
    } else if (((seconds/60) % 60) == 1) {
        self.timeIntervalString = [NSString stringWithFormat:@"%u HR %u MIN", (seconds/3600), ((seconds/60) % 60)];
    } else {
        self.timeIntervalString = [NSString stringWithFormat:@"%u HR %u MINS", (seconds/3600), ((seconds/60) % 60)];
    }
} else {
    if (((seconds/60) % 60) == 0) {
        self.timeIntervalString = [NSString stringWithFormat:@"%u HRS ", (seconds/3600)];
    } else if (((seconds/60) % 60) == 1){
        self.timeIntervalString = [NSString stringWithFormat:@"%u HRS %u MIN", (seconds/3600), ((seconds/60) % 60)];
    } else {
        self.timeIntervalString = [NSString stringWithFormat:@"%u HRS %u MINS", (seconds/3600), ((seconds/60) % 60)];
    }
}
return self.timeIntervalString;

}

有什么想法吗？

Answer 1

return self.timeIntervalString将递归调用相同的timeIntervalString方法。

您可能想要的是return _timeIntervalString。

说明：self.timeIntervalString是属性访问器的合成糖，它与[self timeIntervalString]相同，它将调用您在此处定义的-(NSString *)timeIntervalString方法。 return _timeIntervalString更改将使您直接访问实例变量，而不是递归调用属性访问器。这是您在编写的任何自定义属性访问器方法中应遵循的一般模式。

编辑：根据评论中的讨论，将此标记为只读属性并且从不实际设置值会更好：

在你的.h文件中：

@property (readonly) NSString *timeIntervalString;

在您的.m文件中：

-(NSString *)timeIntervalString {
    NSString *value;
    // insert here the body of your timeIntervalString method, as you
    // originally wrote it, but replace all occurences of:
    // self.timeIntervalString = ...
    // with this: value = ...
    return value;
}