Укажите на "ошибки" в моём коде (аналог скриптового окна)

@Hopeco · Регистрация: 18.03.2014

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

Пытаюсь реализовать что-то похожее на систему скриптов в движке CryEngine 3 (Тык)
Но уже начал ~~сходить с ума,~~ путаться в коде и многое другое.
Привожу код.

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
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
using System;
using System.Collections.Generic;
using System.Drawing;
using System.Drawing.Drawing2D;
using System.Windows.Forms;
 
namespace BuilderScripts
{
    public partial class Form1 : Form
    {
        private static Pen blackPen = new Pen(Color.FromArgb(255, 0, 0, 0));
        private static Font fontArial10 = new Font("Arial", 10);
        private List<Object> objects = new List<Object>();
 
        private Dictionary<string, Object> dict = new Dictionary<string, Object>();
 
        private bool mouseDown = false;
        private MouseButtons button = MouseButtons.None;
        private int startX, startY;
 
        public struct SInput
        {
            public readonly int ID;
            public readonly string name;
 
            public SInput(int ID, string name)
            {
                this.ID = ID;
                this.name = name;
            }
        }
        public struct SOutput
        {
            public readonly int ID;
            public readonly string name;
 
            public SOutput(int ID, string name)
            {
                this.ID = ID;
                this.name = name;
            }
        }
 
        public class Object
        {
            public string title;
            public Rectangle rect;
 
            public List<SInput> inputs;
            public List<SOutput> outputs;
 
            int x, y;
            int width, height;
 
            public Object(string title, List<SInput> inputs, List<SOutput> outputs, int x, int y)
            {
                this.title = title;
 
                this.inputs = inputs;
                this.outputs = outputs;
 
                this.x = x;
                this.y = y;
            }
            public void Draw(Graphics gr)
            {
                height = 15 * inputs.Count + 15;
 
                SizeF size = gr.MeasureString(title, fontArial10);
                foreach(SInput input in inputs)
                {
                    SizeF tmp = gr.MeasureString(input.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
                width = (int)size.Width;
 
                size = new SizeF(0, 0);
                foreach (SOutput output in outputs)
                {
                    SizeF tmp = gr.MeasureString(output.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
 
                for (int i = 0; i < inputs.Count; i++)
                    gr.DrawString(inputs[i].name, fontArial10, new SolidBrush(Color.Black), x, y + (i + 1) * 15);
                for (int i = 0; i < outputs.Count; i++)
                    gr.DrawString(outputs[i].name, fontArial10, new SolidBrush(Color.Black), x + width, y + (i + 1) * 15);
 
                width += (int)size.Width;
 
                gr.DrawRectangle(blackPen, x, y, width, 15);
                gr.DrawString(title, fontArial10, new SolidBrush(Color.Black), x, y);
 
                rect = new Rectangle(x, y + 15, width, 15 * inputs.Count);
                gr.DrawRectangle(blackPen, rect);
            }
        }
 
        public Form1()
        {
            InitializeComponent();
 
            dict.Add("echo", new Object("echo", new List<SInput>() { new SInput(objects.Count, "text") }, new List<SOutput>() { new SOutput(objects.Count, "done") }, 0, 0));
            dict.Add("echo2", new Object("echo2", new List<SInput>() { new SInput(objects.Count, "text") }, new List<SOutput>() { new SOutput(objects.Count, "done") }, 0, 0));
 
            foreach (string key in dict.Keys)
                contextMenuStrip1.Items.Add(key);
        }
 
        void DrawArrow(Graphics gr, Point start, Point end)
        {
            float x1 = start.X, y1 = start.Y, x2 = end.X, y2 = end.Y;
            // вычисляем угол, под которым стрелка повёрнута против часовой
            // вычисляем длину стрелки
            float angle = (float)Math.Atan2(y2 - y1, x2 - x1);
            angle = angle * 180f / (float)Math.PI; // переводим в градусы
            float len = (float)Math.Sqrt((y2 - y1) * (y2 - y1) + (x2 - x1) * (x2 - x1));
            // сохраняем старое состояние Graphics, чтобы потом его восстановить
            GraphicsState state = gr.Save();
            gr.SmoothingMode = SmoothingMode.HighQuality;
            // трансформация: сдвиг к точке start и поворот на угол
            gr.TranslateTransform(x1, y1);
            gr.RotateTransform(angle);
            using (Pen pen = new Pen(Color.Red, 2f))
            {
                int arrowLength = 10; // длина окончания стрелки
                int arrowWidth = 5; // ширина окончания стрелки
 
                // рисуем стрелку в горизонтальном положении
                gr.DrawLine(pen, 0, 0, len, 0);
                gr.DrawLine(pen, len, 0, len - arrowLength, arrowWidth);
                gr.DrawLine(pen, len, 0, len - arrowLength, -arrowWidth);
            }
            // восстанавливаем старое состояние Graphics (убираем наши трансформации)\
            gr.Restore(state);
        }
 
        private void Form1_Paint(object sender, PaintEventArgs e)
        {
            if (mouseDown)
            {
                if (button == MouseButtons.Left)
                {
                    bool contains = false;
                    foreach (Object obj in objects)
                        if (obj.rect.Contains(startX, startY))
                            contains = true;
 
                    if (contains)
                        DrawArrow(e.Graphics, new Point(startX, startY), PointToClient(Cursor.Position));
                }
            }
            if (objects.Count > 0)
                foreach (Object obj in objects)
                    obj.Draw(e.Graphics);
        }
 
        private void Form1_MouseMove(object sender, MouseEventArgs e)
        {
            if (button == MouseButtons.Left)
                Refresh();
        }
 
        private void Form1_MouseDown(object sender, MouseEventArgs e)
        {
            button = e.Button;
 
            mouseDown = true;
 
            startX = e.X;
            startY = e.Y;
 
            Refresh();
        }
 
        private void contextMenuStrip1_ItemClicked(object sender, ToolStripItemClickedEventArgs e)
        {
            Object tmp;
            foreach(string key in dict.Keys)
                if (e.ClickedItem.ToString() == key && dict.TryGetValue(key, out tmp))
                    objects.Add(new Object(tmp.title, tmp.inputs, tmp.outputs, startX, startY));
 
            Refresh();
        }
 
        private void Form1_MouseUp(object sender, MouseEventArgs e)
        {
            startX = e.X;
            startY = e.Y;
 
            if (e.Button == MouseButtons.Right)
                contextMenuStrip1.Show(MousePosition, ToolStripDropDownDirection.Right);
 
            button = e.Button;
 
            mouseDown = false;
 
            Refresh();
        }
    }
}

Требует исправить все повторения, недочёты.

@Storm23 · 07.10.2015, 21:45

Hopeco,
1) Графика мигает.
2) Стрелки рисуются только в момент перетягивания. Я так понимаю, связи должны фиксироваться, и стрелка должна связывать два элемента. Этого нет.
3) Отрисовка стрелок - переусложена. Зачем там все эти вращения канвы? Стрелка ж элементарно рисуется методом DrawLine.
4) Окошки с текстом тягать нельзя.
5) Классы модели почему-то внутри формы.

