對象引用不設置到對象錯誤的實例

Question

private void btnAddStudent_Click(object sender, EventArgs e) 
    { 
     student[counter] = new Student(txtStudentName.Text, txtStudentSurname.Text,  int.Parse(txtExamMark.Text), counter); 
     counter++; 

    } 

    private void btnAverage_Click(object sender, EventArgs e) 
    { 

     for (int i = counter; i <= counter; i++) 
      MessageBox.Show("" + student[i].Average); 
    } 

我的計劃是給我的錯誤：

Object reference not set to an instance of an object.

我只希望循環運行一次只顯示最後計算平均。如果我這樣做： 即：改變int i = counter到i = 0

private void btnAverage_Click(object sender, EventArgs e) 
    { 

     for (int i = 0; i < counter; i++) 
      MessageBox.Show("" + student[i].Average); 
    } 

然後我的程序工作，但它顯示在MessageBox多次根據所輸入的學生i量，與上次值是正確的平均值。

我用一個叫做Student的類來計算平均值。但這不是問題，因爲正在顯示正確的平均值。

我能做些什麼來解決這個錯誤？

Answer 1

首先，你似乎並不需要一個循環，因爲counter會從counter到counter。

其次，我懷疑你有一個out-by-one的錯誤。

如果您嘗試此操作，會發生什麼情況？

private void btnAverage_Click(object sender, EventArgs e) 
{ 
    if (counter > 0) 
     MessageBox.Show("" + student[counter-1].Average); 
} 

Answer 2

我認爲這個問題是因爲你試圖訪問沒有實例的數組/集合中的一個項目（爲什麼你會收到錯誤）。將計數器設置爲0的原因是student[0]的實例爲Student。

如果你只想得到一個項目，你不需要一個循環，你可以通過傳入數組的索引直接訪問student[i]，即0。

MessageBox.Show("" + student[0].Average); 

Answer 3

您的問題是student[counter]爲空，而student[0]是不是，所以我想你counter沒有正確與student陣列對準。

Answer 4

嘗試

if (student.Length > 0) 
{ 
    MessageBox.Show("" + student[student.Length - 1].Average) 
} 

沒有循環 - 這將只顯示數組中的最後平均值。

Answer 5

一個不太容易出錯的方法是使用一個List<T>而不是數組：

List<Student> students = new List<Student>(); 
private void btnAddStudent_Click(object sender, EventArgs e) 
{ 
    students.Add(Student(txtStudentName.Text, txtStudentSurname.Text, 
       int.Parse(txtExamMark.Text), counter); 
    counter++; 
} 

private void btnAverage_Click(object sender, EventArgs e) 
{ 
    if(students.Any()) 
    { 
     MessageBox.Show("" + students.Last().Average); 
    } 
}