iPhone - 自定义对象内的 NSMutableArray。什么时候发布？

Question

我应该什么时候释放[self.activeLocations]，它是我的自定义对象中的一个NSMutableArray？我还在 initWithValue 中遇到内存泄漏。

还有下面的 Location 对象。我是否正确调用并释放了它？

Custom Object.m 中的方法：

- (id)initWithValue:(NSString *)value {
    if ((self = [super init])) {
        self.couponId = [value valueForKey:@"couponId"];
        self.couponName = [value valueForKeyPath:@"couponName"];
        self.qrCode = [value valueForKeyPath:@"description"];
        self.companyName = [value valueForKeyPath:@"companyName"];
        self.categoryName = [value valueForKeyPath:@"categoryName"];
        self.distance = [value valueForKeyPath:@"distance"];

        NSDictionary *activeLocationsDict = [value valueForKeyPath:@"activeLocations"];
        //self.activeLocations = [[[NSMutableArray alloc] init] autorelease];
        self.activeLocations = [NSMutableArray array];
        for (id val in activeLocationsDict) {
            // Add JSON objects to array.
            Location *l = [[Location alloc] initWithValue:val];
            [self.activeLocations addObject:l];
            [l release];
        }
    }

    return self;
}

- (void)dealloc {
    [super dealloc];
    couponId = nil;
    couponName = nil;
    qrCode = nil;
    companyName = nil;
    categoryName = nil;
    distance = nil;
    activeLocations = nil;
}

我的 Custom Object.h

@interface Coupon : NSObject {
    NSNumber *couponId;
    NSString *couponName;
    NSString *qrCode;
    NSString *companyName;
    NSString *categoryName;
    NSString *distance;
    NSMutableArray *activeLocations;
}

@property (nonatomic, copy) NSNumber *couponId;
@property (nonatomic, copy) NSString *couponName;
@property (nonatomic, copy) NSString *qrCode;
@property (nonatomic, copy) NSString *companyName;
@property (nonatomic, copy) NSString *categoryName;
@property (nonatomic, copy) NSString *distance;
@property (nonatomic, retain) NSMutableArray *activeLocations;

- (id)initWithValue:(NSString *)value;

这就是我使用上述 initWithValue 的方式：

- (NSMutableArray *)createArrayOfCoupons:(NSString *)value {
    NSDictionary *responseJSON = [value JSONValue];

    // Loop through key value pairs in JSON response.
    //NSMutableArray *couponsArray = [[NSMutableArray alloc] init] autorelease];
    NSMutableArray *couponsArray = [NSMutableArray array];

    for (id val in responseJSON) {
        // Add JSON objects to array.
        Coupon *c = [[Coupon alloc] initWithValue:val];
        [couponsArray addObject:c];
        [c release];
    }

    return couponsArray;
}

我在上述方法中的 initWithValue 上也遇到了内存泄漏。 ..

位置 自定义对象：

- (id)initWithValue:(NSString *)value {
    if ((self = [super init])) {
        self.locationId = [value valueForKeyPath:@"locationId"];
        self.companyName = [value valueForKeyPath:@"companyName"];
        self.street1 = [value valueForKeyPath:@"street1"];
        self.street2 = [value valueForKeyPath:@"street2"];
        self.suburb = [value valueForKeyPath:@"suburb"];
        self.state = [value valueForKeyPath:@"state"];
        self.postcode = [value valueForKeyPath:@"postcode"];
        self.phoneNo = [value valueForKeyPath:@"phoneNo"];
        self.latitude = [value valueForKeyPath:@"latitude"];
        self.longitude = [value valueForKeyPath:@"longitude"];
    }

    return self;
}

- (void)dealloc {
    [super dealloc];
    locationId = nil;
    companyName = nil;
    street1 = nil;
    street2 = nil;
    suburb = nil;
    state = nil;
    postcode = nil;
    phoneNo = nil;
    latitude = nil;
    longitude = nil;
}

