Add 회원가입 기능 #5

phantom08266 · 2021-03-07T14:23:48Z

회원가입 기능을 통해 Careers를 로그인 한 사용자들만 이용할 수 있도록 한다.
이름, 이메일, 비밀번호 모두를 입력 받는다.(Null 체크)
이메일은 이메일 형식에 맞게 작성해야 한다.
비밀번호는 문자, 숫자, 특수문자로 구성되어야 한다.
비밀번호는 암호화하여 DB에 저장한다.(sha-256)
중복된 이메일인지 체크하여 중복가입을 제한한다.

- 회원가입 기능을 통해 Careers를 로그인 한 사용자들만 이용할 수 있도록 한다. - 이름, 이메일, 비밀번호 모두를 입력 받는다.(Null 체크) - 이메일은 이메일 형식에 맞게 작성해야 한다. - 비밀번호는 문자, 숫자, 특수문자로 구성되어야 한다. - 비밀번호는 암호화하여 DB에 저장한다.(sha-256) - 중복된 이메일인지 체크하여 중복가입을 제한한다. #3

sql/ddl.sql

src/main/java/com/dev/careers/controller/CuratorController.java

src/main/java/com/dev/careers/model/Curator.java

src/main/java/com/dev/careers/service/CuratorService.java

src/main/resources/application.properties

src/main/resources/mybatis/CuratorMapper.xml

src/test/java/com/dev/careers/controller/CuratorControllerTest.java

put -> post

src/main/java/com/dev/careers/controller/CuratorController.java

src/main/java/com/dev/careers/mapper/CuratorMapper.java

src/main/java/com/dev/careers/model/Curator.java

src/main/java/com/dev/careers/service/CuratorService.java

이메일, 비밀번호 위반 시 status code 반환하도록 수정

단일책임원칙 적용

명확한 의도를 설정하기위해 Data 애노테이션 대신 각각 설정

여러 스레드가 해당 부분을 접근할 일이 현재로서는 없으므로 StringBuilder로 수정

src/main/java/com/dev/careers/model/Curator.java

src/main/java/com/dev/careers/service/CuratorService.java

src/main/java/com/dev/careers/service/encryption/SHA256Encryption.java

src/main/java/com/dev/careers/service/error/CuratorExceptionHandler.java

src/main/resources/mybatis/CuratorMapper.xml

…ion.java Fix 코딩 컨벤션 위반 수정 Co-authored-by: f-lab <54677861+f-lab-dev@users.noreply.github.com>

Encryption -> Encryptor 로 변경

src/main/java/com/dev/careers/service/CuratorService.java

f-lab-dev · 2021-03-15T07:18:07Z

src/main/resources/mybatis/CuratorMapper.xml

-        SELECT email from Curator;
-    </select>
+  <select id="getCuratorsEmail" resultType="String">
+    SELECT email


이메일로 이메일을 불러온다라는건 애매하니 존재하는지 확인한다로 가는게 어떨까요?

exists를 사용하여 쿼리성능향상 및 이메일 존재 유무에 대해 명확하게 표현

f-lab-dev

커밋 메세지를 보면 성능 향상이라고 적혀있지만 쿼리 및 메소드 용도의 명확성을 위해서가 더 맞기는 합니다~

ellieMayVelasquez · 2021-03-21T13:17:39Z

src/test/java/com/dev/careers/service/CuratorServiceTest.java