@Hopeco · 08.10.2015, 17:23 **[ТС]**

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

1) Графика мигает.

Проблема есть у многих по неизвестным причинам, у себя я решил эту проблему путём установки у Form1 свойства DoubleBuffered на True

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

2) Стрелки рисуются только в момент перетягивания. Я так понимаю, связи должны фиксироваться, и стрелка должна связывать два элемента. Этого нет.

В процессе так как достаточно сложно придумать систему связей.

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

3) Отрисовка стрелок - переусложена. Зачем там все эти вращения канвы? Стрелка ж элементарно рисуется методом DrawLine.

Попытайтесь.

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

4) Окошки с текстом тягать нельзя.

Спасибо

Только что осознал.

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

5) Классы модели почему-то внутри формы.

Я идиот что ещё сказать

Добавлено через 3 минуты

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

DrawLine

Рисует простую линию, а вот наконечник

если попытаетесь нарисовать то он будет всегда в одну сторону. Так что я просто где-то взял этот код(даже лень было от комментариев отчищать

)

Добавлено через 18 минут

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

4) Окошки с текстом тягать нельзя.

Перетягивание добавил:

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
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
using System;
using System.Collections.Generic;
using System.Drawing;
using System.Drawing.Drawing2D;
using System.Windows.Forms;
 
namespace BuilderScripts
{
    public partial class Form1 : Form
    {
        private static Pen blackPen = new Pen(Color.FromArgb(255, 0, 0, 0));
        private static Font fontArial10 = new Font("Arial", 10);
        private List<Object> objects = new List<Object>();
 
        private Dictionary<string, Object> dict = new Dictionary<string, Object>();
 
        private bool mouseDown = false;
        private MouseButtons button = MouseButtons.None;
        private int startX, startY;
 
