[MVC 구현하기 - 3단계] 에버(손채영) 미션 제출합니다. #865

helenason · 2024-09-30T09:00:46Z

리니 안녕하세요!

어느덧 리니와의 마지막 단계네요

마지막 리뷰도 잘 부탁드려요 😊

linirini

에버~ 마지막 미션까지 수고많으셨습니다.!
코드 변경 사항보다는 궁금해서 남긴 코멘트가 좀 있습니다. 확인하시고 답변 달아주세요:)
다음 리뷰 때는 학습 테스트도 완료 후 요청해주시면 같이 리뷰하도록 하겠습니다~

linirini · 2024-10-01T05:14:14Z

app/src/main/java/com/techcourse/domain/User.java

    public String getAccount() {
        return account;
    }

+    public String getPassword() {


getter는 사용하는 곳이 없는 것 같아요:)

위 getter는 jackson의 ObjectMapper가 User 객체를 Json 문자열로 파싱하기 위한 용도입니다! Getter를 선언하지 않은 필드의 경우는 Json 문자열로 파싱하지 못하더라구요. 혹시 제가 모르는 또 다른 방법이 있는 걸까요? 🤔

말씀하신 부분이 맞는 것 같아요! 그런데 id, password, email은 직렬화하는 곳이 없는 것 같아서 여쭤봤어요!

ObjectMapper는 기본적으로 public으로 선언되어 있지 않은 필드에 대해 직렬화하지 못합니다! ObjectMapper가 private 필드를 직렬화하도록 하기 위해서는 해당 필드에 대한 getter가 필요해요. 즉, 제가 선언한 getter는 저의 비즈니스 코드에서는 사용되지 않지만, ObjectMapper에서 사용된다고 보시면 될 것 같아요 😊

model에 데이터가 1개면 값을 그대로 반환하고 2개 이상이면 Map 형태 그대로 JSON으로 변환해서 반환한다. 라는 요구사항이 있었는데요. User 클래스에 getAccount()만 존재할 경우 아래와 같이 account 필드에 대해서만 직렬화돼요.

이를 아래와 같이 출력하기 위해 나머지 세 필드에 대해서도 getter를 추가했다고 보시면 됩니다!

관련하여 저도 더 자세히 학습할겸, ObjectMapperTest를 추가해두었습니다! 참고 자료도 첨부할게요 :)

P.S. 저는 id, account, password, email 모든 필드에 대해 직렬화하여 반환하는 것을 이번 미션의 요구사항 중 하나라고 생각해서 위와 같이 구현하였습니다!

linirini · 2024-10-01T05:17:24Z

app/src/main/java/com/techcourse/controller/LoginController.java


    private static final Logger log = LoggerFactory.getLogger(LoginController.class);

