Столкнулся с программой написанной на мой неопытный взгляд в странном стиле, а именно:
Почти все действия выполняет большое количество процедур не умеющих IN/OUT параметров. При этом они оперируют глобальными параметрами. В программе уже 4200 строк.
Упрощено программу можно представить следующим образом:
int a;
int b;
int result;
int c = 6;
void Click1() {
proc1();
proc2();
proc3();
}
void Click2() {
proc1();
proc2();
proc4();
}
void proc1() {a = 3;}
void proc2() {b = 5;}
void proc3() {result = a + b + c; Console.Write(result)}
void proc4() {result = a * b / c; Console.Write(result)}
Мне предстоит развивать эту программу и меня интересует вопрос - допустимо ли писать программы таким образом или, я должен приложить усилия, чтобы ее изменить или как минимум не повторять этот стиль в новых блоках кода?
На мой взгляд, программа должна выглядеть следующим образом:
int C = 6;
void Click1() {
proc1(int out a);
proc2(int out b);
proc3(a, b, C);
}
void Click2() {
proc1(int out a);
proc2(int out b);
proc4(a, b, C);
}
void proc1(int out a) {a = 3;}
void proc2(int out b) {b = 5;}
void proc3(int a, int b, C) {
int result;
result = a + b + С;
Console.Write(result)
}
void proc4(int a, int b, C) {
int result;
result = a * b / С;
Console.Write(result)
}