Feature join #5

CUCU7103 · 2024-11-22T04:39:52Z

📌 PR 제목

회원가입 로직 및 프로젝트 로깅 설정

🚀 변경 사항 (Changes)

회원가입 로직 작성
회원가입 시 비밀번호 암호화 하여 저장
회원가입 로직 테스트 코드 작성
프로젝트 로깅 설정 진행
- log4j2 적용
- p6spy 적용

💥 관련 이슈 (Related Issues)

💬 리뷰 요구사항

🚴 체크리스트 (Checklist)

else 키워드 사용하지 않기
getter/setter/property를 쓰지 않기
한 줄에 점을 하나만 찍기 (디미터의 법칙)
테스트 코드 작성여부
네이버 코드 컨벤션 준수여부

jpa,db 연결을 위한 의존성 주입 및 설정 진행과 프로젝트 디렉터리명에 오타를 발견하여 수정함.

회원가입 기능에 필요한 entity , repository ,dto 등 클래스 생성후 기능 구현함

ResponseEntity의 응답 메시지를 커스텀하기 위한 클래스 생성 및 회원가입시 비밀번호를 암호화 하기 위한 클래스 생성

P6spy 적용을 위한 클래스 생성 및 application.yml 설정 진행

로그 프레임 워크를 logback에서 log4j2로 변경함.

Testcontainer를 사용할 수있도록 테스트 환경을 구축하고 실제로 동작하는지 확인하기 위한 테스트 코드 작성

회원 가입 테스트 로직 작성 및 application-test.yml 파일 설정 변경 진행

불필요한 코드와 어노테이션 제거 및 누락된 어노테이션 속성 추가

rest doc 적용하면서 불필요한 테스트 코드 삭제 진행

컨트롤러의 반환값과 비즈니스 로직에서 잘못된 값을 전달하고 반환하는 부분을 확인하여 수정 진행함

f-lab-jd

고생하셨습니다! 리뷰드린 내용 확인 후 답변 부탁드리겠습니다~

src/main/java/spring/academyPlatform/AcademyPlatformApplication.java

src/main/java/spring/academyPlatform/global/mapper/UserMapper.java

src/main/java/spring/academyPlatform/global/util/BcryptPasswordEncryptor.java

src/main/java/spring/academyPlatform/user/domain/User.java

src/main/resources/application.yml

src/main/resources/log4j2-spring.xml

build.gradle

UserMapper의 위치를 global에서 User 도메인 하위로 이동 및 불필요한 어노테이션 삭제

매직넘버를 상수로 변경함

사용하지 않는 api 삭제 진행함

Message 클래스 제거와 이에 따른 코드 수정

DTO 클래스 명칭 변경 및 어노테이션 수정

builder에서 Mapper 사용으로 변경 및 사용하지 않는 로직 삭제

고정 값으로 입력되어지는 값을 enum 클래스로 관리하도록 변경함

불필요한 설정을 삭제함

enum type으로 변경된 컬럼으로 인한 수정

클래스명 변경으로 인한 수정 진행

f-lab-jd · 2024-12-01T12:39:16Z

src/main/java/spring/academyPlatform/user/api/UserController.java