-    @Override
-    public String execute(final HttpServletRequest req, final HttpServletResponse res) throws Exception {
+    @RequestMapping(value = "/login", method = {RequestMethod.POST})


method가 하나 뿐이니, 중괄호는 없어도 될 것 같아요!

linirini · 2024-10-01T05:22:08Z

mvc/src/main/java/com/interface21/webmvc/servlet/HandlerMappingRegistry.java

    private final List<HandlerMapping> handlerMappings = new ArrayList<>();

    public HandlerMappingRegistry() {
    }

    public void initialize() {
-        handlerMappings.add(new ManualHandlerMapping());
-        handlerMappings.add(new AnnotationHandlerMapping());
+        handlerMappings.add(new AnnotationHandlerMapping(CONTROLLER_BASE_PACKAGE));


저는 스캔할 패키지는 프레임워크가 아닌 개발자가 지정해줘야 하는 정보이므로 외부에서 알려줘야 한다고 생각하는데, 에버는 어떻게 생각하세요?

네 리니와 같은 생각이에요! 고민만 하다가 넘어갔었네요 😅

linirini · 2024-10-01T05:29:06Z

mvc/src/main/java/com/interface21/webmvc/servlet/view/JsonView.java

+
+    private static JsonView INSTANCE;
+
+    public static JsonView from() {


싱글톤을 사용하셨네요!
싱글톤으로 구현하는 에버만의 기준이 있나요?

JspView와 같이 객체가 가질 수 있는 상태가 제한되어 있거나, JsonView와 같이 객체가 상태를 가질 수 없는 경우에 싱글톤 패턴을 고려하고 있어요. 이 경우 불필요하게 생성되는 객체가 많다고 생각해 싱글톤 패턴을 도입하고 있습니다!

질문) 싱글톤 패턴을 많이 사용했을 때의 트레이드 오프는 없을까요? 저도 명확한 기준이 없는 부분이라 에버의 의견이 궁금해서 물어봅니다!

같은 상태를 갖는 여러 객체가 중복으로 불필요하게 메모리를 차지하는 경우, 싱글톤 패턴을 도입하곤 하는데요. 싱글톤 패턴을 도입했을 때 발생할 수 있는 단점을 찾아 정리해보았습니다!

싱글톤 패턴을 남용해 사용할 경우 객체지향과는 먼 설계가 될 수 있다는 단점이 있어요. 추상화 클래스가 아닌 구체 클래스에 의존해, 추상화된 인터페이스로의 전환이 어렵기 때문에 DIP를 위반합니다. 또한, 상속을 통한 다형성 구현이 어려워 확장에 닫혀있어, OCP 또한 위반합니다.

멀티스레드 환경에서 동시에 싱글톤 객체를 생성할 경우, 객체의 개수를 하나로 유지하지 못할 수 있습니다. 따라서 synchronized 키워드와 같은 방식으로 동시성을 고려해야 한다는 점에서 추가적인 리소스가 요구될 수 있어요!

싱글톤 객체는 전역적으로(static) 선언되기 때문에 싱글톤 객체가 상태를 갖는 경우, 매 테스트마다 객체를 초기화해주어야 합니다. 하지만 저의 JsonView 클래스는 상태를 갖지 않는다는 전제 조건에서 패턴을 도입했기 때문에 해당 단점은 현재 저의 구조에서는 해당되지 않는다고 생각해요!

이 중 제가 직접적으로 크게 체감한 부분은 아직까지 없는 것 같아요. 따라서 싱글톤의 단점보다 장점을 더 체감하고 있는 상태입니다!

linirini · 2024-10-01T05:31:55Z

...c/main/java/com/interface21/webmvc/servlet/mvc/tobe/annotation/AnnotationHandlerMapping.java

Legacy 클래스가 사라지면서 asis 패키지도 함께 제거되었는데요, 따라서 legacy와 구분하기 위해 있었던 tobe 패키지도 제거해도 될 것 같아요! 해당 패키지를 제거하여 depth를 줄여봐도 좋을 것 같아요!

helenason · 2024-10-02T04:54:02Z

안녕하세요 리니~~~

리뷰 반영하고 학습테스트 추가해서 다시 요청드려요!

감사합니다 :)

linirini · 2024-10-02T08:11:40Z

mvc/src/main/java/com/interface21/webmvc/servlet/DispatcherServlet.java

@@ -13,6 +13,8 @@ public class DispatcherServlet extends HttpServlet {
    private static final long serialVersionUID = 1L;
    private static final Logger log = LoggerFactory.getLogger(DispatcherServlet.class);

+    private static final String CONTROLLER_BASE_PACKAGE = "com.techcourse.controller";


프레임워크에서 스캔할 패키지의 위치가 결정되는군요!
만약, 개발자가 스캔할 위치를 변경하고 싶다면 프레임워크 모듈을 열어봐야겠네요!

엇 그러게요 프레임워크 밖에서 주입하는 게 맞겠네요 ㅎㅎ 생각이 짧았어요

linirini · 2024-10-02T08:13:11Z

app/src/test/java/com/techcourse/controller/UserControllerTest.java

+        assertThat(actual).isInstanceOf(User.class);
+        assertThat(actual).isEqualTo(expected);
+    }
+}