+
+    @Test
+    @DisplayName("중복된 이메일 회원가입 요청")
+    public void DupicatedEmail() throws Exception {


I detect that this code is problematic. According to the Bad practice (BAD_PRACTICE), Nm: Method names should start with a lower case letter (NM_METHOD_NAMING_CONVENTION).
Methods should be verbs, in mixed case with the first letter lowercase, with the first letter of each internal word capitalized.

카멜케이스 적용

phantom08266 added 2 commits March 7, 2021 23:12

Merge remote-tracking branch 'origin/develop' into feature/3/jh

7b3570e

phantom08266 requested a review from f-lab-dev March 7, 2021 14:23

phantom08266 self-assigned this Mar 7, 2021

f-lab-dev reviewed Mar 8, 2021

View reviewed changes

phantom08266 added 6 commits March 9, 2021 23:44

Fix 디비 primary key id로 수정

bff0efd

Fix 큐레이터 테이블 unique index 설정

3770ac2

Fix 회원가입 http method 변경

dc7a4ea

put -> post

Fix 롬복사용하여 생성자 자동주입

b382fc4

Fix 사용하지 않는 요청 메서드 제거

d49212b

Fix 이메일 중복 예외처리

c9b7c1b

f-lab-dev reviewed Mar 10, 2021

View reviewed changes

phantom08266 added 8 commits March 10, 2021 23:16

Fix http status code 처리

10171e7

이메일, 비밀번호 위반 시 status code 반환하도록 수정

Fix model코드 가독성 처리

a0ef85c

Fix 암호화코드와 유저서비스 기능 분리

c8376d0

단일책임원칙 적용

Fix Select쿼리문 email컬럼 명시적으로 표현

78554c7

Fix mysql 사용자계정 변경

d94cac5

Fix 롬복 Data 애노테이션 제거

72e40af

명확한 의도를 설정하기위해 Data 애노테이션 대신 각각 설정

Fix 롬복 애노테이션 정리

748aa39

Fix StringBuffer를 StringBuilder로 변경

e056ccb

여러 스레드가 해당 부분을 접근할 일이 현재로서는 없으므로 StringBuilder로 수정

phantom08266 requested a review from f-lab-dev March 10, 2021 16:33

f-lab-dev reviewed Mar 11, 2021

View reviewed changes

phantom08266 and others added 4 commits March 11, 2021 23:24

Fix 사용하지 않는 애노테이션 제거

ad907a8

Update src/main/java/com/dev/careers/service/encryption/SHA256Encrypt…

95e8c60

…ion.java Fix 코딩 컨벤션 위반 수정 Co-authored-by: f-lab <54677861+f-lab-dev@users.noreply.github.com>

Fix 인터페이스, 클래스 명명규칙 수정

57b7f59

Encryption -> Encryptor 로 변경

Fix 메소드 명명규칙 수정

3535d36

f-lab-dev reviewed Mar 12, 2021

View reviewed changes

src/main/java/com/dev/careers/service/CuratorService.java Outdated Show resolved Hide resolved

phantom08266 added 3 commits March 14, 2021 00:52

Fix 요청에따른 결과 반환 void로 수정

fc65cb6

Fix void반환으로 인한 테스트케이스 수정

56dbcd1

Fix 성능보완을 위해 쿼리로 중복검증 체크

f507d64

f-lab-dev reviewed Mar 15, 2021

View reviewed changes

Fix 성능향상을 위해 쿼리문 수정

510ae49

exists를 사용하여 쿼리성능향상 및 이메일 존재 유무에 대해 명확하게 표현

phantom08266 requested a review from f-lab-dev March 16, 2021 14:41

f-lab-dev approved these changes Mar 17, 2021

View reviewed changes

f-lab-dev previously approved these changes Mar 19, 2021

View reviewed changes

ellieMayVelasquez reviewed Mar 21, 2021

View reviewed changes

phantom08266 dismissed f-lab-dev’s stale review via 1e98461 March 23, 2021 13:43

Fix checkstyle 위반된 부분 수정

1e98461

카멜케이스 적용

phantom08266 requested a review from f-lab-dev March 23, 2021 13:44

f-lab-dev approved these changes Mar 24, 2021

View reviewed changes

dnjwm8612 approved these changes Apr 4, 2021

View reviewed changes

phantom08266 merged commit 60ed0cc into develop Apr 6, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add 회원가입 기능 #5

Add 회원가입 기능 #5

phantom08266 commented Mar 7, 2021

f-lab-dev Mar 15, 2021

f-lab-dev left a comment

ellieMayVelasquez Mar 21, 2021

Add 회원가입 기능 #5

Add 회원가입 기능 #5

Conversation

phantom08266 commented Mar 7, 2021

f-lab-dev Mar 15, 2021

Choose a reason for hiding this comment

f-lab-dev left a comment

Choose a reason for hiding this comment

ellieMayVelasquez Mar 21, 2021

Choose a reason for hiding this comment