Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename domain to entity #41

Merged
merged 2 commits into from
Aug 31, 2021
Merged

Rename domain to entity #41

merged 2 commits into from
Aug 31, 2021

Conversation

neprja
Copy link
Contributor

@neprja neprja commented Aug 31, 2021

Во всех новых проектах я придерживаюсь именования папки со структурами как entity, поэтому предлагаю тут тоже исправить.

@sashamelentyev
Copy link
Contributor

Entity - это прям по дяде Бобу. Может быть тогда и service в usecase, а деливери - в controller? Чтоб прям вообще канонично было

@neprja
Copy link
Contributor Author

neprja commented Aug 31, 2021

Entity - это прям по дяде Бобу. Может быть тогда и service в usecase, а деливери - в controller? Чтоб прям вообще канонично было

service в usecase - OK
А деливери предлагаю в gateway, это более канонично (я у себя в проектах так называю). Controller - отсылает к MVP, поэтому не очень.

@sashamelentyev
Copy link
Contributor

Entity - это прям по дяде Бобу. Может быть тогда и service в usecase, а деливери - в controller? Чтоб прям вообще канонично было

service в usecase - OK
А деливери предлагаю в gateway, это более канонично (я у себя в проектах так называю). Controller - отсылает к MVP, поэтому не очень.

Давай тогда service в usecase переделаем, а над деливери еще подумаем. Хотелось бы в каноничность уйти. Если мы ссылаемся на дядю Боба, нужно следовать его описанию

@neprja neprja merged commit b7cdd1c into master Aug 31, 2021
@sashamelentyev sashamelentyev deleted the refactor/rename_domain_to_entity branch August 31, 2021 10:28
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.

None yet

2 participants