Форум программистов, компьютерный форум, киберфорум
Наши страницы

Обо всем!

Войти
Регистрация
Восстановить пароль
 
 
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
#1

Почему некоторые участники форума считают, что усложнять код - показать высокого интеллекта? - Обо всем!

05.07.2017, 17:39. Просмотров 562. Ответов 29
Метки нет (Все метки)

Почему некоторые участники форума считают, что усложнять код - показать высокого интеллекта? Я лично считаю, что нужно пользоваться минимумом средств, но не в ущерб коду. Однако, мое мнение расходится с одним человеком. Он лично считает, что напихав кучу кода в программу, выплеснув в программу те знания, которые только усложнят код, сделает его круче в глазах других. Я не желаю его осуждать. У меня одна цель - показать как надо программировать.

Добавлено через 57 секунд

Не по теме:

Я не считаю, что я профессионал, но я вижу, что программы которые он пишет можно было бы написать иногда проще.

0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
05.07.2017, 17:39
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Почему некоторые участники форума считают, что усложнять код - показать высокого интеллекта? (Обо всем!):

Почему все считают, что qsort - НЕ из STL? - C++
Странно, что qsort не относят к STL!? Ведь qsort определена в библиотеке cstdlib: C Standard General Utilities Library У кого какие...

Почему все считают, что язык С - низкоуровневый? - C++
Я просто в недоумении. Ходят слухи, что язык С - низкоуровневый, а язык С++ - перетерпел изменения и стал более высокоуровневым! ...

Способы работы с БД - почему некоторые предпочитают код конструкторам? - C#
Всем привет. Меня давно волнует один вопрос. В Visual Studio можно добавлять БД в проект и использовать хранимые процедуры и функции. Зачем...

Почему при использовании компонента TMediaPlayer ,некоторые mp3 открываются а некоторые нет? - Delphi
Почему при использовании компонента TMediaPlayer ,некоторые mp3 открываются а некоторые нет EMCIDeviceError с сообщением "No MCI...

Запросы неправильно считают. Что делаю не так? - MS Access
Добрый день! Пожалуйста, помогите. Раньше мне не приходилось иметь дел с базами данных. Сейчас копаюсь в Ассesse, нужно сделать поля с...

