Probleme mit der .Split() - Methode

Question

Dieser Code wirft die Ausnahme, "Index war außerhalb der Grenzen des Arrays". Sollte dies nicht einfach jeden der aufgeteilten Daten in den vorgesehenen Array-Slot hinzufügen?

while (input != null) 
{ 
    string[] splitInput = inputLine.Split(); 
    EmpNum = int.Parse(splitInput[0]); 
    EmpName = (splitInput[1]); 
    EmpAdd = (splitInput[2]); 
    EmpWage = double.Parse(splitInput[3]); 
    EmpHours = double.Parse(splitInput[4]); 
    inputLine = (myFile.ReadLine()); 
    Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 
} 

ein bisschen Um zu klären, ich bin Lesen von Daten aus einer einfachen Textdatei, die Mitarbeiterdaten (Name, Anschrift, Stunden, Mitarbeiternummer, Löhne) hat.

Ich habe meine gesamte Hauptmethode für Klarheit hinzugefügt.

using System; 
using System.IO; 

class Program 
{ 
static void Main() 
{ 

    //declare an array of employees 
    Employee[] myEmployees = new Employee[10]; 

    //declare other variables 
    string inputLine; 
    string EmpName; 
    int EmpNum; 
    double EmpWage; 
    double EmpHours; 
    string EmpAdd; 

    //declare filepath 
    string environment =   System.Environment.GetFolderPath(System.Environment.SpecialFolder.Personal) + "\\"; 

    //get input 
    Console.Write("\nEnter a file name in My Documents: "); 
    string input = Console.ReadLine(); 
    string path = environment + input; 
    Console.WriteLine("Opening the file..."); 

    //read file 
    StreamReader myFile = new StreamReader(path); 
    inputLine = (myFile.ReadLine()); 

    //split input 
    while (inputLine != null) 
    { 

     string[] splitInput = inputLine.Split(); 
     EmpNum = int.Parse(splitInput[0]); 
     EmpName = (splitInput[1]); 
     EmpAdd = (splitInput[2]); 
     EmpWage = double.Parse(splitInput[3]); 
     EmpHours = double.Parse(splitInput[4]); 
     Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 
    } 

    Console.ReadLine(); 
}//End Main() 
}//End class Program 

Answer 1

Vielleicht wird diese Version für zusätzliche Kredite gut sein :) Aber im Ernst ich bin nicht versuchen, hier zu zeigen - es ist nur, dass selbst ist, obwohl es ein Lern ​​Beispiel ist, wenn Sie einen Job zu bekommen und die Aufgabe gegeben werden Wenn Sie zum Beispiel Code schreiben, der eine CSV-Datei liest, möchten Sie nicht, dass sie abstürzt und Sie schlecht aussehen lassen. Sie werden sich also einen Gefallen tun, um einige Schritte zu kennen, um sie robuster zu machen.

Hinweis - Dies ist kein Versuch, eine Debatte über die perfekte Art und Weise zu beginnen, um Ihr Beispiel zu programmieren - nur versuchen, ein paar Tricks zu zeigen, von denen ich weiß, dass sie hilfreich sind. Ich hoffe, es hilft.

  StreamReader myFile = new StreamReader("TextFile1.txt"); 
      int lineNumber = 0; 
      while (!myFile.EndOfStream) 
      { 
       // Read the next line. 
       string inputLine = myFile.ReadLine(); 
       lineNumber++; 

       // Extract fields line. 
       string[] splitInput = inputLine.Split(); 

       // Make sure the line has the correct number of fields. 
       if (splitInput.Length == 5) 
       { 
        // Parse and validate each field. 

        if (!int.TryParse(splitInput[0], out EmpNum)) 
        { 
         Console.WriteLine("could not parse int " + splitInput[0] + " on line " + lineNumber); 
         continue; 
        } 

        EmpName = (splitInput[1]); 

        EmpAdd = (splitInput[2]); 

        if(!double.TryParse(splitInput[3], out EmpWage)) 
        { 
         Console.WriteLine("could not parse double " + " on line " + lineNumber); 
         continue; 
        } 

        EmpHours = double.Parse(splitInput[4]); 

        if (!double.TryParse(splitInput[4], out EmpHours)) 
        { 
         Console.WriteLine("could not parse double: " + " on line " + lineNumber); 
         continue; 
        } 

        // Output 
        Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 
       } 
       else 
       { 
        Console.WriteLine("Expecting 5 items from split opertation but got " + splitInput.Length + " on line " + lineNumber); 
       } 
      } 
      myFile.Close(); 

TextFile1.txt

1 2 3 4 5 
6 7 8 f9 10 
11 12 

Programm Ausgabe

test 1,5,5 
could not parse double: on line 2 
Expecting 5 items from split opertation but got 2 on line 3 

Answer 2

Überprüfen Sie die Zeichenfolge können Sie nicht mehr als 5 Elemente in der Eingabe bekommen und einige Zeichen in Split-Methode

ändern bieten die inputLine.Split()-inputLine.Split(','), wenn Sie Element Trennung durch Komma

