Comment puis-je libérer() après malloc() lorsque le résultat de malloc() est renvoyé par la fonction?

Question

J'ai la méthode d'instance suivante (adapté de Listing 3-6 de la section Gestion de l'événement dans le Guide de programmation d'applications iPhone):

- (CGPoint)originOfTouch:(UITouch *)touch 
{ 
    CGPoint *touchOriginPoint = (CGPoint *)CFDictionaryGetValue(touchOriginPoints, touch); 
    if (touchOriginPoint == NULL) 
    { 
     touchOriginPoint = (CGPoint *)malloc(sizeof(CGPoint)); // leaks 
     CFDictionarySetValue(touchOriginPoints, touch, touchOriginPoint); 
     *touchOriginPoint = [touch locationInView:touch.view]; 
    } 
    return *touchOriginPoint; 
} 

Chaque fois que dans un certain temps mon application fuites 16 octets par suite de l'appel à malloc(). Je ne suis pas sûr comment retourner touchOriginPoint tandis que free() il aussi bien.

Answer 1

Si vous ne vous souciez pas de légère perte de performance, utilisez un NSMutableDictionary et stocker le point comme NSValue:

NSValue* touchOriginPointValue = [touchOriginPoints objectForKey:touch]; 
if (touchOriginPointValue == nil) { 
    touchOriginPointValue = [NSValue valueWithCGPoint:[touch locationInView:touch.view]]; 
    [touchOriginPoints setObject:touchOriginPointValue forKey:touch]; 
} 
return [touchOriginPointValue CGPointValue]; 

---

Si vous devez utiliser l'approche CFDictionary, vous devez trouver un placer à free ces malloc mémoire -ed lorsque les valeurs ne sont pas nécessaires. Par conséquent, vous devez transmettre les valeurs callbacks lors de la création du dictionnaire

static void free_malloced_memory (CFAllocatorRef allocator, const void *value) { 
    free((void*)value); 
} 
static const CFDictionaryValueCallBacks values_callbacks = {0, NULL, free_malloced_memory, NULL, NULL}; 
... 
touchOriginPoints = CFDictionaryCreateMutable(NULL, 0, &kCFTypeDictionaryKeyCallBacks, & values_callbacks); 
... 

Answer 2

Si vous devez renvoyer la valeur malloc'd de la fonction, vous avez transféré la responsabilité de libérer la mémoire à la fonction appelante ou à l'un de ses appelants.

Puisque nous ne pouvons pas voir les fonctions d'appel, nous ne pouvons plus diagnostiquer.

Answer 3

Si vous allez retourner un objet qui est alloué, alors vous devez avoir l'appelant free(), ou bien vous devez utiliser une sorte de garbage collection (pour qu'il soit libéré automatiquement).

Answer 4

vous ne revenez pas en fait un pointeur, la valeur est copiée à une valeur de température quand il est retourné, de sorte que vous n'êtes pas vraiment restituez le allocation du tout, le problème est que vous ne le libérez pas non plus, vous ajoutez l'allocation au dictionnaire et laissez-le là?

est là comme une fonction EndOfTouch? où vous supprimez le toucher du dictionnaire? s'il y a, appelez gratuitement sur votre allocation là et vous devriez être bien