jungingook / punch_jungingookteam

[2020] OTP-QR코드 방식을 이용한 새로운 출결관리 앱입니다.
0 stars 4 forks source link

a few improvements #39

Open ljh95 opened 3 years ago

ljh95 commented 3 years ago

https://github.com/jungingook/punch_jungingookteam/blob/e95a254640093c98c947180d94a875cc174d4962/BackEnd/routes/account/index.js#L18

  1. js 구조 분해 할당 이용 ex) var o = {p: 42, q: true}; var {p, q} = o;

  2. (err, result1) 여기서 변수 명만 보고도 사람들이 어떤 에러고 어떤 result값인지 확인할 수 있는 변수명으로 변경할 것

  3. js의 throw는 어디로? => catch를 통해 잡거나 안잡으면 바로 프로그램이 종료된다. 따라서 에러 메세지를 좀더 명확하게 해줄 필요가 있다. 오늘 배운 네트워크에서 response code를 통해 client에서 어떤 종류의 error인지 확인시켜 주거나, client에게 보여줄 필요가 없나?(있을 것 같다. )서버에서도 안내 문구를 성실히 작성하자

  4. console.log(result): 이 부분은 만약 다른 사람이 본다면 어떤 것을 확인하기 위해 result를 console에 찍는지 모를 것이다. 더 안내 문자를 추가하자.

  5. res.json({ ... error: "Fail Sign_UP" ----> 이 부분도 왜 실패했는지 추가 안내문을 작성하는 것은 어떨까? })