[Spring 지하철 경로 조회 - 1, 2단계] 검프(김태정) 미션 제출합니다. #100

Livenow14 · 2021-05-14T14:50:51Z

안녕하세요 데이브..! 처음뵙겠습니다. 잘부탁드립니다.

vue.js는 처음이네요.. 프론트 코드를 이해하는데도 꽤나 시간이 걸린거 같아요 😂

백엔드 코드는 기본적인 스켈레톤 코드가 작성되어 있어서 수월했어요.

주로 작성한 부분은.

전역적 ExceptionHandler 처리.

레이어별 테스트 등입니다.

피드백을 통해 성장하겠습니다..! 많은 피드백 기다리겠습니다 감사합니다

…발생한다.

Livenow14 · 2021-05-14T14:51:11Z

검프 학습로그

관계설정

클래스 사이의 관계는 코드에 다른 클래스 이름이 나타나기 때문에 만들어짐.

하지만 객체 사이의 관계는 그렇지 않음. 코드에서 A라는 클래스를 전혀 알지 못하더라도 A 클래스가 구현한 인터페이스를 사용했다면, A 클래스의 객체를 인스턴스 타입으로 받아서 사용할 수 있음.

이는 다형성이라는 특징 덕분임.

즉. 객체 서로가 구현에 의존하는 것이 아닌 역할에 의존한다면, 역할을 구현하는 방법이 바뀌더라도, 상대 객체에는 아무런 영향을 주지 않음.

또한, 객체를 사용하는 클라이언트를 변경하지 않아도, 새로운 기능을 제공할 수 있다는 장점을 얻을 수 잇음.

대체가능성과 확장성을 얻음.

제어의 역전

서비스를 2개의 영역으로 나눠보면 사용 영역, 구성 영역으로 나눌 수 있어요.

제어가 역전됐다는 것은 프로그램의 제어 흐름 구조가 뒤 바뀐 것이라 할 수 있어요.

일반적인 프로그램은 main() 메소드와 같이 프로그램이 시작되는 지점에서 다음에 사용할 객체를 결정하고, 결정할 객체를 생성하고, 만들어진 객체에 있는 메소드를 호출하고 등의 작업이 반복되요.

즉, 모든 종류의 작업을 사용 영역에서 제어하는 구조에요.

제어의 역전이란 이런 제어 흐름의 개념을 거꾸로 뒤집는 것이에요.

제어의 역전에서는 객체가 자신이 사용할 오브젝트를 스스로 선택하지 않고, 생성하지도 않아요. 또한 자신이 어떻게 만들어지고 어디서 사용되는지를 알 수 없어요. 모든 제어 권한을 자신이 아닌 다른 대상에게 위임하기 때문이에요.

즉, 모든 종류의 작업을 구성 영역에서 제어하는 구조에요.

라이브러리와 프레임워크의 차이가 여기서 나요

라이브러리를 사용하는 애플리케이션 코드는 애플리케이션 흐름을 직접 제어해요. 단지 동작하는 중에 필요한 기능이 있을 때 능동적으로 라이브러리를 사용할 뿐이에요.

반면에 프레임워크는 애플리케이션 코드가 프레임워크에 의해 사용돼요. 보통 프레임워크 위에 개발한 클래스를 등록해두고, 프레임워크가 흐름을 주도하는 중에 개발자가 만든 애플리케이션 코드를 사용하도록 만드는 방식이에요

프레임워크에는 분명한 제어의 역전(IOC) 개념이 적용되어야해요

즉, 애플리케이션 코드는 프레임워크가 짜놓은 틀에서 수동적으로 동작해야해요.

금방 작성했던 코드도 IoC를 적용한 것이에요.

# 구성영역
		public class DaoFactory {

    public UserDao userDao() {
        return new UserDao(connections());
    }

    public AccountDao accountDao() {
        return new AccountDao(connections());
    }

    public MessageDao messageDao() {
        return new MessageDao(connections());
    }

    private Connections connections() {
        return new DConnections();
    }
}

# 사용영역
public class UserDaoTest {
    public static void main(String[] args) {
        UserDao userDao = new DaoFactory().userDao();
    }
}

