释放快速

Question

如何简洁地处理这种情况。我没有在if 声明中正确释放contactDictionary...

    NSNumber *pIDAsNumber;
     ...        
    NSMutableDictionary *contactDictionary = [NSMutableDictionary dictionaryWithDictionary:[defaults dictionaryForKey:kContactDictionary]];
    if (!contactDictionary) {
            contactDictionary = [[NSMutableDictionary alloc] initWithCapacity:1];
    }
    [contactDictionary setObject:pIDAsNumber forKey:[myClass.personIDAsNumber stringValue]];
    [defaults setObject:contactDictionary forKey:kContactDictionary];

Answer 1

通常，使用 [NSMutableDictionary dictionaryWithCapacity:1] 代替 alloc/init。这将为您提供一个自动释放的字典，从内存管理的角度来看，它的行为与上述字典相同。不过……

在这种特定情况下，您的 if 子句永远不会为真（除非您的内存不足，在这种情况下您会遇到更大的问题）。 -dictionaryWithDictionary: 如果传入 nil，则返回一个空字典而不是 nil。因此，即使 -dictionaryForKey: 返回 nil，-dictionaryWithDictionary: 仍会创建一个空的可变字典供您添加。

Answer 2

您可以完全删除if 语句，因为-[NSMutableDictionary dictionaryWithDictionary:] 总是返回一个字典。

另外，不要使用[NSMutableDictionary dictionaryWithCapacity:1] 来获取空的、自动释放的可变字典。只需使用[NSMutableDictionary dictionary]。