Не удалось устранить утечку, обнаруженную инструментом утечки.Могу ли я игнорировать это? - PullRequest
1 голос
/ 22 сентября 2011

Я не могу найти способ устранить утечку, обнаруженную инструментом утечки. Вот моя проблема ... В моем делегате есть одноэлементный объект, который хранит данные на глобальном уровне. Теперь у меня есть массив объектов, которые я поддерживаю здесь и добавляю или изменяю их из контроллеров. Ниже приведена функция, которая заполняет объекты и устанавливает вышеуказанный глобальный массив, Теперь выделенные линии (отмеченные // LEAK) - это место, где инструмент утечки сообщает мне, что это утечка. Мне нужен этот массив для моей сессии. Я выпускаю массив в конце, когда я выхожу. Должен ли я беспокоиться о такой утечке?

-(LayoutInfo *) fillLayout: (GDataXMLElement *) layoutElement {

        LayoutInfo *layout = [[LayoutInfo alloc] init];
        layout.dataTableCount = 0;
        layout.chartsCount = 0;
        NSArray *templateNameArr = [layoutElement elementsForName:@"TemplateName"];


        NSMutableArray *chartElements = [[NSMutableArray alloc] init];   // LEAK 
        NSMutableArray *dtElements = [[NSMutableArray alloc] init];
        NSArray *charts = [layoutElement elementsForName:@"chart"];     // LEAK 
        if (charts.count > 0) {
            for (GDataXMLElement *singleChart in charts) {
                chart *chartInfo = [[chart alloc] init];  // LEAK 
                layout.chartsCount = layout.chartsCount + 1;
                NSArray *imageName = [singleChart elementsForName:@"imageName"];
                if (imageName.count > 0) {
                    GDataXMLElement *imageNameStr = (GDataXMLElement *) [imageName objectAtIndex:0];
                    chartInfo.imageName = imageNameStr.stringValue; // LEAK 
                }           
                NSArray *imagePath = [singleChart elementsForName:@"imagePath"];
                if (imagePath.count > 0) {
                    GDataXMLElement *imagePathStr = (GDataXMLElement *) [imagePath objectAtIndex:0];
                    chartInfo.imagePath = imagePathStr.stringValue;  // LEAK 
                } 

                NSArray *imageFileName = [singleChart elementsForName:@"imageFileName"];
                if (imageFileName.count > 0) {
                    GDataXMLElement *imageFileNameStr = (GDataXMLElement *) [imageFileName objectAtIndex:0];
                    chartInfo.imageFileName = imageFileNameStr.stringValue;
                } 
                ASIHTTPRequest *request = [ASIHTTPRequest requestWithURL:[NSURL URLWithString:chartInfo.imagePath]];
                [request setDownloadDestinationPath:[[NSHomeDirectory() stringByAppendingPathComponent:@"Documents"] 
                                                     stringByAppendingPathComponent:chartInfo.imageFileName]];

                [request setDidFinishSelector:@selector(fillLayout_requestDone:)];
                [request setDidFailSelector:@selector(fillLayout_requestWentWrong:)];
                [request startSynchronous];

                NSString *imagePath1 = [[NSHomeDirectory() stringByAppendingPathComponent:@"Documents"] stringByAppendingPathComponent:chartInfo.imageFileName];
                if([[NSFileManager defaultManager] fileExistsAtPath:imagePath1]) {
                    NSLog(@" --- IMAGE SAVED -- %@", imagePath1);
                }
                [chartElements addObject:chartInfo];
            } //for
            layout.chartElement = chartElements;   // THIS IS WHERE I ASSIGN THE GLOBAL ARRAY
            //[chartElements release];
        } 
        return layout;
    }

1 Ответ

1 голос
/ 22 сентября 2011
-(LayoutInfo *) fillLayout: (GDataXMLElement *) layoutElement {

    LayoutInfo *layout = [[LayoutInfo alloc] init];
    layout.dataTableCount = 0;
    layout.chartsCount = 0;
    NSArray *templateNameArr = [layoutElement elementsForName:@"TemplateName"];


    NSMutableArray *chartElements = [[NSMutableArray alloc] init];   // LEAK 
    //NSMutableArray *dtElements = [[NSMutableArray alloc] init];
    NSArray *charts = [layoutElement elementsForName:@"chart"];     // LEAK 
    if (charts.count > 0) {
        for (GDataXMLElement *singleChart in charts) {
            chart *chartInfo = [[chart alloc] init];  // LEAK 
            layout.chartsCount = layout.chartsCount + 1;
            NSArray *imageName = [singleChart elementsForName:@"imageName"];
            if (imageName.count > 0) {
                GDataXMLElement *imageNameStr = (GDataXMLElement *) [imageName objectAtIndex:0];
                chartInfo.imageName = imageNameStr.stringValue; // LEAK 
            }           
            NSArray *imagePath = [singleChart elementsForName:@"imagePath"];
            if (imagePath.count > 0) {
                GDataXMLElement *imagePathStr = (GDataXMLElement *) [imagePath objectAtIndex:0];
                chartInfo.imagePath = imagePathStr.stringValue;  // LEAK 
            } 

            NSArray *imageFileName = [singleChart elementsForName:@"imageFileName"];
            if (imageFileName.count > 0) {
                GDataXMLElement *imageFileNameStr = (GDataXMLElement *) [imageFileName objectAtIndex:0];
                chartInfo.imageFileName = imageFileNameStr.stringValue;
            } 
            ASIHTTPRequest *request = [ASIHTTPRequest requestWithURL:[NSURL URLWithString:chartInfo.imagePath]];
            [request setDownloadDestinationPath:[[NSHomeDirectory() stringByAppendingPathComponent:@"Documents"] 
                                                 stringByAppendingPathComponent:chartInfo.imageFileName]];

            [request setDidFinishSelector:@selector(fillLayout_requestDone:)];
            [request setDidFailSelector:@selector(fillLayout_requestWentWrong:)];
            [request startSynchronous];

            NSString *imagePath1 = [[NSHomeDirectory() stringByAppendingPathComponent:@"Documents"] stringByAppendingPathComponent:chartInfo.imageFileName];
            if([[NSFileManager defaultManager] fileExistsAtPath:imagePath1]) {
                NSLog(@" --- IMAGE SAVED -- %@", imagePath1);
            }
            [chartElements addObject:chartInfo];
            [chartInfo release];                   // it's retained in chartElements until removed, or until chartElements is deallocced
        } //for
        if(layout.charElement){
            [layout.charElement release];     // you should however consider in making charElement property as retain;
            layout.charElement = nil;         // this isn't required here (since you're assigning it a new value), but you should usually set it to nil after a release to prevent EXC_BADACCESS
        }
        layout.chartElement = chartElements;   // THIS IS WHERE I ASSIGN THE GLOBAL ARRAY
        //[chartElements release];
    } 
    return [layout autorelease];   // in case you don't want it autoreleased you should call your method something like: createFilledLayout ('create' is usually used so anyone that uses the method knows it's responsible for releasing the return value)
}

Вы должны взглянуть на Руководство по программированию управления памятью

...