dave915

검프 안녕하세요! 리뷰를 맡게된 데이브 입니다!
리뷰가 늦었네요 🙇‍♂️
몇가지 피드백 남겼어요. 확인 부탁드릴게요!
궁금한 점 있으면 언제든 편하게 질문 주세요 :)

dave915 · 2021-05-16T08:42:18Z

src/main/java/wooteco/subway/GlobalExceptionHandler.java

+    @ExceptionHandler({AuthException.class})
+    public ResponseEntity<String> authException(RuntimeException exception) {
+        return ResponseEntity.status(HttpStatus.UNAUTHORIZED).body(exception.getMessage());


dave915 · 2021-05-16T08:43:05Z

src/main/java/wooteco/subway/member/ui/MemberController.java

        return ResponseEntity.ok().build();
    }

    // TODO: 구현 하기
    @DeleteMapping("/members/me")
-    public ResponseEntity<MemberResponse> deleteMemberOfMine() {
+    public ResponseEntity<MemberResponse> deleteMemberOfMine(@AuthenticationPrincipal LoginMember loginMember) {


noContent 면 응답을 Void로 해놓아도 좋을 것 같아요 :)

Suggested change

public ResponseEntity<MemberResponse> deleteMemberOfMine(@AuthenticationPrincipal LoginMember loginMember) {

public ResponseEntity<Void> deleteMemberOfMine(@AuthenticationPrincipal LoginMember loginMember) {

body를 포함하지 않는 메소드, 전체 Void를 반환하도록 변경했습니다 :)

f97da8a

dave915 · 2021-05-16T08:45:19Z

src/main/java/wooteco/subway/auth/ui/AuthenticationPrincipalArgumentResolver.java

-        // TODO: 유효한 로그인인 경우 LoginMember 만들어서 응답하기
-        return null;
+        String token = AuthorizationExtractor.extract(Objects.requireNonNull(webRequest.getNativeRequest(HttpServletRequest.class)));
+        return  authService.findByToken(token);


지금 같은 경우 @AuthenticationPrincipal 어노테이션이 있을때만 로그인 검증을 할 텐데 intercepter 를 사용해서 @AuthenticationPrincipal를 사용하지 않는 api 들도 인증한 사용자인지 체크하도록 해보면 좋을 것 같아요!

@AuthenticationPrincipal을 사용하지 않는 api또한 인증된 사용자인지 체크하도록 변경했어요 😊

@Override public void addInterceptors(InterceptorRegistry registry) { registry.addInterceptor(new LoginInterceptor(authService)) .addPathPatterns("/**") .excludePathPatterns("/login/token", "/members"); }

여기서 질문이 있어요..!

저는 post의 /members만 inteceptor의 영향을 받고싶지 않은데,

지금 같은경우는 members의 path는 메소드에 상관없이 다 허용하고있어요.

이런경우 어떻게 해야하는걸까요...?

bcf2449

인터셉터는 Path로만 제어 가능 한 것으로 알고 있어요 😭
아래 링크 처럼 인터셉터 동작 후 처리 하던가, 원하지 않는 api의 path를 변경 해야 할 것 같네요.

https://stackoverflow.com/questions/48306597/spring-mvc-interceptor-pattern

dave915 · 2021-05-16T08:47:04Z

frontend/src/pages/member/LoginPage.vue

+          body: JSON.stringify(jsonData)
+        };
+
+        const response = await fetch("http://localhost:8080/login/token", option);


http://localhost:8080 같은경우 환경에 따라 달라질 수 있기 때문에 보통 Path로만 요청하고 있어요 :)

Suggested change

const response = await fetch("http://localhost:8080/login/token", option);

const response = await fetch("/login/token", option);

view.config의 수정을 통해, path로 요청할 수 있게 번경했습니다.

const path = require('path'); module.exports = { outputDir: path.resolve(__dirname, '../src/main/resources/static/'), devServer: { proxy: { '/api': { target: 'http://localhost:8080/', ws: true, changeOrigin: true, }, }, historyApiFallback: true, }, transpileDependencies: [ 'vuetify' ] }

또한 기본 path에 /api를 추가했어요.

72d7456

dave915 · 2021-05-16T08:49:24Z

frontend/src/pages/member/LoginPage.vue

+        const response = await fetch("http://localhost:8080/login/token", option);
+        let resJson = await response.json();
+        const accessToken = resJson.accessToken;
+        this.$store.commit(SET_ACCESS_TOKEN, accessToken);


오호.. store!
store에 토큰을 저장 할 경우 새로고침하면 store가 초기화 되지 않나요? 🤔

새로고침을 전혀 생각하지 않았네요..!

vuex-persistedstate를 사용하며,
실질적으로 사용하는 member, auth 모듈의 state만 저장되게 번경햇습니다.😊

좋은 피드백 감사합니다!

72d7456

dave915 · 2021-05-16T08:51:12Z

src/main/java/wooteco/subway/GlobalExceptionHandler.java

+
+@RestControllerAdvice
+public class GlobalExceptionHandler {
+    @ExceptionHandler({AuthException.class})


지금 미션과 상관은 없지만 Exception.class 를 처리하는 handler도 만들어보면 좋을 것 같아요 :)

사용자 잘못이 아닌, 서버 자체의 예외 처리를 말씀하시는 거죠..?

우선 이번 서비스에서 중요 로직을 담당하는 데이터베이스의 예외를 잡기위해

DataAccesException으로 서버 내부 예외에 대한 처리를 완료했어요. 😊

@ExceptionHandler({DataAccessException.class}) public ResponseEntity<String> databaseException(RuntimeException exception) { logger.info(exception.getMessage()); return ResponseEntity.status(HttpStatus.UNAUTHORIZED).body(exception.getMessage()); }

bf54781

DataAccesException.class 이외의 에러가 발생했을때는 어떻게 해야할까요?
보통 @ExceptionHandler({Exception.class}) 을 만들어서 개발자가 예상하지 못한 에러가 발생시 처리해주고 있어요 :)

