Suchen Sie nach einer einfacheren Objective-C-Lösung

Question

Vielleicht einfacher ist nicht das richtige Wort; knapp könnte es auch tun. Ich bin mir bewusst, dass Objective-C eine ausführliche Sprache ist; Ich frage mich nur, ob es so ausführlich sein muss wie in der Lösung, die ich unten zeige. Ich bin relativ neu in der Sprache.

Dieser Codeabschnitt zählt die Nummer jedes Feldtyps in einem Datensatz gemäß der Generierung einer UITableViewCell mit variabler Höhe, die ein Label pro Feld enthält. Meine Frage ist nicht, wie man das macht - ich habe das schon herausgefunden - ich frage mich nur: gibt es nicht einen einfacheren oder weniger verbindlichen Weg, dies in Objective-C zu tun, mit einem NSDictionary-Objekt? Ist das so kurz, wie es mit einem NSDictionary funktioniert? Lösungen, die andere Objekttypen verwenden, sind ebenfalls willkommen.

Hier ist mein -PartsRecord.countFields-Methode. Ich habe den Code etwas vereinfacht.

-(NSDictionary*) countFields { 
NSDictionary* dict = [[NSDictionary alloc] initWithObjectAndKeys: 
    @"s", [NSNumber numberWithUnsignedInt: [self.struts count]], 
    @"h", [NSNumber numberWithUnsignedInt: [self.headAssemblies count]], 
    @"l", (self.leftVent == nil) ? [NSNumber numberWithUnsignedInt: 0] : 
             [NSNumber numberWithUnsignedInt: 1], 
    @"r", (self.rightVent == nil) ? [NSNumber numberWithUnsignedInt: 0] : 
             [NSNumber numberWithUnsignedInt: 1], 
    nil ]; 
return dict; 

}

Alle Fehler in dem obigen Code sind Fehler in der Transkription, keine echten Fehler im Code.
TIA,
Howard

Answer 1

@"l", [NSNumber numberWithUnsignedInt: self.leftVent ? 1 : 0], 
@"r", [NSNumber numberWithUnsignedInt: self.rightVent ? 1 : 0], 

wird es durch zwei Linien verkürzen

Answer 2

Sie auch [[NSDictionary alloc] initWithObjectsAndKeys:] zu [NSDictionary dictionaryWithObjectsAndKeys:] ändern könnte.

Dies wäre sowieso eine gute Idee, da Ihr Code eine NSDictionary zuweist, ohne sie zu veröffentlichen. -dictionaryWithObjectsAndKeys: gibt ein automatisch freigegebenes Wörterbuch zurück.

Answer 3

-[NSNumber numberWithInt: 0] und -[NSNumber numberWithUnsignedInt: 0] sind die gleiche Anzahl (tatsächlich können sie sogar das gleiche Objekt auf einigen Implementierungen von Cocoa sein).

Answer 4

Sofern Sie nicht beabsichtigen, die Zahlen einzugeben, ist Ihre Objektbestellung falsch. Es sollte Objekt und dann Schlüssel sein.

dict = [NSDictionary dictionaryWithObjectsAndKeys: 
    [NSNumber numberWithUnsignedInt: [self.struts count]], @"s", 
    [NSNumber numberWithUnsignedInt: [self.headAssemblies count]], @"h", 
    [NSNumber numberWithUnsignedInt: self.leftVent ? 1 : 0], @"l", 
    [NSNumber numberWithUnsignedInt: self.rightVent ? 1 : 0], @"r", 
    nil ]; 

Auf diese Weise würde die [self.struts count] Nummer Objekt [dict objectForKey:@"s"] Aufruf zurück.

Answer 5

Muss es ein Wörterbuch sein? Wenn nicht, wäre vielleicht der Rückgriff auf ein einfaches C leichter zu verstehen.

typedef struct { 
    unsigned strutCount; 
    unsigned headAssembliesCount; 
    BOOL leftVent; 
    BOOL rightVent; 
} Fields; 


- (void) countFields:(Fields *) fields 
{ 
    fields->strutCount = [self.struts count]; 
    fields->headAssembliesCount = [self.headAssemblies count]; 
    fields->leftVent = (self.leftVent != nil); 
    fields->rightVent = (self.rightVent != nil); 
} 

Answer 6

Ich denke, dass mit Hilfe von Makros ganz in Ordnung ist Eingabe zu vereinfachen und erhöht die Lesbarkeit

#define SET_KEY_NUMBER(_key, _value) \ 
    [NSNumber numberWithUnsignedInt:_value], _key 
#define SET_KEY_BOOL(_key, _bool) \ 
    [NSNumber numberWithUnsignedInt: (_bool) ? 1 : 0], _key 

-(NSDictionary*) countFields { 
    NSDictionary* dict = [NSDictionary dictionaryWithObjectsAndKeys: 
     SET_KEY_NUMBER(@"s", [self.struts count]), 
     SET_KEY_NUMBER(@"h", [self.headAssemblies count]), 
     SET_KEY_BOOL( @"l", self.leftVent == nil), 
     SET_KEY_BOOL( @"r", self. rightVent == nil), 
     nil ]; 
    return dict; 
} 

... und @ Matt Ball ist richtig, wenn ein Objekt der Rückkehr sollte es seine Autoreleased