Answer 1

- (id)init {
   ....
}

摆脱这个。它什么也不做。

- (id)initWithValue:(NSString *)value {
    [super init];

您应该使用一种特定的模式进行初始化：

- (id)initWithValue:(NSString *)value {
    if (( self = [super init] )) {
      // everything except the return
    }
    return self;
}

最后，为了回答您的实际问题，假设您在属性中使用 retain ，那么您需要释放两个位置。

这是第一个：

self.activeLocations = [[NSMutableArray alloc] init];

为什么：[[NSMutableArray alloc] init] 使您的代码通过保留该对象来拥有该对象。但该财产集也通过保留它来声明所有权。您并不真的希望此 NSMutableArray 由代码和您的自定义对象拥有，您希望它由您的对象拥有。

我的建议是只使用这个：

self.activeLocations = [NSMutableArray array];

第二个地方是在你的 dealloc 中：（

- (void)dealloc {
    self.activeLocations = nil;

    // ...and everything else you've set as a property using retain

    [super dealloc];
}

就我个人而言，我一直在考虑是否在 dealloc 中使用点符号而不是 [activeLocations release];。我我现在倾向于使用该属性将所有内存管理规则设置为 nil，这会将所有内存管理规则放在一个位置。）

Apple 有一份关于内存管理的精彩文档，您应该阅读：内存管理编程指南：对象所有权和处置。

Answer 2

首先，你重写的 -init 方法完全没有必要，因为默认情况下，当调用一个方法时，运行时会执行继承层次结构的向上遍历，直到指定的方法找到了，所以它会找到NSObject的-init方法并调用它。

其次，您应该在覆盖的 -dealloccopy 或 retain 的属性）调用 release代码>方法。

第三，在您的情况下，当您调用传递本地已拥有的对象的属性 setter 时，您必须在调用 setter 后向该对象发送 release 消息，以正确地将对象的所有权移交给接收器。

有两种方法可以做到这一点：

一种方法是使用 alloc、copy 或 new 创建您拥有的对象，然后调用属性设置器，传入该对象，然后向其发送 release 消息。

另一种方法是将自动释放的对象传递给属性设置器，然后属性设置器将保留或复制其参数，从而获得所有权

Answer 3

何时应该释放它的答案是 activeLocations 数组和该数组中的所有元素（记住数组中的每个元素都由数组本身保留）是否是必需的问题Location 对象的生命周期。

如果您将 activeLocations 数组用于某些临时目的，例如在方法或方法链中，则不再需要它，或者您计划在下次需要时刷新其成员，那么这是有意义的当您使用完数组（及其元素，这是自动的）后，在最后使用该数组的任何函数中释放该数组。您将使用约定

self.activeLocations = nil;

来让运行时系统释放数组并将成员设置为 nil。

另一方面，如果 activeLocations 数组数据对于 Locations 对象的功能是必需的，并且只要 Location 对象存在就必须存在，那么您将需要在 Location 对象的 dealloc 方法中释放该数组，例如：

- (void) dealloc {
    [activeLocations release];

    [super dealloc];
}

碰巧的是，您几乎总是希望在 dealloc 方法中释放成员对象，例如 activeLocations。这确保了当 Location 对象被释放时，它包含的成员被清除。请记住，Objective-C 不会调用空指针上的方法，因此如果您之前将 activeLocations 设置为 nil，则 dealloc 中的调用是安全的无操作。

鉴于您总是会在 dealloc 中将事物设置为释放，现在您实际上只需要问自己是否需要在对象生命周期中的某个位置释放/重新创建阶段（同样，由使用频率要求决定）。

Answer 4

这取决于你要问什么。在您共享的 initWithValue: 方法中，您双重保留了数组。它应该在 initWithValue: 内释放或自动释放一次。

应在自定义对象的 dealloc 方法中第二次释放该数组。