Удаление динамически созданных элементов

@Черный ворон · Регистрация: 31.01.2012

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

программа строит график и необходимо нанести на оси координаты. для этого создаю динамически label. однако при изменении границ графика необходимо и изменить положение отметок координатной сетки. старую надо удалить, но у меня она удаляется почему-то избирательно-рандомно=(

весь код:

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
using System;
 
 
using System.Drawing;
using System.Linq;
 
using System.Windows.Forms;
 
namespace lines
{
    public partial class MainForm : Form
    {
        private double maxZ, minZ;
        
        public MainForm()
        {
            InitializeComponent();
            drawOsiX();
            drawOsiY();
        }
 
        private void exitToolStripMenuItem_Click(object sender, EventArgs e)
        {
            Application.Exit();
        }
 
        private double function(double x, double y) {
            double result=0;
            //result = Math.Cos(x + y);
            //result = Math.Tan(x + y);
            result = Math.Cos(x * x + y * y);
            //result = x * x + y * y;
            return result;
        }
 
        private void extremalZ  () { 
            double temp=function(double.Parse(minXBox.Text), double.Parse(minYBox.Text));
            maxZ = temp;
            minZ = temp;
            double ystep = scaleY();
            double xstep = scaleX();
            for (double y = double.Parse(minYBox.Text); y <= double.Parse(maxYBox.Text); y += ystep)
            for (double x = double.Parse(minXBox.Text); x <= double.Parse(maxXBox.Text); x += xstep) {
                temp = function(x, y);
                if (temp > maxZ) maxZ = temp;
                if (temp < minZ) minZ = temp;
            }                        
        }
 
        private double scaleX() {
            double result=0;
            result = (double.Parse(maxXBox.Text)-double.Parse(minXBox.Text))/isolinesBox.Size.Width;
            return result;
        }
 
        private double scaleY()
        {
            double result = 0;
            result = (double.Parse(maxYBox.Text) - double.Parse(minYBox.Text)) / isolinesBox.Size.Height;
            return result;
        }
 
        private void drawOsiX() {
            double xMin=double.Parse(minXBox.Text);
            double xMax=double.Parse(maxXBox.Text);
            int i = 0;
            Label[] xLabels = new Label[(int)xMax - (int)xMin + 1];
            for (double x = xMin; x <=xMax ; x += 1.0) {
                xLabels[i] = new Label();
                xLabels[i].Name = x.ToString();
                xLabels[i].Text = x.ToString();
                xLabels[i].Top = 620;
                xLabels[i].Left = 23 + (int)(i / scaleX());
                this.Controls.Add(xLabels[i]);
                xLabels[i].BringToFront();
                i++;
            }        
        }
 
        private void drawOsiY()
        {
            double yMin = double.Parse(minYBox.Text);
            double yMax = double.Parse(maxYBox.Text);
            int i = 0;
            Label[] yLabels = new Label[(int)yMax - (int)yMin + 1];
            for (double y = yMin; y <= yMax; y += 1.0)
            {
                yLabels[i] = new Label();
                yLabels[i].Name = (-y).ToString();
                yLabels[i].Text = (-y).ToString();
                yLabels[i].AutoSize = true;
                yLabels[i].Left = 0;
                yLabels[i].Top = 90 + (int)(i / scaleY());
                this.Controls.Add(yLabels[i]);
                yLabels[i].BringToFront();
                i++;
            }
        }
 
 
        private void isolinesToolStripMenuItem_Click(object sender, EventArgs e)
        {
            clearToolStripMenuItem_Click(this, e);            
            extremalZ();
            float ystep = (float)scaleY();
            float xstep = (float)scaleX();
            int i = 0, j = 0;
            for (float y = float.Parse(minYBox.Text); y <= float.Parse(maxYBox.Text); y += ystep)
            {
                for (float x = float.Parse(minXBox.Text); x <= float.Parse(maxXBox.Text); x += xstep)
                {
                    double t1 = function(x, y);
                    double t = 255 * ((t1 - minZ) / (maxZ - minZ));
                    if (t < 0) t = 0;
                    if (t > 255) t = 255;
                    int temp = (int)t;
                    Graphics g = isolinesBox.CreateGraphics();
                    SolidBrush sb = new SolidBrush(Color.FromArgb(255, temp, temp, temp));
                    g.FillRectangle(sb, i, isolinesBox.Size.Height-j, 1, 1);
                    i++;
                }
                j++;
                i = 0;
            }
        }
 
        private void isolinesButton_Click(object sender, EventArgs e)
        {
            isolinesToolStripMenuItem_Click(this, e);
        }
 
        private void toolStrip1_ItemClicked(object sender, ToolStripItemClickedEventArgs e)
        {
 
        }
 
        private void clearToolStripMenuItem_Click(object sender, EventArgs e)
        {
            foreach (Label labels in this.Controls.OfType<Label>()) {
                if (labels.Text != "x" | labels.Text != "y")
                {
                    labels.Hide();
                    labels.Dispose();
                }                
            }
            Graphics g = isolinesBox.CreateGraphics();
            SolidBrush sb = new SolidBrush(Color.FromArgb(255, 255, 255, 255));
            g.FillRectangle(sb, 0, 0, isolinesBox.Size.Width, isolinesBox.Size.Height);
            drawOsiX();
            drawOsiY();
        }
    }
}

добавляю label так:

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
private void drawOsiX() {
            double xMin=double.Parse(minXBox.Text);
            double xMax=double.Parse(maxXBox.Text);
            int i = 0;
            Label[] xLabels = new Label[(int)xMax - (int)xMin + 1];
            for (double x = xMin; x <=xMax ; x += 1.0) {
                xLabels[i] = new Label();
                xLabels[i].Name = x.ToString();
                xLabels[i].Text = x.ToString();
                xLabels[i].Top = 620;
                xLabels[i].Left = 23 + (int)(i / scaleX());
                this.Controls.Add(xLabels[i]);
                xLabels[i].BringToFront();
                i++;
            }        
        }

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
private void drawOsiY()
        {
            double yMin = double.Parse(minYBox.Text);
            double yMax = double.Parse(maxYBox.Text);
            int i = 0;
            Label[] yLabels = new Label[(int)yMax - (int)yMin + 1];
            for (double y = yMin; y <= yMax; y += 1.0)
            {
                yLabels[i] = new Label();
                yLabels[i].Name = (-y).ToString();
                yLabels[i].Text = (-y).ToString();
                yLabels[i].AutoSize = true;
                yLabels[i].Left = 0;
                yLabels[i].Top = 90 + (int)(i / scaleY());
                this.Controls.Add(yLabels[i]);
                yLabels[i].BringToFront();
                i++;
            }
        }

удаляю так:

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

C#
1
2
3
4
5
6
7
foreach (Label labels in this.Controls.OfType<Label>()) {
                if (labels.Text != "x" | labels.Text != "y")
                {
                    labels.Hide();
                    labels.Dispose();
                }                
            }

но почему-то перебираются не все созданные мною label's. не могу понять почему...

@adm_loro · 24.02.2013, 03:50

вызвав labels.Dispose() контрол удаляется из коллекции Controls, а в foreach этого делать нельзя, думаю понятно почему.

посмотрел рефлектором как это делается.

C#
1
2
3
4
5
6
7
8
9
10
  protected override void Dispose(bool disposing){
 
.................................
if (this.parent != null)
            {
              this.parent.Controls.Remove(this);
            }
..............................
 
}

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

@ViterAlex · 24.02.2013, 04:21

Ты добавляешь label в массив при создании. Почему бы из него не удалять?

C#
1
Label[] yLabels = new Label[(int)yMax - (int)yMin + 1];

Объяви массив глобально и используй для удаления:

C#
1
2
3
4
5
6
7
8
            for (int i = 0; i<xLabels.Count(); i++)
            {
                xLabels[i].Dispose();
            }
            for (int i = 0; i<yLabels.Count(); i++)
            {
                yLabels[i].Dispose();
            }

А вообще, я бы советовал отказаться от меток и рисовать цифры вместе с осями и графиком. Создаёшь два битмапа: один для осей с цифрами, второй для графика. Рисуешь на них, потом совмещаешь в один битмап и уже этот совмещённый помещаешь в пикчербокс

@aleks_pilipenko · 24.02.2013, 04:37

Только что сделал такой код:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
        private void Form1_Load(object sender, EventArgs e)
        {
            Button[] Butons = new Button[16];
            int i = 0;
            for (int y = 0; y < 4; y++)
            {
                for (int x = 0; x < 4; x++)
                {
                    Butons[i] = new Button();
                    Butons[i].Size = new Size(50, 50);
                    //Butons[i].Click += new EventHandler(Buttons_Click);
                    Butons[i].Location = new Point(12 + x * 56, 41 + y * 56);
                    this.Controls.Add(Butons[i]);
                    Butons[i].Text = ((i++) + 1).ToString();
                }
            }
        }
        private void button1_Click(object sender, EventArgs e)
        {
            foreach (var item in this.Controls)
            {
                if (item is Button) { Button f = (Button)item; if(f.Text!="Удалить") f.Dispose(); }
            }
        }

Удаляет элементы строго через один, причем, я добавил на форму вручную элементы, убрал добавление в Form1_Load, а они все равно удаляются через один. причем при повторном нажатии опять через один, и так пока все не удалятся...
Лучше все таки через перебор массива это делать:

C#
1
2
3
4
            for (int i = 0; i < 16; i++)
            {
                Butons[i].Dispose();
            }

@Vitall · 24.02.2013, 14:23

Правелный порядок удаления контрола .
Сначала он отписывапется от всех событий

C#
1
  label1.Click -=new EventHandler(label1_Click);

потом удаляется из контрола родителя методом

C#
1
this.Controls.Remove(label1);

только после этого вызывается

C#
1
 label1.Dispose();

@aleks_pilipenko · 24.02.2013, 17:50

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

Правильный порядок удаления контрола .

Переписал свою прогу, добавил Remove, а кнопки все равно через одну удаляются, почему?

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
 
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }     
        Button[] Butons = new Button[16];
 
        private void Form1_Load(object sender, EventArgs e)
        {
            int i = 0;
            for (int y = 0; y < 4; y++)
            {
                for (int x = 0; x < 4; x++)
                {
                    Butons[i] = new Button();
                    Butons[i].Size = new Size(50, 50);
                    Butons[i].Location = new Point(12 + x * 56, 41 + y * 56);
                    this.Controls.Add(Butons[i]);
                    Butons[i].Text = ((i++) + 1).ToString();
                }
            }
        }
        private void button1_Click(object sender, EventArgs e)
        {
            foreach (var item in this.Controls)
            {
                if (item is Button) { Button f = (Button)item; if (f.Text != "Удалить") { this.Controls.Remove(f); f.Dispose(); } }
            }
        }
    }
}