dave915 · 2021-05-16T08:52:14Z

src/main/java/wooteco/subway/member/ui/MemberController.java

    @PutMapping("/members/me")
-    public ResponseEntity<MemberResponse> updateMemberOfMine() {
+    public ResponseEntity<TokenResponse> updateMemberOfMine(@AuthenticationPrincipal LoginMember loginMember, @RequestBody MemberRequest request ) {


@Valid 어노테이션을 이용해서 MemberRequest의 유효성 체크도 해보면 좋을 것 같아요 :)

spring starterr의 validatiion의존성을 추가해, 바인딩시 유효셩 검사를 진행했어요.

6c7f050

src/main/java/wooteco/subway/member/ui/MemberController.java

dave915 · 2021-05-16T08:56:26Z

src/main/java/wooteco/subway/member/application/MemberService.java

    public void updateMember(Long id, MemberRequest memberRequest) {
        memberDao.update(new Member(id, memberRequest.getEmail(), memberRequest.getPassword(), memberRequest.getAge()));
    }

    public void deleteMember(Long id) {
        memberDao.deleteById(id);
    }
+
+    public Member findMemberByEmail(TokenRequest request) {


TokenRequest 채로 받는게 맞을지 고민되네요 🤔
email로만 Member를 조회하고 login 메소드에서 비밀번호 체크하는건 어떠신가요?

Service와 도메인의 책임을 더 생각해보게 됐어요..!
수정했습니다..! 좋은 피드백 감사합니다.

e03250f

Livenow14 · 2021-05-17T09:01:56Z

안녕하세요 데이브! 두번째 pr요청입니다..!
세세한 부분들을 깊게 생각해보지 못했던거 같아요!! 피드백을 통해 세세한 부분들을 많이 체크하게 됐어요 ㅎㅎ
좋은 피드백 감사합니다 😊

질문 사항은 👀 를 통해 남겼어요..
이번에도 많은 피드백 부탁드려요..!🙇🏻‍♂️🙇🏻‍♂️🙇🏻‍♂️

dave915

검프 안녕하세요!
피드백 반영 한 부분 확인했어요 👍
질문 주신 부분에 대해서도 답변 남겼어요. 확인 부탁드릴게요!
이번 단계는 이만 머지할게요 :)

Livenow14 and others added 19 commits May 12, 2021 15:42

feat: Initialize

54c9dfe

docs: 서로에 대해 알아가기

e10a9d5

docs: 기능 요구사항 정리

4410d4a

docs: 기능 요구 사항 정리

4a8c27f

feat(AuthController.class): 액세스 토큰 발급 테스트

2dffaf0

feat: 토큰 로그인 시 아이디 비밀번호 검증 추가

7deb7cf

feat(AuthException.class): 토큰 발급시 등록된 이메일이 아니거나, 패스워드가 일치하지 않을 시 예외가 …

4d4ba05

…발생한다.

feat(AuthService.class): jwt 토큰 유효성 검증 추가

fc16f06

refactor(LineResponse.class): 기본생성자 추가

752677e

feat: Front login, 화원가입 구현 완료

f5c1d62

fix: 로그인 예외 수정 및 todo정리

7c6fc1a

feat: Member 대신 LoginMember 반환 로직 변경

5d6e048

feat: update 구현 및 Test작성

6dd5eac

feat: delete 구현 및 Test 작성

de73e71

feat: 프론트 login api 작성

1878b3c

featL 프론트 멤버 수정 로직 작성

902fe3b

feat: 프론트 삭제 api 작성

08b7172

refactor(MyPage.class): store에 저장된 member 삭제

6583195

docs: 기능목록 완료

fc28397

Livenow14 changed the base branch from master to livenow14 May 14, 2021 14:51

dave915 suggested changes May 16, 2021

View reviewed changes

Livenow14 added 8 commits May 17, 2021 12:18

refactor: ResponseEntity 반환값 변경

f97da8a

feat: @AuthenticationPrincipal을 사용하지 않는 api또한 인증된 사용자인지 체크

bcf2449

refactor: lcathost부분 제거 및 controller 메서드 path 변경

72d7456

feat: vuex-persistedstate 적용

b6f81fc

feat: 데이터베이스 예외 핸들링 추가

bf54781

feat(MemberRequest.class): validation 의존성 추가 및 적용

6c7f050

feat: 역이름 validation생성

698fe25

feat: 노선이름 validation생성

802e8da

Livenow14 added 2 commits May 17, 2021 17:36

refactor: 사용하지 않는 출력문 제거

1481aa5

reafctor: 비밀번호 체크 로직 변경

e03250f

dave915 approved these changes May 19, 2021

View reviewed changes

dave915 merged commit 08a9293 into woowacourse:livenow14 May 19, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Spring 지하철 경로 조회 - 1, 2단계] 검프(김태정) 미션 제출합니다. #100

