Как создать массив экземпляров класса

@DEA7H · Регистрация: 05.10.2011

Author24 — интернет-сервис помощи студентам

Здравствуйте.
Есть класс:

C#

public class Figure
    {
        private int x;
        private int y;
        private string inf;
 
        public int X
        {
            set { x = value; }
            get { return x; }
        }
        public int Y
        {
            set { y = value; }
            get { return y; }
        }
 
        public string INF
        {
            set { inf = value; }
            get { return inf; }
        }
    }

Мне нужно много экземпляров этого класса. Как их создать?

Я подумал сделать так, но ошибка:

C#

Figure [] mas = new Figure [5];
mas[0].X = 11;
mas[0].Y = 12;
mas[0].INF = "pos1";

Подскажите как правильно.

@becky · 14.10.2012, 19:27

C#

 Figure[] mas = new Figure[5];
            mas[0] = new Figure();
            mas[0].X =11;
            mas[0].Y = 12;
            mas[0].INF = "pos1";

@DEA7H · 14.10.2012, 19:29 **[ТС]**

Тьфу ты... Так просто...

@xZooYx · 14.10.2012, 19:35

Можно еще проще (в плане записи):

C#

var mas = new[]
{
    new Figure { X = 11, Y = 12, INF = "pos1" },
    new Figure { X = 11, Y = 12, INF = "pos1" },
    new Figure { X = 11, Y = 12, INF = "pos1" }
};

Будет создан массив типа Figure, состоящий из 3 элементов.

@DEA7H · 14.10.2012, 20:09 **[ТС]**

Еще 1 проблема.
Вот весь код:

C#

namespace ConvexHull
{
    public partial class GeneralForm : Form
    {
        public Element[] masEl = new Element[100];
        public int counter = 0;
        public GeneralForm()
        {
            InitializeComponent();
        }
 
        private void clickedPanel_Click(object sender, EventArgs e)
        {
            if (e is MouseEventArgs)
            {
                masEl[counter] = new Element();
                masEl[counter].posX = ((MouseEventArgs)e).X;
                masEl[counter].posY = ((MouseEventArgs)e).Y;
                masEl[counter].infEl = "Element # " + counter;
 
                this.Invalidate();
 
                counter++;
            }
        }
 
        private void clickedPanel_Paint(object sender, PaintEventArgs e)
        {
            Bitmap bmp = new Bitmap(1, 1);
            bmp.SetPixel(0, 0, Color.Black);
            e.Graphics.DrawImage(bmp, masEl[counter].posX, masEl[counter].posY);
        }
    }
}

Я создаю экземпляр класса в методе "clickedPanel_Click".

Теперь когда я пытаюсь получить доступ полю в другом методе "clickedPanel_Paint" - вылетает ошибка.

Не работает вот эта строчка:

C#

1	e.Graphics.DrawImage(bmp, masEl[counter].posX, masEl[counter].posY);

Ошибка:

Как исправить?

@turbanoff · 14.10.2012, 20:21

У вас в counter количество элементов в массиве, а в c# по умолчанию индексация с нуля. Отсюда и ошибка.

@DEA7H · 14.10.2012, 20:28 **[ТС]**

Сообщение от turbanoff

У вас в counter количество элементов в массиве, а в c# по умолчанию индексация с нуля. Отсюда и ошибка.

Количество элементов тут:

C#

1	public Element[] masEl = new Element[100];

а counter у меня индекс массива:

C#

1	public int counter = 0, temp;

Если написать так:

C#

private void clickedPanel_Click(object sender, EventArgs e)
        {
            if (e is MouseEventArgs)
            {
                temp = counter;
                masEl[counter] = new Element();
                masEl[counter].posX = ((MouseEventArgs)e).X;
                masEl[counter].posY = ((MouseEventArgs)e).Y;
                masEl[counter].infEl = "Element # " + counter;
 
                MessageBox.Show(Convert.ToString(masEl[counter].posX));
 
                this.Invalidate();
 
                counter++;
            }
        }

то:

C#

1	MessageBox.Show(Convert.ToString(masEl[counter].posX));

работает! Поскольку он в:
private void clickedPanel_Click

а в другом методе не пашет, даже тот самый:

C#

1	MessageBox.Show(Convert.ToString(masEl[counter].posX));

я думаю тут в этом проблема... Хотя может и нет, если что то скажите конкретно что исправить.

@xZooYx · 14.10.2012, 20:41

В вашем примере вы в методе clickedPanel_Click инициируете элемент массива с индексом 0 (counter = 0), в конце метода увеличиваете значение counter и в методе clickedPanel_Paint пытаетесь обратиться уже в элементу с индексом 1, а он не инициализирован и равен null, отсюда и ошибка.

@DEA7H · 14.10.2012, 20:53 **[ТС]**

Сообщение от xZooYx

он не инициализирован и равен null, отсюда и ошибка.

Давайте сделаем так:

C#

1	e.Graphics.DrawImage(bmp, masEl[0].posX, masEl[0].posY);

Та-же ошибка!

Добавлено через 7 минут
Все понял! Спасибо. Исправил.
Сделал немножко по тупому но работает:

C#

namespace ConvexHull
{
    public partial class GeneralForm : Form
    {
        public Element[] masEl = new Element[100];
        public int counter = 0, temp = 0;
        public GeneralForm()
        {
            InitializeComponent();
        }
 
        private void clickedPanel_Click(object sender, EventArgs e)
        {
            if (e is MouseEventArgs)
            {
                masEl[counter] = new Element();
                masEl[counter].posX = ((MouseEventArgs)e).X;
                masEl[counter].posY = ((MouseEventArgs)e).Y;
                masEl[counter].infEl = "Element # " + counter;
                
                clickedPanel.Invalidate();
 
                counter++;
            
                temp = 1;
            }
        }
 
        private void clickedPanel_Paint(object sender, PaintEventArgs e)
        {
            if (temp == 1)
            {
                Bitmap bmp = new Bitmap(1, 1);
                bmp.SetPixel(0, 0, Color.Black);
                e.Graphics.DrawImage(bmp, masEl[counter-1].posX, masEl[counter-1].posY);
            }
        }
    }
}

@turbanoff · 14.10.2012, 21:11

зачем еще одна переменная?
можно ведь просто

C#

1	if (counter > 0)

И вообще, пользуйтесь нормальными динамическими коллекциями, List<Element> например

@DEA7H .NET ONLY 71 / 71 / 7 Регистрация: 05.10.2011 Сообщений: 495
	14.10.2012, 19:29 [ТС]	3
	Тьфу ты... Так просто... 0

@turbanoff 4091 / 3825 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 11
	14.10.2012, 20:21	6
	У вас в counter количество элементов в массиве, а в c# по умолчанию индексация с нуля. Отсюда и ошибка. 0

@xZooYx 120 / 101 / 9 Регистрация: 12.06.2012 Сообщений: 322 Записей в блоге: 12
	14.10.2012, 20:41	8
	В вашем примере вы в методе clickedPanel_Click инициируете элемент массива с индексом 0 (counter = 0), в конце метода увеличиваете значение counter и в методе clickedPanel_Paint пытаетесь обратиться уже в элементу с индексом 1, а он не инициализирован и равен null, отсюда и ошибка. 0