EOL을 추가해주세요!

linirini · 2024-10-02T08:13:25Z

mvc/src/test/java/com/interface21/webmvc/servlet/view/JsonViewTest.java

+
+        // then
+        verify(response).setContentType(MediaType.APPLICATION_JSON_UTF8_VALUE);
+        assertThat(writer.toString()).isEqualTo("{\"key1\":\"value1\",\"key2\":\"value2\"}");


linirini · 2024-10-02T08:27:51Z

study/src/test/java/di/stage3/context/DIContainer.java

+    }
+
+    private boolean canAssign(Field field, Object bean) {
+        return bean.getClass().equals(field.getType()) || field.getType().isAssignableFrom(bean.getClass());


에버는 isAssignableFrom을 쓰셨네요!
저는 리뷰어에게 질문 받고서야 이런 메서드가 있다는 걸 알았어요 ㅋㅋㅋ
제가 받았던 질문을 한번 드려보겠습니다:)
isInstance와 isAssignableFrom은 어떤 차이가 있을까요?

제가 isAssignableFrom 을 사용했던 이유는 구현클래스 여부를 확인하기 위함이었습니다. 일단, 다시 코드를 확인하니 클래스의 equals 비교는 isAssignableFrom 역할에 포함되어 제거하였습니다. 추가로 리니가 제시해준 isInstance와 isAssignableFrom, 더하여 instanceof까지 비교해보자면!

A instanceof B : A는 인스턴스, B는 클래스이며, B 클래스가 A의 타입보다 더 generic할 경우 true를 반환합니다.

A.isInstance(B) : A는 클래스, B는 인스턴스이며, 역할은 instanceof와 동일합니다. 다만 A 클래스가 B의 타입보다 더 generic할 경우 true를 반환합니다.

A.isAssignableFrom(B) : A와 B 모두 클래스이며, 클래스의 타입이 동등하거나 B가 더 구체적일 경우 true를 반환합니다.

그런데, 개인적으로 isAssignableFrom보다 isInstance가 잘 읽혀서 해당 메서드로 수정하였습니다!

참고: https://www.baeldung.com/java-isinstance-isassignablefrom

linirini · 2024-10-02T08:29:41Z

study/src/test/java/di/stage4/annotations/DIContainer.java

