即使使用if/else语句，空值也会导致崩溃？

Question

我遇到了最奇怪的问题。我使用下面的代码来显示登录用户的个人资料照片（我正在从Drupal数据库的字段中检索图像）。

.M

//USER PHOTO 

    NSDictionary *user = [[DIOSSession sharedSession] user]; 
    NSString *secondLink = user[@"field_photo_path"][@"und"][0][@"safe_value"]; 

    NSLog(@"This is second link %@", secondLink); 

if([secondLink length]>0) { 


    NSString *ImageURL = secondLink; 
    NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
// self.imageView.image = [UIImage imageWithData:imageData]; 
    self.imageView.image = [[UIImage alloc] initWithData:imageData]; 

    } else { 

    NSString *ImageURL = @"http://url.com/default.png"; 
    NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
    self.imageView.image = [UIImage imageWithData:imageData]; 
} 

当然，如果是secondlink为空（或空的，例如没有照片已上传），我收到以下错误：

*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[__NSArray0 objectForKeyedSubscript:]: unrecognized selector sent to instance 0x14f602fe0'

我想我解决这个问题与if/else语句，但显然不是。我怎样才能解决这个问题？

干杯！

Answer 1

NSString *secondLink = user[@"field_photo_path"][@"und"][0][@"safe_value"]; 

您需要先验证用户字典是否有效，然后再读取它。

if (user[@"field_photo_path"]) { 

    // you can look for [@"und"] here 
    // providing it is a dictionary and not an array 
    if([user[@"field_photo_path"] isKindOfClass:[NSDictionary class]]){ 
     if (user[@"field_photo_path"][@"und"]) { 
      // etc 
     } 
    } 
} 

您可以通过建模User类为NSObject子类减少这种嵌套的代码，并包括在类方法来安全地分析这些数据，让您只在一个地方写这个验证代码。

编辑 正如评论中提到：

如果数据在user[@"field_photo_path"][@"und"]是NSDictionary，而不是一个NSArray你不能[0]访问它，并会得到你的崩溃。您有一个索引（array[0]，array[1]等）和NSDictionary用钥匙（dict[@"name"]，dict[@"und"]等）

Answer 2

解决方案是将检查你的关键的

//用户提供的照片访问NSArray's

NSDictionary *user = [[DIOSSession sharedSession] user]; 
if (user[@"field_photo_path"]) { 


    if ([user[@"field_photo_path"] isKindOfClass:[NSDictionary class]]) { 

     if (user[@"field_photo_path"][@"und"]) { 
      if ([user[@"field_photo_path"][@"und"]isKindOfClass:[NSArray class]]) { 

       if ([user[@"field_photo_path"][@"und"] count]>0) { 

        if (user[@"field_photo_path"][@"und"][0][@"safe_value"]) { 

         if ([user[@"field_photo_path"][@"und"][0][@"safe_value"] isKindOfClass:[NSDictionary class]]){ 

          NSString *secondLink = user[@"field_photo_path"][@"und"][0][@"safe_value"]; 

          NSLog(@"This is second link %@", secondLink); 

          if([secondLink length]>0) { 


           NSString *ImageURL = secondLink; 
           NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
           // self.imageView.image = [UIImage imageWithData:imageData]; 
           self.imageView.image = [[UIImage alloc] initWithData:imageData]; 

          } else { 

           NSString *ImageURL = @"http://url.com/default.png"; 
           NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
           self.imageView.image = [UIImage imageWithData:imageData]; 
          } 
         } 


        } 
       } 

      } 
     } 

    } 
} 

您还可以使用try catch块来避免崩溃

@try { 
    //USER PHOTO 

    NSDictionary *user = [[DIOSSession sharedSession] user]; 
    NSString *secondLink = user[@"field_photo_path"][@"und"][0][@"safe_value"]; 

    NSLog(@"This is second link %@", secondLink); 

    if([secondLink length]>0) { 


     NSString *ImageURL = secondLink; 
     NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
     // self.imageView.image = [UIImage imageWithData:imageData]; 
     self.imageView.image = [[UIImage alloc] initWithData:imageData]; 

    } else { 

     NSString *ImageURL = @"http://url.com/default.png"; 
     NSData *imageData = [NSData dataWithContentsOfURL:[NSURL URLWithString:ImageURL]]; 
     self.imageView.image = [UIImage imageWithData:imageData]; 
    } 
} 
@catch (NSException *exception) { 
    //Handle expecption 
} 
@finally { 

}