        public struct SInput
        {
            public readonly int ID;
            public readonly string name;
 
            public SInput(int ID, string name)
            {
                this.ID = ID;
                this.name = name;
            }
        }
        public struct SOutput
        {
            public readonly int ID;
            public readonly string name;
 
            public SOutput(int ID, string name)
            {
                this.ID = ID;
                this.name = name;
            }
        }
 
        public class Object
        {
            public string title;
 
            public Rectangle rectTitle;
            public Rectangle rectInpOut;
 
            public List<SInput> inputs;
            public List<SOutput> outputs;
 
            public int x, y;
            public int width, height;
 
            public Object(string title, List<SInput> inputs, List<SOutput> outputs, int x, int y)
            {
                this.title = title;
 
                this.inputs = inputs;
                this.outputs = outputs;
 
                this.x = x;
                this.y = y;
            }
            public void Draw(Graphics gr)
            {
                height = 15 * inputs.Count + 15;
 
                SizeF size = gr.MeasureString(title, fontArial10);
                foreach(SInput input in inputs)
                {
                    SizeF tmp = gr.MeasureString(input.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
                width = (int)size.Width;
 
                size = new SizeF(0, 0);
                foreach (SOutput output in outputs)
                {
                    SizeF tmp = gr.MeasureString(output.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
 
                for (int i = 0; i < inputs.Count; i++)
                    gr.DrawString(inputs[i].name, fontArial10, new SolidBrush(Color.Black), x, y + (i + 1) * 15);
                for (int i = 0; i < outputs.Count; i++)
                    gr.DrawString(outputs[i].name, fontArial10, new SolidBrush(Color.Black), x + width, y + (i + 1) * 15);
 
                width += (int)size.Width;
 
 
                rectTitle = new Rectangle(x, y, width, 15);
                gr.DrawRectangle(blackPen, rectTitle);
 
                gr.DrawString(title, fontArial10, new SolidBrush(Color.Black), x, y);
 
                rectInpOut = new Rectangle(x, y + 15, width, 15 * inputs.Count);
                gr.DrawRectangle(blackPen, rectInpOut);
            }
        }
 
        public Form1()
        {
            InitializeComponent();
 
            dict.Add("echo", new Object("echo", new List<SInput>() { new SInput(objects.Count, "text") }, new List<SOutput>() { new SOutput(objects.Count, "done") }, 0, 0));
            dict.Add("echo2", new Object("echo2", new List<SInput>() { new SInput(objects.Count, "text") }, new List<SOutput>() { new SOutput(objects.Count, "done") }, 0, 0));
 
            foreach (string key in dict.Keys)
                contextMenuStrip1.Items.Add(key);
        }
 
        void DrawArrow(Graphics gr, Point start, Point end)
        {
            float x1 = start.X, y1 = start.Y, x2 = end.X, y2 = end.Y;
            // вычисляем угол, под которым стрелка повёрнута против часовой
            // вычисляем длину стрелки
            float angle = (float)Math.Atan2(y2 - y1, x2 - x1);
            angle = angle * 180f / (float)Math.PI; // переводим в градусы
            float len = (float)Math.Sqrt((y2 - y1) * (y2 - y1) + (x2 - x1) * (x2 - x1));
            // сохраняем старое состояние Graphics, чтобы потом его восстановить
            GraphicsState state = gr.Save();
            gr.SmoothingMode = SmoothingMode.HighQuality;
            // трансформация: сдвиг к точке start и поворот на угол
            gr.TranslateTransform(x1, y1);
            gr.RotateTransform(angle);
            using (Pen pen = new Pen(Color.Red, 2f))
            {
                int arrowLength = 10; // длина окончания стрелки
                int arrowWidth = 5; // ширина окончания стрелки
 
                // рисуем стрелку в горизонтальном положении
                gr.DrawLine(pen, 0, 0, len, 0);
                gr.DrawLine(pen, len, 0, len - arrowLength, arrowWidth);
                gr.DrawLine(pen, len, 0, len - arrowLength, -arrowWidth);
            }
            // восстанавливаем старое состояние Graphics (убираем наши трансформации)\
            gr.Restore(state);
        }
 
        int contains = 0; Object tmp = null;
        private void Form1_Paint(object sender, PaintEventArgs e)
        {
            if (mouseDown)
            {
                if (button == MouseButtons.Left)
                {
                    if (contains == 0 || tmp == null) { contains = 0; tmp = null; }
 
                    if (contains == 0)
                    {
                        foreach (Object obj in objects)
                        {
                            if (obj.rectInpOut.Contains(startX, startY))
                                contains = 1;
                            else if (obj.rectTitle.Contains(startX, startY))
                            {
                                contains = 2;
                                tmp = obj;
                            }
                        }
                    }
 
 
                    Point mousePos = PointToClient(Cursor.Position);
                    if (contains == 1)
                        DrawArrow(e.Graphics, new Point(startX, startY), mousePos);
                    else if (contains == 2 && tmp != null)
                    {
                        tmp.x = mousePos.X - tmp.width / 2;
                        tmp.y = mousePos.Y - tmp.height / 4;
                    }
                }
            }
            if (objects.Count > 0)
                foreach (Object obj in objects)
                    obj.Draw(e.Graphics);
        }
 
        private void Form1_MouseMove(object sender, MouseEventArgs e)
        {
            if (button == MouseButtons.Left)
                Refresh();
        }
 
        private void Form1_MouseDown(object sender, MouseEventArgs e)
        {
            button = e.Button;
 
            mouseDown = true;
 
            startX = e.X;
            startY = e.Y;
 
            Refresh();
        }
 
        private void contextMenuStrip1_ItemClicked(object sender, ToolStripItemClickedEventArgs e)
        {
            Object tmp;
            foreach(string key in dict.Keys)
                if (e.ClickedItem.ToString() == key && dict.TryGetValue(key, out tmp))
                    objects.Add(new Object(tmp.title, tmp.inputs, tmp.outputs, startX, startY));
 
            Refresh();
        }
 
        private void Form1_MouseUp(object sender, MouseEventArgs e)
        {
            startX = e.X;
            startY = e.Y;
 
            if (e.Button == MouseButtons.Right)
                contextMenuStrip1.Show(MousePosition, ToolStripDropDownDirection.Right);
 
            button = e.Button;
 
            mouseDown = false;
 
            contains = 0;
            tmp = null;
 
            Refresh();
        }
    }
}

Добавлено через 25 минут

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

5) Классы модели почему-то внутри формы.

Я вынес в отдельный класс класс Object

Структуры туда же засунуть или ...?

Добавлено через 15 часов 7 минут

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

сложно придумать систему связей.

Подскажите если что есть на уме. Сейчас подумываю над записей ID входов и выходов в отдельный List<Link>
а собственно сам Link <= int idInput, idOutput; , а потом просто при отрисовки проходить по этому массиву и рисовать связи между объектами, но мне кажется это как-то мудрёно.

@Hopeco · 11.10.2015, 08:34 **[ТС]**

Готовый код, графическая оболочка для скриптового окна:

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
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
using System;
using System.Collections.Generic;
using System.Drawing;
using System.Drawing.Drawing2D;
using System.Windows.Forms;
 
namespace BuilderScripts
{
    public partial class Form1 : Form
    {
        private static int nextId = 0;
 
