ios - 目标 C：如何在没有大量 if 语句的情况下检查对象类型

Question

我正在尝试将我的text字符串设置为 URL。以下代码有效，但我觉得我可以重构它以使其看起来更整洁。

NSString *text = @“”;

id json = [NSJSONSerialization JSONObjectWithData:[data dataUsingEncoding:NSUTF8StringEncoding] options:0 error:nil];

if ([json isKindOfClass:[NSDictionary class]]) {

   id data = json[@“data”];

    if ([data isKindOfClass:[NSDictionary class]]) {
       id value = data[@"value"];

       if ([value isKindOfClass:[NSArray class]]) {
            id url = [value valueForKey:@"url"];

          if ([url isKindOfClass:[NSString class]]) {
             text  = url;
         }
       }
     }
   } 

到目前为止，它已经完成了整个“厄运之山”，我想知道如何在不使用这么多 if 语句的情况下检查对象类型是否正确。任何提示或建议表示赞赏。

编辑：这是我的代码的精简版，但概念是一样的。

标签： iosobjective-cif-statementrefactoring

Answer 1

在我看来，有两种方法可以让它看起来更整洁并忽略 if-else-nesting-hell。

• 使用return.

  NSString *text = @“”;
  
  id json = [NSJSONSerialization JSONObjectWithData:[data dataUsingEncoding:NSUTF8StringEncoding] options:0 error:nil];
  
  if (![json isKindOfClass:[NSDictionary class]]) {
    return;
  }
  id data = json[@“data”];
  
  if (![data isKindOfClass:[NSDictionary class]]) {
    return;
  }
  id value = data[@"value"];
  
  if (![value isKindOfClass:[NSArray class]]) {
    return;
  }
  id url = [value valueForKey:@"url"];
  
  if (![url isKindOfClass:[NSString class]]) {
    return;
  }
  text  = url;
  

• 创建一个检查类类型并返回安全值的通用方法

  - (id)safeValueFromObject:(id)object forKey:(NSString *)key class:(Class)valueClass {
    if (![object respondsToSelector:@selector(valueForKey:)]) {
      return [[valueClass alloc] init];
    }
  
    id result = [object valueForKey:key];
    return [result isKindOfClass:valueClass] ? result : [[valueClass alloc] init];
  }
  

  利用

  NSString *text = @"";
  
  id json = [NSJSONSerialization JSONObjectWithData:[data dataUsingEncoding:NSUTF8StringEncoding] options:0 error:nil];
  id data = [self safeValueFromObject:json forKey:@"data" class:NSDictionary.class];
  id value = [self safeValueFromObject:data forKey:@"value" class:NSArray.class];
  id url = [self safeValueFromObject:value forKey:@"url" class:NSString.class];
  text  = url;