Project/codestates-final-project

4일차 / SR 피드백

fullfish 2022. 4. 23. 02:37

피드백 내용

에러로그 중요함

프론트엔드는 플로우차트가 정말 중요

백엔드는 API가 중요

DB는 수정하기 힘드니까 처음에 상세하게 짜야함

회원가입 중복의 경우 400말고 다른번호가 낫다

 

 

그리고 다음에 만났을때 다시 여쭤봐야할것이

2가지 조언받은부분에 대해서인데

1. /trip/:trip_id보다 trip/:id가 낫겠다

2. /user/:user_id/trip/:trip_id/diary/:diary_id를 /diary/:diary_id로만 만들어라인데

지금은 엔드포인트가

위의 2번경우처럼 다이어리를 삭제한다면

user : trip = 1 : N
trip = diary = 1: N이라서

/user/:user_id/trip/:trip_id/diary/:diary_id (앞으로 a라고 하겠음)인데 

/diary/:diary (앞으로 b라고 하겠음)로 바꾼다면 user_id와 trip_id는 body에 담아보내야하는것인지?

만약 그렇다면 이점이 무엇일지

 

당장드는 생각은

b의 경우는 버튼 클릭으로만 요청을 보낼 수 있는데 반해

a의 경우는 주소창에 직접 입력하는것으로도 요청을 보낼 수 있음 그래서 a는 로그인안해도 접근가능 -> 토큰인증 도입하면 해결될거라고 생각

또한 a의 경우는 id번호 노출됨

 

그리고 만약 a의 경우로 쓴다면 /:trip_id를 /:id로 바꾸면 user_id와 diary_id와의 구분을 못하지 않나...

 

restful api 어렵다