Ссылка на объект не указывает на экземпляр класса

Question

Хотел сделать класс(Box), в котором поля(name, age, length) вводились бы с form-ы, а потом находилось бы максимальное значение поля lenght и через MessageBox отправлялось бы сообщение имени самого высокого человека.

     public Box(string name, int width, int length, int weight, int age) => NewPers = name;
    public int Len //как я понял, здесь я добавляю свойство, чтобы обращаться к приватному параметру length с других блоков программы 
    {
        get {
            return length;
        }
        set {
            length = value; 
        }
    }
    public string NewPers 
    {
        get => name[counter];
        set
        {
            name[counter] = value;
    }
}


}
private void button1_Click(object sender, EventArgs e)
    {
    Box[] NameBox = new Box[5]; //создаю объекты класса Box
    for (int i = 0; i < 5; i++)
    {
        if (NameBox[i] != null) //если значение не равно нулю, то продолжаем
            continue;

        NameBox[i] = new Box(textBox1.Text, int.Parse(textBox2.Text), int.Parse(textBox3.Text), int.Parse(textBox4.Text), int.Parse(textBox5.Text));
            break;
    }
    for (int j = 0; j <= NameBox.Length; j++)
    {
        if (max < NameBox[j].Len)
        {
            max = NameBox[j].Len; 
        }
    }
    MessageBox.Show( $"Максимальный рост у человека {max}");
}

}

P.S. Вообщем я позавчера решил познакомиться поближе, пока еще мало что понимаю => возможно, вообще ерунду пишу. Не забывайте про комментарии, пожалуйста

Answer 1

Для простоты я ваш класс Box урезал до двух свойств (остальные там просто не использовались) будет не сложно их добавить. Во первых зачем вы каждый раз создаете массив Box[] NameBox = new Box[5]; ? цикл заполнения тоже очень странный, да и не нужный в принципе. Вообще проще использовать список, советую их изучить. Сделайте вторую кнопку "сброс" подклчите к ней обработчик button2_Click для сброса массива.

• свойства можно объявлять без переменных(их создаст компилятор);

• для сравнения удобно использовать шаблон IComparable;

• конвертировать типы желательно через TryParse с проверкой на ошибку;

• ф-я Max() - из linq, если не знакомы очень советую изучить, это очень мощный инструмент;

   private int _curIndex;
   private Box[] _boxes = new Box[5];
  private void button1_Click(object sender, EventArgs e)
   {
       if (_curIndex == _boxes.Length)
       {
           MessageBox.Show($"Массив полностью заполнен");
           return;
       }
       if (!int.TryParse(textBox2.Text, out var len))
       {
           MessageBox.Show($"Рост указан не верно");
           return;
       }
   _boxes[_curIndex] = new Box(textBox1.Text, len);
   _curIndex++;
  
   var maxHighBox = _boxes.Max();
  
   MessageBox.Show($"Максимальный рост у человека {maxHighBox.Name} - {maxHighBox.Len}");
  
  }
  private void button2_Click(object sender, EventArgs e)
   {
       _curIndex = 0;
       _boxes = new Box[5];
   }
  public class Box : IComparable<Box>
   {
       public int Len { get; }
   public string Name { get; }
  
   public Box(string name, int len)
   {
       Len = len;
       Name = name;
   }
  
   public int CompareTo(Box other) =&gt; Len.CompareTo(other.Len);
  
  }
  

Answer 2

После создания массива все его элементы будут null - поэтому нет смысла проверять в цикле

NameBox[i] != null

это выражение всегда вернет false

break - прерывает выполнение цикла. В данном случае цикл всегда прерывается после первой итерации.

Таким образом инициализируется всегда только нулевой элемент.

Поэтому во втором цикле попытка взять

NameBox[j].Len

у любого элемента с индексом больше 0 будет приводить к указанной ошибке.

Answer 3

Ссылка на объект не указывает на экземпляр класса - вы обращаетесь к объекту который не инициализирован

вот как должно быть

По порядку сам класс

class Human
{
    public string Name { get; set; }       
    public double Width { get; set; }
    public double Height { get; set; }
    public double Weight { get; set; }
    public int Age { get; set; }
//ctor empty
public Human()
{

}

//ctor with params
public Human(string name, string width, string height, string weight, string age)
{
    Name = name;
    Width = Convert.ToDouble(width);
    Height = Convert.ToDouble(height);
    Weight = Convert.ToDouble(weight);
    Age = Convert.ToInt32(age);
}

//ctor with params for textbox
public Human(TextBox name, TextBox width, TextBox height, TextBox weight, TextBox age)
{
    Name = name.Text;
    Width = Convert.ToDouble(width.Text);
    Height = Convert.ToDouble(height.Text);
    Weight = Convert.ToDouble(weight.Text);
    Age = Convert.ToInt32(age.Text);
}



}

теперь его реализация

public partial class Form1 : Form
    {
        List<Human> _humans = new List<Human>();//создаем список с нашим классом
        public Form1()
        {
            InitializeComponent();
        }
    private void Form1_Load(object sender, EventArgs e)
    {

        Human human1 = new Human(&quot;imja1&quot;, &quot;10&quot;, &quot;110&quot;, &quot;10&quot;, &quot;10&quot;);// создаем экземпляр класса
        _humans.Add(human1);//добавляем его в список 
        Human human2 = new Human(&quot;imja2&quot;, &quot;20&quot;, &quot;120&quot;, &quot;20&quot;, &quot;20&quot;);
        _humans.Add(human2);
        Human human3 = new Human(&quot;imja3&quot;, &quot;30&quot;, &quot;130&quot;, &quot;30&quot;, &quot;30&quot;);
        _humans.Add(human3);
        Human human4 = new Human(&quot;imja4&quot;, &quot;40&quot;, &quot;140&quot;, &quot;40&quot;, &quot;40&quot;);
        _humans.Add(human4);
        Human human5 = new Human(&quot;imja5&quot;, &quot;50&quot;, &quot;150&quot;, &quot;50&quot;, &quot;50&quot;);
        _humans.Add(human5);


    }

    //кнопка добавить человека в список
    private void button1_Click(object sender, EventArgs e)
    {

        Human human = new Human(textBox1,textBox2,textBox3,textBox4,textBox5);//тут указываем переменные Texbox
        _humans.Add(human);

    }
    //кнопка найти человака с самым большим ростом 
    private void button2_Click(object sender, EventArgs e)
    {
        Human result = _humans.Find(x =&gt; x.Height == _humans.Max(z =&gt; z.Height));// ишем человека с саммым высоким ростом

        MessageBox.Show(result.Name);// показываем его имя
    }
}