[Springboot] 15 예외처리 및 코드 리팩토링 ~ing

HTTP 상태 코드 - HTTP | MDN

HTTP 응답 상태 코드는 특정 HTTP 요청이 성공적으로 완료되었는지 알려줍니다. 응답은 5개의 그룹으로 나누어집니다: 정보를 제공하는 응답, 성공적인 응답, 리다이렉트, 클라이언트 에러, 그리고 서버 에러. 상태 코드는 section 10 of RFC 2616에 정의되어 있습니다.

https://developer.mozilla.org/ko/docs/Web/HTTP/Status

package shop.mtcoding.blog.core.error;

import org.springframework.web.bind.annotation.ExceptionHandler;
import org.springframework.web.bind.annotation.RestControllerAdvice;
import shop.mtcoding.blog.core.error.ex.*;
import shop.mtcoding.blog.core.util.Script;

@RestControllerAdvice
public class GlobalExceptionHandler {

    // 유효성 검사 실패 (잘못된 클라이언트의 요청)
    @ExceptionHandler(Exception400.class)
    public String ex400(Exception e) {
        return Script.back(e.getMessage());
    }

    // 인증 실패 (클라이언트가 인증없이 요청했거나, 인증을 하거나 실패했거나)
    @ExceptionHandler(Exception401.class)
    public String ex401(Exception e) {
        return Script.back(e.getMessage());
    }

    // 권한 실패 (인증은 되어 있는데, 삭제하려는 게시글이 내가 적은게 아니다)
    @ExceptionHandler(Exception403.class)
    public String ex403(Exception e) {
        return Script.back(e.getMessage());
    }

    // 서버에서 리소스(자원) 찾을 수 없을때
    @ExceptionHandler(Exception404.class)
    public String ex404(Exception e) {
        return Script.back(e.getMessage());
    }

    // 서버에서 심각한 오류가 발생했을때
    @ExceptionHandler(Exception500.class)
    public String ex500(Exception e) {
        return Script.back(e.getMessage());
    }
}

   // 서버에서 심각한 오류가 발생했을때 (모를 때)
    @ExceptionHandler(Exception.class) // 그냥 exception
    public String ex(Exception e) {
        return Script.back(e.getMessage());
    }

User 리팩토링 시작

UserRepositoryTest

javascript코드 복사
let singleQuote = 'This is a string';
let doubleQuote = "This is also a string";
let templateLiteral = `This is a string with a variable: ${variable}`;

javascript코드 복사
let multiLineString = `
This is a
multi-line
string.
`;

let name = "John";
let greeting = `Hello, ${name}!`; // 템플릿 리터럴에서 변수를 삽입

public static String href(String msg, String url) {
    String errMsg = """
            <script>
                alert('$msg');
               location.href = '$url';
            </script>
            """.replace("$msg", msg)
            .replace("$url", url);
    return errMsg;
    // 자바스크립트 문자를 만들어서 리턴하는 것
} 

java코드 복사
public static String href(String msg, String url) {

    String errMsg = """
            <script>
                alert('$msg');
                location.href = '$url';
            </script>
            """.replace("$msg", msg)
            .replace("$url", url);
    return errMsg;
}

java코드 복사
href("Invalid input", "https://example.com");

html코드 복사
<script>
    alert('Invalid input');
    location.href = 'https://example.com';
</script>

• 401 은 이전 페이지로 back 보다 로그인하라고 보내주는게 낫다.

• 그리고 e.getMesasge 해서 에러메세지를 그대로 보여주지 말고,
인증되지 않았습니다! 와 같이 msg 작성하기. 안그럼 해킹당한다.

BoardCotroller ( 리팩토링 ~ ing)

package shop.mtcoding.blog.board;

import jakarta.servlet.http.HttpServletRequest;
import jakarta.servlet.http.HttpSession;
import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestParam;
import shop.mtcoding.blog.core.error.ex.Exception401;
import shop.mtcoding.blog.user.User;

import java.util.List;

// 책임 : 식별자 요청 받기 & 응답하기
// 컨트롤러는 REPOSITORY에 의존하고 있다. (Repository가 없으면 일을 못하기 때문에)
@RequiredArgsConstructor
@Controller // 식별자 요청을 받을 수 있다.
public class BoardController {

