Skip to content

Commit fcad67e

Browse files
committed
refactor: LoginMemberArgumentResolver DI방식으로 변경
- 직접 생성자를 호출하게 되면 스프링 컨테이너 관리 대상이 아니게 되어 유지보수면에서 불편함 - 일관성 있게 adminInterceptor와 같이 DI방식으로 변경
1 parent 5549795 commit fcad67e

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

src/main/java/com/yourssu/roomescape/auth/LoginMemberArgumentResolver.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,14 @@
88
import com.yourssu.roomescape.util.JwtTokenProvider;
99
import jakarta.servlet.http.HttpServletRequest;
1010
import org.jetbrains.annotations.NotNull;
11+
import org.springframework.context.annotation.Configuration;
1112
import org.springframework.core.MethodParameter;
1213
import org.springframework.web.bind.support.WebDataBinderFactory;
1314
import org.springframework.web.context.request.NativeWebRequest;
1415
import org.springframework.web.method.support.HandlerMethodArgumentResolver;
1516
import org.springframework.web.method.support.ModelAndViewContainer;
1617

18+
@Configuration
1719
public class LoginMemberArgumentResolver implements HandlerMethodArgumentResolver {
1820

1921
private final MemberRepository memberRepository;

src/main/java/com/yourssu/roomescape/config/WebConfig.java

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
package com.yourssu.roomescape.config;
22

33
import com.yourssu.roomescape.auth.LoginMemberArgumentResolver;
4-
import com.yourssu.roomescape.member.MemberRepository;
5-
import com.yourssu.roomescape.util.JwtTokenProvider;
64
import org.springframework.context.annotation.Configuration;
75
import org.springframework.web.method.support.HandlerMethodArgumentResolver;
86
import org.springframework.web.servlet.config.annotation.*;
@@ -12,19 +10,17 @@
1210
@Configuration
1311
public class WebConfig implements WebMvcConfigurer {
1412

15-
private final MemberRepository memberRepository;
16-
private final JwtTokenProvider jwtTokenProvider;
1713
private final AdminInterceptor adminInterceptor;
14+
private final LoginMemberArgumentResolver loginMemberArgumentResolver;
1815

19-
public WebConfig(MemberRepository memberRepository, JwtTokenProvider jwtTokenProvider, AdminInterceptor adminInterceptor) {
20-
this.memberRepository = memberRepository;
21-
this.jwtTokenProvider = jwtTokenProvider;
16+
public WebConfig(AdminInterceptor adminInterceptor, LoginMemberArgumentResolver loginMemberArgumentResolver) {
2217
this.adminInterceptor = adminInterceptor;
18+
this.loginMemberArgumentResolver = loginMemberArgumentResolver;
2319
}
2420

2521
@Override
2622
public void addArgumentResolvers(List<HandlerMethodArgumentResolver> resolvers) {
27-
resolvers.add(new LoginMemberArgumentResolver(memberRepository, jwtTokenProvider));
23+
resolvers.add(loginMemberArgumentResolver);
2824
}
2925

3026
@Override

0 commit comments

Comments
 (0)