[Spring 지하철 경로 조회 - 1, 2단계] 검프(김태정) 미션 제출합니다. #100

Livenow14 commented May 14, 2021

Livenow14 commented May 14, 2021

dave915 left a comment

dave915 May 16, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 19, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 19, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

dave915 May 16, 2021

Livenow14 May 17, 2021

Livenow14 commented May 17, 2021

dave915 left a comment

	public ResponseEntity<MemberResponse> deleteMemberOfMine(@AuthenticationPrincipal LoginMember loginMember) {
	public ResponseEntity<Void> deleteMemberOfMine(@AuthenticationPrincipal LoginMember loginMember) {

	const response = await fetch("http://localhost:8080/login/token", option);
	const response = await fetch("/login/token", option);

[Spring 지하철 경로 조회 - 1, 2단계] 검프(김태정) 미션 제출합니다. #100

[Spring 지하철 경로 조회 - 1, 2단계] 검프(김태정) 미션 제출합니다. #100

Conversation

Livenow14 commented May 14, 2021

주로 작성한 부분은.

Livenow14 commented May 14, 2021

검프 학습로그

관계설정

제어의 역전

라이브러리와 프레임워크의 차이가 여기서 나요

dave915 left a 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

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

Choose a reason for hiding this comment

Livenow14 commented May 17, 2021

dave915 left a comment

Choose a reason for hiding this comment