Skip to content

Логирование и конфигурирование в приложении#9

Merged
antons3310 merged 2 commits intomasterfrom
Homework09
Mar 27, 2022
Merged

Логирование и конфигурирование в приложении#9
antons3310 merged 2 commits intomasterfrom
Homework09

Conversation

@antons3310
Copy link
Copy Markdown
Owner

Логирование и конфигурирование в приложении

Copy link
Copy Markdown

@OKrylov OKrylov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом все практически идеально, но есть одно замечание по конфигурации, просьба стоит обратить внимание

import org.springframework.stereotype.Service;

@Service
@Profile("test2")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@antons3310 antons3310 merged commit 44283d5 into master Mar 27, 2022
@antons3310 antons3310 deleted the Homework09 branch March 27, 2022 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants