自定义 SQLite 包装器中的内存泄漏

Question

在我的应用程序中，我有一个到 sqlite3 数据库的连接。我做了一个包装类，在这个包装类中我有一个 NSMutableDictionary 和 NSMutableArray。

每次运行查询时，我都会从字典和说唱歌手类中的数组中removeAllObjects（我不释放它）。然后我将查询的结果添加到数组和字典中。该词典包含另一个子词典。

我有一个 tableViewController，在这个类中，我使用我的 rapper 类从数据库中获取数据并将其复制到我的 tableviewcontroller 变量中：

.h

     @interface BrandViewController : UIViewController
<UITableViewDataSource , UITableViewDelegate>
{
    FairPriceDatabaseView *FairPriceDB;
    NSArray *brandsIDs;
    NSMutableDictionary *brandsRecords;
    UITableView *tableView;  
}

.m

    - (void)viewWillAppear:(BOOL)animated
{
    [super viewWillAppear:animated];
    [self loadBrandsIDs];
    [tableView reloadData];
}
- (void)dealloc {[brandsRecords release];
    [brandsIDs release];
    [super dealloc];
}
-(NSArray *) loadBrandsIDs
{
    [self loadBrandsDB];

    [brandsIDs release];
    brandsIDs = [[FairPriceDB getBrandIDs]copy];

    [brandsRecords release];
    **brandsRecords = [[FairPriceDB getBrandIDs_NSDictionary]copy];**

    [FairPriceDB release];
    FairPriceDB = nil;
    return brandsIDs;
}
- (FairPriceDatabaseView *) loadBrandsDB {
    if (!FairPriceDB) 
        FairPriceDB = [[FairPriceDatabaseView alloc] initWithFairPriceDatabaseViewFilename:@"b.db"];
    return FairPriceDB;
}

在测试时，我在星号行出现内存泄漏 (brandsRecords = [[FairPriceDB getBrandIDs_NSDictionary]copy];) 当我更改 tableviewcontroller 并回到这些 tableviewcontrollers 时发生内存泄漏....

我想知道，我这样做是否正确？为什么会漏水？

另外，我每次发布NSMutableDictionary时，是否还需要发布其中包含的子词典？

FairPriceDataBaseViewController.h（包装类）

@interface FairPriceDatabaseView {
    NSMutableArray * idList;
    NSMutableDictionary * recordList;
}

FairPriceDataBaseViewController.m（包装类）

            - (NSArray *) getBrandIDs {
                NSDictionary * row;
                [idList removeAllObjects];  // reset the array
                for (row in [self getQuery:@"SELECT productID,brandName FROM product GROUP BY brandName;"]) 
                    [idList addObject:[row objectForKey:@"productID"]];
                return idList;
            }

            -(NSDictionary *) getBrandIDs_NSDictionary{
                [recordList removeAllObjects];
                [idList removeAllObjects];
                [self getBrandIDs];

                NSNumber * rowid;
                for(rowid in [self idList])
                    [recordList setObject:[self getProductRow:rowid]  forKey:rowid];

                return recordList; 
            }
        - (NSDictionary *) getProductRow: (NSNumber *) rowid {
            self.tableName = @"select * from product where productID = ?";
            return [self getRow:rowid];
        }
        -(FairPriceDatabaseView *) initWithFairPriceDatabaseViewFilename: (NSString *) fn
         {
             if((self = (FairPriceDatabaseView *) [Super initWithDBFilename:fn]))
             {
                    idList = [[NSMutableArray alloc] init];
                    recordList = [[NSMutableDictionary alloc]init];
             }
              [self setDefaults];
              return self;
         }

Answer 1

请阅读objective-C memory management guidelines。总之，您必须平衡所有保留操作 (retain,new,init,copy) 与发布。

由于您是清空并重新填充结果字典和数组，而不是创建一个新的然后复制结果，因此您将保留此对象而不是释放它。如果您多次执行此操作，那么您将丢失对先前值的引用，因此您现在有内存泄漏。

您需要在分配新值之前释放以前的值，或者最好还是创建一个保留属性并通过合成访问器添加新值。

Answer 2

[super dealloc]; 应该是-dealloc 方法的最后一行。在上面的代码中，您首先放置它。

这可能会导致意外行为，例如在此之后没有正确释放您所做的对象，包括brandsRecords。当 BrandViewController 被释放时，这可能会导致您的泄漏，因为它的实例变量可能永远不会被释放。

另外，正如我评论 jrturton 的回答，不要这样做：

if(brandsIDs)
        [brandsIDs release];

这是一个无用的条件，因为如果 brandsIDs 已被释放，则默认情况下它不会为 nil（除非您使用弱指针，但您没有在这里）并且您的应用程序此时将崩溃。如果它是 nil，则向 nil 发送一个 release 不会做任何事情，那么为什么还要在那里使用 if 语句呢？只需使用[brandsIDs release];