[항해99] TIL D+2

항해에는 좋은 사람들이 많다.

• 당장 우리 팀원분들만 해도 참 좋은 분들이고
• 요근래 만난 매니저님들도 다들 사람냄새나는 좋은 분들이셨다.

하지만 매니저님의 코드리뷰는 자비가 없지

• 오늘 매니저님께 강렬한 코드리뷰를 받아봤다.
• 진짜 어제 밤을 샌 영향도 있겠지만 영혼까지 탈탈탈 털렸다.
• (근데 우리 팀원분의 말씀으로는 이게 진짜 화목하게 진행된편이라고 한다…)
• (매니저님이 심적으로 많이 봐주신듯..)
• (탈탈 털린게 인격적으로 털린게 아니라 진짜 코드적으로만 털린것이다)
• 특히 오늘 코드 리뷰의 최고, 베스트 하이라이트는

하이라이트

• “파일 이름은 왜 날짜로 하셨어요?” 라는 질문에
  • 아주 그냥 막 정말 오래간만에 아는 거 나왔다고 신나가지고
• “사용자가 아무리 빨라도 1초 내에 사진을 두번 올릴 수는 없으니까 중복된 파일명을 가질 수 없게 하기 위해서 입니다! 😃”
  • 라고 자신있게 말하니 매니저님께서
• “그럼 사용자 2명이 동시에 업로드 하는 경우에는요? 아니면 한 사용자가 브라우저 두개 키고 이이용하면요?” 라는 지극히 당연한 질문에
• 내가 순간 할 말을 잃어서 어버버버버버벙버벙버어버 하던게 최고 백미였다 ㅋㅋㅋㅋㅋ…ㅠ
• (사실 내가 한 답은 어느 정도 정식적으로 배운 것이다. 즉 기본에 해당하는 답이고 개발자라면 항상 그 앞을, 한발 더 내다봐야한다는 매니저님의 깊은 뜻이 아니였을까 추측해본다.)

너무 유익하고 뜻깊고 빠른 성장 경험이었다.

• 다시 한번 이 자리를 빌어 진심으로 우리팀 담당 매니저님께 감사인사를 전한다.
• 매니저님 덕분에 코드리뷰를 어떻게 해야 더 잘 할수 있을지 알게 되었다.
• 앞으로 또 기회가 된다면 계속해서 탈탈 털렸으면 좋겠다.
• 그리고 고질병인 멘탈도 튼튼…(그렇다고 일상생활에서까지 털리면 안되지만..) 코드리뷰에서만큼은 탈탈 털려 앞으로 내 코딩 멘탈을 튼튼하게도 하고싶다.

---

코드리뷰 정리

1.

• 파일입출력의 흐름 bscustom이 init됐을때는 안궁금하고 뭔가 데이터가 뽑아지고 되는 것이다.
• 저 라이브러리를 안이용했으면 좋겠다.
• 아니면 라이브러리를 알아야된다.
• 대강이라도 대충이라도 트러블슈팅과 라이브러리는 잘 알아둬야 면접에서 잘 할 수 있다.

2.

• let 을 쓰면 휴먼에러가 나올수 있다.
• 이왕 습관적으로 쓸거면 let보다는 const를 쓰자

3.

• header의 contentType이 제대로 설정해줘야 좋다.
• header 부분이 명확하게, contentType, queryString 나중에 알게 된다.

4.

• api 에서 input 이란 이름은 많이 API end point를 잘 설정하는 것이 작명이 제일 어렵다.
• 명시적으로 눈에잘띄고 의미를 설명해줄수있는 end point를 작성할 수 있도록 하자.

5.

• 사용자가 동시에 사용하거나 할 수 있기 때문에, 시간을 사용하기보다,
• 난수를 만들어주어 유니크한 값을 해주는것이 더 좋다.