iCulture forum | iPhone, iPad,  iPod touch, Apple TV en iOS

iCulture forum | iPhone, iPad, iPod touch, Apple TV en iOS (https://forum.iculture.nl/)
-   Ontwikkelen voor iOS (https://forum.iculture.nl/f133/development/f58/ontwikkelen-voor-ios/)
-   -   Het probleem van de veranderende NSString (https://forum.iculture.nl/f133/development/f58/ontwikkelen-voor-ios/80246-probleem-de-veranderende-nsstring.html)

Nikooos 17-12-10 11:47

Het probleem van de veranderende NSString
 
Ik heb een xml met deze opbouw:
<category name="Carhifi">
<subcat code="AC-ACCESS">Accesoires</subcat>
<subcat code="AC-HEADUN">Autoradio</subcat>
<subcat code="AC-NAVIGA">Portable Navigatie</subcat>
</category>

Dan heb ik een object filteritem met een string en een mutable array:
@interface FilterItem : NSObject {
NSString *categorie;
NSMutableArray *subcat;
}

Via een xmlcontroller (nsxmlparser) vul ik die string en array. Maar als ik die later wil uitlezen, dan is op een of andere vage reden (af en toe) het laatste object in de array veranderd van een nsstring naar een calayer. Dit zorgt voor een crash als ik dit wil uitputten als string.

Heel begrijpelijk dat hij crash, maar waarom het opeens geen string meer is??

Iemand een ideetje die me op de goede weg kan helpen?

XMLFilterController.h
Code:

#import <Foundation/Foundation.h>
@class FilterItem;

@interface XMLFilterController: NSObject <NSXMLParserDelegate> {
    NSMutableString *currentNodeContent;
    NSMutableArray *items;
    NSXMLParser *parser;
    FilterItem *currentItem;
   
    int counter;
    NSMutableArray *subcats;
    NSString *attribuut;
}

@property (readonly,retain) NSMutableArray *items;

- (id)loadXMLByURL:(NSString *)urlString;
- (id)loadXMLByFile:(NSString *)file;

@end

XMLFilterController.m
Code:

#import "XMLFilterController.h"
#import "FilterItem.h"

@implementation XMLFilterController

@synthesize items;

- (id)loadXMLByURL:(NSString *)urlString {
    //NSLog(@"parsen!");
   
    items = [[NSMutableArray alloc] init];
    currentItem = [FilterItem alloc];
    currentItem.categorie = @"<kies filter>";
    [items addObject:currentItem];
   
   
    NSURL *url = [NSURL URLWithString: urlString];
    parser = [[NSXMLParser alloc] initWithContentsOfURL:url];
    parser.delegate = self;
    [parser parse];
    [parser release];
   
    currentItem = nil;
   
    return self;
}

- (id)loadXMLByFile:(NSString *)file {
    //NSLog(@"loadXMLByFile");
    items = [[NSMutableArray alloc] init];
    //NSURL *url = [NSURL URLWithString: urlString];
    //parser = [[NSXMLParser alloc] initWithContentsOfURL:url];
    parser = [[NSXMLParser alloc] initWithData:[NSData  dataWithContentsOfFile:[[NSBundle mainBundle] pathForResource:file  ofType:@"XML"]]];
    parser.delegate = self;
    [parser parse];
    [parser release];
    currentItem = nil;   
   
    return self;
}

- (void)parser:(NSXMLParser *)parser didStartElement:(NSString  *)elementName namespaceURI:(NSString *)namespaceURI  qualifiedName:(NSString *)qName attributes:(NSDictionary *)attributeDict  {
    if([elementName isEqualToString:@"category"]){
        //NSLog(@"item found!");
        currentItem = [FilterItem alloc];
        currentNodeContent = [[NSMutableString alloc] init];
        //NSLog(@"attribuut: %@",[attributeDict objectForKey:@"name"]);
        attribuut = [attributeDict objectForKey:@"name"];
       
        currentItem.categorie = attribuut;
        counter = 1;
        subcats = [[NSMutableArray alloc] init];
        //NSLog(@"%@",subcats);
        [subcats addObject:@"<kies filter>"];
    } else if ([elementName isEqualToString:@"subcat"]) {
        //NSLog(@"Een van deze ook gevonden!");
        currentNodeContent = [NSMutableString string];
    }
   
}

- (void)parser:(NSXMLParser *)parser didEndElement:(NSString  *)elementName namespaceURI:(NSString *)namespaceURI  qualifiedName:(NSString *)qName {   
    if ([elementName isEqualToString:@"category"]) {
        if (currentItem == nil) {
           
        } else {
            currentItem.subcat = subcats;
            NSLog(@"%@",subcats);
           
            [currentNodeContent release];
            currentNodeContent = nil;
           
            [items addObject:currentItem];
            //NSLog(@"%d",[items count]);
            [currentItem release];
            currentItem = nil;
        }
       
    } else if ([elementName isEqualToString:@"subcat"]) {
       
        if ([currentNodeContent isEqualToString:@""]){
            //[subcats addObject:currentNodeContent];
        } else {
            NSString *cat = currentNodeContent;
            [subcats addObject:cat];
            //NSLog(@"subcat: %@",currentNodeContent);
        }
       
    }
   
}

- (void)parser:(NSXMLParser *)parser foundCharacters:(NSString *)string { 
    [currentNodeContent appendString:[string  stringByTrimmingCharactersInSet:[NSCharacterSet  whitespaceAndNewlineCharacterSet]]];
}

- (void)dealloc {
    [currentNodeContent release];
    [items release];
    [super dealloc];
}

@end

FilterItem.h
Code:

#import <Foundation/Foundation.h>


@interface FilterItem : NSObject {
    NSString *categorie;
    NSMutableArray *subcat;

}

@property (retain, nonatomic) NSString *categorie;
@property (retain, nonatomic) NSMutableArray *subcat;


@end

FilterItem.m
Code:

#import "FilterItem.h"


@implementation FilterItem

@synthesize categorie;
@synthesize subcat;


- (void)dealloc {
    [super dealloc];
}

@end

Dan uiteindelijk vul ik een array met die object door deze funtie aan te roepen:
filterArray = [[xmlcontr1 loadXMLByURL:url1] items];

en vul ik mijn array met subcategorieen
subArray = [[filterArray objectAtIndex:selectedCat] subcat];

Als ik die subarray wil outputten, dan gaat het mis.

Whacko 17-12-10 11:56

currentNodeContent = [NSMutableString string];

die een retain geven, of autorelease zetten. Je releaset hem namelijk zelf in DidEndElement.

Nikooos 17-12-10 12:17

Whacko held!!! Dat was dé oplossing!!!

Echt, anderhalve dag aan het zoeken geweest en daar had ik simpelweg niet aan gedacht. Wie had gedacht dat het zo simpel kon zijn. Ik zat me echt blind te staren..

Whacko 17-12-10 12:23

You are welcome :)

wubbe 17-12-10 16:05

Volgens mij had je hier op de volgende manier achter kunnen komen:
1) Doe Appeltje-I op je App in 'Executables'
2) Ga naar de tab 'Arguments'
3) Voeg onderin toe: NSZombieEnabled en NSDebugEnabled, beide met value 'YES'
4) Als je vage resultaten krijgt zet je het vinkje voor deze twee aan en draai je je Applicatie nog een keer.

Als je nu een al gereleased object benaderd krijg je een foutmelding: "Message sent to deallocated instance."

Nikooos 18-12-10 13:59

Bedankt voor de tip Wubbe. Ik zal het eens proberen van de week.

wubbe 20-12-10 10:11

Let wel op dat je het vinkje bij 'gewoon' werken weer uit zet. De grap is dat iedere nil-pointer wijst naar zo'n Zombie object. Dat betekent wel dat:

Code:

if (!object) {
    object = [[Object alloc] init];
}

Niet meer werkt. if (object) is dan namelijk altijd TRUE.


Alle tijden zijn GMT +2. Het is nu 04:06.