CC
Это нечитабельный говнокод
Size: a a a
2020 September 06
CC
Такое нужно рефакторить
CC
Ты нарушаешь принцип единой ответственности, который существует не просто так
CC
Разработка ПО это управление сложностью
RM
Ты нарушаешь принцип единой ответственности, который существует не просто так
Чего это
CC
Ты создаешь себе проблемы на ровном месте
RM
Логом управляют через класс плагина
RM
Это и так единая ответственность
CC
Чего это
А какого черта у тебя процедура и конфиг парсит и определяет логику логгирования для однопоточного и многопоточного логгирования
CC
Единая отвественность должна быть у произвольного програмного элемента
CC
Функции/процедуры/методы не являются исключением
RM
А какого черта у тебя процедура и конфиг парсит и определяет логику логгирования для однопоточного и многопоточного логгирования
Потому что там выбор стоит либо однопоток либо многопоток
CC
Потому что там выбор стоит либо однопоток либо многопоток
А почему это все в одной процедуре?
RM
Читаем файл, парсим атрибуты и задаём дефолтный логгер
CC
Каким образом ты декомпозировал задачу разработки этого модуля?
CC
Читаем файл, парсим атрибуты и задаём дефолтный логгер
Вот, 3 хайлевел задачи
CC
Почему одна процедура?
IL
Читаем файл, парсим атрибуты и задаём дефолтный логгер
вот это должно быть 3 процедуры
IL
как минимум
CC
На самом деле больше