在 dealloc 中释放 NSMutableArray 会导致应用崩溃

Question

我有一个UIView，其中包含一个UITableView。此表由 NSMutableArray 填充。

我在 UIView 的 initWithFrame: 方法中初始化了数组和表格。

- (id)initWithFrame:(CGRect)frame
{
  self = [super initWithFrame:frame];
  if (self)
  {
    bookTable = [[UITableView alloc]initWithFrame:CGRectMake(100, 108, 260, 535)];
    bookArray = [[NSMutableArray alloc]init]; //this array will be populated later

    //other codes
  }
  return self;
}

我在dealloc发布它们：

- (void)dealloc
{
   [bookTable release];      
   [bookArray release];

   [super dealloc];
}

问题是当应用程序在dealloc 中崩溃时。它产生错误：[CALayer release]: message sent to deallocated instance 0x82db660

我在我的代码中到处查看，我确信我没有过度发布任何东西。我决定删除dealloc 中的[bookArray release]，它就停止了崩溃。我运行了Analyze，它没有给我任何潜在的泄漏。

谁能解释一下为什么在 dealloc 中释放数组会导致崩溃？

注意： 表和数组都是视图的实例变量。

编辑： 填充数组的代码：

NSString *sql = [NSString stringWithFormat:@"SELECT BookName FROM books WHERE UserID=%d", userId];
booksArray = [sqlite executeQuery:sql];

请注意，我使用的是 sql 包装器。

Answer 1

我建议像这样为 booksArray 创建一个属性：

@property (nonatomic, retain) NSMutableArray *bookArray;

当您填充它时，请尝试以下代码：

self.bookArray = [NSMutableArray arrayWithArray:[sqlite executeQuery:sql]];

这将创建返回数组的自动释放副本，然后由属性“自动”保留。

当然你应该确保 [sqlite executeQuery:sql] 返回的数组是自动释放的。

Answer 2

1. 更改executeQuery 方法的返回语句，如[array autorelease]
2. 将booksArray = [sqlite executeQuery:sql]; 替换为booksArray = [[sqlite executeQuery:sql] retain];

如果您为booksArray 写了property，例如@property(nonatomic, retain) NSMutableArray *bookArray;，那么您可以将步骤2替换为self.bookArray = [sqlite executeQuery:sql];

您的问题是您正在从 executeQuery 方法返回一个已发布的对象（您在评论中提到了它）。这就是您收到错误的原因

Answer 3

我认为问题只是在这里。

booksArray = [sqlite executeQuery:sql];

executeQuery 方法可能返回一个自动释放的 NSMutableArray 或者类似的东西可能写在 executeQuery 中

NSMutableArray* tempArray = [NSMutableArray arrayWithCapacity:0];

如果是这样，它应该已经处于自动释放模式，因此dealloc方法中没有release。

更新

另外，为数组分配 0sign 属性。

@property (nonatomic, retain) NSMutableArray* booksArray;

然后，无论您在哪里使用它，都将其用作self.booksArray

self.booksArray = [sqlite executeQuery:sql];

这样就可以进行适当的内存管理，然后在dealloc中释放数组。

Answer 4

问题来了

booksArray = [sqlite executeQuery:sql];

sqlite 返回的数组是一个自动释放的对象。您必须保留它或从 `dealloc' 中删除释放。

booksArray = [[sqlite executeQuery:sql] retain];

由于它是自动释放的对象，当系统也尝试release它时，在dealloc中释放它会导致崩溃。

并删除init方法中的数组初始化，这是多余的。