Discussion :

[cypher90]

Bonjour,

Je suis en train d'apprendre l'objective C et le développement Iphone. Ma bête noire en ce moment, c'est la gestion de la mémoire.

Voici, un exemple qui me pose problème parmi d'autres :

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
- (void)setBalise:(id)sender{
	NSMutableString *str = [self.text mutableCopy];
	NSRange r = [self selectedRange];
 
	if (r.location == NSNotFound) {
		return;
	}
 
	NSString *selectedStr = [str substringWithRange:r];
	NSString *newStr = [NSString stringWithFormat:@"<mabalise>%@</mabalise>", selectedStr];
	[str replaceCharactersInRange:r
						withString:newStr];
 
	self.text = str;
 
}

Dans ce code, peut importe son fonctionnement, je vois que l'on a recours à des NSString, à des NSRange, mais à aucun moment on ne fais de release une fois qu'on a fini de les utiliser.

Pourriez-vous m'expliquer pourquoi ?

[the_nmy]

le release faut le mettre dans la fonction viewdidunload....

[JeitEmgie]

Bonjour,

Je suis en train d'apprendre l'objective C et le développement Iphone. Ma bête noire en ce moment, c'est la gestion de la mémoire.

Voici, un exemple qui me pose problème parmi d'autres :

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
- (void)setBalise:(id)sender{
	NSMutableString *str = [self.text mutableCopy];
	NSRange r = [self selectedRange];
 
	if (r.location == NSNotFound) {
		return;
	}
 
	NSString *selectedStr = [str substringWithRange:r];
	NSString *newStr = [NSString stringWithFormat:@"<mabalise>%@</mabalise>", selectedStr];
	[str replaceCharactersInRange:r
						withString:newStr];
 
	self.text = str;
 
}

Dans ce code, peut importe son fonctionnement, je vois que l'on a recours à des NSString, à des NSRange, mais à aucun moment on ne fais de release une fois qu'on a fini de les utiliser.

Pourriez-vous m'expliquer pourquoi ?

1. vous avez le "Build And Analyze" qui va vous dire si vous faites des erreurs dans la gestion mémoire en tenant compte de comment vos "property" sont déclarées (ce qui manque ici dans ce que vous montrez) (analyse statique de votre code source)

2. NSRange n'est pas un NSObject mais une simple "struct" et n'est donc pas concerné par la gestion mémoire

3. mutableCopy renvoie un objet avec un retain count de 1, sortir par "return" sans le "release" est donc un memory leak
et a priori en considérant les conventions habituelles pour une property comme "text", qui devrait être "retain",
self.text = str va augmenter encore le retain count de 1.

Solution:

Code :

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

1
2
 
	NSMutableString *str = [[self.text mutableCopy] autorelease] ;

et il serait mieux de copier l'objet seulement s'il y a quelque chose à faire : déplacez cette ligne après le test sur r.location.

4. substringWithRange et stringWithFormat renvoient des autoreleased : donc pas de problème pour eux

5. replaceCharactersInRange est une méthode de NSMutableString et non de NString : vous devez donc utiliser stringByReplacingCharactersInRange :

Code :

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

1
2
 
self.text = [str stringByReplacingCharactersInRange:r withString:newStr] ;

(si "text" est bien une property avec l'attribut "retain" - sinon il faut un ajouter un appel à "retain").

NB
a. Toutes les conventions de noms des méthodes en fonction du retour "retain-ed" ou "autorelease-d" sont expliquées dans la documentation sur le site d'Apple.

b. avec XCode 4, il est aussi possible d'utiliser les techniques liées à l'"Automatic Reference Counting" : voir la session 323 de la WWDC 2011 : le compilateur va s'occuper des retain/release et forcer le respect des conventions.