    /*
    private final BoardRepository boardRepository;

    public BoardController(BoardRepository boardRepository) {
        System.out.println("BoardController 생성자");
        this.boardRepository = boardRepository;
    }
    */

    private final HttpSession session;
    private final BoardService boardService;
    // 다고치고 boardRepository 지우기

    // url : http://localhost:8080/board/1/update
    // body : title=제목1변경&content=내용1변경
    // content-type : x-www-form-urlencoded
    // 버퍼 -> 보조스트림
    // 바구니에 담아서 줌. 10개를 한꺼번에 주면 받는사람은 하나씩 꺼내쓰면되고 주는사람은 기다리지않고 다른일할수있다.
    // 서로 통신하면 내 버퍼(Write buffer), 상대방 버퍼 (read buffer) -- bufferedWrite(->자바에서 편하게 쓰려고 prinwrite를 씀), bufferedRead
    // 수정하기 버튼을 클릭하면 브라우저의 application 버퍼에 먼저 쌓인다.
    // 스프링의 버퍼 (readBuffer)
    // request 로 버퍼에 접근할 수 있다.
    @PostMapping("/board/{id}/update")
    public String update(@PathVariable("id") int id, @RequestParam("title") String title, @RequestParam("content") String content) {
        //인증체크하고 업데이트 해라

        //boardRepository.updateById(title, content, id);
        return "redirect:/board/" + id;

        //return "/board/detail" 하면 안됨 -> view에서 model.id 뿌려줘야하니까. 아까 request 객체에 담아서 보낸건 이미 사라짐ㄴ


    }


    @PostMapping("/board/{id}/delete")
    public String delete(@PathVariable("id") int id) {

        User sessionUser = (User) session.getAttribute("sessionUser");

        // 인증 체크
        if (sessionUser == null) {
            throw new Exception401("로그인이 필요합니다"); // 나중에 이 매서드 프록시 안에 넣을 예정 ! 그럼 이 코드 작성 안해도 되니 편리함
        }

        boardService.게시글삭제(id, sessionUser);

        return "redirect:/board";
    }


    @PostMapping("/board/save")
    public String save(BoardRequest.SaveDTO saveDTO) { // 스프링 기본전략 x-www-encoded
        User sessionUser = (User) session.getAttribute("sessionUser"); // session에 있는건 Object 여서 User 로 다운캐스팅 해주어야 한다.

        // 인증 체크 필요함. 안하면 누가 강제로 들어와서 글을 작성할 수 있다. User_id 에 null 이 들어가지 못하도록 설정해야함

        if (sessionUser == null) {
            throw new Exception401("로그인이 필요합니다");
            // 로그인 페이지로 리다이렉트 하지말고, 무조건 exception으로 터트리기!
        }

        boardService.게시글쓰기(saveDTO, sessionUser);

        return "redirect:/board"; //redirect는 다른 컨트롤러를 때리는 것

    }


    // get, post, put, delete
    // insert, delete, update 할때 일단 지금은 post 를 쓰자
    @GetMapping("/board") // 식별자 요청
    public String list(HttpServletRequest request) {

        //System.out.println(request.getRemoteAddr()); // 내가 내껄 때려서 정확한 ip 주소가 안뜸 0 0 0 0 0 이렇게 뜨고
        //System.out.println(request.getRequestURI());


        List<Board> boardList = boardService.게시글목록보기(); // request 객체 저장
        request.setAttribute("models", boardList);

        HttpSession session = request.getSession();
        session.setAttribute("num", 1);

        return "board/list"; // return의 위치가 templates 폴더로 고정되어 있다. / 가 없어도 바로 templates 폴더로 연결된다.
        // 확장자도 mustache 로 자동설정 되어있기 때문에 파일명만 적어주면 거기로 리턴한다.
    }

    // 1. 매서드 : Get
    // 2. 주소 : /board/1
    // 3. 응답 : board/detail

