Discussion :

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[JeitEmgie]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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]

[sculpteur]

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

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

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:

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)

[JeitEmgie]

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

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


si je le laisse:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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…

[sculpteur]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

myParser = [[… alloc] initXXXX:…]

initXXXX ??


Merci beaucoup JeitEmgie pour tes éclaircissements

[JeitEmgie]

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 ?

comment je dois procéder pour completer celle ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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] ;

[sculpteur]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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:

[JeitEmgie]

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


J'ai essayé çà:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
+ (id)parseXMLAtURL: …
{
    return [[[self alloc] initWithXMLAtURL: …] autorelease] ;
}

qu'on invoquerait alors ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

sans devoir faire de [myParser release] plus loin.

[sculpteur]

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

Merci beaucoup