        private const int StandartHeightBlock = 15;
 
        public struct Link
        {
            public readonly int ID_Output;
            public readonly int ID_Input;
 
            public Link(int ID_Output, int ID_Input)
            {
                this.ID_Input = ID_Input;
                this.ID_Output = ID_Output;
            }
        }
        public struct SInput
        {
            public readonly int ID;
            public readonly string name;
 
            public SInput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
        }
        public struct SOutput
        {
            public readonly int ID;
            public readonly string name;
 
            public SOutput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
        }
 
        public class Object
        {
            private Pen blackPen = new Pen(Color.FromArgb(255, 0, 0, 0));
            private Font fontArial10 = new Font("Arial", 10);
 
            public string title;
 
            public Rectangle rect;
            public Rectangle rectTitle;
            public Rectangle rectInpOut;
            public Rectangle rectInp;
            public Rectangle rectOut;
 
            public List<SInput> inputs;
            public List<SOutput> outputs;
 
            public int x, y;
            public int width, height;
 
            public Object(string title, List<SInput> inputs, List<SOutput> outputs, int x, int y)
            {
                this.title = title;
 
                this.inputs = inputs;
                this.outputs = outputs;
 
                this.x = x;
                this.y = y;
            }
 
            public Object(string title, List<SInput> inputs, List<SOutput> outputs)
            {
                this.title = title;
 
                this.inputs = inputs;
                this.outputs = outputs;
            }
 