    @GetMapping("/board/{id}")
    public String detail(@PathVariable("id") Integer id, HttpServletRequest request) {
    /*    Board board = boardRepository.findById(id);
        request.setAttribute("model", board); // 1건 이니까 그냥 model로 적어주기!
        request.setAttribute("isOwner", false);
    */
        User sessionUser = (User) session.getAttribute("sessionUser");

        BoardReponse.DetailDTO detailDTO = boardService.상세보기(id, sessionUser);
        request.setAttribute("model", detailDTO);

        return "board/detail";
    }

    // 1. 매서드 : Get
    // 2. 주소 : /board/save-form
    // 3. 응답 : board/save-form

    @GetMapping("/board/save-form")
    public String saveForm() {
        return "board/save-form";
    }

    // 1. 매서드 : Get
    // 2. 주소 : /board/1/update-form
    // 3. 응답 : board/update-form

    @GetMapping("/board/{id}/update-form")
    public String updateForm(@PathVariable("id") int id, HttpServletRequest request) { // PathVariable은 정규표현식으로 처리하는걸 해줌 -> /board/1 이든 /board/2 이든 다 받아줌

        // 주석처리한거 나중에 만들쟈
        //Board board = boardRepository.findById(id); // 만약 여기서 못찾으면 exception이 터짐
        //request.setAttribute("model", board); // request 객체(model)에 담음
        return "board/update-form";
    }


}

BoardService (리팩토링 ~ing)

package shop.mtcoding.blog.board;


// C -> S -> R

import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;
import shop.mtcoding.blog.core.error.ex.Exception403;
import shop.mtcoding.blog.user.User;

import java.util.List;

@RequiredArgsConstructor
@Service
public class BoardService {

    // DI 생성자 주입
    private final BoardRepository boardRepository;

    // 수정하기
    // 게시글 목록보기 해보기

    // v3 페이징이랑 댓글쓰기, 검색 배울예정
    // 파이어베이스, 몽고디비, 웹소켓

    public List<Board> 게시글목록보기() { // 나중에 페이징 처리하는거 여기 추가할 예정
        List<Board> boardList = boardRepository.findAll();

        return boardList;
    }


    @Transactional
    public void 게시글삭제(int id, User sessionUser) {
        // 1. 컨트롤러에서 게시글 id 받기 & 권한체크에 필요한 sessionUser 도 받기

        // 2. 게시글 존재 여부 확인 (404)
        Board board = boardRepository.findById(id);

        // 3. 권한 체크 (내가 쓴 글인지 확인하기. 403(권한없음))
        if (board.getUser().getId() != sessionUser.getId()) {
            throw new Exception403("권한이 없습니다.");

        }
        // 4. 게시글 삭제
        boardRepository.deleteById(id);

    }

    @Transactional
    public void 게시글쓰기(BoardRequest.SaveDTO saveDTO, User sessionUser) {
        // 누가 적었는지 알아야하니까 session이 필요
        boardRepository.save(saveDTO.toEntity(sessionUser));

        // sessionUser 인증체크는 컨트롤러에서 함
        // 그래서 여기 매개변수로 null이 들어올 경우는 없음
        // sessionUser == null 이런조건은 컨트롤러에서


    }


    // 기능명은 한글로 적는다.
    // 매서드는 행위 -> 동사
    public BoardReponse.DetailDTO 상세보기(int id, User sessionUser) { // 매개변수는 컨트롤러로부터 받는다.
        // final Httpsession으로 받으면 session.getAttriute해서 user로 다운캐스팅 해야해서 귀찮아지니까 이렇게 안하고
        // 세션 인증 체크는 컨트롤러에서 할거고 지금 서비스에서는 권한 체크(Board의 user랑 session의 user가 일치하는지)
        // 를 할거니까 sessionuSER는 매개변수로 받아도 충분
        Board board = boardRepository.findById(id); // 조인한 쿼리 쓰니까 (Board - User) 둘다 있음
        // 여기 null 처리 할 필요가 없음. findById 에서 null 이 들어오면 throw로 던지니까 서비스에서 board 값으로 null을 받을 일이 없음

        return new BoardReponse.DetailDTO(board, sessionUser); // board 와 boolean 2개 리턴 불가능 해 dto 만들기

    }

}