iPhone - JSONKit - 働いていますが、私はそれを正しく使用していますか？

Question

私はJSONKitを使用してデシリアライゼーションを少し行いました。それは働いていますが、私が書いたことはかなりかすかな感じです。誰もが改善をお願いします。

//call web service with url and staff id 
    NSString *result = [MobileAppDelegate webServiceExecuteGetSingleWithId:StaffId 
    atUrl:@"XXXXXXXX"]; 

    //create NSDictionary from JSON return type 
    NSDictionary *items = [result objectFromJSONString]; 

    NSArray *ar = (NSArray*)[items objectForKey:@"SummaryJsonResult"]; 

    for(int i = 0; i < [ar count]; i++){ 
     NSDictionary *tmpDict = (NSDictionary*)[ar objectAtIndex:i]; 
     AlertItem *tmpItem = [[AlertItem alloc] init]; 
     tmpItem.Description = [tmpDict objectForKey:@"Description"]; 
     tmpItem.NumItems = [tmpDict objectForKey:@"ItemCount"]; 
     tmpItem.ModuleGroup = [tmpDict objectForKey:@"ModuleGroup"]; 
     tmpItem.StaffID = [tmpDict objectForKey:@"StaffID"]; 
     tmpItem.status - [tmpDict objectForKey:@"Status"]; 
     [array addObject: tmpItem]; 
     [tmpItem release]; 
    } 

Answer 1

これは基本的にどのように行う必要がありますか。あなたは少し物事をクリーンアップすることができます。ファクトリメソッドに辞書からAlertItemを作成

for (NSDictionary *tmpDict in [items objectForKey:@"SummaryJsonResult"]) { 
    ... 

リファクタリング：全ての

+ (AlertItem *)alertItemWithDictionary:(NSDictionary *)dict { 
    AlertItem *item = [[self alloc] init]; 
    ...do the same stuff you do in the loop 
    return [item autorelease]; 
} 

... then in the loop you just do: 
[array addObject:[AlertItem alertItemWithDictionary:tempDict]]; 

Answer 2

まず、それはそう

はfor-inループを使用します同期リクエストを実行しているかのように、通常は悪い考えです。メインスレッドでこれを実行している場合は、リクエストが完了するまですべてのユーザーのやりとりをブロックしています。

あなたのコードでは、不正なデータも処理されません。あなたのアプリケーションは、あなたのJSON内のオブジェクトのいずれかがあなたが期待するものと異なるタイプであればクラッシュします。 "SummaryJsonResult"が配列の代わりに辞書を指している場合は、objectForKey:というメッセージを送信しようとすると「認識できないセレクタ」例外が発生します。 isKindOfClass:またはrespondsToSelector:を使用してこれを防ぐことができます。