AKitavtsev / telegram-vk-bot

BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Handle для конфига #19

Closed olegromashin closed 3 years ago

olegromashin commented 3 years ago

Не могу понять зачем ты решил для получения конфига сделать хэндл. Предлагаю обычную IO-функцию сделать. Ну или если есть причина, то напиши)

AKitavtsev commented 3 years ago

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

olegromashin commented 3 years ago

В твоём проекте во всём этом нет необходимости. Конфиг используется только в одном месте. Если поменяется реализация, то даже без хэндла тебе нужно будет поменять только одно место в функции main. Логгер используется повсеместно, поэтому если тебе потребовалось бы поменять логгер, то без использования хэндла пришлось бы рефакторить весь проект.

AKitavtsev commented 3 years ago

Исправлено