Форум программистов, компьютерный форум, киберфорум
C# Windows Forms
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.93/15: Рейтинг темы: голосов - 15, средняя оценка - 4.93
30 / 29 / 15
Регистрация: 28.09.2011
Сообщений: 449

Класс для рисования треугольников. Предупреждение "Полю нигде не присваивается значение"

29.03.2014, 13:42. Показов 3304. Ответов 7
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
сделал класс для рисования треугольников
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
class Triangle
    {
        Graphics Graph;//тут ошибка
        Pen MyPen;
        int st = 1, x = 1, y = 1, x2 = 1, y2 = 1;
 
        public Triangle(int st, int x, int y, int x2, int y2)
        {
            this.st = st;
            this.x = x;
            this.y = y;
            this.x2 = x2;
            this.y2 = y2;
        }
        
        public void DrawTri(int st, int x, int y, int x2, int y2, MouseEventArgs e)
        {
            if (st == 0)
            {
                x = e.X;
                y = e.Y;
                st++;
            }
            else if (st == 1)
            {
                x2 = e.X;
                y2 = e.Y;
                Graph.DrawLine(MyPen, x, y, e.X, e.Y);
                st++;
            }
            else if (st == 2)
            {
                Graph.DrawLine(MyPen, x2, y2, e.X, e.Y);
                Graph.DrawLine(MyPen, x, y, e.X, e.Y);
                st = 0;
            }
        }
вызываю его
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
    public partial class Form1 : Form
    {
        Graphics Graph;
        Pen MyPen;
 
        int st = 0;
        int x=1, y=1, x2=1, y2=1;
        public Form1()
        {
            InitializeComponent();
            Graph = CreateGraphics();
            MyPen = new Pen(Color.Blue, 2);
        }
 
        private void Form1_MouseDown(object sender, MouseEventArgs e)
        {
            Triangle res = new Triangle(st, x, y, x2, y2);
            res.DrawTri(st, x, y, x2, y2, e);
        }
    }
но приложение не работает, так как в классе выдает такую ошибку: Полю "kr.Triangle.Graph" нигде не присваивается значение, поэтому оно всегда будет иметь значение по умолчанию null
и то же самое про MyPen. Что здесь необходимо указать?
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
29.03.2014, 13:42
Ответы с готовыми решениями:

Полю нигде не присваивается значение, поэтому оно всегда будет иметь значение null
Пишу программу которая должна png картинки преобразовывать в матрицу (матрица заносится в массив).Т.к. картинки имеют разный размер...

Полю Form1. clicks нигде не присваивается значение, поэтому оно всегда будет иметь значение 0
Делаю игру кликер, не могу никак сделать сохранение. В форм клоседе есть параметр int clicksp и сейв в лоаде никак не могу понять что...

Полю не присваивается значение
Создаю игру. Осталось реализовать рейтинг, но столкнулась с проблемой, очки не начисляются за победу. Начисление очков выглядит таким...

7
 Аватар для netBool
325 / 304 / 173
Регистрация: 16.11.2010
Сообщений: 1,069
Записей в блоге: 9
29.03.2014, 14:32
Тебе в конструктор Triangle нужно помимо координат передавать цвет ручки (Pen) и объект Graphics, примерно так:
C#
1
2
3
4
5
6
7
8
9
10
        public Triangle(int st, int x, int y, int x2, int y2, Pen p, Graphics g)
        {
            this.st = st;
            this.x = x;
            this.y = y;
            this.x2 = x2;
            this.y2 = y2;
            Graph = g;
            MyPen = p;
        }
И при вызове:
C#
1
           Triangle res = new Triangle(st, x, y, x2, y2,Graph,MyPen );
Скорее всего так
0
30 / 29 / 15
Регистрация: 28.09.2011
Сообщений: 449
29.03.2014, 15:16  [ТС]
нет, все равно не работает.
0
 Аватар для tarasalk
1992 / 1216 / 440
Регистрация: 13.06.2013
Сообщений: 4,115
29.03.2014, 16:25
Все работает. Просто код кривой.
Вот ты создаешь объект res класса Triangle. Пишешь ему st, x, y, x2, y2. А потом в методе DrawTri опять эти же параметры передаешь, это бессмысленно.
Потом в самом начале ты пишешь st=0. Смотрим класс Triangle, если st=0, то запоминаем координаты нажатия, НО ничего не рисуем, метод на этом заканчивается.
Вообще, класс весь неверный.
Я так понимаю нужно нарисовать треугольник там куда кликнули?Можно например так:
Класс треугольник. Конструктор должен принимать объект класса Graphics, примерно так как описал netBool, только без всяких левых параметров. Также есть метод Draw. Этот метод должен принимать точку Point, эта точка куда вы кликнули мышкой. Дальше метод рисует три линии, относительно этой точки. Эта точка может быть например центром треугольника, или вершиной, в общем как захочешь. При желании можно усложнить, в метод Draw можно передавать кисточку, длины треугольника итд...
0
 Аватар для netBool
325 / 304 / 173
Регистрация: 16.11.2010
Сообщений: 1,069
Записей в блоге: 9
29.03.2014, 16:26
я перепутал Graph и MyPen надо поменять местами здесь:
C#
1
Triangle res = new Triangle(st, x, y, x2, y2,MyPen ,Graph  );
0
30 / 29 / 15
Регистрация: 28.09.2011
Сообщений: 449
29.03.2014, 22:33  [ТС]
netBool, да это понятно. но все равно криво. что имел ввиду tarasalk я не особо понял.
0
 Аватар для tarasalk
1992 / 1216 / 440
Регистрация: 13.06.2013
Сообщений: 4,115
30.03.2014, 08:09
С какого места не понятно?
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
class Triangle
    {
        Graphics Graph;//тут ошибка
        Pen MyPen;
        int st = 1, x = 1, y = 1, x2 = 1, y2 = 1;//здесь не нужно задавать значения
 
        public Triangle(int st, int x, int y, int x2, int y2)
        {
            this.st = st;//переменная st не нужна вообще, нужна только одна точка
            this.x = x;
            this.y = y;
            this.x2 = x2;
            this.y2 = y2;
        }
        
        public void DrawTri(int st, int x, int y, int x2, int y2, MouseEventArgs e)//здесь тоже не нужны все эти переменные, только одна точка
        {
            if (st == 0)
            {
                x = e.X;
                y = e.Y;
                st++;
            }
            else if (st == 1)
            {
                x2 = e.X;
                y2 = e.Y;
                Graph.DrawLine(MyPen, x, y, e.X, e.Y);
                st++;
            }
            else if (st == 2)
            {
                Graph.DrawLine(MyPen, x2, y2, e.X, e.Y);
                Graph.DrawLine(MyPen, x, y, e.X, e.Y);
                st = 0;
            }
        }
Добавил несколько комментариев. В первую очередь тебе надо почитать про конструкцию if else if. Судя по коду, ты ее не правильно понимаешь. Код не читается строчка за строчкой. Сначала проверяется условие st==0, если верно то выполняется
C#
1
2
3
x = e.X;
y = e.Y;
st++;
Если не верно и st==2, то выполняется
C#
1
2
3
4
x2 = e.X;
y2 = e.Y;
Graph.DrawLine(MyPen, x, y, e.X, e.Y);
st++;
Повторяю, только если st не равен 0, то только тогда будет проверяться условие st==1. Ты не можешь выполнить сразу и if(st==0) {...} и сразу же else if(st==1) {...}. Слово else говорит о том, что делать в ином случае, т.е. если st не равен 0. То что там написано st++; это не играет роли, так как если код дойдет до туда, то все что идет после else игнорируется.

Дальше нужно очень внимательно перечитать про классы, конструкторы, объекты. В общем про ООП.
Потом нужно прочитать про класс Point. Вкратце- это точка, для которой ты можешь задать координаты. Это более правильно, чем передавать всякие x,y ,x2 ,y2. Пользоваться им можно так:
C#
1
2
3
4
5
6
7
8
9
10
11
Point clickPoint1=new Point(e.X,e.Y);//создали объект класса точка, задали ему координаты клика мыши.
Point clickPoint2=new Point();//по умолчанию точка с координатами 0,0. 
clickPoint2.X=100;Зададим ему координаты
clickPoint2.Y=200;
res.Draw(clickPoint1, clickPoint2);//передали точку в метод Draw объекта res
 
//в самом методе нужно прописать
public void Draw(Point clickPoint1, Point clickPoint2)
{
Graph.DrawLine(MyPen, clickPoint1, clickPoint2);//нарисовали линию между двумя точками
}
Пусть клик мыши будет центром фигуры, тогда тебе нужны всего лишь найти три точки-вершины треугольника и нарисовать три линии. Простейшая геометрия.

Потом я подозреваю что отладкой пользоваться ты не умеешь. Как это делается. Вот у тебя ошибка на какой то строке, ты нажимаешь мышкой на эту строку и нажимаешь F9, строка выделяется красным. Потом запускаешь программу, и она выполнится до этой строке. В это время ты можешь задерживать курсор над кодом и будет появляться различная информация. Например значение переменной. Так гораздо легче найти ошибку.

Ну вроде все разжевал.
1
 Аватар для netBool
325 / 304 / 173
Регистрация: 16.11.2010
Сообщений: 1,069
Записей в блоге: 9
31.03.2014, 20:25
Согласен с tarasalk, назначения классов не понятны.
Просто я глянул, что
C#
1
        Graphics Graph;//тут ошибка
ну и указал, в чем она...
Если уж говорить о самой реализации, то прежде всего я бы сделал прорисовку в событии Paint, а не в MouseDown...

Счас, момент, попробую у себя изобразить то, что сделал автор..

Добавлено через 23 минуты
Хм.. вопроизвел, проникся так сказать...
В общем tarasalk все правильно подсказывает... тут много замечаний. Но все их озвучивать не вижу смысла...
Помимо if else if нужно еще почитать, как работают события, классы, параметры классов... как бы азы, азбуку, так сказать... Без этой азбуки не поймешь, что говорит tarasalk... Он практически написал тебе весь код (правда для одной линии),
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
31.03.2014, 20:25
Помогаю со студенческими работами здесь

Не присваивается значение полю
Здравствуйте. Почему полю не присваивается значение? Есть класс самолетов и их конструктор, присваивающий значение полям. Пытаюсь прочитать...

Не присваивается значение полю?
Всем доброго времени суток. У меня возникла проблемма с кодом: Private Sub Form_Open(Cancel As Integer) DoCmd.Maximize Dim rst As...

Не присваивается значение полю в бд
Связываю два объекта через @ManyToOne и @OneToMany, две таблицы, связывание через axisid Но когда я грубо прогонял тестовым кодом, но...

Полю структуры не присваивается значение
Последовательность, в которой вызываются процедуры и описание ошибки: 1) _private_tls_hkdf_extract() вызывает hmac_init(&hmac,...

Возвращаемое значение метода, присваивается полю (автоматическому свойству). Но почему срабатывает сам метод?
Добрый день...подскажите пожалуйста почему так? код : class Program { static void Main(string args) ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
8
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer
8Observer8 08.02.2026
Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств
8Observer8 07.02.2026
Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru