Question de fonctionnalité NSDictionary

Question

Donc j'essaye d'enregistrer des tableaux dans un NSDictionary à la volée. Laissez-moi vous montrer le code et expliquer ce qui se passe.

for (int x= 0; x <[appDelegate.people count]; x++) { 
    Person *aPerson = [[Person alloc] init]; 
    aPerson = [appDelegate.people objectAtIndex:x]; 
    if ([appDelegate.groupedBusiness objectForKey:aPerson.business_name] == nil) { 
     NSMutableArray *newBusiness = [[NSMutableArray alloc] init]; 
     //if the business does not exist in the dict, add the person to the business and add it to dict. 

        [newBusiness addObject:aPerson]; 

     [appDelegate.groupedBusiness setObject:newBusiness forKey:aPerson.business_name]; 

     [newBusiness release]; 
     newBusiness = nil; 
     //NSLog(@"%@", appDelegate.groupedBusiness); 
    } else { 
     NSMutableArray *existingBusiness= [appDelegate.groupedBusiness objectForKey:aPerson.business_name]; 
     [existingBusiness addObject:aPerson]; 
        //THIS IS THE LINE I AM NOT SURE ABOUT!!! 
     [appDelegate.groupedBusiness setObject:existingBusiness forKey:aPerson.business_name]; 

     [existingBusiness release]; 
     existingBusiness = nil; 
     //NSLog(@"%@", appDelegate.groupedBusiness); 
    } 

} 

Bon, le appDelegate a un tableau de « peuple » qui a un tas d'attributs sur une personne. J'essaie de mettre en place un dictionnaire pour les trier par leur nom d'entreprise. Je fais cela en créant un tableau et en l'enregistrant dans le dictionnaire avec le nom de l'entreprise comme clé. Chaque itération de la boucle que je vérifie pour voir si la clé existe, si c'est le cas, retirez le tableau existant, ajoutez la personne que vous vérifiez et réenregistrez-la dans le dictionnaire. Cependant, cela ne semble pas se produire. Y a-t-il un comportement exotique dans la classe NSDictionary qui empêcherait cela? J'ai versé sur la page Web de la classe et ne trouve rien. Désolé si c'est une question noobie, j'essaie toujours de comprendre les classes objectives-c. Merci!

Answer 1

Pourquoi lancez-vous l'entreprise existante? Vous ne créez pas d'objet, prenez simplement le pointeur d'un tableau. Lorsque vous invoquez release, retainCount devient 0 et les objets deallocs.

Il suffit de retirer les deux lignes suivantes:

[existingBusiness release]; 
existingBusiness = nil; 

et tout devrait bien fonctionner.

Answer 2

[appDelegate.groupedBusiness setObject:existingBusiness forKey:aPerson.business_name]; 

    [existingBusiness release]; 
    existingBusiness = nil; 

Tout devrait être enlevé. L'objet existant est déjà dans la dict, et il s'agit d'un objet mutable - lorsque vous ajoutez une personne, cela sera reflété dans le dictionnaire et c'est le même objet que vous avez affaire. En dehors de cela, vous avez quelques fuites de mémoire comme le souligne Daniel.

Answer 3

Vous êtes trop compliqué, pour ne pas mentionner quelques fuites.

for (Person *aPerson in appDelegate.people) { 
    NSMutableArray *business = [appDelegate.groupedBusiness objectForKey:aPerson.business_name]; 
    if (!business) { 
     business = [NSMutableArray array]; 
     [appDelegate.groupedBusiness setObject:business forKey:aPerson.business_name]; 
    } 

    [business addObject:aPerson]; 
} 

Answer 4

Pas une réponse, mais quelques problèmes de style de codage.

Utilisez itération rapide si vous n'avez pas besoin de l'index:

for (Person *aPerson in appDelegate.people) { 

utilisation constructeurs de proximité; il rend votre code plus lisible (souvenez-vous de retirer le « libération » à la fin):

NSMutableArray *newBusiness = [NSMutableArray arrayWithObject:aPerson]; 

Évitez la logique en double, si possible:

NSMutableArray * business = [appDelegate.groupedBusiness objectForKey:aPerson.business_name; 
if (!business) { 
    business = [NSMutableArray array]; 
} 
[business addObject:aPerson]; 
[appDelegate.groupedBusiness setObject:business forKey:aPerson.business_name]; 

Le « setObject: existingBusiness » appel ne ne change rien à part de gaspiller des cycles CPU, mais dans le cas ci-dessus, il rend le code un peu plus lisible.