返回对象使 iPhone 崩溃

Question

我对 iPhone 开发者的整个世界还很陌生，所以如果这很简单，请原谅我。我有一个对象Card，上面有6 个Question 对象。当我说[card getQuestion:@"Art"] 时，我目前正在返回一个Question 对象，就像这样

- (Question*) getQuestion: (NSString*) questionType {
    Question *q = [questions objectForKey:questionType];
    return [q autorelease];
}

Question 的属性为text（NSString 类型），它允许我查看问题的文本是什么。所以我想用这个文本来更新viewController中的一个UILabel

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
    NSLog(@"%@", [[self.card getQuestion:@"Art"] qText]);
    self.myQuestion.text = [[self.card getQuestion:@"Art"] qText];
}

这会使 iPhone 崩溃，而如果我将对象 Card 中的函数更改为此

- (NSString*) getQuestion: (NSString*) questionType {
    return [[questions objectForKey:questionType] qText];
}

我在 viewController 中调用

- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];
    NSLog(@"%@", [self.card getQuestion:@"Art"]);
    self.myQuestion.text = [self.card getQuestion:@"Art"];
}

这很好用.. 任何人都可以解释我做错了什么，在这两种情况下，对 NSLog 的调用都会返回相关文本。在这两种情况下，显示器都会加载，但在第一种情况下它会在不久之后崩溃，而在另一种情况下它会保持稳定。

任何帮助表示赞赏。

Answer 1

- (Question*) getQuestion: (NSString*) questionType {
    Question *q = [questions objectForKey:questionType];
    return [q autorelease];
}

根据内存管理规则，您不应该自动释放该对象，这几乎肯定是导致崩溃的原因。您正在放弃您不拥有的对象的所有权，这导致它过早地被解除分配。这意味着当另一个假定拥有该问题的对象尝试访问它时，该问题消失了，您会因EXC_BAD_ACCESS 错误而崩溃。

此外，该方法可能应该称为questionForType: 或questionOfType:。使用get 前缀意味着该对象将通过一个输出参数返回，但事实并非如此。

Answer 2

由于您是新手 - 您绝对应该确保运行您的应用程序以寻找僵尸，并使用仪器应用程序运行它，以确保您不会在任何地方泄漏。

你应该每隔几天阅读 Objective - C 中关于内存管理的非常简短的解释 - 有一天它会完全融入并成为第二天性。

http://macdevelopertips.com/objective-c/objective-c-memory-management.html

Answer 3

我可以看到两个错误，

1：您正在发布您不拥有的东西（问题“q”） - 如果您执行分配、复制或保留，您应该释放它。如果没有，你不应该。

2：setter-line 应该这样写：

Question *q = (Question *)[questions objectForKey:questionType];

这样可以避免收到警告或错误。