[레거시 코드 리팩터링 - 4단계] 다즐(최우창) 미션 제출합니다. #789

woo-chang · 2023-10-29T04:11:43Z

안녕하세요 밀리!
벌써 마지막 미션이네요 🥲

이전 리뷰에 말씀해주신 양방향 의존을 제거하고,
컨택스트 별로 모듈을 나누는 과정을 진행해 보았습니다!

모듈 간 의존 관계는 아래와 같습니다 :)

menu -> product
order -> menu, product
product
table

리뷰 잘 부탁드려요 🙇🏻‍♂️

miseongk

안녕하세요 다즐!
의존성 분리와 멀티 모듈 적용하신 것 모두 잘 봤습니다!
그동안 미션하느라 너무 고생많으셨어요 👍
이제 우테코에서 하는 미션이 모두 끝났네요..🥲
바쁘실텐데 준비 잘하시고 화이팅입니다!!

miseongk · 2023-10-29T11:05:53Z

app/src/main/java/kitchenpost/Application.java

@@ -1,4 +1,4 @@
-package kitchenpos;
+package kitchenpost;


패키지명에 오타가 있어요!

miseongk · 2023-10-29T11:08:59Z

order/src/main/java/kitchenpos/application/TableValidatorImpl.java

 import org.springframework.stereotype.Component;
 import org.springframework.util.CollectionUtils;

 @Component
-public class TableValidator {
+public class TableValidatorImpl implements TableValidator {


인터페이스와 구현체를 만들어서 의존성을 끊어주셨군요 👍

woo-chang added 2 commits October 29, 2023 05:14

feat: 멀티 모듈 분리

966dd3f

refactor: 코드 컨벤션 정리

32ea0f2

woo-chang requested a review from miseongk October 29, 2023 04:11

woo-chang self-assigned this Oct 29, 2023

miseongk approved these changes Oct 29, 2023

View reviewed changes

miseongk merged commit a44f5a6 into woowacourse:woo-chang Oct 29, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[레거시 코드 리팩터링 - 4단계] 다즐(최우창) 미션 제출합니다. #789

[레거시 코드 리팩터링 - 4단계] 다즐(최우창) 미션 제출합니다. #789

woo-chang commented Oct 29, 2023

miseongk left a comment

miseongk Oct 29, 2023

miseongk Oct 29, 2023

[레거시 코드 리팩터링 - 4단계] 다즐(최우창) 미션 제출합니다. #789

[레거시 코드 리팩터링 - 4단계] 다즐(최우창) 미션 제출합니다. #789

Conversation

woo-chang commented Oct 29, 2023

miseongk left a comment

Choose a reason for hiding this comment

miseongk Oct 29, 2023

Choose a reason for hiding this comment

miseongk Oct 29, 2023

Choose a reason for hiding this comment