[✍️ Project Dev Log] 2022-01-12

[✍️ Dev Log] 2022-01-12

오늘은 어떻게 프로젝트에 기여했나요?

• admin: 확인한 lounge report 삭제 기능 추가
• model assosiate 이용하여 특정 lounge를 리펙토링

오늘의 프로젝트에서 힘든 점 또는 기억에 남는 점은 무엇인가요?

- 코드에는 정답이 없다 & 에러처리는 확실하게

어제 sequelize model들 간의 관계설정을 해준 뒤, 모델을 한번의 쿼리를 통해 작업을 처리하는 것이 아닌 두 번에 걸쳐 쿼리 해주는 코드를 찾아 새롭게 한 번의 쿼리를 통해 작업을 끝낼 수 있게 리펙토링 해보기로 했다.

lounge // 여기서 첫 번째 쿼리
  .findOne({
    where: { id: req.params.loungeId },
  })
  .then((data) => {
    const { image, address, avgRating } = data.dataValues;
    comment // 여기서 두 번째 쿼리
      .findAll({
        where: { loungeId: req.params.loungeId },
        attributes: ["userId", "contents", "rating", "createdAt"],
      })
      .then((data) => {
        const newData = data.map((element) => element.dataValues);
        console.log(newData);
        return res.status(200).send({
          data: { image, address, avgRating, comments: newData },
        });
      });
  });

이 코드가 기존에 작성된 코드였는데 lounge에서 한 번 쿼리하고 comment에서 또 한 번 쿼리하는 식으로 작성되어 있었는데 이 부분을 include 옵션을 사용해서 한 번에 처리해주는 코드로 새롭게 작성해봤다

lounge
  .findOne({
    where: {
      id: req.params.loungeId,
    },
    include: {
      model: comment,
      require: true,
      where: {
        loungeId: req.params.loungeId,
      },
    },
  })
  .then((data) => {
    const { image, address, avgRating } = data.dataValues;
    const comments = data.dataValues.comments.map((element) => {
      return {
        userId: element.dataValues.userId,
        contents: element.dataValues.contents,
        rating: element.dataValues.rating,
        createdAt: element.dataValues.createdAt,
      };
    });
    console.log(comments);
    return res
      .status(200)
      .send({ data: { image, address, avgRating, comments } });
  });

이런식으로 작성 했는데 사용하면서 느낀점이 코드 길이도 늘어나고 복잡해지는 느낌이었다.
확장성이 있는 서비스를 만든다면 이후에 관계가 더 복잡해질 때 장점이 있을 수 있겠지만 지금 프로젝트로 진행중인 정도의 복잡도면 기존의 두 번에 걸쳐 쿼리하는 식의 코드가 더 가독성도 좋고 에러핸들링 하기도 좋은 것 같았다.
(내가 제대로 관계를 이용한 쿼리를 잘 하지 못해 이렇게 느끼는 걸 수도 있음 ...)

새로 리펙토링하면서 기존에 발생하지 않았던 오류들이 클라이언트와 연결 할 때 생겨서 에러처리를 새로 추가해줘야하는 부분들이 생겼다

TypeError: Cannot read properties of null (reading 'dataValues') <= 이런 부분들
오늘 두 번 이상 발생한 에러가 이 부분인데 발생한 이유는,
테스트를 할 때 데이터가 있는 상황에서만 테스트를 해서 데이터가 null값일 때 에러가 발생할 수 있다는 점을 간과하고 코드를 짜서 생긴 문제였다.
이 부분은 코드를 짤 때 항상 염두에 두고 진행해야겠다.

내일은 프로젝트에 기여하기 위해 무엇을 해야 하나요?

• 에러처리 끝내고 null값인 상태에서 테스트 진행
• aws 배포 연습