ファイルから読み取るときにクラッシュログなしでiPadアプリがクラッシュする

Question

私のプログラムの基本的な構造では、ユーザーが格納されたテキストファイルに対応するUITableViewから項目を選択します。ファイルは配列と辞書に読み込まれます。配列にはキーがあります（私は辞書自体からキーを取得できますが、これは私の質問ではありません）。

ビューは、マスタビューにキーがあるUISplitViewに変更され、ディテールビューには、そのキーに関連付けられているディクショナリ内の項目があります。この場合、ユーザーが回答を選択する一連の「はい/いいえ」の質問です。

私の問題はこれです：UITableView（最初の画面）のセルをクリックすると、正常に動作し、データは完全に読み込まれます。私がUITableViewに戻り、同じセルをもう一度クリックすると、プログラムがクラッシュします。ここで、リードイン-からのファイルの方法は次のとおりです。

-(NSArray *)readFromFile:(NSString *)filePath{ 
    // NSLog(@"Path was: %@", filePath); 
    NSString *file = [[NSString alloc] initWithContentsOfFile:filePath]; 

    // NSLog(@"File was: %@", file); 
    NSScanner *fileScanner = [[NSScanner alloc] initWithString:file]; 

    NSString *held; 
    NSString *key; 
    NSMutableArray *detailStrings; 
    NSMutableArray *keys = [[NSMutableArray alloc] init]; 
    NSMutableDictionary *details = [[NSMutableDictionary alloc] init]; 
    /** 
    This is where the fun stuff happens! 
    **/ 
    while(![fileScanner isAtEnd]){ 
     //Scan the string into held 
     [fileScanner scanUpToString:@"\r" intoString:&held]; 
     NSLog(@"Inside the while loop"); 
     // If it is a character, it's one of the Key points, so we do the management necessary 
     if ([[NSCharacterSet lowercaseLetterCharacterSet] characterIsMember:[[held lowercaseString] characterAtIndex: 0]]){ 
      NSArray *checkers = [[NSArray alloc] initWithArray:[held componentsSeparatedByString:@"\t"]]; 
      NSLog(@"Word at index 2: %@", [checkers objectAtIndex:2]); 

      if(detailStrings != nil){ 
       [details setObject:detailStrings forKey:key]; 
       [detailStrings release]; 
      } 
      NSLog(@"After if statement"); 
      key = [checkers objectAtIndex:2]; 
      [keys addObject:(NSString *) key]; 
      detailStrings = [[NSMutableArray alloc] init]; 
     } 
     else if ([[NSCharacterSet decimalDigitCharacterSet] characterIsMember:[[held lowercaseString] characterAtIndex: 0]]){ 

      NSArray *checkers = [[NSArray alloc] initWithArray:[held componentsSeparatedByString:@"\t"]]; 
      NSLog(@"Word at index 1: %@", [checkers objectAtIndex:1]); 
      [detailStrings addObject:[checkers objectAtIndex:1]]; 
     } 
    } 
    NSLog(@"File has been read in"); 
    [details setObject:detailStrings forKey:key]; 
    NSArray *contents = [[NSArray alloc] initWithObjects:(NSMutableArray *) keys, (NSMutableDictionary *) details, nil]; 

    [detailStrings release]; 
    return contents; 
} 

私は

if(detailStrings != nil) 

文の中のプログラムがクラッシュすることを決定しました。これは、私がやっているはずのメモリ管理が行方不明だからだと思うが、どこが間違っているのか分からないからだ。問題についてのアイデア、または私にログを与えずにクラッシュする理由は何ですか？

Answer 1

detailStringsは、whileループを入力すると初期化されません。メソッド内にNSMutableArray *detailStrings;を宣言すると、detailStringsは自動的にnilに設定されません。ですから、

if (detailStrings != nil) { .. } 

を行うときには、if文を入力し、それが初期化されていないので、あなたがdetailStringsにアクセスすると、それがクラッシュします。

detailStringsは、ループの最初の部分にelseが入ると初期化されません。それもクラッシュを引き起こします。だからあなたの要件に基づいて、いずれかの

NSMutableArray *detailStrings = nil; 

を行うか、whileループに入る前に、それを初期化します。

Answer 2

Deepakは真実を言った。まず、detailStringをnilで初期化する必要があります。

しかし、第二の可能な問題があります：

次のループであなたがnilでメモリの存在しない部分をテストすることがあるので、私は、リリース後にnilを設定することもお勧めします。

if(detailStrings != nil){ 
      [details setObject:detailStrings forKey:key]; 
      [detailStrings release]; 
      detailStrings = nil; 
} 

そして、第3の可能な問題：あなたは、IF文は初めての第二部に移動し、非初期化された配列にのaddObjectにしようとして入ってくるデータから垂れ下がります。

第四（最後の希望）：この時点で

//read in the file 
NSString *file = [[NSString alloc] initWithContentsOfFile:filePath]; 
//create the scanner 
NSScanner *fileScanner = [[NSScanner alloc] initWithString:file]; 

//declare some uninitialized stuff 
NSString *held; 
NSString *key; 
NSMutableArray *detailStrings; 

//initialize some stuff 
NSMutableArray *keys = [[NSMutableArray alloc] init]; 
NSMutableDictionary *details = [[NSMutableDictionary alloc] init]; 

//begin loop 
while(![fileScanner isAtEnd]){ 
    //scan up to a newline 
    [fileScanner scanUpToString:@"\r" intoString:&held]; 
    //see if you scanned a lowercase string 
    if ([[NSCharacterSet lowercaseLetterCharacterSet] characterIsMember:[[held lowercaseString] characterAtIndex: 0]]){ 
     //make an array 
     NSArray *checkers = [[NSArray alloc] initWithArray:[held componentsSeparatedByString:@"\t"]]; 

     //do a check... against an uninitialized value 
     if(detailStrings != nil){ 
      //set a potentially uninitialized value into an array with an uninitialized key 
      [details setObject:detailStrings forKey:key]; 

が、あなたはかなりまずいしている：あなたは「チェッカー」配列

Answer 3

ここで私が見ているものだとメモリリークを持っています。

修正： `detailStrings`は` nil`ある場合

• 適切
• あなたの変数を初期化するには、
• 静的アナライザを実行するには、メモリ管理プログラミングガイドに