Системы искусственного интеллекта - разобрать код - C++/CLI
Доброго времени суток. Помогите в следующем вопросе... Нужно сделать какое либо задание по система искуственного интелекта (нейронные...

29
RaevskiAnatoly
20 / 19 / 6
Регистрация: 04.03.2017
Сообщений: 249
05.07.2017, 17:41 #2
Усложнять код - это как?
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
05.07.2017, 17:41  [ТС] #3
Например, писал я сегодня код:
Pascal
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
{$reference System.Windows.Forms.dll} 
{$reference System.Drawing.dll} 
uses System.Windows.Forms;
 
const
  n = 11;
 
type
  OperationType = (PLUS, MINUS, DIVIDE, MULT);
 
var
  f: Form;
  ///Some buttons: "0".."9", "." and "=".
  Btns: array [0..n] of Button;
  ///Field for typing.
  Field: System.Windows.Forms.Label;
  ///Buttons Btns's index.
  i: byte;
  ///Properties for number.
  FirstN, HasDot: Boolean;
  ///Numbers.
  a, b: string;
  ///Running operation.
  Operation: OperationType;
  ///Operation's buttons.
  PlusBtn, MinusBtn, DivBtn, MultBtn: Button;
  ///Extra buttons.
  ResetBtn, ClearBtn: Button;
 
procedure AddFreeChar(c: char);
begin
  if FirstN then
  begin
    a := a + c;
    Field.Text := a;
  end
    else
  begin
    b := b + c;
    Field.Text := b;
  end;
end;
 
procedure AddChar(c: char);
begin
  if c <> '.' then AddFreeChar(c) else
  if not HasDot and ((FirstN and (a <> '')) or (not FirstN and (b <> ''))) then
  begin
    AddFreeChar(c);
    HasDot := true;
  end;
end;
 
procedure Btn0Click(sender: Object; e: System.EventArgs):=AddChar('1');
 
procedure Btn1Click(sender: Object; e: System.EventArgs):=AddChar('2');
 
procedure Btn2Click(sender: Object; e: System.EventArgs):=AddChar('3');
 
procedure Btn3Click(sender: Object; e: System.EventArgs):=AddChar('4');
 
procedure Btn4Click(sender: Object; e: System.EventArgs):=AddChar('5');
 
procedure Btn5Click(sender: Object; e: System.EventArgs):=AddChar('6');
 
procedure Btn6Click(sender: Object; e: System.EventArgs):=AddChar('7');
 
procedure Btn7Click(sender: Object; e: System.EventArgs):=AddChar('8');
 
procedure Btn8Click(sender: Object; e: System.EventArgs):=AddChar('9');
 
procedure Btn9Click(sender: Object; e: System.EventArgs):=AddChar('0');
 
procedure BtnDotClick(sender: Object; e: System.EventArgs):=AddChar('.');
 
 
procedure Reset();
begin
  FirstN := false;
  HasDot := false;
end;
 
procedure BtnPlusClick(sender: Object; e: System.EventArgs);
begin
  if a <> '' then
  begin
    Reset();
    Operation := OperationType.PLUS;
  end;
end;
 
procedure BtnMinusClick(sender: Object; e: System.EventArgs);
begin
  if a <> '' then
  begin
    Reset();
    Operation := OperationType.MINUS;
  end;
end;
 
procedure BtnDivClick(sender: Object; e: System.EventArgs);
begin
  if a <> '' then
  begin
    Reset();
    Operation := OperationType.DIVIDE;
  end;
end;
 
procedure BtnMultClick(sender: Object; e: System.EventArgs);
begin
  if a <> '' then
  begin
    Reset();
    Operation := OperationType.MULT;
  end;
end;
 
 
procedure BtnEqualClick(sender: Object; e: System.EventArgs);
begin
  if b <> '' then
  begin
    FirstN := true;
    HasDot := false;
    if Operation = OperationType.PLUS then
      Field.Text := FloatToStr(StrToFloat(a) + StrToFloat(b));
    if Operation = OperationType.MINUS then
      Field.Text := FloatToStr(StrToFloat(a) - StrToFloat(b));
    if Operation = OperationType.DIVIDE then
      Field.Text := FloatToStr(StrToFloat(a) / StrToFloat(b));
    if Operation = OperationType.MULT then
      Field.Text := FloatToStr(StrToFloat(a) * StrToFloat(b));
    a := '';b := '';
  end;
end;
 
 
procedure BtnResetClick(sender: Object; e: System.EventArgs);
begin
  FirstN := true;
  HasDot := false;
  a := '';b := '';Field.Text := '';
end;
 
procedure BtnClearClick(sender: Object; e: System.EventArgs);
begin
  if FirstN then a := '' else b := '';
  Field.Text := '';
end;
 
begin
  FirstN := true;
  HasDot := false;
  
  f := new Form();
  f.Width := 310;
  f.Height := 350;
  f.Text := 'Calculator';
  
  i := 0;
  for var y := 1 to 4 do
    for var x := 1 to 3 do
    begin
      if i > n then break;
      Btns[i] := new Button();
      Btns[i].Width := 50;
      Btns[i].Height := 50;
      Btns[i].Left := 60 * x - 50;
      Btns[i].Top := 60 + 60 * y - 50;
      f.Controls.Add(Btns[i]);
      Inc(i);
    end;
  
  for i := 0 to n - 2 do Btns[i].Text := IntToStr(i + 1);
  
  Btns[n - 2].Text := '0';
  Btns[n - 1].Text := '.';
  
  Btns[n].Text := '=';
  Btns[n].Click += BtnEqualClick;
  
  Btns[0].Click += Btn0Click;
  Btns[1].Click += Btn1Click;
  Btns[2].Click += Btn2Click;
  Btns[3].Click += Btn3Click;
  Btns[4].Click += Btn4Click;
  Btns[5].Click += Btn5Click;
  Btns[6].Click += Btn6Click;
  Btns[7].Click += Btn7Click;
  Btns[8].Click += Btn8Click;
  Btns[9].Click += Btn9Click;
  Btns[10].Click += BtnDotClick;
  
  
  Field := new System.Windows.Forms.Label();
  Field.Width := 170;
  Field.Height := 50;
  Field.Left := 10;
  Field.Top := 10;
  Field.BackColor := System.Drawing.Color.WhiteSmoke;
  Field.BorderStyle := System.Windows.Forms.BorderStyle.Fixed3D;
  f.Controls.Add(Field);
  
  
  PlusBtn := new Button();
  PlusBtn.Width := 100;
  PlusBtn.Height := 50;
  PlusBtn.Left := 190;
  PlusBtn.Top := 10;
  PlusBtn.Text := 'Add';
  PlusBtn.ForeColor := System.Drawing.Color.Red;
  PlusBtn.Click += BtnPlusClick;
  f.Controls.Add(PlusBtn);
  
  MinusBtn := new Button();
  MinusBtn.Width := 100;
  MinusBtn.Height := 50;
  MinusBtn.Left := 190;
  MinusBtn.Top := 70;
  MinusBtn.Text := 'Minus';
  MinusBtn.ForeColor := System.Drawing.Color.Red;
  MinusBtn.Click += BtnMinusClick;
  f.Controls.Add(MinusBtn);
  
  DivBtn := new Button();
  DivBtn.Width := 100;
  DivBtn.Height := 50;
  DivBtn.Left := 190;
  DivBtn.Top := 130;
  DivBtn.Text := 'Divide';
  DivBtn.ForeColor := System.Drawing.Color.Red;
  DivBtn.Click += BtnDivClick;
  f.Controls.Add(DivBtn);
  
  MultBtn := new Button();
  MultBtn.Width := 100;
  MultBtn.Height := 50;
  MultBtn.Left := 190;
  MultBtn.Top := 190;
  MultBtn.Text := 'Multiple';
  MultBtn.ForeColor := System.Drawing.Color.Red;
  MultBtn.Click += BtnMultClick;
  f.Controls.Add(MultBtn);
  
  ResetBtn := new Button();
  ResetBtn.Width := 45;
  ResetBtn.Height := 50;
  ResetBtn.Left := 190;
  ResetBtn.Top := 250;
  ResetBtn.Text := 'Reset';
  ResetBtn.ForeColor := System.Drawing.Color.Green;
  ResetBtn.Click += BtnResetClick;
  f.Controls.Add(ResetBtn);
  
  ClearBtn := new Button();
  ClearBtn.Width := 45;
  ClearBtn.Height := 50;
  ClearBtn.Left := 245;
  ClearBtn.Top := 250;
  ClearBtn.Text := 'Clear';
  ClearBtn.ForeColor := System.Drawing.Color.Green;
  ClearBtn.Click += BtnClearClick;
  f.Controls.Add(ClearBtn);
  
  System.Windows.Forms.Application.EnableVisualStyles();
  Application.Run(f); 
end.
Я не согласен с тем, что он идеален. Но я и не старался использовать что-то лишнее.
Вот код того товарища:
Pascal
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
{$reference System.Windows.Forms.dll} 
{$reference System.Drawing.dll} 
uses System.Windows.Forms , System.Drawing , System.Reflection;
 
const
  n = 11;
 
type
  OperationType = (PLUS, MINUS, DIVIDE, MULT);
 
type
  CalculationForm = class(Form)
  private 
    Btns: array [0..n] of Button;
    Field: System.Windows.Forms.Label;
    i: byte;
    FirstN, HasDot: Boolean;
    a, b: string;
    Operation: OperationType;
    PlusBtn, MinusBtn, DivBtn, MultBtn: Button;
    ResetBtn, ClearBtn: Button;
    //Добавляет все приват поля приводимые к Control
    procedure AddComponentsInForm();
    var
      FormFields := GetType().GetFields(BindingFlags.NonPublic or BindingFlags.Instance);
    begin
      foreach var el in FormFields do
      begin
        var Component := el.GetValue(Self) as Control;
        if (Component <> nil) then Self.Controls.Add(Component );
      end;
    end;
    
    procedure AddFreeChar(c: char);
    begin
      if FirstN then
      begin
        a := a + c;
        Field.Text := a;
      end
      else
      begin
        b := b + c;
        Field.Text := b;
      end;
    end;
    
    procedure AddChar(c: char);
    begin
      if c <> '.' then AddFreeChar(c) else
      if not HasDot and ((FirstN and (a <> '')) or (not FirstN and (b <> ''))) then
      begin
        AddFreeChar(c);
        HasDot := true;
      end;
    end;
    
    procedure Btn0Click(sender: Object; e: System.EventArgs) := AddChar('1');
    
    procedure Btn1Click(sender: Object; e: System.EventArgs) := AddChar('2');
    
    procedure Btn2Click(sender: Object; e: System.EventArgs) := AddChar('3');
    
    procedure Btn3Click(sender: Object; e: System.EventArgs) := AddChar('4');
    
    procedure Btn4Click(sender: Object; e: System.EventArgs) := AddChar('5');
    
    procedure Btn5Click(sender: Object; e: System.EventArgs) := AddChar('6');
    
    procedure Btn6Click(sender: Object; e: System.EventArgs) := AddChar('7');
    
    procedure Btn7Click(sender: Object; e: System.EventArgs) := AddChar('8');
    
    procedure Btn8Click(sender: Object; e: System.EventArgs) := AddChar('9');
    
    procedure Btn9Click(sender: Object; e: System.EventArgs) := AddChar('0');
    
    procedure BtnDotClick(sender: Object; e: System.EventArgs) := AddChar('.');
    
    
    procedure Reset();
    begin
      FirstN := false;
      HasDot := false;
    end;
    
    procedure BtnPlusClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.PLUS;
      end;
    end;
    
    procedure BtnMinusClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.MINUS;
      end;
    end;
    
    procedure BtnDivClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.DIVIDE;
      end;
    end;
    
    procedure BtnMultClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.MULT;
      end;
    end;
    
    
    procedure BtnEqualClick(sender: Object; e: System.EventArgs);
    begin
      if b <> '' then
      begin
        FirstN := true;
        HasDot := false;
        if Operation = OperationType.PLUS then
          Field.Text := FloatToStr(StrToFloat(a) + StrToFloat(b));
        if Operation = OperationType.MINUS then
          Field.Text := FloatToStr(StrToFloat(a) - StrToFloat(b));
        if Operation = OperationType.DIVIDE then
          Field.Text := FloatToStr(StrToFloat(a) / StrToFloat(b));
        if Operation = OperationType.MULT then
          Field.Text := FloatToStr(StrToFloat(a) * StrToFloat(b));
        a := '';b := '';
      end;
    end;
    
    
    procedure BtnResetClick(sender: Object; e: System.EventArgs);
    begin
      FirstN := true;
      HasDot := false;
      a := '';b := '';Field.Text := '';
    end;
    
    procedure BtnClearClick(sender: Object; e: System.EventArgs);
    begin
      if FirstN then a := '' else b := '';
      Field.Text := '';
    end;
    
    procedure InitializeComponent();
    begin
      FirstN := true;
      HasDot := false;
      
      self.Width := 310;
      self.Height := 350;
      self.Text := 'Calculator';
      
      i := 0;
      for var y := 1 to 4 do
        for var x := 1 to 3 do
        begin
          if i > n then break;
          Btns[i] := new Button();
          Btns[i].Width := 50;
          Btns[i].Height := 50;
          Btns[i].Left := 60 * x - 50;
          Btns[i].Top := 60 + 60 * y - 50;
          self.Controls.Add(Btns[i]);
          Inc(i);
        end;
      
      for var i := 0 to n - 2 do Btns[i].Text := IntToStr(i + 1);
      
      Btns[n - 2].Text := '0';
      Btns[n - 1].Text := '.';
      
      Btns[n].Text := '=';
      Btns[n].Click += BtnEqualClick;
      
      Btns[0].Click += Btn0Click;
      Btns[1].Click += Btn1Click;
      Btns[2].Click += Btn2Click;
      Btns[3].Click += Btn3Click;
      Btns[4].Click += Btn4Click;
      Btns[5].Click += Btn5Click;
      Btns[6].Click += Btn6Click;
      Btns[7].Click += Btn7Click;
      Btns[8].Click += Btn8Click;
      Btns[9].Click += Btn9Click;
      Btns[10].Click += BtnDotClick;
      
      
      Field := new System.Windows.Forms.Label();
      Field.Width := 170;
      Field.Height := 50;
      Field.Left := 10;
      Field.Top := 10;
      Field.BackColor := System.Drawing.Color.WhiteSmoke;
      Field.BorderStyle := System.Windows.Forms.BorderStyle.Fixed3D;
      self.Controls.Add(Field);
      
      
      PlusBtn := new Button();
      PlusBtn.Width := 100;
      PlusBtn.Height := 50;
      PlusBtn.Left := 190;
      PlusBtn.Top := 10;
      PlusBtn.Text := 'Add';
      PlusBtn.ForeColor := System.Drawing.Color.Red;
      PlusBtn.Click += BtnPlusClick;
      self.Controls.Add(PlusBtn);
      
      MinusBtn := new Button();
      MinusBtn.Width := 100;
      MinusBtn.Height := 50;
      MinusBtn.Left := 190;
      MinusBtn.Top := 70;
      MinusBtn.Text := 'Minus';
      MinusBtn.ForeColor := System.Drawing.Color.Red;
      MinusBtn.Click += BtnMinusClick;
      self.Controls.Add(MinusBtn);
      
      DivBtn := new Button();
      DivBtn.Width := 100;
      DivBtn.Height := 50;
      DivBtn.Left := 190;
      DivBtn.Top := 130;
      DivBtn.Text := 'Divide';
      DivBtn.ForeColor := System.Drawing.Color.Red;
      DivBtn.Click += BtnDivClick;
      self.Controls.Add(DivBtn);
      
      MultBtn := new Button();
      MultBtn.Width := 100;
      MultBtn.Height := 50;
      MultBtn.Left := 190;
      MultBtn.Top := 190;
      MultBtn.Text := 'Multiple';
      MultBtn.ForeColor := System.Drawing.Color.Red;
      MultBtn.Click += BtnMultClick;
      self.Controls.Add(MultBtn);
      
      ResetBtn := new Button();
      ResetBtn.Width := 45;
      ResetBtn.Height := 50;
      ResetBtn.Left := 190;
      ResetBtn.Top := 250;
      ResetBtn.Text := 'Reset';
      ResetBtn.ForeColor := System.Drawing.Color.Green;
      ResetBtn.Click += BtnResetClick;
      self.Controls.Add(ResetBtn);
      
      ClearBtn := new Button();
      ClearBtn.Width := 45;
      ClearBtn.Height := 50;
      ClearBtn.Left := 245;
      ClearBtn.Top := 250;
      ClearBtn.Text := 'Clear';
      ClearBtn.ForeColor := System.Drawing.Color.Green;
      ClearBtn.Click += BtnClearClick;
      self.Controls.Add(ClearBtn);
      
      System.Windows.Forms.Application.EnableVisualStyles();
      Application.Run(self);
    end;
    
    public constructor();
    begin
      InitializeComponent;
      AddComponentsInForm;
    end;
    
    procedure StartApplication();
    begin
      Application.Run(Self);
    end;
  end;
 
begin
  var f := new CalculationForm;
  f.StartApplication;
end.
Один вопрос. Зачем было создавать класс, если можно без него?
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
05.07.2017, 17:47  [ТС] #4
Pascal
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
{$reference System.Windows.Forms.dll} 
{$reference System.Drawing.dll} 
uses System.Windows.Forms , System.Drawing , System.Reflection;
 
const
  n = 11;
 
type
  OperationType = (PLUS, MINUS, DIVIDE, MULT);
 
type
  CalculationForm = class(Form)
  protected
  Operation: OperationType;
  i: byte;
  FirstN, HasDot: Boolean;
  a, b: string;
  private 
    Field: System.Windows.Forms.Label;
    PlusBtn, MinusBtn, DivBtn, MultBtn: Button;
    ResetBtn, ClearBtn: Button;
 
    //Добавляет все приват поля приводимые к Control
    procedure AddComponentsInForm();
    var
      FormFields := GetType().GetFields(BindingFlags.NonPublic or BindingFlags.Instance);
    begin
      foreach var el in FormFields do
      begin
        var Component := el.GetValue(Self) as Control;
        if (Component <> nil) then Self.Controls.Add( Component );
      end;
    end;
    
    procedure AddFreeChar(c: char);
    begin
      if FirstN then
      begin
        a := a + c;
        Field.Text := a;
      end
      else
      begin
        b := b + c;
        Field.Text := b;
      end;
    end;
    
    procedure AddChar(c: char);
    begin
      if c <> '.' then AddFreeChar(c) else
      if not HasDot and ((FirstN and (a <> '')) or (not FirstN and (b <> ''))) then
      begin
        AddFreeChar(c);
        HasDot := true;
      end;
    end;
    
    //Universal event-e
    procedure BtnNumericsClick(sender: Object; e: System.EventArgs);
    var btn := sender as button;
    begin
      AddChar( char.Parse(btn.Text) );
    end;
    
    procedure Reset();
    begin
      FirstN := false;
      HasDot := false;
    end;
    
    procedure BtnPlusClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.PLUS;
      end;
    end;
    
    procedure BtnMinusClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.MINUS;
      end;
    end;
    
    procedure BtnDivClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.DIVIDE;
      end;
    end;
    
    procedure BtnMultClick(sender: Object; e: System.EventArgs);
    begin
      if a <> '' then
      begin
        Reset();
        Operation := OperationType.MULT;
      end;
    end;
    
    
    procedure BtnEqualClick(sender: Object; e: System.EventArgs);
    begin
      if b <> '' then
      begin
        FirstN := true;
        HasDot := false;
        case Operation of
        OperationType.PLUS :
          Field.Text := FloatToStr(StrToFloat(a) + StrToFloat(b));
          OperationType.MINUS :
          Field.Text := FloatToStr(StrToFloat(a) - StrToFloat(b));
        OperationType.DIVIDE :
          Field.Text := FloatToStr(StrToFloat(a) / StrToFloat(b));
        OperationType.Mult :
          Field.Text := FloatToStr(StrToFloat(a) * StrToFloat(b));
          end;
        a := '';b := '';
      end;
    end;
    
    procedure BtnResetClick(sender: Object; e: System.EventArgs);
    begin
      FirstN := true;
      HasDot := false;
      a := '';b := '';Field.Text := '';
    end;
    
    procedure BtnClearClick(sender: Object; e: System.EventArgs);
    begin
      if FirstN then a := '' else b := '';
      Field.Text := '';
    end;
    
    procedure InitializeComponent();
    begin
      FirstN := true;
      HasDot := false;
      
      self.Width := 310;
      self.Height := 350;
      self.Text := 'Calculator';
      
      i := 0;
      for var y := 1 to 4 do
        for var x := 1 to 3 do
        begin
          if i > n then break;
          var btn := new Button();
          Btn.Click += BtnNumericsClick;
          Btn.Width := 50;
          Btn.Height := 50;
          Btn.Left := 60 * x - 50;
          Btn.Top := 60 + 60 * y - 50;
          Btn.Text := i.ToString();
          Btn.Name := i.ToString();
          Controls.Add(Btn);
          Inc(i);
        end;
      //
      var BtnDot := Controls.Item[10];
      BtnDot.Text := '.';
      var btnEqual := Controls.Item[11];
      BtnEqual.Text := '=';
      BtnEqual.Click -= BtnNumericsClick;
      BtnEqual.Click += BtnEqualClick;
      //   
      Field := new System.Windows.Forms.Label();
      Field.Width := 170;
      Field.Height := 50;
      Field.Left := 10;
      Field.Top := 10;
      Field.BackColor := System.Drawing.Color.WhiteSmoke;
      Field.BorderStyle := System.Windows.Forms.BorderStyle.Fixed3D;
      
      PlusBtn := new Button();
      PlusBtn.Width := 100;
      PlusBtn.Height := 50;
      PlusBtn.Left := 190;
      PlusBtn.Top := 10;
      PlusBtn.Text := 'Add';
      PlusBtn.ForeColor := System.Drawing.Color.Red;
      PlusBtn.Click += BtnPlusClick;
      
      MinusBtn := new Button();
      MinusBtn.Width := 100;
      MinusBtn.Height := 50;
      MinusBtn.Left := 190;
      MinusBtn.Top := 70;
      MinusBtn.Text := 'Minus';
      MinusBtn.ForeColor := System.Drawing.Color.Red;
      MinusBtn.Click += BtnMinusClick;
 
      DivBtn := new Button();
      DivBtn.Width := 100;
      DivBtn.Height := 50;
      DivBtn.Left := 190;
      DivBtn.Top := 130;
      DivBtn.Text := 'Divide';
      DivBtn.ForeColor := System.Drawing.Color.Red;
      DivBtn.Click += BtnDivClick;
      
      MultBtn := new Button();
      MultBtn.Width := 100;
      MultBtn.Height := 50;
      MultBtn.Left := 190;
      MultBtn.Top := 190;
      MultBtn.Text := 'Multiple';
      MultBtn.ForeColor := System.Drawing.Color.Red;
      MultBtn.Click += BtnMultClick;
      
      ResetBtn := new Button();
      ResetBtn.Width := 45;
      ResetBtn.Height := 50;
      ResetBtn.Left := 190;
      ResetBtn.Top := 250;
      ResetBtn.Text := 'Reset';
      ResetBtn.ForeColor := System.Drawing.Color.Green;
      ResetBtn.Click += BtnResetClick;
      
      ClearBtn := new Button();
      ClearBtn.Width := 45;
      ClearBtn.Height := 50;
      ClearBtn.Left := 245;
      ClearBtn.Top := 250;
      ClearBtn.Text := 'Clear';
      ClearBtn.ForeColor := System.Drawing.Color.Green;
      ClearBtn.Click += BtnClearClick;
 
    end;
    
    public constructor();
    begin
      InitializeComponent;
      AddComponentsInForm;
    end;
    
    procedure StartApplication();
    begin
      Application.Run(Self);
    end;
  end;
 
begin
  var f := new CalculationForm;
  f.StartApplication;
end.
Основано не его идеи, хотя я был против этого.

Добавлено через 5 минут
По его мнению - введение нового класса добавит легкости написания кода в многооконных приложениях. Однако, я считаю, что если у меня калькулятор однооконный, то делать "лишнее" не следует.
0
kolorotur
Эксперт .NET
9312 / 7828 / 1266
Регистрация: 17.09.2011
Сообщений: 13,489
06.07.2017, 12:06 #5
У меня тоже вопрос:
Как из вот этого:
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
По его мнению - введение нового класса добавит легкости написания кода в многооконных приложениях.
Вы пришли вот к этому выводу:
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Он лично считает, что напихав кучу кода в программу, выплеснув в программу те знания, которые только усложнят код, сделает его круче в глазах других.
Выглядит малость надуманно и предвзято.
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
06.07.2017, 12:52  [ТС] #6
Цитата Сообщение от kolorotur Посмотреть сообщение
Выглядит малость надуманно и предвзято.
Я понимаю, что это так выглядит. Я говорил об этом:
Он пишет программы, пытаясь сделать их более универсальными, но не осознает того, что часто из-за универсальности может падать производительность программ. К примеру, давайте возьмем тот же самый Python. Там динамическая типизация, там все попытались упростить до не могу. Попытались сделать более удобно для пользователей-программистов. Но, производительность то приложения на Python значительно отличается от того же аналога, допустим на Pascal. Я не к тому, что кто-то пишет полностью приложения на Python. Это плохо. А к тому, что производительность разная у языков.

Нет, не поймите неправильно, надумывать я не собираюсь ничего - это было бы нарушение правил форума.

Цитата Сообщение от kolorotur Посмотреть сообщение
Он лично считает, что напихав кучу кода в программу, выплеснув в программу те знания, которые только усложнят код, сделает его круче в глазах других.
Ну, например, вот на тех кодах, которые я скинул. Зачем вводить класс для универсальности и удобства, если это не будет использоваться?
0
RaevskiAnatoly
20 / 19 / 6
Регистрация: 04.03.2017
Сообщений: 249
06.07.2017, 14:41 #7
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Зачем вводить класс для универсальности и удобства, если это не будет использоваться?
Если все-таки хоть чуть-чуть посмотреть на исходник , то очевидно , класс - скорее попытка получить шаблон формы , как это делается во всех нормальных приложениях .(Схожий шаблон создается в случае создания проекта в PascalAbc.Net WF)Во - вторых , без класса усложняется реализация метода AddComponentsInForm . Из чего выходит необходимость обертывания в класс.
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
06.07.2017, 14:53  [ТС] #8
Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Если все-таки хоть чуть-чуть посмотреть на исходник , то очевидно , класс - скорее попытка получить шаблон формы , как это делается во всех нормальных приложениях .(Схожий шаблон создается в случае создания проекта в PascalAbc.Net WF)Во - вторых , без класса усложняется реализация метода AddComponentsInForm . Из чего выходит необходимость обертывания в класс.
Это попытка улучшения была бы, на мой взгляд, полезна лишь тогда, когда было бы целесообразно применять этот класс несколько раз, но не один.

Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Из чего выходит необходимость обертывания в класс.
Не не обходимость, а твое желание. Без этого класса тут запросто можно обойтись. А если можно - почему бы и нет?

Добавлено через 12 секунд
Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Если все-таки хоть чуть-чуть посмотреть на исходник , то очевидно , класс - скорее попытка получить шаблон формы , как это делается во всех нормальных приложениях .(Схожий шаблон создается в случае создания проекта в PascalAbc.Net WF)Во - вторых , без класса усложняется реализация метода AddComponentsInForm . Из чего выходит необходимость обертывания в класс.
Это попытка улучшения была бы, на мой взгляд, полезна лишь тогда, когда было бы целесообразно применять этот класс несколько раз, но не один.

Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Из чего выходит необходимость обертывания в класс.
Не не обходимость, а твое желание. Без этого класса тут запросто можно обойтись. А если можно - почему бы и нет?
0
CoderHuligan
698 / 401 / 128
Регистрация: 30.06.2015
Сообщений: 2,134
Записей в блоге: 3
06.07.2017, 14:57 #9
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Ну, например, вот на тех кодах, которые я скинул. Зачем вводить класс для универсальности и удобства, если это не будет использоваться?
Просто он по-видимому, не может по другому. Вероятно начал изучать программирование прямо с ооп-языков. Так учили с самого начала. Поэтому для таких, ради того чтобы сложить 2 и 2 надо писать класс. Любая хорошая идея может исказиться до неузнаваемости в соответствии с искажениями психики её прилагающих на практике.
Один хороший программист как-то сказал: пишите код так, как-будто его будет читать маньяк-психопат. То бишь - пишите проще, без понтов, тогда и самолёты падать не будут вместе с пассажирами, и с психикой у многих проблемы исчезнут...
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
06.07.2017, 16:02  [ТС] #10
Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Если все-таки хоть чуть-чуть посмотреть на исходник , то очевидно , класс - скорее попытка получить шаблон формы , как это делается во всех нормальных приложениях .(Схожий шаблон создается в случае создания проекта в PascalAbc.Net WF)Во - вторых , без класса усложняется реализация метода AddComponentsInForm . Из чего выходит необходимость обертывания в класс.
Это попытка улучшения была бы, на мой взгляд, полезна лишь тогда, когда было бы целесообразно применять этот класс несколько раз, но не один.

Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Из чего выходит необходимость обертывания в класс.
Не необходимость, а твое желание. Без этого класса тут запросто можно обойтись. А если можно - почему бы и нет?
0
RaevskiAnatoly
20 / 19 / 6
Регистрация: 04.03.2017
Сообщений: 249
06.07.2017, 17:35 #11
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Не не обходимость, а твое желание.
Для невеж , получить метаинформацию , с помощью рефлексии ,полей класса , можно только если это поля класса. Да , возможно, ты не знал , но весь код (даже если в нем нет и строчки class) обертывается в процессе компиляции в класс (Дизассемблер в помощь).Но получать поля этого класса нерационально , да и как , он фактически скрыт по задумке. И это (напомню , создание наследника формы) необходимость. Нет , писать везде f.Controls.Add - счастье идиота. А если честно , можно и через класс, а если можно - почему бы и нет? Ах , это усложнение . А почему не через класс? Ну пиши везде add , усложняй внедрение многоокнности. Ааа , у меня же одно окно... А настоящий программист смотрит в будующее и пишет код расширяемым , а не шлако-бетонным.

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

Не по теме:

Удаляем свои дубли сами , форум идеален , во всем виноваты мы.

0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
06.07.2017, 17:38  [ТС] #12
Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Для невеж , получить метаинформацию , с помощью рефлексии ,полей класса , можно только если это поля класса. Да , возможно, ты не знал , но весь код (даже если в нем нет и строчки class) обертывается в процессе компиляции в класс (Дизассемблер в помощь).Но получать поля этого класса нерационально , да и как , он фактически скрыт по задумке. И это (напомню , создание наследника формы) необходимость. Нет , писать везде f.Controls.Add - счастье идиота. А если честно , можно и через класс, а если можно - почему бы и нет? Ах , это усложнение . А почему не через класс? Ну пиши везде add , усложняй внедрение многоокнности. Ааа , у меня же одно окно... А настоящий программист смотрит в будующее и пишет код расширяемым , а не шлако-бетонным.
Настоящий программист видит будущее программы отчасти, а не придумывает его. Почему я могу сказать что ты его придумал (будущее)? Потому, что ты не учел, что приложение не будет никогда многооконным. Однако, ты почему то решил, что оно все таки будет многооконным, и зачем-то по непонятной мне причине написал класс.

Я не вижу ничего плохого писать:
Pascal
1
f.Controls.Add(Control)
Чуть длиннее - да, но больше гарантии, что в коде не будет ошибок потому что не стараюсь писать замудренный код, который чуть-чуть может что-то и улучшит, но принесет за собой столько ошибок, что мама не горюй.
0
RaevskiAnatoly
20 / 19 / 6
Регистрация: 04.03.2017
Сообщений: 249
06.07.2017, 17:44 #13
Фишка класса не только в облегчении многооконности , а в поддержке метода AddComponents.
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
да, но больше гарантии, что в коде не будет ошибок
Гарантия того что ошибок будет больше. В данном случае , ты можешь просто забыть где-то (в киллометровом коде) поставить add. К моему варианту это не относится.
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
писать замудренный код
Замудренный - это тот который ты не понимаешь , так и говори. Этот код для меня не представляет ничего сложного , так что .
0
Volobuev Ilya
125 / 98 / 44
Регистрация: 25.07.2014
Сообщений: 1,865
Записей в блоге: 2
06.07.2017, 18:08  [ТС] #14
Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
В данном случае , ты можешь просто забыть где-то (в киллометровом коде) поставить add.
Километровый код обычно разделяют на модули. Чтобы он не был слишком громоздким. Также, пишут в едином стиле. Например, как у меня - под конец определения свойств каждого компонента формы сразу добавляю его на форму. И сразу видно - где Add не дописан.

Цитата Сообщение от RaevskiAnatoly Посмотреть сообщение
Замудренный - это тот который ты не понимаешь , так и говори. Этот код для меня не представляет ничего сложного , так что .
Многие те, кто плохо пишет программы говорят подобные слова. Для них codestyle является чуждым. А желание писать хорошо коды для них просто напросто может означать написать громоздкий код, который будет медленно работать, который будут плохо понимать другие и который будет сложно редактировать.

Добавлено через 3 минуты
Если говорить про конструирование интерфейса формы, то классы для каких-то объектов было бы целесообразно создавать лишь если они будут использоваться часто. Например, форма для входа пользователя - два поля и две метки и кнопка "Log In".

Добавлено через 15 минут
На другом примере. Совершенно не понятно зачем писать это:
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
/*
 * Name - Консольный таймер
 * Пользователь: RAE
 * Дата: 04.07.2017
 * Время: 17:56
 * 
 * (c) Raevski Anatoliy
 */
using System;
using System.Windows;
namespace Timer
{
    class Program
    {
        Int64 ElapsedTime;
        void UpdateTitle(){
            long minutes_Elapsed , seconds_Elapsed , hours_Elapsed = Math.DivRem(ElapsedTime,3600,out minutes_Elapsed);
            minutes_Elapsed = Math.DivRem(minutes_Elapsed,60,out seconds_Elapsed);
            Console.Title = String.Format( "Осталось : {0}ч {1}мин {2}сек " , hours_Elapsed,minutes_Elapsed,seconds_Elapsed);
        }
        void FonStart(Action act){
            var T = new System.Threading.Tasks.Task(act);
            T.Start();
        }
        void TimerEnd(){
                Console.ForegroundColor = ConsoleColor.Red;
                Console.WriteLine("Время вышло!");
                FonStart( () => {while (true) Console.Beep(3200,300);} );
                MessageBox.Show("Время вышло!","Внимание");
        }
        void Start(){
            Console.Write("Введите время ожидания в минутах : ");
            ElapsedTime = Int64.Parse(Console.ReadLine())*60;
            if (ElapsedTime == 0) 
                throw new ArgumentException("Недопустимое значение таймера!");
            Console.WriteLine("Нажмите Enter для запуска!");
            while (Console.ReadKey(true).Key != ConsoleKey.Enter) {}
            Console.ForegroundColor = ConsoleColor.Green;
            Console.WriteLine("Время пошло!");
            while (true) {
                UpdateTitle();
                System.Threading.Thread.Sleep(1000);
                ElapsedTime--;
                if (ElapsedTime == 0) {
                    TimerEnd();
                    return;
                }
            }
        }
        public static void Main(string[] args)
        {
            Console.BufferHeight = 50;
            Console.ForegroundColor = ConsoleColor.White;
            Console.Title = "Timer";
            var ExecuteProgram = new Program();
            ExecuteProgram.Start();
            Console.ReadKey(true);
        }
    }
}
Если можно вот так:
Pascal
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
uses crt;
var
  t, time: integer;
 
begin
  while true do
  begin
    time := 0;
    while time <= 0 do
    begin
      crt.ClrScr;
      Write('Time: ');
      try
        Readln(time);
      except
        on System.FormatException do continue end;
    end;
    t := 0;
    crt.ClrScr;
    while t <> time do
    begin
      WritelnFormat('Seconds {0}.', time - t);
      Inc(t);
      Sleep(1000);
      crt.ClrScr;
    end;
    Writeln('The time is gone.');
    Sleep(1000);
    crt.ClrScr;
  end;
end.
0
RaevskiAnatoly
20 / 19 / 6
Регистрация: 04.03.2017
Сообщений: 249
06.07.2017, 21:56 #15
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Совершенно не понятно зачем писать это
Если тебе не понятен код , вариант1 - ты не знаешь языка и вообще как можешь тогда рационально рассуждать( Не ну есть еще один вариант , ты ума лишенный , но я решил его отбросить)

Добавлено через 2 минуты
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Если можно вот так:
И что в результате получаем? Мой вариант лучше , и в 3 раза меньше весит.

Добавлено через 2 минуты
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Километровый код обычно разделяют на модули. Чтобы он не был слишком громоздким.
Сколько не разделяй , а меньше он не станет.

Добавлено через 1 минуту
Цитата Сообщение от Volobuev Ilya Посмотреть сообщение
Для них codestyle является чуждым.
Вот именно , свой codestyle , похожий на свалку мусора , ты уважаешь , а чужой в стиле ооп это не то.
0
06.07.2017, 21:56
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
06.07.2017, 21:56
Привет! Вот еще темы с ответами:

Считает почему то только первый код, на остальные пишет что код не найден, хотя он есть в списке, не понимаю п - VBA
Написать процедуру поиска в списке, расположенном на рабочем листе с именем «Цены» активной рабочей книги, кода товара, введенного с...

Скрыть/показать некоторые поля формы - MS Access
Я не силен, к сожалению в VBA. Подскажите пожалуйста как сделать так что если поставить галочку для поля &quot;скрыть параметры для колесных...

Скрыть (показать) по некоторые поля (элементы) формы - MS Access
Подскажите пожалуйста как скрыть или наоборот показать некоторые поля, элементы в форме по условию. Допустим если в форме А пустое...

сегодня наконец то понял что такое КЛАСС, и ОБЪЕКТ. понято всё, кроме одного - зачем всё это? в смысле, можно же без этого? так зачем жизнь усложнять? - C++
сегодня наконец то понял что такое КЛАСС, и ОБЪЕКТ. понято всё, кроме одного - зачем всё это? в смысле, можно же без этого? так зачем жизнь...


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

Или воспользуйтесь поиском по форуму:
15
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.