Project 2 - 5일 차 (6) : 내가 push 한 회원가입 함수 : pull request 팀원 리뷰 + 수정본 ** 해야 함

Wecode - Project 2 (부트캠프)/Project 2 과정

Project 2 - 5일 차 (6) : 내가 push 한 회원가입 함수 : pull request 팀원 리뷰 + 수정본 ** 해야 함

Queen Julia 2023. 9. 22. 16:18

username === undefined 는 !username 으로 축약

수정본

필수사항 및 선택사항 정보 받아온 거 저장하기

수정본

그런데, 선택사항은 어디서 받아오지?
key error는 필수사항에 대한 건데?

--> insert into query문에서
선택 + 필수 정보 모두 포함

만약, 유저가 선택 정보 기입을 안 했다면 'Undefined/ -'로 뜨겠지만,
일단 쓸 경우를 대비해서 가져오긴 해야지

[const error = new Error("KEY_ERROR");
error.statusCode = 400;
throw error;

해당 부분은 utils 폴더의 throwError(400, 'KEY_ERROR') 함수를 사용해서 축약될 수 있을 것 같습니당]

계속 반복되는 부분이라,

아마 이것을 활용하면 될 거 같다!

수정본

아직...

[ !emailRegex.test(email)

해당 부분은 utils 폴더의 isValidData(emailRegex, email) 함수를 사용해서 축약될 수 있을 것 같습니당]

수정본

아직...

pr comment 에는 없지만, 내가 발견한 오류 1

수정본

SELECT databaseId ->SELECT id

근데 select * FROM users WHERE email = '${email}'
하면 안되는지?
--> 최대한, * 은 안 쓰는 게 좋음. 쓰지 않을 데이터까지 다 가져오는 데이터 낭비가 되니까.
--> select id, email FROM users WHERE email = '${email}'