1

我正在尝试使用通讯录在联系人 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循环语句上显示内存泄漏，它说：

4

2 回答 2

1

就像我在回答您的其他问题时提到的那样，在 Core Foundation 中，创建和复制函数返回所有权引用。

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

self.filteredData= CFArrayCreateMutable(…);

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

通常，每个属性、ivar 或其他强引用（例如局部变量）应该只拥有每个对象一次。任何额外的所有权（例如来自 Create 和 Copy 函数）都是您需要清理的东西，这就是为什么最好在 CF-land 中做的事情越少越好：ARC 会为您清理东西，但它不会触及 CF除非你告诉它。

说到属性和局部变量，您不需要将所有内容都设为属性。person在搜索代码中应该是该方法中的局部变量，因为这是该搜索状态的一部分，而不是您的对象需要无限期保留的东西。你已经有str一个局部变量，所以我不确定你为什么使用person.

您告诉 ARC 通过桥接模型与 CF 世界进行交互：

  • __bridge_transfer（或CFBridgingRelease）告诉 ARC “当我完成它时为我清理这个所有权”。
  • __bridge_retained（或CFBridgingRetain）告诉 ARC “我要把这个物体扔进 CF-land；除非我另有说明，否则不要放手”（然后您必须在某个时候使用转移或立即执行此操作CFRelease）。

而且，同样，您通常希望将事情从 CF 世界中取出，并让 ARC 尽可能多地处理它们。

因此，我建议制作self.filteredDataNSMutableArray，而不是 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？

于 2013-08-03T04:58:36.833 回答
0

显然[addContactButton release]，您在viewDidLoad. 另外，为什么这些CFRelease电话被注释掉了？他们应该平衡CFArrayCreateMutable

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

于 2013-08-02T09:34:08.263 回答