NSArray运行时数组

Question

我有我有两种方法在不同的类。一个是类方法，另一个是实例方法。我从实例方法调用类方法。当实例方法完成时，会给运行时错误“EXC_BAD_ACCESS”。

#import "xmlObject.h" 
#import "textmeAppDelegate.h" 
@implementation Class1 
    - (void)method1 { 
       textmeAppDelegate *del = (textmeAppDelegate *)[[UIApplication sharedApplication] delegate]; 

       NSArray *bgColor = [[NSArray alloc] initWithArray:[xmlObject fetchImmediateChildrenValues:[del.navigationbarStyle objectForKey:@"backgroundcolor"]]]; 
       UIColor *color = [UIColor colorWithRed:[[bgColor objectAtIndex:3] floatValue] green:[[bgColor objectAtIndex:2] floatValue] blue:[[bgColor objectAtIndex:1] floatValue] alpha:[[bgColor objectAtIndex:0] floatValue]]; 
       CGContextSetFillColor(context, CGColorGetComponents([color CGColor])); 
       CGContextFillRect(context, rect); 
       [bgColor release]; 

     } 
    @end 

@implementation xmlObject 
     + (NSArray *) fetchImmediateChildrenValues:(NSMutableDictionary *) node { 
     NSMutableDictionary *tmp = [[node objectForKey:@"children"] retain]; 
     NSArray *keys = [[NSArray alloc] initWithArray:[tmp allKeys]]; 
     keys = [keys sortedArrayUsingSelector:@selector(caseInsensitiveCompare:)]; 
     NSMutableArray *pushArr = [[[NSMutableArray alloc] init] autorelease]; 
     NSString *val = [[NSString alloc] init]; 
     for(NSString *str in keys) { 
      val = (NSString *)[[tmp objectForKey:str] objectForKey:@"innertext"]; 
      [pushArr addObject:val]; 
     } 
     [val release]; 
     [keys release]; 

     return [NSArray arrayWithArray:pushArr]; 
     } 

@end 

代码有什么问题？同时也是应用崩溃的这行代码 应用程序崩溃，如果我加入这一行

NSArray *bgColor = [[NSArray alloc] initWithArray:[xmlObject fetchImmediateChildrenValues:[del.navigationbarStyle objectForKey:@"backgroundcolor"]]]; 

如果我删除应用程序运行速度流畅。

Answer 1

我对你的代码有几点评论。其中之一是你的崩溃的直接原因，但你至少还需要解决另一个问题。简短的答案是，你通过释放val和键。

NSArray *bgColor = [[NSArray alloc] initWithArray:[xmlObject fetchImmediateChildrenValues:[del.navigationbarStyle objectForKey:@"backgroundcolor"]]]; 

你并不需要在这里创建一个新的数组，你可以简单地写：

NSArray *bgColor = [xmlObject fetchImmediateChildrenValues:[del.navigationbarStyle objectForKey:@"backgroundcolor"]]; 

如果你这样做，你不需要[bgColor release]进一步下跌。

NSArray *keys = [[NSArray alloc] initWithArray:[tmp allKeys]]; 
keys = [keys sortedArrayUsingSelector:@selector(caseInsensitiveCompare:)]; 

这两行泄漏第一个NSArray，你分配它，但你用排序后的版本直接覆盖它。事实上，你可以简单的写：

keys = [[tmp allKeys] sortedArrayUsingSelector:@selector(caseInsensitiveCompare:)]; 

注意，你没有自己的密钥，所以你可以摆脱[keys release]线的进一步下跌。

NSString *val = [[NSString alloc] init]; 
for(NSString *str in keys) { 
    val = (NSString *)[[tmp objectForKey:str] objectForKey:@"innertext"]; 
    [pushArr addObject:val]; 
} 
[val release]; 

这是您的直接问题的来源。你首先分配一个新的字符串。然后，在循环的每次迭代中立即覆盖它。所以分配的NSString泄漏。在每次迭代中，您都不拥有由[[tmp objectForKey:str] objectForKey:@"innertext"];返回的val，因此循环后的版本ov val不应该在那里。

在附注上，objectForKey:返回一个id - 对NSString *的转换是多余的。大多数人都把它排除在外。

  [keys release]; 

回到上面的位置，我告诉你，你正在泄漏你的分配键？那么你与你重写的新版本密钥并不拥有。因此你不能在这里释放密钥。

return [NSArray arrayWithArray:pushArr]; 

这很好。我的偏好是：

return [[pushArray copy] autorelease]; 

但它只是一个风格问题。你也可以返回pushArray，但pushArray是可变的，调用者可以依赖返回值是不可变的。

Answer 2

用NSZombieEnabled设置测试你的代码...它应该给你足够的信息来解决你的问题。