[JavaScript] : post.destroy() / await Posts.destroy({where : {postId}})

 

https://sequelize.org/docs/v6/core-concepts/model-instances/

Model Instances | Sequelize

 

제목의 두 코드는 정확히 같은 동작을 수행하는 코드이다.

너무 짜여진 형식에만 얽매여있었나 하는 느낌을 굉장히 크게 받았다.

오늘 코드 피드백을 받으면서 "이렇게도 할 수 있었네?" 라는 느낌을 크게 받아 남겨둔다.

기존에 게시글을 삭제하던 코드이다.

// 5. post(게시글) 삭제

router.delete('/posts/:postId', authMiddleware, async (req, res) => {
	
    const { postId } = req.params;
    const { userId } = res.locals.user;
    try {
        const post = await Posts.findOne({ where: { postId } });
        
        // 해당 게시글이 존재하지 않는 경우
        if (!post) {
            return res.status(403).json({ errorMessage: '게시글이 존재하지 않습니다..' });
        }
        
        // 게시글의 userId와 현재 로그인된 user의 userId가 일치하지 않은 경우
        if (!userId || post.UserId !== userId) {
            return res.status(403).json({ errorMessage: '게시글 삭제 권한이 존재하지 않습니다.' });
        }
        
        // 게시글 삭제
        await Posts.destroy({ where: { postId } }).catch((err) => {
            res.status(401).json({ errorMessage: '게시글이 삭제되지 않았습니다.' });
        });
        return res.status(200).json({ message: '게시글을 성공적으로 삭제하였습니다.' });
    } catch (error) {
        console.log(error.message);
        res.status(400).json({ errorMessage: '게시글 삭제에 실패하였습니다.' });
    }
});

예외처리에서 해당 게시글의 존재여부를 확인했고, 그 과정에서 이미 한 번 db에 접근해 우리가 삭제하고자 하는 post를 긁어왔다.

난 이렇게 긁어온 post를 게시글이 존재하는지 안하는지의 한 번의 예외처리에 사용했을 뿐이다.

근데 이렇게 가져온 post를 실질적인 게시글 삭제 로직에서 사용할 생각을 하지 못했다.

이미 긁어온 post를 destroy() 시킨 코드

// 5. post(게시글) 삭제
router.delete('/posts/:postId', authMiddleware, async (req, res) => {
    const { postId } = req.params;
    const { userId } = res.locals.user;
    try {
        const post = await Posts.findOne({ where: { postId } });
        
        // 게시글이 존재하지 않는 경우
        if (!post) {
            return res.status(403).json({ errorMessage: '게시글이 존재하지 않습니다..' });
        }
        
        // 게시글의 userId와 현재 로그인된 user의 userId가 일치하지 않는 경우
        if (!userId || post.UserId !== userId) {
            return res.status(403).json({ errorMessage: '게시글 삭제 권한이 존재하지 않습니다.' });
        }
		
        // 게시글 삭제
        post.destroy().catch((err) => {
            res.status(401).json({ errorMessage: '게시글이 삭제되지 않았습니다.' });
        });
        return res.status(200).json({ message: '게시글을 성공적으로 삭제하였습니다.' });
    } catch (error) {
        console.log(error.message);
        res.status(400).json({ errorMessage: '게시글 삭제에 실패하였습니다.' });
    }
});

이런식으로 똑같이 destroy메서드를 사용하여 긁어온 post를 삭제가 가능하다.

또한 DB에서 조건으로 찾을 필요없이 이미 찾아온 데이터를 삭제하는 것이기 때문에 await을 사용하지 않아도 된다.

#destroy #sequelize_logging