[Merge] 1주차 머지 리퀘스트 입니다 - [merged] #110

Closed seonghbae closed 2 years ago

In GitLab by @about2weeks on May 27, 2022, 23:28

Merges dev -> master

상세한 내용은 1st Merge Request를 확인 부탁드립니다.

vm에서 접속 가능하신 주소는 http://kdt-sw2-seoul-team13.elicecoding.com:5000/ 입니다.

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 27

위 코드를 아래와 같이 바꾸면 어떨까요?

const {fullName, email, password, address, phoneNumber, gender, role} = req.body;

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 29

불필요한 console.log 코드는 삭제해주시면 좋아요! 그럼 왜 console.log 코드를 삭제해주는 것이 좋을까요? 만약 console.log 코드를 작성하고 배포한다면, 어떤 일이 벌어질까요!?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 61

이 코드도 const {email, password} = req.body; 처럼 변경해보는 것은 어떨까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 78

res라는 응답객체에 cookie와 status, json을 한 줄로 보낼 수 있지 않을까요?!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 77

쿠키에 이 설정을 해야할 이유가 있을까요? 어떤 이유에서 login, true라는 설정이 필요할까요? 로그인됐다는 상태를 기록해야 할 이유가 있을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 125

shortId라는 것은 어떤 의미일까요? 변수를 선언할 때, 변수가 어떤 의미인지 이름만으로도 알 수 있어야 합니다!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 114

edit이라는 url을 설정해야 하는 이유가 있을까요?

Patch '/' 라는 API와 Patch 'edit' 라는 API 중, 어떤 API가 더 Rest API에 가까울까요? 한번 고민해보면 좋겠습니다.

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 133

가능하다면 여기도 코드의 양을 줄여보는 것은 어떠할까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/user-router.js line 199

deleteUser에 userInfoRequired라는 변수를 주입했는데요!

userInfoRequired라는 변수를 만들지 않고, 바로 { shortId } 라는 값을 할당한다면 userInfoRequired라는 변수를 선언함으로써 메모리가 사용될텐데, 값을 바로 할당한다면 메모리를 사용하지 않을 수 있다고 생각해요!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/product-router.js line 14

categoryIds에 let을 활용한 이유가 있을까요? const를 활용하면 어떨까요? let과 const의 차이는 무엇일까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/product-router.js line 15

이 부분도 위와 동일합니다.

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 6

import { orderService, userService } from '../services';

위와 같이 모듈을 불러올 때 있어서 코드를 간결하게 해줄 수 있습니다.

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 35

코드를 간결하게 해볼까요? 간결하게 만드는 연습을 꾸준히 해봅시다!!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 12

purchase라는 url이 필요할까요? order라는 도메인에 Post방식으로 요청하는 것이라면 '/'로 경로를 지정하더라도 그 의미가 잘 보일 수 있지 않을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 57

여기서도 orderlist가 필요할까요? 목록을 조회할 때, Get '/'으로 요청하면 어떨까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 68

!= 와 !==의 차이는 무엇일까요? == 와 ===의 차이를 비교하면서, 어떤 것을 적용하면 좋을지 고민해봅시다!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 70

에러가 날 경우, 밑에서 return하는게 아닌, 지금의 코드를 바로 return하는 것은 어떠할까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/routers/order-router.js line 140

cancel이라는 주소가 필요할까요?! 바로 delete '/:shortId'로 할 수는 없을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/middlewares/error-handler.js line 8

백엔드를 개발하면서 생각해보셔야 할 것은, 만약 예상하지 못한 에러가 날 경우 어떻게 처리해야 하는가 입니다.

지금 서버에서는 예상하지 못한 에러가 날 경우엔 400 응답코드를 출력하고 있는데, 과연 400 응답코드를 주는 것이 맞을까요? 아니면 예상하지 못한 에러가 날 경우, 어떤 처리를 해주어야 할까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/user-service.js line 76

여기서 shortId란 어떤 의미일까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/user-service.js line 174

불필요한 콘솔 코드와 주석 코드는 삭제해주시는 것이 좋습니다! 여기서, 질문! 왜 불필요한 콘솔과 주석 코드를 삭제해야 할까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/product-service.js line 59

여기서 toUpdate라는 코드는 어떤 의미를 담고 있을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/order-service.js line 15

