Index war außerhalb der Grenzen des Arrays auf einem Wörterbuch

Question

Ich habe eine App, die eine Datei lädt, die wie key:value formatiert ist, und fügt sie dann zu Dictionary. Das funktioniert gut mit kleinen Dateien, aber wenn ich versuche, eine Datei zu laden, die 65.000 Zeilen hat, wird es nicht funktionieren und wirft Index was outside the bounds of the array. auf Dictionary.TryAdd().

Ich kompiliere meine App für 64-Bit-Architekturen und ich setzte auch die <gcAllowVeryLargeObjects enabled="true" /> in der 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

Dies ist fast sicher, weil eine Ihrer Zeilen : nicht enthält, so dass die Indexierung in das resultierende Split string Array fehlschlagen wird, da es nicht 2 Teile dazu gibt.

Sie Linien überspringen könnte, die zwei Teile, wie nicht so haben:

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 
    } 
} 

Als beiseite, gibt es keine Notwendigkeit ConcurrentDictionary zu verwenden, wenn Sie den gleichzeitigen Zugriff haben. Möglicherweise befindet sich an anderer Stelle in Ihrer Anwendung, aber es gibt keine in dem Code, den Sie angezeigt haben.

Answer 2

Bitte, versuchen Sie die foreach Körper auf die folgende Codesegment ändern:

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

Entfernen leere Einträge nicht neccesary ist jedoch macht es sicher, dass Sie nicht mit leeren Schlüssel oder Werte zum Wörterbuch hinzufügen würde. Ich würde auch vorschlagen, um sicherzustellen, dass das Wörterbuch nicht den gleichen Schlüssel enthält.