            public void Draw(Graphics gr)
            {
                height = StandartHeightBlock * ((inputs.Count >= outputs.Count ? inputs.Count : outputs.Count) + 1);
 
                SizeF size = gr.MeasureString(title, fontArial10);
                foreach (SInput input in inputs)
                {
                    SizeF tmp = gr.MeasureString(input.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
                width = (int)size.Width;
 
                rectInp = new Rectangle(x, y + StandartHeightBlock, width, height - StandartHeightBlock);
                
 
                size = new SizeF(0, 0);
                foreach (SOutput output in outputs)
                {
                    SizeF tmp = gr.MeasureString(output.name, fontArial10);
                    if (tmp.Width > size.Width)
                        size = tmp;
                }
 
                for (int i = 0; i < inputs.Count; i++)
                    gr.DrawString(inputs[i].name, fontArial10, new SolidBrush(Color.Black), x, y + (i + 1) * StandartHeightBlock);
                for (int i = 0; i < outputs.Count; i++)
                    gr.DrawString(outputs[i].name, fontArial10, new SolidBrush(Color.Black), x + width, y + (i + 1) * StandartHeightBlock);
 
                width += (int)size.Width;
 
                rectTitle = new Rectangle(x, y, width, StandartHeightBlock);
                gr.DrawRectangle(blackPen, rectTitle);
 
                gr.DrawString(title, fontArial10, new SolidBrush(Color.Black), x, y);
 
                rectInpOut = new Rectangle(x, y + StandartHeightBlock, width, height - StandartHeightBlock);
                gr.DrawRectangle(blackPen, rectInpOut);
 
                rectOut = new Rectangle(x + rectInp.Width, y + StandartHeightBlock, width - rectInp.Width, height - StandartHeightBlock);
 
                rect = new Rectangle(x, y, width, height);
            }
        }
 
        private Dictionary<string, Object> patterns;
 
        private List<Object> objects;
        private List<Link> links;
 
        private bool mouseDown;
        private MouseButtons button;
        private int startX, startY;
 
        private int ID_START;
 
        public Form1()
        {
            patterns = new Dictionary<string, Object>();
 
            objects = new List<Object>();
            links = new List<Link>();
 
            mouseDown = false;
            button = MouseButtons.None;
 
            InitializeComponent();
 
            /* Настройка формы */
            FormBorderStyle = FormBorderStyle.FixedToolWindow;
            DoubleBuffered = true;
            Width = 800;
            Height = 600;
            /* ~Настройка формы */
 
            patterns.Add("start", new Object("start", new List<SInput>() { new SInput() }, new List<SOutput>() { new SOutput("output") }));
            patterns.Add("echo", new Object("echo", new List<SInput>() { new SInput("text") }, new List<SOutput>() { new SOutput("done") }));
 
            foreach (string key in patterns.Keys)
                contextMenuStrip1.Items.Add(key);
        }
 
        void DrawArrow(Graphics gr, Point start, Point end)
        {
            float x1 = start.X, y1 = start.Y, x2 = end.X, y2 = end.Y;
            float angle = (float)Math.Atan2(y2 - y1, x2 - x1);
            angle = angle * 180f / (float)Math.PI;
            float len = (float)Math.Sqrt((y2 - y1) * (y2 - y1) + (x2 - x1) * (x2 - x1));
            GraphicsState state = gr.Save();
            gr.SmoothingMode = SmoothingMode.HighQuality;
            gr.TranslateTransform(x1, y1);
            gr.RotateTransform(angle);
            using (Pen pen = new Pen(Color.Red, 2f))
            {
                int arrowLength = 10;
                int arrowWidth = 5;
 
                gr.DrawLine(pen, 0, 0, len, 0);
                gr.DrawLine(pen, len, 0, len - arrowLength, arrowWidth);
                gr.DrawLine(pen, len, 0, len - arrowLength, -arrowWidth);
            }
            gr.Restore(state);
        }
 
        int contains = 0; Object tmp = null;
        private void Form1_Paint(object sender, PaintEventArgs e)
        {
            if (mouseDown)
            {
                if (button == MouseButtons.Left)
                {
                    if (contains == 0 || tmp == null)
                    {
                        contains = 0;
                        tmp = null;
 
                        foreach (Object obj in objects)
                        {
                            if (obj.rectOut.Contains(startX, startY))
                            {
                                contains = 1;
                                ID_START = obj.outputs[Math.Abs(startY - obj.rectOut.Y) / StandartHeightBlock].ID;
                            }
                            else if (obj.rectTitle.Contains(startX, startY))
                            {
                                contains = 2;
                                tmp = obj;
                            }
                        }
                    }
 
                    Point mousePos = PointToClient(Cursor.Position);
                    if (contains == 1)
                        DrawArrow(e.Graphics, new Point(startX, startY), mousePos);
                    else if (contains == 2 && tmp != null)
                    {
                        tmp.x = mousePos.X - tmp.width / 2;
                        tmp.y = mousePos.Y - tmp.rectTitle.Height / 2;
                    }
                }
            }
            foreach (Object obj in objects)
                obj.Draw(e.Graphics);
 
            Point startPoint = Point.Empty, endPoint = Point.Empty;
            
            foreach(Link link in links)
            {
                foreach (Object obj in objects)
                {
                    for (int i = 0; i < obj.inputs.Count; i++)
                        if (obj.inputs[i].ID == link.ID_Input)
                            endPoint = new Point(obj.x, obj.rectInp.Y + obj.rectInp.Height / StandartHeightBlock * i + StandartHeightBlock / 2);
                    for (int i = 0; i < obj.inputs.Count; i++)
                        if (obj.outputs[i].ID == link.ID_Output)
                            startPoint = new Point(obj.x + obj.width, obj.rectOut.Y + obj.rectOut.Height / StandartHeightBlock * i + StandartHeightBlock / 2);
 
                    if (startPoint != Point.Empty && endPoint != Point.Empty)
                    {
                        DrawArrow(e.Graphics, startPoint, endPoint);
 
                        startPoint = Point.Empty;
                        endPoint = Point.Empty;
                    }
                }
            }
        }
 
        private void Form1_MouseMove(object sender, MouseEventArgs e)
        {
            if (button == MouseButtons.Left)
                Refresh();
        }
 
        private void Form1_MouseDown(object sender, MouseEventArgs e)
        {
            button = e.Button;
 
            mouseDown = true;
 
            startX = e.X;
            startY = e.Y;
 
            Refresh();
        }
 
        private void contextMenuStrip1_ItemClicked(object sender, ToolStripItemClickedEventArgs e)
        {
            Object tmp;
            foreach (string key in patterns.Keys)
                if (e.ClickedItem.ToString() == key && patterns.TryGetValue(key, out tmp))
                {
                    List<SInput> inputs = new List<SInput>();
                    foreach (SInput input in tmp.inputs)
                        inputs.Add(new SInput(input.name));
 
                    List<SOutput> outputs = new List<SOutput>();
                    foreach (SOutput output in tmp.outputs)
                        outputs.Add(new SOutput(output.name));
 
                    objects.Add(new Object(tmp.title, inputs, outputs, startX, startY));
                }
 
            Refresh();
        }
 
        private void Form1_MouseUp(object sender, MouseEventArgs e)
        {
            startX = e.X;
            startY = e.Y;
 
            if (e.Button == MouseButtons.Right)
                contextMenuStrip1.Show(MousePosition, ToolStripDropDownDirection.Right);
 
            foreach (Object obj in objects)
            {
                if (obj.rectInp.Contains(startX, startY))
                {
                    Point mousePos = PointToClient(Cursor.Position);
                    links.Add(new Link(ID_START, obj.inputs[Math.Abs(obj.rectInp.Y - mousePos.Y) / StandartHeightBlock].ID));
                    ID_START = 0;
                }
            }
            button = e.Button;
 
            mouseDown = false;
 
            contains = 0;
            tmp = null;
 
            Refresh();
        }
    }
}

@Storm23 · 11.10.2015, 09:04

Hopeco,
1) Классы модели по прежнему внутри формы.
2) Стрелка почему-то конектится не с первого раза.
3) В саму стрелку тыкнуться нельзя, соответственно нельзя ни удалить, ни перенести на другой объект.
4) Отрисовка стрелки по прежнему переусложнена.
5) Ссылки объектов друг на друга по ID - это не рационально, не удобно и при большом количестве блоков - будет тормозить. (хотя этот пункт не так очевиден, зависит от контекста).
Ну и по коду - я особо не смотрел, но выглядит страшновато.