@Vitall · 24.02.2013, 18:43

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

Переписал свою прогу, добавил Remove, а кнопки все равно через одну удаляются, почему?

Здесь уже на это вопрос ответели .Нельзя изменять коллекцию во время перечесления
,а я просто добавил принятый порядок удаления !

@aleks_pilipenko · 24.02.2013, 20:04

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

Нельзя изменять коллекцию во время перечИсления

Спасибо, разобрался, переписал вот так, автору в тему будет:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
private void button1_Click(object sender, EventArgs e)
        {
            Button[] Butons = new Button[16];
            int i = 0;
            foreach (var item in this.Controls)
            {
                if (item is Button) { Button f = (Button)item; if (f.Text != "Удалить") { Butons[i] = f; i++; } }
            }
            for (i = 0; i < 16; i++)
            {
                this.Controls.Remove(Butons[i]);
                Butons[i].Dispose();
            }
        }

@Vitall · 24.02.2013, 20:21

Что бы было меншьше гемороя ,автору лучше кинуть на форму панель и наполнить ее лейблэми .После чего их удалять можно одной строчкой

C#
1
this.panel1.Controls.Clear();

@nik_1712 · 27.10.2018, 15:15

Столкнулся с такой же проблемой. Пришел к таком решению проблемы удаления контролов через один. Возможно кому нибудь подойдет.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
        string[] CleanButtons = new string[1];
        void clearToolStripMenuItem_Click(object sender, EventArgs e)
        {
            int count=0;
                foreach(System.Windows.Forms.Control ctr in this.Controls)
                {
                    if(ctr.GetType()==typeof(Button)){
                    CleanButtons[count] = ctr.Name; //при перечислении спросить как его имя и занести имя в массив
                     count++;
                    Array.Resize(ref CleanButtons, count+1);
                    }
                }
                for(int i=0;i<CleanButtons.Length;i++){
                this.Controls.RemoveByKey(CleanButtons[i]); //Затем по имени удалить
                }
             Array.Resize(ref CleanButtons, 1);
        }

@nik_1712 · 28.10.2018, 10:34

Как оказалось, в результате некоторых исключительных случаев, имя контрола бывает пустым ctr.Name="", и понятно, что он не удаляется.

Пришлось дополнить выше приведённый код, чтобы исправить это недоразумение

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
        string[] CleanButtons = new string[1];
        void clearToolStripMenuItem_Click(object sender, EventArgs e)
        {
            int count=0;
                foreach(System.Windows.Forms.Control ctr in this.Controls)
                {
                    if(ctr.GetType()==typeof(Button)){
                       if(ctr.Name==""){
                           ctr.Name = "Name" + count.ToString();//Раз уж он безымянный, то назовём его как нибудь, но без тёзок, если он не один такой безымянный
                    }
                    CleanButtons[count] = ctr.Name; //при перечислении спросить как его имя и занести имя в массив
                     count++;
                    Array.Resize(ref CleanButtons, count+1);
                    }
                }
                for(int i=0;i<CleanButtons.Length;i++){
                this.Controls.RemoveByKey(CleanButtons[i]); //Затем по имени удалить
                }
             Array.Resize(ref CleanButtons, 1);
            CleanButtons[0]="";
        }

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .