인프런 커뮤니티 질문&답변

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
여기에 질문 내용을 남겨주세요.

현재 스프링mvc와 jpa를 활용해서 게시판 프로젝트를 만들고있습니다.

로그인 처리는 세션으로 구현했습니다.

한 유저가 로그인을 해서 게시글을 작성했을때,

그 게시글은 작성한 해당 유저만이 수정 및 삭제를 할수있게 구현을 하는 중입니다.

@GetMapping("/{boardId}/edit")
public String editForm(@PathVariable("boardId") Long boardId, Model model, HttpServletRequest request, HttpServletResponse response) throws IOException {

    Optional<Board> findBoardOptional = boardRepository.findByIdJoinFetchMember(boardId);
    if (findBoardOptional.isEmpty()) {
        response.sendError(HttpServletResponse.SC_NOT_FOUND);
        return null;
    }

    MemberSessionDto memberSessionDto = getMemberSessionDto(request);
    Board findBoard = findBoardOptional.get();

    if (!memberSessionDto.getId().equals(findBoard.getMember().getId())) {
        response.sendError(HttpServletResponse.SC_FORBIDDEN);
        return null;
    }

    model.addAttribute("board", findBoard);
    return "boards/editForm";
}

현재 이런식으로 해당 게시글의 번호(boardId)로 해당 게시글이 있는지 먼저 검증을 한 후, 세션에 담아둔 유저의 정보를 가져와 해당 게시글을 작성한 유저의 id와 세션의 유저 정보의 id를 비교를 해서 다 검증이 끝나면은 로직이 정상 수행이 되게 만들었는데

막상 이렇게 만드니 수정 페이지이동, 수정, 삭제 에서도

검증하는 로직이 계속 중복됩니다.

@PostMapping("/{boardId}/delete")
public String deleteBoard(@PathVariable("boardId") Long boardId, HttpServletRequest request, HttpServletResponse response) throws IOException {

    Optional<Board> findBoardOptional = boardRepository.findByIdJoinFetchMember(boardId);
    if (findBoardOptional.isEmpty()) {
        response.sendError(HttpServletResponse.SC_NOT_FOUND);
        return null;
    }

    MemberSessionDto memberSessionDto = getMemberSessionDto(request);
    Board findBoard = findBoardOptional.get();

    if (!memberSessionDto.getId().equals(findBoard.getMember().getId())) {
        response.sendError(HttpServletResponse.SC_FORBIDDEN);
        return null;
    }
    
    boardRepository.deleteById(boardId);        

    return "redirect:/boards";
}

이렇게 똑같이 중복이 됩니다.

이렇게 구현을 하는게 맞는건지, 혹은 더 추천하는 방법이 잇는지, 실무에서는 어떻게 검증을 하는지 여쭈어보고싶습니다.