-    public DIContainer(final Set<Class<?>> classes) {
-        this.beans = Set.of();
+    public static DIContainer createContainerForPackage(final String rootPackageName) throws Exception {
+        Set<Class<?>> classes = ClassPathScanner.getAllClassesInPackage(rootPackageName);


LMS에 Repository와 Service를 스캔한다는 요구사항이 있어요!

네 그 요구사항은 ClassPathScanner 클래스 내부에서 처리해주고 있어요!

하지만 생각해보니 Service와 Repository를 스캔한다는 사실은 외부로 추출해야겠네요!

제가 클래스 스캐너를 놓쳤군요!😅

linirini · 2024-10-02T08:31:54Z

study/src/test/java/di/stage4/annotations/UserService.java

@@ -8,10 +8,11 @@ class UserService {
    @Inject
    private UserDao userDao;

+    public UserService() {


public으로 접근제어자를 바꾼 이유는 뭔가요?

해당 클래스를 빈으로 등록할 때 private 기본생성자에 대해 인스턴스를 생성해주기 위함이었습니다!

하지만 public으로 열지 않고 constructor.setAccessible(true)로 잠시 접근제어자를 열어주는 방법이 있었네요 :) 해당 방식으로 수정하였습니다!

linirini

에버~ 학습테스트까지 확인했습니다! 궁금했던 부분과 수정사항 코멘트가 좀 있어서 RC 했습니다. 반영 후 재리뷰요청해주세요:>

helenason · 2024-10-03T13:10:48Z

리니 리뷰 반영하고, 구구가 구현해두신 함수형 인터페이스로 리팩터링까지 완료한 후 재리뷰 요청 드립니다~~~
좋은 리뷰 감사해요 :)

linirini

에버~ 충분히 학습한 것 같아서 이만 merge 합니다!
이제 미션 하나만 남겨두고 있네요...ㅎ 마지막까지 화이팅입니다💛

helenason added 4 commits September 30, 2024 17:46

feat: JsonView 구현

c36eff8

test: JsonView 및 UserController

17d9af6

refactor: Legacy MVC 제거

3643262

test: 전체적인 테스트 보완

f5d5126

linirini requested changes Oct 1, 2024

View reviewed changes

helenason added 4 commits October 2, 2024 13:35

test: 학습테스트 IoC/DI

aac5307

refactor: 컨트롤러 메서드 시그니처 통일

3dac7a8

refactor: 스캔할 패키지 경로 외부에서 주입

31859ab

refactor: tobe 패키지 제거

a3850dd

linirini reviewed Oct 2, 2024

View reviewed changes

linirini requested changes Oct 2, 2024

View reviewed changes

helenason added 7 commits October 3, 2024 14:24

refactor: ControllerScanner 클래스 추출

fb0c1bd

test: ObjectMapper 파싱 방식 테스트

211fbec

refactor: UserService 기본생성자 접근제어자 닫기

95a7556

refactor: 어노테이션 필터링 로직 외부 추출

761923d

refactor: canAssign 메서드 로직 수정

e6e71ea

refactor: 컨트롤러 패키지 상수 app 모듈로 추출

d1d98ce

refactor: 예외 처리 함수형 인터페이스 적용

6dfc3da

linirini approved these changes Oct 4, 2024

View reviewed changes

linirini merged commit e2c04d1 into woowacourse:helenason Oct 4, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[MVC 구현하기 - 3단계] 에버(손채영) 미션 제출합니다. #865

[MVC 구현하기 - 3단계] 에버(손채영) 미션 제출합니다. #865

helenason commented Sep 30, 2024 •

edited

Loading

linirini left a comment

linirini Oct 1, 2024

helenason Oct 2, 2024 •

edited

Loading

linirini Oct 2, 2024

helenason Oct 3, 2024 •

edited

Loading

linirini Oct 1, 2024

linirini Oct 1, 2024

helenason Oct 2, 2024

linirini Oct 1, 2024

helenason Oct 2, 2024

linirini Oct 2, 2024

helenason Oct 3, 2024

linirini Oct 1, 2024

helenason commented Oct 2, 2024

linirini Oct 2, 2024

helenason Oct 3, 2024

linirini Oct 2, 2024

linirini Oct 2, 2024

linirini Oct 2, 2024

helenason Oct 3, 2024 •

edited

Loading

linirini Oct 2, 2024

helenason Oct 3, 2024

linirini Oct 3, 2024

linirini Oct 2, 2024

helenason Oct 3, 2024

linirini left a comment

helenason commented Oct 3, 2024

linirini left a comment


		private static JsonView INSTANCE;

		public static JsonView from() {

[MVC 구현하기 - 3단계] 에버(손채영) 미션 제출합니다. #865

[MVC 구현하기 - 3단계] 에버(손채영) 미션 제출합니다. #865

Conversation

helenason commented Sep 30, 2024 • edited Loading

linirini left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

helenason Oct 2, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

helenason Oct 3, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

helenason commented Oct 2, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

helenason Oct 3, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

linirini left a comment

Choose a reason for hiding this comment

helenason commented Oct 3, 2024

linirini left a comment

Choose a reason for hiding this comment

helenason commented Sep 30, 2024 •

edited

Loading

helenason Oct 2, 2024 •

edited

Loading

helenason Oct 3, 2024 •

edited

Loading

helenason Oct 3, 2024 •

edited

Loading