Question

我正在尝试使用地址簿对联系人APP做一个程序并且它工作正常但是当我分析有几个内存泄漏时我已经设法将内存泄漏最小化，现在我发现了2个主内存泄漏警告，一个在我的地址簿重新加载功能中，我添加了注释以显示我尝试过的所有内容

-(void)reloadAddressBook
{
   //if(self.addressBook)
      //CFRelease(self.addressBook);
    self.addressBook = (__bridge ABAddressBookRef) CFBridgingRelease(ABAddressBookCreate());

    if(ABAddressBookHasUnsavedChanges(self.addressBook))
    {

        ABAddressBookSave(self.addressBook,NULL);


    }


    //if(self.contactAdd)
        //CFRelease(self.contactAdd);
    self.contactAdd= ABAddressBookCopyArrayOfAllPeople(self.addressBook);


**//Memory warning here and says: call to function ABAddressBookCopyArrayOfAllPeople returns a core foundation object with a +1 retain count**     
        //self.contactAdd= (__bridge ABAddressBookRef) CFBridgingRelease(ABAddressBookCopyArrayOfAllPeople(self.addressBook));
        **// If I use this format my memory leak issue solves here but I get error in my program**
    }

- (void)viewDidLoad
{**//Memory warning here and says :object leaked :allocated object is not retained lated in this execution path and has retain count +1**
    [super viewDidLoad];

    self.contactSearchBar.delegate=self;
    self.contactTableView.delegate=self;
    self.contactTableView.dataSource=self;

    UIBarButtonItem *addContactButton=[[UIBarButtonItem alloc]initWithTitle:@"Add" style:UIBarButtonItemStyleBordered target:self action:@selector(newContact:)];
    self.navigationItem.rightBarButtonItem=addContactButton;
    self.navigationItem.title=@"My Contacts";


}

此搜索栏功能中存在另一个内存泄漏

-(void)searchBar:(UISearchBar *)searchBar textDidChange:(NSString *)searchText
{

    if(searchText.length==0)
    {
        isFiltered=NO;

    }
    else
    {
        isFiltered=YES;

        int j=0,i=0;
        self.filteredData= CFArrayCreateMutable(kCFAllocatorDefault, 0,&kCFTypeArrayCallBacks);**// Memory warning here and says: call to function CFArrayCreateMutable returns a core foundation object with a +1 retain count**

       for(i=0;i<CFArrayGetCount(self.contactAdd);i++)**//Memory warning here and says :object leaked :allocated object is not retained lated in this execution path and has retain count +1**
        {
            self.person=CFArrayGetValueAtIndex(self.contactAdd,i);
            NSString *str=[[NSString stringWithFormat:@"%@", (__bridge_transfer NSString *)ABRecordCopyValue(self.person, kABPersonFirstNameProperty)] stringByTrimmingCharactersInSet:[NSCharacterSet whitespaceCharacterSet]];
            NSRange contactRange= [str rangeOfString: searchText options:NSCaseInsensitiveSearch];

            NSLog(@"i=%d, j=%d",i,j);

            if(contactRange.location!=NSNotFound)
            {
                CFArrayInsertValueAtIndex(self.filteredData,j++,self.person);
               CFArrayGetValueAtIndex(self.filteredData,j-1);
            }

        }
         //CFRelease(self.contactAdd);
    //CFRelease(self.filteredData);
    }

for loop语句中显示内存泄漏，它说：

Answer 1

就像我在my answer on your other question，in Core Foundation, Create and Copy functions return an ownership reference.

中提到的那样

保存对象的属性通常也拥有引用，除非您另外声明它们（通常使用weak）。这意味着，在这样的声明中：

self.filteredData= CFArrayCreateMutable(…);

您现在拥有该对象两次：一次因为您创建了它，一次因为您的属性保留了它。

对于每个属性，ivar或其他强引用（例如局部变量），通常应该只拥有每个对象一次。任何额外的所有权（例如来自创建和复制功能）都是你需要清理的东西，这就是为什么最好尽可能少地在CF-land上做：ARC为你清理东西，但它不会触及CF东西，除非你告诉它。

说到属性和局部变量，您不需要将所有内容都设为属性。搜索代码中的person应该是该方法中的局部变量，因为这是该搜索状态的一部分，而不是您的对象需要无限期保留的内容。您已将str作为本地变量，因此我不确定您为何使用person的属性。

你告诉ARC通过桥牌演员与CF世界互动：

__bridge_transfer（或CFBridgingRelease）告诉ARC“当我完成它时，我会清理这个所有权。”
__bridge_retained（或CFBridgingRetain）告诉ARC“我要把这个物体扔进CF-land;在我另有说法之前不要放过它“（你必须在转移或立即CFRelease的情况下做某事。）

而且，您通常希望从CF世界中解脱出来，让ARC尽可能地处理它们。

因此，我建议将self.filteredData设为NSMutableArray，而不是CFMutableArray。使用Foundation创建数组意味着它已经处于ARC的控制之下，无需桥接它。

对于self.contactAdd，您可以在检索时将其桥接到NSArray，然后从那时起将其视为NSArray。这包括使用快速枚举而不是索引来迭代它，或者更好的是，使用谓词搜索而不是自己迭代。（我不会指望内置的比较谓词能够与ABPersons一起工作，但这就是predicateWithBlock:的用途。）

基于谓词的搜索代码版本看起来像这样（非常未经测试）：

self.filteredData = [self.contactAdd filteredArrayUsingPredicate:[NSPredicate predicateWithBlock:^BOOL(id evaluatedObject, NSDictionary *bindings){
    ABPersonRef person = (__bridge ABPersonRef)evaluatedObject;
    NSString *firstName = [[NSString stringWithFormat:@"%@", (__bridge_transfer NSString *)ABRecordCopyValue(self.person, kABPersonFirstNameProperty)] stringByTrimmingCharactersInSet:[NSCharacterSet whitespaceCharacterSet]];

    NSRange contactRange = [str rangeOfString:searchText options:NSCaseInsensitiveSearch];
    return (contactRange.location != NSNotFound);
}];

（块和基于它的谓词，返回每个对象是否匹配。filteredArrayUsingPredicate:创建一个数组，其中包含谓词评估为true的每个对象。）

还有一件事：你考虑过使用ABPeoplePickerNavigationController吗？

Answer 2

显然，您在[addContactButton release]结束时遗漏了viewDidLoad。另外，为什么CFRelease来电被注释掉？他们应该平衡CFArrayCreateMutable

关于self.contactAdd - 如果它被定义为＆＃34;保留＆＃34;属性，不要直接为其分配ABAddressBookCopyArrayOfAllPeople的返回值。相反，实例化一个局部变量，将其分配给属性，然后使用CFRelease

释放它

解决Xcode中的内存泄漏问题

2 个答案: