j'ai un Memory leak Method returns an objective-C obbject with a +1

Bonjour
je suis entrain de déboguer mon application et j'ai un Memory leak dans la ligne en rouge

Method returns an objective-C obbject with a +1 retain count(owning reference)

->Object allocated on line 201 is no longer referenced after this point and has a retain count of +1(object leaked)

j'utilise la classe XMLToObjectParser
http://www.ipup.fr/page.php?id=332


Voici mon code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

- (void)viewDidLoad { tableau = [[NSMutableArray alloc] init]; NSURL *url = [NSURL URLWithString: @"http://www.site.com/feed.xml"]; XMLToObjectParser *myParser = [[XMLToObjectParser alloc] parseXMLAtURL:url toObject:@"eventLoc" andDenominator:@"eventLoc" parseError:nil]; for(int i = 0; i < [[myParser items] count]; i++) { //eventLoc *new = [[eventLoc alloc] init]; eventLoc *new = (eventLoc *)[[myParser items] objectAtIndex:i]; [tableau addObject:new]; } //[myParser release]; [super viewDidLoad]; }

---

je ne sais pas vraiment comment résoudre ce memory leak

Merci d'avance pour votre m'aider

Citation:

---

Envoyé par sculpteur

Bonjour
je suis entrain de déboguer mon application et j'ai un Memory leak dans la ligne en rouge

Method returns an objective-C obbject with a +1 retain count(owning reference)

->Object allocated on line 201 is no longer referenced after this point and has a retain count of +1(object leaked)

j'utilise la classe XMLToObjectParser
http://www.ipup.fr/page.php?id=332


Voici mon code

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

- (void)viewDidLoad { tableau = [[NSMutableArray alloc] init]; NSURL *url = [NSURL URLWithString: @"http://www.site.com/feed.xml"]; XMLToObjectParser *myParser = [[XMLToObjectParser alloc] parseXMLAtURL:url toObject:@"eventLoc" andDenominator:@"eventLoc" parseError:nil]; for(int i = 0; i < [[myParser items] count]; i++) { //eventLoc *new = [[eventLoc alloc] init]; eventLoc *new = (eventLoc *)[[myParser items] objectAtIndex:i]; [tableau addObject:new]; } //[myParser release]; [super viewDidLoad]; }

---

je ne sais pas vraiment comment résoudre ce memory leak

Merci d'avance pour votre m'aider

---

enlevez le // devant [myParser release]

oui j'ai déjà essayé ça

si j'enlève le commentaire(//):

Citation:

---

Method returns an objective-C obbject with a +0 retain count(non-owning reference)

->incorrect decrement of the reference count of an object is not owned at this point by the caller

---

si je le laisse:

Citation:

---

Method returns an objective-C obbject with a +1 retain count(owning reference)

->Object allocated on line 201 is no longer referenced after this point and has a retain count of +1(object leaked)

---

Citation:

---

Envoyé par sculpteur

oui j'ai déjà essayé ça

si j'enlève le commentaire(//):


si je le laisse:

---

Code:

---

1 2	XMLToObjectParser *myParser = [[XMLToObjectParser alloc] parseXMLAtURL:url toObject:@"eventLoc" andDenominator:@"eventLoc" parseError:nil];

---

ne respecte pas les conventions : c'est pour cela que l'analyseur ne peut donner de messages cohérents…

soit

vous avez myParser = [[… alloc] initXXXX:…]
et ensuite [myParser parseXMLAtURL:…]
et enfin [myParser release]

soit parseXMLAtURL est une méthode de classe et non d'instance et vous avez

[XMLToObjectParser parseXMLAtURL:…]

et rien d'autre car dans ce cas la convention est que parseXMLAtURL retourne une instance autoreleased…

Citation:

---

Envoyé par JeitEmgie

Code:

---

1 2	XMLToObjectParser *myParser = [[XMLToObjectParser alloc] parseXMLAtURL:url toObject:@"eventLoc" andDenominator:@"eventLoc" parseError:nil];

---

ne respecte pas les conventions : c'est pour cela que l'analyseur ne peut donner de messages cohérents…

soit

vous avez myParser = [[… alloc] initXXXX:…]
et ensuite [myParser parseXMLAtURL:…]
et enfin [myParser release]

soit parseXMLAtURL est une méthode de classe et non d'instance et vous avez

[XMLToObjectParser parseXMLAtURL:…]

et rien d'autre car dans ce cas la convention est que parseXMLAtURL retourne une instance autoreleased…

---

et vous pensez que c'est nécessaire de respecter les conventions?
et si je dois les respecter?
comment je dois procéder pour completer celle ligne

Code:

---

myParser = [[… alloc] initXXXX:…]

---

initXXXX ??


Merci beaucoup JeitEmgie pour tes éclaircissements

Citation:

---

Envoyé par sculpteur

et vous pensez que c'est nécessaire de respecter les conventions?
et si je dois les respecter?

---

faut-il répondre à une telle question ?

Citation:

---

Envoyé par sculpteur

comment je dois procéder pour completer celle ligne

Code:

---

myParser = [[… alloc] initXXXX:…]

---

initXXXX ??

---

aucune idée : je n'ai pas les sources de XMLToObjectParser, à vous de lire sa documentation…
et s'il n'y a pas de initXXX… faites simplement
myParser = [[XMLToObjectParser alloc] init] ;

Citation:

---

faut-il répondre à une telle question ?

---

je parle lors de l'envoie de l'application pour être validée, est ce qu'ils exigent çà


J'ai essayé çà:

Code:

---

myParser = [[XMLToObjectParser alloc] init] ;

---

mais j'ai une autre erreur:
Value stored to 'myParser' during its initialization is never read

la classe myparser est ici:

Citation:

---

Envoyé par sculpteur

je parle lors de l'envoie de l'application pour être validée, est ce qu'ils exigent çà


J'ai essayé çà:

Code:

---

myParser = [[XMLToObjectParser alloc] init] ;

---

mais j'ai une autre erreur:
Value stored to 'myParser' during its initialization is never read

la classe myparser est ici:

---

Passons sur le fait que le problème n'est pas reproductible ici…
ce qui laisse supposer qu'entre ce que vous faites réellement et ce que vous postez il y a une différence.

Pour faire au plus court :

1. Changer le nom de la méthode parseXMLAtURL par initWithXMLAtURL et faites

Code:

---

1 2	XMLToObjectParser *myParser = [[XMLToObjectParser alloc] initWithXMLAtURL:…] ;

---

2. laissez la ligne [myParser release] SANS commentaire
3. pour être strict la première ligne de initWithXMLAtURL devrait être :

Code:

---

1 2 3 4 5

- (id)initWithXMLAtURL: … { self = [super init] ; …

---

et il n'y aura plus de warning de l'analyseur (qui ne sont QUE des warnings…)


NB

Le code sur ipub.fr étant un exemple de mauvaises pratiques,
donc toujours en ayant renommé parseXMLAtURL en initWithXMLAtURL :
une méthode s'appelant "parseXMLAtURL" devant être une méthode de classe renvoyant une instance auto-released :

Code:

---

1 2 3 4 5

+ (id)parseXMLAtURL: … { return [[[self alloc] initWithXMLAtURL: …] autorelease] ; }

---

qu'on invoquerait alors ainsi :

Code:

---

1 2	XMLToObjectParser *myParser = [XMLToObjectParser parseWithXMLAtURL:…] ;

---

sans devoir faire de [myParser release] plus loin.

c'est résolu grâce a toi JeitEmgie

Merci beaucoup