+	private final UserService userService;
+
+	@PostMapping("/user")
+	public ResponseEntity<Object> getUser(@Valid @RequestBody UserCreateRequest dto) {


f-lab-jd · 2024-12-01T12:39:40Z

src/main/java/spring/academyPlatform/user/api/UserController.java

+		try {
+			UserCreateResponse result = userService.createUser(dto);
+			Map<String, Object> resultMap = new HashMap<>();
+			resultMap.put("user", result);


f-lab-jd · 2024-12-01T12:40:31Z

src/main/java/spring/academyPlatform/user/dto/UserCreateRequest.java

+@Getter
+public class UserCreateRequest {
+	@NotBlank
+	String userId;


f-lab-jd · 2024-12-01T12:40:59Z

src/main/java/spring/academyPlatform/user/mapper/UserMapper.java

+import spring.academyPlatform.user.dto.UserCreateRequest;
+import spring.academyPlatform.user.dto.UserCreateResponse;
+
+@Component


f-lab-jd · 2024-12-01T12:41:18Z

src/main/java/spring/academyPlatform/user/model/TypeCode.java

+package spring.academyPlatform.user.model;
+
+public enum TypeCode {
+	student, teacher;


f-lab-jd · 2024-12-01T13:13:27Z

src/main/java/spring/academyPlatform/user/dto/UserCreateResponse.java

+	@NotBlank
+	String userName;
+	@NotBlank
+	TypeCode userType;


f-lab-jd · 2024-12-01T13:13:58Z

src/main/java/spring/academyPlatform/user/dto/UserCreateResponse.java

+import lombok.Builder;
+import lombok.Getter;
+import spring.academyPlatform.user.model.TypeCode;
+import spring.academyPlatform.user.model.YnCode;


YnCode 클래스 패키지 이동 및 TypeCode 클래스 UserTypeCode로 명칭 변경함

기존 Message 클래스를 return 값으로 받는 로직에서 dto 클래스를 받는 로직으로 변경함.

기존type 에서 enum type으로의 변경에 따른 로직 수정 및 변환 메서드 추가

dto 클래스에 누락된 접근제어자를 추가하고 enum type으로 의 변경 진행

f-lab-jd

👍

CU7103 added 10 commits November 14, 2024 23:16

Chore: 라이브러리 의존성 주입과, 프로젝트 디렉터리명 변겅

94d67fe

jpa,db 연결을 위한 의존성 주입 및 설정 진행과 프로젝트 디렉터리명에 오타를 발견하여 수정함.

Feat: 회원가입 기능 생성

251a7b5

회원가입 기능에 필요한 entity , repository ,dto 등 클래스 생성후 기능 구현함

Feat: ResponseEntity 응답 커스텀 및 회원가입시 암호화 기능추가

80c6187

ResponseEntity의 응답 메시지를 커스텀하기 위한 클래스 생성 및 회원가입시 비밀번호를 암호화 하기 위한 클래스 생성

Init: P6spy 적용

ab31016

P6spy 적용을 위한 클래스 생성 및 application.yml 설정 진행

Init: log4j2 적용

b8a4576

로그 프레임 워크를 logback에서 log4j2로 변경함.

Test: Testcontainer를 사용하여 테스트 환경 구축

70444a7

Testcontainer를 사용할 수있도록 테스트 환경을 구축하고 실제로 동작하는지 확인하기 위한 테스트 코드 작성

Test: 회원가입 테스트 로직 작성

7e758a9

회원 가입 테스트 로직 작성 및 application-test.yml 파일 설정 변경 진행

Refactor: 회원가입 관련 클래스 수정

dcf0726

불필요한 코드와 어노테이션 제거 및 누락된 어노테이션 속성 추가

Chore: 불필요한 테스트 코드 삭제

a591383

rest doc 적용하면서 불필요한 테스트 코드 삭제 진행

Refactor: 회원가입 로직 리펙터링 진행

2c7b6e4

컨트롤러의 반환값과 비즈니스 로직에서 잘못된 값을 전달하고 반환하는 부분을 확인하여 수정 진행함

CUCU7103 requested a review from f-lab-jd November 22, 2024 04:40

CUCU7103 self-assigned this Nov 22, 2024

f-lab-jd reviewed Nov 23, 2024

View reviewed changes

CU7103 added 10 commits November 28, 2024 02:12

Refactor: UserMapper 리펙터링

911eea4

UserMapper의 위치를 global에서 User 도메인 하위로 이동 및 불필요한 어노테이션 삭제

Refactor: 매직넘버 변환

d7ceb07

매직넘버를 상수로 변경함

Refactor: 사용하지 않는 api 삭제

8cb857e

사용하지 않는 api 삭제 진행함

Refactor: Message 클래스 제거

a10d23b

Message 클래스 제거와 이에 따른 코드 수정

Refactor: DTO 클래스 리펙터링

d0f6463

DTO 클래스 명칭 변경 및 어노테이션 수정

Refactor: UserService 클래스 리펙터링

a11dd83

builder에서 Mapper 사용으로 변경 및 사용하지 않는 로직 삭제

Feat: enum 클래스 생성

f1391e2

고정 값으로 입력되어지는 값을 enum 클래스로 관리하도록 변경함

Chore: 주석 처리된 설정 삭제

d862cda

불필요한 설정을 삭제함

Refactor: User 클래스 수정

bd9a434

enum type으로 변경된 컬럼으로 인한 수정

Chore: 변경된 클래스명에 의한 수정 진행

24d1dc2

클래스명 변경으로 인한 수정 진행

f-lab-jd reviewed Dec 1, 2024

View reviewed changes

CU7103 added 4 commits December 2, 2024 22:35

Refactor: enum 클래스 수정

5eeba06

YnCode 클래스 패키지 이동 및 TypeCode 클래스 UserTypeCode로 명칭 변경함

Refactor: ResponseEntity return type 수정

9c921f6

기존 Message 클래스를 return 값으로 받는 로직에서 dto 클래스를 받는 로직으로 변경함.

Refactor: enum type으로의 변경

8f0a1d5

기존type 에서 enum type으로의 변경에 따른 로직 수정 및 변환 메서드 추가

Refactor: dto 접근제어자 추가 및 수정

1d20d60

dto 클래스에 누락된 접근제어자를 추가하고 enum type으로 의 변경 진행

f-lab-jd approved these changes Dec 7, 2024

View reviewed changes

CUCU7103 merged commit b94e40c into develop Dec 7, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature join #5

Feature join #5

CUCU7103 commented Nov 22, 2024

f-lab-jd left a comment

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd Dec 1, 2024

f-lab-jd left a comment

Feature join #5

Feature join #5

Conversation

CUCU7103 commented Nov 22, 2024

📌 PR 제목

🚀 변경 사항 (Changes)

💥 관련 이슈 (Related Issues)

💬 리뷰 요구사항

🚴 체크리스트 (Checklist)

f-lab-jd left a comment

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd Dec 1, 2024

Choose a reason for hiding this comment

f-lab-jd left a comment

Choose a reason for hiding this comment