インデックスが辞書の配列の境界外にありました

Question

key:valueのようなフォーマットのファイルを読み込み、Dictionaryに追加するアプリがあります。これは小さなファイルではうまくいきますが、65,000行のファイルを読み込もうとするとうまく動作せず、Dictionary.TryAdd()にIndex was outside the bounds of the array.がスローされます。

私のアプリケーションを64ビットアーキテクチャ用にコンパイルし、もapp.configに設定します。

private void LoadFile() 
{ 
    ConcurrentDictionary<string, string> Dict = new ConcurrentDictionary<string, string>(); 

    OpenFileDialog dlgFile = new OpenFileDialog(); 
    dlgFile.Filter = "All Files (*.*)|*.*"; 
    dlgFile.FilterIndex = 1; 

    if (dlgFile.ShowDialog() == DialogResult.OK) 
    { 
     foreach (string line in File.ReadLines(dlgFile.FileName)) 
     { 
      // Index was outside the bounds of the array. 
      Dict.TryAdd(line.Split(':')[0], line.Split(':')[1]); 
     } 
    } 
} 

Answer 1

これはほぼ確実にあなたの行の1つに:が含まれていないため、結果の分割string配列へのインデックス付けは2つの部分ではないので失敗します。あなたがそうのような二つの部分持っていない行をスキップでき

：余談として

foreach (string line in File.ReadLines(dlgFile.FileName)) 
{ 
    var parts = line.Split(':'); 

    if (parts.Length == 2) 
    { 
     var key = parts[0]; 
     var value = parts[1]; 
     Dict.TryAdd(key, value); 
    } 
    else 
    { 
     // log that line was ignored 
    } 
} 

を、あなたは同時アクセスを持っていない限り、ConcurrentDictionaryを使用する必要はありません。あなたのアプリケーションには別の場所があるかもしれませんが、あなたが示したコードには何もありません。

Answer 2

、次のコードセグメントにforeachの本体を変更してみてください：空のエントリを削除

 var parts = line.Split(new char[] { ':' }, StringSplitOptions.RemoveEmptyEntries); 
     if (parts.Length > 1) 
     { 
      Dict.TryAdd(parts[0], parts[1]); 
     } 
     else 
     { 
      // ERROR logging 
     } 

がneccesaryではありません、しかし、あなたが辞書に空のキーや値を追加しませんを確認します。また、辞書には同じキーが含まれていないことを確認することをお勧めします。