@Hopeco · 11.10.2015, 10:17 **[ТС]**

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

1) Классы модели по прежнему внутри формы.

Довольно сложно вынести в отдельный файл. Будет сделано потом.

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

2) Стрелка почему-то конектится не с первого раза.

Странно у меня конектится с первого раза.

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

) В саму стрелку тыкнуться нельзя, соответственно нельзя ни удалить, ни перенести на другой объект.

В процессе.

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

4) Отрисовка стрелки по прежнему переусложнена.

Другого способа не нашёл.

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

5) Ссылки объектов друг на друга по ID - это не рационально, не удобно и при большом количестве блоков - будет тормозить. (хотя этот пункт не так очевиден, зависит от контекста).
Ну и по коду - я особо не смотрел, но выглядит страшновато.

Ничего другого на ум не пришло. Буду думать. Если есть предложения слушаю, а по по поводу страшновато это да ))

Добавлено через 48 минут
Storm23, CryEngine 3 при сериализации получается такой файл, тут всё основано тоже на ID:

XML
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
<Graph Description="" Group="Third Person" enabled="1" MultiPlayer="ClientServer">
 <Nodes>
  <Node Id="1" Class="Game:Start" pos="-570,190,0" flags="0">
   <Inputs InGame="1" InEditor="1"/>
  </Node>
  <Node Id="2" Class="FeatureTest:SimulateInput" pos="-120,60,0" flags="0">
   <Inputs ActionInput="thirdperson" Value="0"/>
  </Node>
  <Node Id="3" Class="Debug:ConsoleVariable" pos="-120.88382,154.04108,0" flags="0">
   <Inputs CVar="cl_tpvDist" Value="2"/>
  </Node>
  <Node Id="5" Class="Debug:ConsoleVariable" pos="-120.88382,234.04108,0" flags="0">
   <Inputs CVar="cl_tpvOffsetLeftRight" Value="0.6"/>
  </Node>
  <Node Id="6" Class="Debug:ConsoleVariable" pos="-120.88382,314.04108,0" flags="0">
   <Inputs CVar="cl_tpvOffsetUpDown" Value="0"/>
  </Node>
  <Node Id="7" Class="Debug:ConsoleVariable" pos="-120,390,0" flags="0">
   <Inputs CVar="r_displayinfo" Value="0"/>
  </Node>
  <Node Id="9" Class="Debug:ConsoleVariable" pos="344.94797,325.83728,0" flags="0">
   <Inputs CVar="cl_tpvPitch" Value="0"/>
  </Node>
  <Node Id="11" Class="entity:SpawnPoint" pos="-120,-20,0" flags="0" GraphEntity="0">
   <Inputs entityId="0" Disable="0" Enable="0" Spawn="0"/>
  </Node>
  <Node Id="12" Class="Debug:ConsoleVariable" pos="340,230,0" flags="0">
   <Inputs CVar="cl_tpvYaw" Value="8.5"/>
  </Node>
  <Node Id="14" Class="Math:Mul" pos="190,320,0" flags="0">
   <Inputs A="0" B="6.9"/>
  </Node>
  <Node Id="15" Class="Math:Mul" pos="190,230,0" flags="0">
   <Inputs A="0" B="6.9"/>
  </Node>
 </Nodes>
 <Edges>
  <Edge nodeIn="2" nodeOut="1" portIn="Press" portOut="output" enabled="1"/>
  <Edge nodeIn="3" nodeOut="1" portIn="Set" portOut="output" enabled="1"/>
  <Edge nodeIn="5" nodeOut="1" portIn="Set" portOut="output" enabled="1"/>
  <Edge nodeIn="6" nodeOut="1" portIn="Set" portOut="output" enabled="1"/>
  <Edge nodeIn="7" nodeOut="1" portIn="Set" portOut="output" enabled="1"/>
  <Edge nodeIn="11" nodeOut="1" portIn="Spawn" portOut="output" enabled="1"/>
  <Edge nodeIn="15" nodeOut="5" portIn="A" portOut="CurValue" enabled="1"/>
  <Edge nodeIn="14" nodeOut="6" portIn="A" portOut="CurValue" enabled="1"/>
  <Edge nodeIn="9" nodeOut="14" portIn="Set" portOut="out" enabled="1"/>
  <Edge nodeIn="9" nodeOut="14" portIn="Value" portOut="out" enabled="1"/>
  <Edge nodeIn="12" nodeOut="15" portIn="Set" portOut="out" enabled="1"/>
  <Edge nodeIn="12" nodeOut="15" portIn="Value" portOut="out" enabled="1"/>
 </Edges>
 <GraphTokens />
