Ссылка на объект не указывает на экземпляр ошибки объекта, используя для цикла

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.

Во-вторых, я подозреваю, что у вас есть ошибка «один за другим».

Что произойдет, если вы попробуете это?

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