#5번 이슈까지 진행한 코드를 개선합니다 #26

wisdom08 · 2023-07-31T06:27:14Z

작업 목적

코드 안정성을 올리기 위해 메서드 파라미터에 final 키워드 추가
이전 PR에서 커밋하지 않은 map-underscore-to-camel-case 설정 추가
테스트 코드 내 변수명 변경
쓰지 않는 코드 제거

주요 변경점

final 키워드

참고

이슈 작성한 것과는 무관하게 진행한 거라 브랜치명을 임의로 refactor 로 지정했습니다.

- 추후 로그인유저 객체를 고려

ccommit-dev

피드백 드린 내용 댓글 추가 후 머지 부탁드립니다.

src/main/java/com/bookbook/controller/UserController.java

src/main/resources/application.yaml

- 변수의 재할당을 막음으로써 예측 가능한 코드를 만드는데 도움이 될 수 있지만 코드의 가독성을 해친다고 생각해 제거합니다

wisdom08 added 4 commits July 31, 2023 15:18

예측 가능한 코드를 만들기 위해 메소드 파라미터에 final 키워드를 붙입니다

192340c

map-underscore-to-camel-case 설정을 추가합니다

a2e470d

더이상 쓰지 않은 응답 코드 상수를 제거합니다

4c6b109

테스트코드에서 사용하는 유저의 변수명을 좀 더 구체적으로 변경합니다

2ef0741

- 추후 로그인유저 객체를 고려

wisdom08 added the enhancement New feature or request label Jul 31, 2023

wisdom08 requested a review from ccommit-dev July 31, 2023 06:27

wisdom08 self-assigned this Jul 31, 2023

ccommit-dev changed the title ~~작성한 코드를 개선합니다~~ 5번 이슈 까지 코드를 개선합니다 Jul 31, 2023

ccommit-dev reviewed Jul 31, 2023

View reviewed changes

src/main/java/com/bookbook/controller/UserController.java Outdated Show resolved Hide resolved

src/main/resources/application.yaml Show resolved Hide resolved

wisdom08 changed the title ~~5번 이슈 까지 코드를 개선합니다~~ #5번 이슈까지 진행한 코드를 개선합니다 Jul 31, 2023

메서드 파라미터의 final 키워드를 제거합니다

1a9a91b

- 변수의 재할당을 막음으로써 예측 가능한 코드를 만드는데 도움이 될 수 있지만 코드의 가독성을 해친다고 생각해 제거합니다

wisdom08 merged commit 40d4395 into main Jul 31, 2023

wisdom08 deleted the refactor branch July 31, 2023 17:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#5번 이슈까지 진행한 코드를 개선합니다 #26

#5번 이슈까지 진행한 코드를 개선합니다 #26

wisdom08 commented Jul 31, 2023 •

edited

Loading

ccommit-dev left a comment

#5번 이슈까지 진행한 코드를 개선합니다 #26

#5번 이슈까지 진행한 코드를 개선합니다 #26

Conversation

wisdom08 commented Jul 31, 2023 • edited Loading

작업 목적

주요 변경점

참고

ccommit-dev left a comment

Choose a reason for hiding this comment

wisdom08 commented Jul 31, 2023 •

edited

Loading