NSDictionary stringForKey: и литье [NSNull null] в NSString *

Question

Мне недавно сказали, что кастинг [NSNull null] до (NSString*) был «ужасным».

Однако метод -[NSDictionary stringForKey:] вернет [NSNull null], если ключ не находится в словаре, и если я не выполняю бросок, компилятор кричит на меня.

Я что-то не хватает?

EDIT:

Моя ошибка ... и я думаю, что я мог бы начинаю видеть проблему ...

Как все уже отмечалось, нет stringForKey: метод на NSDictionary, да Я думал о том, по умолчанию пользователя, когда я задал вопрос ... так что я вернулся и посмотрел на то, что я делаю ... вот это:

NSString * someValue = (NSString*)[myDictionary objectForKey:@"key"]; 
if (someValue == (NSString*)[NSNull null]) ... 

, если я не делаю бросок в в этом случае я получаю следующую ошибку:

warning: comparison of distinct Objective-C types 'struct NSNull *' and 'struct NSString *' lacks a cast

Кастинг значение «решает» проблему, и я предпочел бы не написать 10 строк, в которых один будет делать ...

это плохо? Где я столкнулся с проблемами?

ТАКЖЕ:

Словари приходят из библиотеки JSON ... и Нуль- действительные значения в этом случае, поэтому, возможно, это не то, что NSDictionary возвращает их, если ключ отсутствует, а то, что ключ , на самом деле, там, и значение на самом деле является нулевым.

Answer 1

NSDictionary не возвращает экземпляры NSNull, если вы или какой-либо другой источник не поставили их там. Кроме того, NSDictionary не отвечает на -stringForKey:. Возможно, вы думаете о NSUserDefaults?

Если у вас есть экземпляр NSDictionary, и все, что вы хотите сделать, это получить строку из него, вы можете попробовать что-то вроде следующего кода.

NSString *MyGetString(NSDictionary *dict, id key, NSString *fallback) { 
    id result = [dict objectForKey: key]; 

    if (!result) 
     result = fallback; 
    else if (![result isKindOfClass: [NSString class]]) 
     result = [result description]; 

    return result; 
} 

Передаёт строку, как резервный fallback --the функция будет использовать, что если объект не связан с ключом key.

Answer 2

Как показывает код Джонатана, лучше использовать метод isKindOfClass:. Он защитит вас, даже если вы измените метод извлечения объекта и т. Д., И это более точная проверка.

Также помните, что [someValue isKindOfClass:[NSString class]] возвращается YES только тогда, когда someValue не nil И это в NSString, что означает, что также достаточно для тестирования nil значений.

Answer 3

Отливка отличная, потому что NSNullне a NSString. Кастинг не изменяет фактический тип объекта, на который он указывает, он только изменяет отношение компилятора, um, к нему.

В показанном конкретном случае, где только , что вы собираетесь делать с объектом, является прямым испытанием для равенства указателя, ваш подход будет работать, но это источник потенциальных будущих ошибок. Такой хриплый бросок, чтобы отключить предупреждения компилятора, вызывает беспокойство, и его следует избегать, если есть лучший способ.

Реальная проблема, конечно, является предыдущей литой, где NSObject* от objectForKey: отливает прямо на NSString* без вопросов. В настоящее время вы можете уйти от него, потому что считаете, что вам гарантировано получить только строки или NSNull, но что произойдет, если библиотека JSON изменится или просто не сможет выполнить этот контракт? Вы бы действительно лучше делать что-то вроде:

NSObject* someValue = [myDictionary objectForKey:@"key"]; 
NSString* someStr = nil; 
if (someValue == [NSNull null]) 
    ... 
else if ([someValue isKindOfClass:[NSString class]]) 
    someStr = (NSString*) someValue; 
.... 

Answer 4

Хороший тест для проверки NULL, особенно при тестировании проанализированных результатов от сетевых интерфейсов (JSON особенно) является isEqual: метод. Я проверить как наличие, так и для полезной стоимости с использованием ...

NSString *someValue = [response objectForKey:MyAPIKeySomeValue]; 
if (!!someValue && ![someValue isEqual:[NSNull null]]) { 
    // optionally test for class type 
    // use someValue 
} 
else { 
    // error out for required values; skip for optional values 
} 

На более общем плане, если вы ожидаете JSON в определенном формате, и API не удовлетворяет этому требованию, вы могли бы считают, что кандидат на обертку в @try/@catch. Затем вы можете обрабатывать ненормальный/незаконный формат ответа и устранять жесткие сбои в отношении недопустимых форматов данных.

Если формат не определен, вы можете позволить своим процедурам разбора более грациозно скомпоновать, построив содержательный NSError.

Answer 5

Вы можете использовать расширение этой категории, чтобы обернуть эту логику вверх. это на GitHub

NSDictionary + Verified.h:

// NSDictionary+Verified.h 
// 
// Created by alexruperez on 08/05/13. 
// Copyright (c) 2013 alexruperez. All rights reserved. 
// 

#import <Foundation/Foundation.h> 

@interface NSDictionary (Verified) 

- (id)verifiedObjectForKey:(id)aKey; 

@end 

NSDictionary + Verified.m:

NSDictionary+Verified.m 

#import "NSDictionary+Verified.h" 

@implementation NSDictionary (Verified) 

- (id)verifiedObjectForKey:(id)aKey 
{ 
    if ([self objectForKey:aKey] && ![[self objectForKey:aKey] isKindOfClass:[NSNull class]]) return [self objectForKey:aKey]; 
    return nil; 
} 

@end