C＃並行配列と順次配列を使用した混乱

Question

私はイントロコースを取っており、メソッドと配列に導入されるまでは大きな問題はありませんでした。私は、ユーザー入力郵便番号を取ることができる配列の配列検索を実行する方法を理解するのが難しいです。郵便番号のテキストファイル内にあるかどうか確認してください。 もしそうなら、別のテキストファイル "AGI Text File"内にあるインデックスサブスクリプトを確認し、AGI番号を私のagiOutputLabelに出力する必要があります。

誰かが私の誤解を解消したり、私の教科書よりも理にかなっていると私に説明してくれれば、私はそれを感謝します。

using System; 
using System.Collections.Generic; 
using System.ComponentModel; 
using System.Data; 
using System.Drawing; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 
using System.Windows.Forms; 
using System.IO; 

namespace AGI_Florida 
{ 
    public partial class Form1 : Form 
    { 
     public Form1() 
     {   
      InitializeComponent(); 

     } 
     // Creating an array method 
     private void SetZipArray(int[] zips) 
     { 

     } 
     private void SetAGIArray(double[] agi) 
     { 

     } 

     private void FindAGI(int[] zips, int agi) 
     { 
      bool found = false; 
      int index = 0; 
      int position = -1; 

      while (!found && index < zips.Length) 
      { 
       if (zips[index] == agi) 
       { 
        found = true; 
        position = index; 
       } 
       index++; 
      } 
      return; 
     } 



     private void getAgiButton_Click(object sender, EventArgs e) 
     { 
      try 
      { 
       // Initializing the array with 917 spaces 
       int[] zips = new int[917]; 
       int[] agi = new int[917]; 

       // Pulling array from text file 
       StreamReader inputFile = File.OpenText("AGI_FL_ZipCodes-    1.txt"); 
       // Setting index to 0 so the loop knows to stop at 917 spaces 
       int index = 0; 
       // Running loop 
       while ((index < zips.Length) && (!inputFile.EndOfStream)) 
       { 
        // Creating the index from the zip and parsing it 
        zips[index] = int.Parse(inputFile.ReadLine()); 
        index++; 
       } 
       inputFile.Close(); 

       int zipInput; 


       if (int.TryParse(zipInputTextBox.Text, out zipInput)) 
       { 
        if (zipInput > 32003 && zipInput < 34997) 
        { 
         if (zipInput != -1) 
        { 
          MessageBox.Show(agi.ToString()); 

        } 

        } 
        else 
        { 
         MessageBox.Show("Please enter a valid zip code"); 
        } 

       } 


       else 
       { 
        MessageBox.Show("Please enter a valid zip code"); 

        zipInputTextBox.Text = " "; 
        zipInputTextBox.Focus(); 
       } 

      } 
      catch (Exception ex) 
      { 
       MessageBox.Show(ex.Message); 
      } 

     } 
     private void getAverageAgiButton_Click(object sender, EventArgs e) 
     { 
      // initialize new array 
      double[] agi = new double[917]; 

      StreamReader inputFile = File.OpenText("AGI_FL-1-1.txt"); 
      // Setting index to 0 so the loop knows to stop at 917 spaces 
      int index = 0; 

      while ((index < agi.Length) && (!inputFile.EndOfStream)) 
      { 
       // Creating the index from the zip and parsing it 
       agi[index] = double.Parse(inputFile.ReadLine()); 
       index++; 
      } 
      inputFile.Close(); 

      // create a variable sum 
      double sum = 0; 

      // create a loop to read through the text file 
      for (index = 0; index < agi.Length; index++) 
      { 
       // sum the index of values 
       sum += agi[index]; 
      } 

      // perform calculations 
      double averageAgi = sum/(double) agi.Length; 
      // print to label 
      agiLabel.Text = averageAgi.ToString("c"); 

     } 
    } 
    } 

Answer 1

まず、コーディングにようこそ！

改善の機会はたくさんあります。いくつかのコードを少しきれいにします。

マジックナンバー。

既知の長さの配列を作成し、その中のunknowデータを持つファイルに移動し、それらが整列することを期待するのではなく、ファイル内のデータの長さ

var filePath = "AGI_FL_ZipCodes-1.txt"; 
var allLines = File.ReadAllLines(filePath); //Nicer version of File.OpenText +ReadLine+EndOfStream+whileloop 
int[] zips = new int[allLines.Length]; //instead of hard coded magic value of 917; 

解析と

ループは、しかし、今、あなたはまだ読んで、すべての値を解析する必要があります。

for (int i = 0; i < allLines.Length; i++) 
{ 
    var line = allLines[i]; 
    zips[i] = int.Parse(line); //Chance of an error here is the file is invalid :-/ 
} 

条件チェック

あなたは大丈夫です、条件付きチェックif (zipInput > 32003 && zipInput < 34997)を持っています。しかし、私たちのコードが書かれているよりも頻繁に読んでいることを考えると、何が起こっているのかが視覚的に分かりやすい形でフォーマットするのは良いことです。同じ方法を指しているすべての比較演算子を文の端に境界値を移動したことで、私たちは 32003と（排他的）34997

if (32003 < zipInput && zipInput < 34997) 

デッドコードの間zipInput値を望むことがわかります

次に、zipInputが-1に等しくないかどうかを確認します。これはその直前のチェックのために不要です。 32003〜34997の間のすべての値は-1に等しくありません。

agi変数も宣言し、使用しないでください。あなたは

あなたFindAGI方法はかなり良い見て、配列内の値を検索このライン

int[] agi = new int[917]; 

を削除することができます。しかし、私はあなたが探している値を返す関数になるように "アップグレード"します。

private int IndexOf(int[] source, int value) 
{ 
    for (int i = 0; i < source.Length; i++) 
    { 
     if (source[i] == value) 
     { 
      return i; 
     } 
    } 
    return -1; 
} 

要素が見つかるとすぐに終了することに注意してください。一致しない場合、-1を返します（これはかなり標準的な動作です）。

これはあなたの手助けになるかもしれませんが、今はこのコードの実際の使用場所はわかりません。FindAGI（現在はIndexOf）は参照されていないようです。