</Graph>

@Storm23 · 11.10.2015, 11:34

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

Довольно сложно вынести в отдельный файл. Будет сделано потом.

Так если у вас уже сейчас сложно разлепить классы, значит у вас уже проблемы. А потом разделить их будет еще сложнее. Они собственно изначально и проектируются максимально независимо, чтобы не было проблемы их разделить.

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

В процессе.

Оно не может быть у вас в процессе. Потому что у вас изначально неправильная архитектура. Стрелка у вас не является объектом. Поэтому тыкнуться в нее нельзя будет. Теперь вам придется переделывать весь механизм стрелок целиком. А все потому что вы изначально не продумали архитектуру.

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

Другого способа не нашёл.

Стрелка рисуется так:

C#
1
2
            using (var pen = new Pen(Color.Red, 2) { CustomEndCap = new AdjustableArrowCap(5, 5) })
                e.Graphics.DrawLine(pen, 10, 10, 200, 200);

Название: 1.png
Просмотров: 27

Размер: 12.5 Кб

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

CryEngine 3 при сериализации получается такой файл, тут всё основано тоже на ID:

Так там сделано так, потому что при сериализации нельзя сделать по-другому.
Ваши классы должны хранить не ID связанных блоков, а сами блоки. То есть не вот так:

C#
1
2
3
4
5
        public struct Link
        {
            public readonly int ID_Output;
            public readonly int ID_Input;
        }

