从循环中将 NSMutableArray 添加到另一个似乎会创建重复项

Question

我正在解析 json 编码事件的 NSDictionary ，并根据月份将它们放入二维 NSMutableArray 中，以便在分段表视图中显示。

由于我将项目添加到数组中，然后将该数组放入循环中的数组 (event_container) 中，event_container 显示了正确的数组数量，但是，它们都出现了是上次迭代的重复项，因此 event_container 的所有内容都是相同的数组。

我相信这是因为它是一个指针和/或没有被释放。我不确定是否有适当的方法来解决这个问题，甚至不确定是否有更好的解决方案。我正在使用ARC。

int month = 0;
int current_month = 0;
int counter = 0;

event_container = [[NSMutableArray alloc] init];

temp_array = [[NSMutableArray alloc] init];


for (NSDictionary *result in results)
{

    NCEvent *anEvent = [[NCEvent alloc] init];

    anEvent.title = [result objectForKey:@"title"];
    anEvent.startdate = [result objectForKey:@"startdate"];
    anEvent.enddate = [result objectForKey:@"enddate"];

    NSDateFormatter *importDate = [[NSDateFormatter alloc] init];
    [importDate setDateFormat:@"yyyy-M-d H:m:ss"];
    anEvent.dateStart = [importDate dateFromString:anEvent.startdate];
    anEvent.dateEnd = [importDate dateFromString: anEvent.enddate];

    NSDateFormatter *exportDate = [[NSDateFormatter alloc] init];
    [exportDate setDateFormat:@"d"];
    anEvent.text_date = [exportDate stringFromDate: anEvent.dateStart];

    NSDateFormatter *exportMon = [[NSDateFormatter alloc] init];
    [exportMon setDateFormat:@"MMM"];
    anEvent.text_mon = [exportMon stringFromDate: anEvent.dateStart];

    NSDateFormatter *monthInt = [[NSDateFormatter alloc] init];
    [monthInt setDateFormat:@"M"];
    month = [[monthInt stringFromDate: anEvent.dateStart] intValue];

    if(counter == 1){                           //first month
        current_month = month;
        NSLog(@"I'm the first month: %i", month); 
        [temp_array addObject:anEvent];

    }
    else if(month > current_month){             //new month
        NSLog(@"This is a new month");                    
        current_month = month;

        //add the events array to events container and reset the events array
        [self.event_container addObject: temp_array];

        [temp_array removeAllObjects];

        [temp_array addObject:anEvent];

    }
    else{
        NSLog(@"Same Month");                   //same month
        [temp_array addObject:anEvent];


    }

    NSLog(@"Event month integer: %i", month);

    anEvent = nil;

    counter++;

}

这些数组被声明为属性：

@property (nonatomic, retain) NSMutableArray *event_container;
@property (nonatomic, retain) NSMutableArray *temp_array;

Answer 1

在该行中：

[self.event_container addObject: temp_array];

您始终将相同的实例 temp_array 添加到 self.event_container 中。这就是为什么您会看到同一个数组重复多次。

您可以通过执行以下操作来解决此问题，例如：

- 在 for 循环之前添加以下内容

for (int i = 0; i < 12; i++) {
    [event_container addObject:[NSMutableArray array]];
}
for (NSDictionary *result in results)
...

- 删除

if(counter == 1){                           //first month
     current_month = month;
     NSLog(@"I'm the first month: %i", month); 
     [temp_array addObject:anEvent];
}

- 并将其后的代码更改为：

tmp_array = [event_container objectAtIndex:month];
[temp_array addObject:anEvent];

Answer 2

您对数组是指针的怀疑基本上是正确的。问题是您的 temp_array 并不是那么临时——实际上每次循环时它都是同一个数组对象。

您在循环外部创建它，每当您发送 addObject: 或 removeAllObjects 时，它都会影响您已放入其中的内容。

但关键在于，当您将 temp_array 添加到 event_container 时，它是完全相同的对象。它不是被复制的； event_container 数组仅获取指向 temp_array 的指针。当你再次添加它时，它是一样的。由于 event_container 仅保存一大堆指针，因此当您检查它时，您最终会看到同一个对象。

这就是正在发生的事情。为了解决这个问题，您需要为每个月创建一个单独的数组；我认为sch的答案对你有用。

快速演示：

NSMutableArray * container = [NSMutableArray array];
NSMutableArray * temp = [NSMutableArray array];

int i;
for( i = 0; i < 5; i++ ){
    [temp addObject:[NSNumber numberWithInt:i]];
    [container addObject:temp];    // Doesn't copy; just adds pointer to temp
    [temp removeAllObjects];
}
// Inspecting container now, we find that it has five arrays, all empty.
NSLog(@"%@", container);

Answer 3

temp_array 是一个指针类型（与 Objective c 中的所有对象一样）。因此，通过此调用：

[self.event_container addObject: temp_array];

...您将指向该对象的指针添加到 event_container。您不是在创建新数组，而只是向同一个对象添加多个指针。您最可能想要做的是添加对象的（指向）副本，如下所示：

[self.event_container addObject: [temp_array mutableCopy]];