ArgumentOutOfRangeException

Question

私はC＃でプロジェクトを始めました。オブジェクト（CLAというクラス）があります。 CLAはセット（リスト）を作成します。

List<Cla>で機能し、HighValue,LowValueなどが必要です。しかしArgumentOutOfRangeExceptionはここに入っています。

コンストラクタでメソッドを呼び出すことはできますか？このコードは正しいですか？あなたの問題である

class Program 
{ 
    static void Main(string[] args) 
    { 
     List<Cla> ClaList = new List<Cla>(); 
     ClaList.Add(new Cla("a", 10.3, "Priznak1", 1)); 
     ClaList.Add(new Cla("b", 3.3, "Priznak1", 1)); 
     ClaList.Add(new Cla("c", 7.3, "Priznak1", 1)); 
     ClaList.Add(new Cla("d", 9.3, "Priznak1", 1)); 
     ClaList.Add(new Cla("e", 8.3, "Priznak1", 1)); 

     CritListSmall NewTest = new CritListSmall(ClaList); 
     Console.WriteLine("ddd {0}", NewTest.HighValue); 
    } 
} 


class CritListSmall 
{ 
    public double HighValue; 
    public double LowValue; 

    public CritListSmall(List<Cla> p) 
    { 
     HighValue = p[0].ObjectValue; 
     LowValue = p[0].ObjectValue; 

     int NumberElements; 

     NumberElements = p.Count; 
     for (int i = 1; i <= NumberElements; i++) 
     { 
      double m = p[i].ObjectValue; //Exception occurs here 
      if (HighValue < m) 
       HighValue = m; 
     } 
     for (int l = 1; l <= NumberElements; l++) 
     { 
      double n = p[l].ObjectValue; //and here 
      if (LowValue > n) 
       LowValue = n; 
     } 
    } 


    public class Cla 
    { 
     public string ObjectName; 
     public double ObjectValue; 
     public string PriznakName; 
     public int ClassNumber; 

     public Cla(string on, double ov, string pn, int cn) 
     { 
      ObjectName = on; 
      ObjectValue = ov; 
      PriznakName = pn; 
      ClassNumber = cn; 
     } 

     public double CritValue; 
     public double ExpValue; 

     public bool Outlier; 
     public double AbsoluteValue; 
    } 
} 

Answer 1

for(int i = 1; i <= NumberElements; i++) 

。それは：

for(int i = 0; i < NumberElements; i++) 

for(int l = 0; l < NumberElements; l++) 

どうしてですか？ NumberElementsは、実施例10のために、Pのカウントです。しかしインデックスはC＃配列のインデックス0から9

Answer 2

にありません1、0で開始するので、あなたはこのようにあなたのループを変更する必要があります。

for (int i = 0; i < NumberElements; i++) 

そして：

for (int l = 0; l < NumberElements; l++) 

Answer 3

C＃のコレクションでは、ゼロベースのインデックスを持っています。リストに5つのアイテムがある場合、そのインデックスは0,1,2,3,4になります。だからではなく、

for(int i = 1; i <= NumberElements; i++) 

のあなたは

for(int i = 0; i < NumberElements; i++) 
      ^^
    from zero and less than count of items