Подскажите,почему не рисует прямоугольник с заданными параметрами?

@_Lis_ · Регистрация: 16.12.2016

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

Вот само задание:
Создать меню с командами Size, Color, Paint, Quit.
Команда Paint недоступна. При выборе команды Quit приложение
завершается.
При выборе команды Size открывается диалоговое окно, содержащее:
• два поля типа TextBox для ввода длин сторон прямоугольника;
• группу из трех флажков (Red, Green, Blue) типа CheckBox;
• кнопку типа Button.
Обеспечить возможность:
• введение длины сторон прямоугольника в пикселях в поля ввода;
• выбора его цвета с помощью флажков.
После установки параметров команда Paint становится доступной.
При выборе команды Paint в главном окне программы выводится
прямоугольник заданного размера и сочетание цветов или выдается
сообщение, если введенные размеры превышают размер окна.
Я его реализовала,но после задания параметров и выбора команды Paint прямоугольник не рисует,что я делаю неправильно,подскажите пожалуйста.

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
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
 
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        int height;
        int width;
        Color color;
        private void Form1_Load(object sender, EventArgs e)
        {
 
        }
 
        private void MenuItemQuit_Click(object sender, EventArgs e)
        {
            Application.Exit();
        }
 
        private void MenuItemSize_Click(object sender, EventArgs e)
        {
            this.IsMdiContainer = true;
            Form2 newMdiChild = new Form2();
            newMdiChild.MdiParent = this;
            newMdiChild.Show();
            height = newMdiChild.GetHeight;
            width = newMdiChild.GetWidth;
            color = newMdiChild.RectangleColor;
            
        }
 
        
        
        private void MenuItemPaint_Click(object sender, EventArgs e)
        {
            if (width > this.Width || height > this.Height)
            {
                MessageBox.Show("Введенные размеры превышают размер окна");
            }
            else
            {
                Graphics g = this.CreateGraphics();
                Pen pen = new Pen(color, 3f);
                g.DrawRectangle(pen, 0, 30, width, height);
            }
        }
    }
}

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
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
 
namespace WindowsFormsApplication1
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }
 
 
        int width;
        int height;
        Color color;
        private void InputSize_Click(object sender, EventArgs e)
        {
            width = int.Parse(MyWidth.Text);
            height = int.Parse(MyHeight.Text);
            if (Red.Checked)
                color = Color.Red;
            else if (Green.Checked)
                color = Color.Green;
            else if (Blue.Checked)
                color = Color.Blue;
        }
 
        public int GetWidth
        {
            get { return width; }
        }
        public int GetHeight
        {
            get { return height; }
        }
        public Color RectangleColor
        {
            get { return color; }
        }
 
 
    }
 
    }

WindowsFormsApplication1.rar

@ViterAlex · 01.05.2017, 18:42

Введённые во второй форме значения не присваиваются. В этом коде:

C#
1
2
3
4
5
6
7
8
9
10
11
private void MenuItemSize_Click(object sender, EventArgs e)
{
    this.IsMdiContainer = true;
    Form2 newMdiChild = new Form2();
    newMdiChild.MdiParent = this;
    newMdiChild.Show();
    height = newMdiChild.GetHeight;
    width = newMdiChild.GetWidth;
    color = newMdiChild.RectangleColor;
            
}

7,8 и 9 строки выполнятся сразу, а не после закрытия формы, как ты, вероятно, думала. Лучше всего эти параметры считывать непосредственно перед рисованием

@_Lis_ · 01.05.2017, 20:07 **[ТС]**

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

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
 private void MenuItemSize_Click(object sender, EventArgs e)
        {
            this.IsMdiContainer = true;
            Form2 newMdiChild = new Form2();
            newMdiChild.MdiParent = this;
            newMdiChild.Show();
            height = newMdiChild.GetHeight;
            width = newMdiChild.GetWidth;
            color = newMdiChild.RectangleColor;
 
        }
 
 
 
        private void MenuItemPaint_Click(object sender, EventArgs e)
        {
            Form2 newMdiChild = new Form2();
            
            height = newMdiChild.GetHeight;
            width = newMdiChild.GetWidth;
            color = newMdiChild.RectangleColor;
 
            if (width > this.Width || height > this.Height)
            {
                MessageBox.Show("Введенные размеры превышают размер окна");
            }
            else
            {
                Graphics g = this.CreateGraphics();
                Pen pen = new Pen(color, 3);
                g.DrawRectangle(pen, 0, 30, width, height);
            }

@ViterAlex · 01.05.2017, 20:39

_Lis_, нет, так тоже неправильно. Если ты вызываешь форму задания параметров, то эти параметры потом нужно куда-то сохранить. Сейчас у тебя так:
MenuItemSize_Click создаёт форму для параметров, показывает её и присваивает полям width, height, color пустые значения. После этого, чтобы ты в форме параметров ни поменяла, основная форма об этом не узнает.
MenuItemPaint_Click создаёт форму для параметров, не показывает её и присваивает полям width, height, color пустые значения.
Основная форма опять ничего не знает.
Как бы я сделал? При вызове формы параметров я бы подписался на её событие FormClosing и в нём прочитал бы введённые значения. Это во-первых.
Во-вторых, чтобы что-то нарисовать на форме, она не должна быть IsMdiContainer, поэтому перед рисованием это свойство нужно установить в false.
Ну и в-третьих, рисовать нужно в контексте рисования, а не просто создавая объект Graphics.
Подводя итог, можно переписать твои методы так, добавив метод рисования:

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
private void MenuItemSize_Click(object sender, EventArgs e)
{
    this.IsMdiContainer = true;
    Form2 newMdiChild = new Form2 { MdiParent = this };
    //При закрытии формы считываем введённые параметры
    newMdiChild.FormClosing += (o, args) =>
    {
        height = newMdiChild.GetHeight;
        width = newMdiChild.GetWidth;
        color = newMdiChild.RectangleColor;
    };
    newMdiChild.Show();
}
 
//Метод рисования
protected override void OnPaint(PaintEventArgs e)
{
    base.OnPaint(e);
    using (Pen pen = new Pen(color, 3f))
    {
        e.Graphics.DrawRectangle(pen, 0, 30, width, height);
    }
}
 
private void MenuItemPaint_Click(object sender, EventArgs e)
{
    IsMdiContainer = false;
    if (width > this.Width || height > this.Height)
    {
        MessageBox.Show("Введенные размеры превышают размер окна");
    }
    else
    {
        //Вызов рисования
        Invalidate();
    }
}

Не самый идеальный код, но чтобы сделать его идеальным, нужно менять архитектуру приложения.

@_Lis_ · 01.05.2017, 21:08 **[ТС]**

очень доступно объяснили.

большое спасибо.

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

@_Lis_ 1 / 1 / 0 Регистрация: 16.12.2016 Сообщений: 53
	01.05.2017, 21:08 [ТС]
	очень доступно объяснили. большое спасибо. 0

Подскажите,почему не рисует прямоугольник с заданными параметрами?

Решение