Ihre Eingaben werden wie "erste", "zweite", "dritte", "weiter", "fünfte"

Answer 3

1) Sollte nicht die input sein?

2) Fügen Sie für jedes Array-Element einen Null-Check hinzu, bevor Sie es verwenden.

Auch denke ich,

while (input != null) 
    { 
     string[] splitInput = inputLine.Split(); 
     EmpNum = int.Parse(splitInput[0]); 
     EmpName = (splitInput[1]); 
     EmpAdd = (splitInput[2]); 
     EmpWage = double.Parse(splitInput[3]); 
     EmpHours = double.Parse(splitInput[4]); 
     inputLine = (myFile.ReadLine()); 
     Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 
    } 

sollte

while (input != null) 
    { 
     inputLine = (myFile.ReadLine()); 
     string[] splitInput = inputLine.Split(); 
     EmpNum = int.Parse(splitInput[0]); 
     EmpName = (splitInput[1]); 
     EmpAdd = (splitInput[2]); 
     EmpWage = double.Parse(splitInput[3]); 
     EmpHours = double.Parse(splitInput[4]); 

     Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 

}

Zuerst Lesen aus der Datei sein inputLine = (myFile.ReadLine()); und dann den Split-Betrieb durchführen ...

3) Wie von @Aaron Anodide vorgeschlagen, addin ga-Check für Länge sollte wie der Trick ..

etwas tun ..

inputLine = (myFile.ReadLine()); 
string[] splitInput = inputLine.Split(); 
if(splitInput!=null && splitInput.length ==5) 
{ 
EmpNum = int.Parse(splitInput[0]); 
     EmpName = (splitInput[1]); 
     EmpAdd = (splitInput[2]); 
     EmpWage = double.Parse(splitInput[3]); 
     EmpHours = double.Parse(splitInput[4]); 
     Console.WriteLine("test {0},{1},{2}", EmpNum, EmpWage, EmpHours); 
} 

Answer 4

einen Haltepunkt der Linie hinzufügen, nachdem Sie den Eingang aufgeteilt, dann können Sie die Maus über das resultierende Array und klicken Sie auf das Pluszeichen . Auf diese Weise können Sie genau sehen, wie die Daten aufgeteilt werden. Dies ist besonders hilfreich, wenn versteckte Zeichen vorhanden sind, die den Split auslösen (\ n, \ t, \ r).

Answer 5

Sie haben eine Zeile, die nicht genügend Elemente enthält. Überprüfen Sie die Länge des Feldes, bevor die Elemente zu lesen:

string[] splitInput = inputLine.Split(); 
if (splitInput.Length >= 5) { 
    EmpNum = int.Parse(splitInput[0]); 
    EmpName = (splitInput[1]); 
    EmpAdd = (splitInput[2]); 
    EmpWage = double.Parse(splitInput[3]); 
    EmpHours = double.Parse(splitInput[4]); 
} else { 
    // not enough items - show an error message or something 
} 

Auch sind die Überprüfung Sie die Variable input statt inputLine im where, aber das ist nicht die Ursache für den Fehler, die Sie erhalten. Wenn Sie bis zum Ende der Datei lesen, erhalten Sie eine NULL-Referenzausnahme, wenn Sie versuchen, die NULL-Referenz in der Teilung zu verwenden.

Answer 6

Sie haben ein paar Probleme. Das erste Problem ist das Split(). Sie müssen inputLine.Split() zu inputLine.Split(',') ändern. Jetzt rufen Sie die Überladung von System.String.Split(params char[]) an und da Sie keine zu spaltenden Zeichen angeben, wird die gesamte Zeichenfolge zurückgegeben.

Die anderen Probleme (als CS Student), sollten Sie wirklich an Ihren Namenskonventionen und Fehlerprüfung arbeiten. Der Code ist ziemlich spröde und wird sehr leicht brechen. Sie sollten früh anfangen, gute Softwareentwicklungspraktiken zu erlernen und qualitativ hochwertigen Code zu schreiben.

using (FileStream fstream = new FileStream("path", FileMode.Open)) 
using (StreamReader reader = new StreamReader(fstream)) { 
    string line; 

    while (!reader.EndOfStream && (line = reader.ReadLine()) != null) { 
     string[] data = line.Split(','); 

     if (data.Length < 5) { 
      // You will have IndexOutOfRange issues 
      continue; // skip processing the current loop 
     } 

     int employeeNumber; 
     string employeeName; 
     string employeeAddress; 
     double employeeWage; 
     double employeeHours; 

     // Will be used to check validity of fields that require parsing into a type. 
     bool valid; 

     valid = int.TryParse(data[0], out employeeNumber); 

     if (!valid) { 
      // employee number is not parsable 
     } 

     employeeName = data[1]; 
     employeeAddress = data[2]; 

     valid = double.TryParse(data[3], out employeeWage); 

     if (!valid) { 
      // employee wage is not parsable 
     } 

     valid = double.TryParse(data[4], out employeeHours); 

     if (!valid) { 
      // employee hours are not parsable 
     } 
    } 
}