А вот так:

C#
1
2
3
4
5
        public struct Link
        {
            public readonly Object Output;
            public readonly Object Input;
        }

Ну или в крайнем случае завести себе Dictionary<int, Object> который будет связывать Id объектов и сами объекты.
(тут я не могу точно сказать как лучше, потому что все завязано на контекст. если например вы собираетесь открывать файлы CryEngine, то Id придется оставить, если вы делаете свое с нуля - вам слудет задуматься, нужны ли вам эти Id)

@Hopeco · 12.10.2015, 07:11 **[ТС]**

Storm23,

C#
1
2
3
4
5
public struct Link
        {
            public readonly Object Output;
            public readonly Object Input;
        }

Это не подойдёт ведь входов и выходов может быть несколько, а так не ясно что именно мы конектили. Так что наверное лучше будет как-то так:

C#
1
2
3
4
5
public struct Link
        {
            public readonly SOutput Output;
            public readonly SInput Input;
        }

Добавлено через 24 минуты

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

Оно не может быть у вас в процессе. Потому что у вас изначально неправильная архитектура.

Можно

Но придётся очень жёстко мудрить

Способ такой: ловим нажатие на вход проверяем есть ли соединение, если есть то разрушаем его, но запоминаем от куда это соединение идёт ну а дальше думаю всё ясно

Но лучше конечно переделать.

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

Так если у вас уже сейчас сложно разлепить классы, значит у вас уже проблемы.

Разбил с помощью partial class, но это бред

Добавлено через 52 минуты
Проект с разделением, перетягивание, но без новой системы ID: Тык
Надо будет ещё добавить, чтобы к одному входу мог подходить только один выход.

Добавлено через 15 часов 51 минуту

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

Ссылки объектов друг на друга по ID - это не рационально, не удобно и при большом количестве блоков - будет тормозить.

Я задумался вот над чем если убрать систему ID, то всё станет намного печальней чем сейчас так как так или иначе придётся каким-то образом различать разные входы и выходы, а вот каким

Ведь у нас останется в входе и выходе лишь одно поле name так что это тупиковая ситуация систему ID придётся оставить.

Новые блоги и статьи Все статьи Все блоги /
Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

@Storm23 10425 / 5155 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34
	07.10.2015, 21:45
	Hopeco, 1) Графика мигает. 2) Стрелки рисуются только в момент перетягивания. Я так понимаю, связи должны фиксироваться, и стрелка должна связывать два элемента. Этого нет. 3) Отрисовка стрелок - переусложена. Зачем там все эти вращения канвы? Стрелка ж элементарно рисуется методом DrawLine. 4) Окошки с текстом тягать нельзя. 5) Классы модели почему-то внутри формы. 1

@Storm23 10425 / 5155 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34
	11.10.2015, 09:04
	Hopeco, 1) Классы модели по прежнему внутри формы. 2) Стрелка почему-то конектится не с первого раза. 3) В саму стрелку тыкнуться нельзя, соответственно нельзя ни удалить, ни перенести на другой объект. 4) Отрисовка стрелки по прежнему переусложнена. 5) Ссылки объектов друг на друга по ID - это не рационально, не удобно и при большом количестве блоков - будет тормозить. (хотя этот пункт не так очевиден, зависит от контекста). Ну и по коду - я особо не смотрел, но выглядит страшновато. 1