바로 await this.orderModel.create(orderInfo); 코드를 리턴하면 어떨까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/order-service.js line 31

위 메서드들을 포함해서, 변수를 만들고, 값을 할당해서, 그 변수를 리턴할 필요가 있을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/services/category-service.js line 29

addCategory가 아닌, AddCategory로 설정한 이유가 있을까요!?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/db/schemas/order-schema.js line 11

여기서 index는 어떤 역할을 할까요? index란 무엇이며, 왜 사용해야 할까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/db/models/category-model.js line 13

바로 await Category.find(query); 코드를 리턴하면 어떨까요?

바로 리턴할 수 있는 코드는 바로 리턴하는 것이 좋습니다! 아래의 메서드들도 동일하게 수정해주시면 좋을 것 같아요!!

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/db/models/product-model.js line 9

여기서 $in은 어떤 의미일까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/db/models/product-model.js line 43

findOneAndUpdate를 활용해주셨는데, updateOne, findOneAndUpdate, findByIdAndUpdate는 어떤 차이가 있을까요?

In GitLab by @constP on May 28, 2022, 24:39

Commented on src/db/models/product-model.js line 54

deleteOne과 findOneAndDelete는 어떤 차이가 있을까요?

In GitLab by @easton on May 30, 2022, 14:01

Commented on src/views/addproduct/addproduct.html line 39

jQuery 를 사용하지 않고 순수한 dom 메소드만을 활용한다면 어떤 방식으로 navbar, footer 를 공통화 시킬 수 있을까요 ?

In GitLab by @easton on May 30, 2022, 14:01

Commented on src/views/product-list/product-list.js line 35

이 부분 index 가 4의 배수일 때의 대한 처리는 왜 들어간걸까요 ?

한 줄에 4개씩 표시되는 상품 목록에 대한 레이아웃을 잡기 위함이라면 grid 를 한번 참고해보세요:)

https://studiomeal.com/archives/533

In GitLab by @easton on May 30, 2022, 14:01

Commented on src/views/api.js line 9

token 을 쿠키로 관리하는거 같은데 sessionStorage 에서 가져오나요 ?!

In GitLab by @easton on May 30, 2022, 14:01

Commented on src/views/api.js line 132

이건 무슨 용도의 API 일까요 ?!

In GitLab by @easton on May 30, 2022, 14:01

Commented on src/views/useful-functions.js line 17

Intl.NumberFormat 을 통해 세자리 콤마를 표기해줄 수 도 있습니다. 이건 참고해주세요 !

https://developer.mozilla.org/ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat

In GitLab by @about2weeks on May 30, 2022, 16:23

mentioned in commit b759039ee28b4919f6db4299f838ce96cce36318

In GitLab by @about2weeks on May 30, 2022, 23:07

Commented on src/routers/user-router.js line 29

다음 유지 보수를 하는 개발자에게 혼동이 있을 수 있을 것 같습니다. @constP

In GitLab by @about2weeks on May 30, 2022, 23:07

Commented on src/routers/user-router.js line 77

프론트엔드에서 jwt 토큰이 있는지 없는지를 판단하기보다 우선 로그인값을 가져다 쓰기로 결정했었습니다. 메모리 차원에서 불필요하다면 추후 모든 기능이 완성된 후에 바꾸면 괜찮을 것 같습니다. @constP

In GitLab by @about2weeks on May 30, 2022, 23:25

Commented on src/db/schemas/order-schema.js line 11

index는 서치 쿼리시에 검색을 더 빨리하는 역할을 합니다. user와 관련된 질의를 할 경우 검색 성능이 향상된다고 알고 있습니다. userId로 주문 내역을 조회할 때가 많을 것 같아 index를 넣었습니다. @constP

In GitLab by @about2weeks on May 30, 2022, 23:27

Commented on src/db/models/category-model.js line 13

관련해서 이해는 했습니다! 하지만 저런 코드에 있어서 추후 다른 개발자가 코드를 읽을 때 바로 코드를 이해할 수 있을지 궁금합니다. @constP

In GitLab by @about2weeks on May 30, 2022, 23:38

Commented on src/views/api.js line 9

header를 전혀 사용하지 않을 때 해당 코드를 어떻게 수정해야할지 모르겠습니다! @easton

seonghbae / Usually

[Merge] 1주차 머지 리퀘스트 입니다 - [merged] #110