.Split（）メソッドで問題が発生しました

Question

このコードでは、「インデックスが配列の境界外です」という例外がスローされています。これは分割された各データを指定されたアレイスロットに追加するだけでいいですか？

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

ビットを明確にするために、私は、従業員データ（名前、住所、時間、従業員番号、賃金を）持っている、単純なテキストファイルからデータを読んでいます。

わかりやすくするために私の主な方法をすべて追加しました。

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

たぶん、このバージョンでは、余分な信用のために良いだろう:)真剣に、私はここに披露するつもりはないよけれども - それは学習の例だにもかかわらず、それだけだ、あなたが仕事を得ると、タスクを与えられている場合たとえば、CSVファイルを読み込むコードを書く場合は、クラッシュさせて見栄えを悪くしないようにして、より堅牢にするためのいくつかのステップについて知っておくとよいでしょう。

注 - これは、あなたの例をコードする完璧な方法についての議論を開始しようとする試みではありません。それが助けてくれることを願って。その他の情報が必要な場合

  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 

プログラム出力

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

あなたが入力中に5つの要素を取得することはできませんあなたの文字列をチェックして、分割方法でいくつかの文字を提供

変更inputLine.Split()からinputLine.Split(',')あなたは

コンマ

して入力を要素を分離している場合は、意志

Answer 3

1）inputはinputLineである必要がありますか？1つは、「1」、「2」、「3」、

2）使用する前に、すべての配列要素にnullチェックを追加します。

また、私は

...

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

はまずinputLine = (myFile.ReadLine());を使用してファイルから読み込まれた後、分割操作を行う

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); 

}

する必要があり、推測します3）@Aaron Anodideによって提案されているように、addinトリックを行う必要があります長さのためのGAチェック..

何かのような..

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

は、あなたが入力を分割した後、結果の配列かけてすることができますマウス、ブレークポイントにラインを追加し、プラス記号をクリックしてください。この方法で、データの分割方法を正確に把握できます。これは、分割（\ n、\ t、\ r）を捨てる隠し文字がある場合に特に役立ちます。

Answer 5

十分な品物が含まれていない行があります。項目を読む前に、配列の長さを確認してください：

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 
} 

はまた、あなたがwhere内の変数inputの代わりinputLineをチェックしているが、それはあなたが得るエラーの原因ではありません。ファイルの最後まで読み込んだ場合、分割でヌル参照を使用しようとするとヌル参照例外が発生します。

Answer 6

あなたにはいくつか問題があります。最初の問題はSplit（）です。 inputLine.Split()からinputLine.Split(',')に変更する必要があります。今はSystem.String.Split(params char[])のオーバーロードを呼び出しており、分割する文字を指定していないので、文字列全体が返されます。

その他の問題（CS学生の方）では、命名規則とエラーチェックを実際に行う必要があります。コードはかなり脆く、非常に簡単に壊れます。優れたソフトウェアエンジニアリングの習得を学び、高品質のコードを書くためには、早めに始めるべきです。

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