🌱<인프런 워밍업 스터디 클럽 0기> BE - 나혼자 실습(도서관리 애플리케이션)

간단히 리팩토링 하는글을 써보려구 한다.
오늘자 강의 듣기전에 코드부터 먼저 짜보고 싶어서..^^

우선 리팩토링 전 BookController 클래스이다.

package com.group.libraryapp.controller.book;

import com.group.libraryapp.dto.book.request.LoanBookRequest;
import com.group.libraryapp.dto.book.request.SaveBookRequest;
import com.group.libraryapp.dto.book.response.BookResponse;
import org.springframework.jdbc.core.JdbcTemplate;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.PutMapping;
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RestController;

import java.util.List;

@RestController
public class BookController {


    private JdbcTemplate jdbcTemplate;

    public BookController(JdbcTemplate jdbcTemplate) {
        this.jdbcTemplate = jdbcTemplate;
    }

    //책 등록 API
    @PostMapping("/book")
    public void saveBook(@RequestBody SaveBookRequest request){
      String sql = "INSERT INTO book(name) VALUES(?)";
      jdbcTemplate.update(sql,request.getName());

    }

    //책 대출 API
    @PostMapping("/book/loan")
    public void loanBook(@RequestBody LoanBookRequest request){
        String sql = "SELECT * FROM book WHERE name = ?";
        Boolean isBookNotExist = jdbcTemplate.query(sql, (rs, rowNum) -> 0,request.getBookName()).isEmpty();
        if(isBookNotExist){
            throw new IllegalArgumentException();
        }
        String readSql = "SELECT loan_user FROM book WHERE name = ?";

     List<BookResponse> loanUser = jdbcTemplate.query(readSql, (rs, rowNum) -> {
            String name = rs.getString("loan_user");
            return new BookResponse(name);
        },request.getBookName());


        if(loanUser.get(0).getName() == null){
            String insertSql = "UPDATE book SET loan_user = ? WHERE name = ?";
            jdbcTemplate.update(insertSql,request.getUserName(),request.getBookName());
        }else{
            System.out.println("이미 대출한 사람이 있습니다.");
        }



    }


    //책 반납 API
    @PutMapping("/book/return")
    public void returnBook(@RequestBody LoanBookRequest request){
        String sql = "SELECT * FROM book WHERE name = ?";
        Boolean isBookNotExist = jdbcTemplate.query(sql, (rs, rowNum) -> 0,request.getBookName()).isEmpty();
        if(isBookNotExist){
            throw new IllegalArgumentException();
        }


            String insertSql = "UPDATE book SET loan_user = ? WHERE name = ?";
            jdbcTemplate.update(insertSql,null,request.getBookName());



    }





}

이 코드에는 크나큰 문제가 있다. 바로바로,, 레이어드 아키텍쳐로 구현되지 않았다. 강의에서 나온 말이라 한번 써봤는데,, 간단히 말하면 코드가 지금 뭉탱이로 컨트롤러에 들어있어서 이럴 거면 회사 못들어간다고 보면 된다.
그래서 이 컨트롤러 안에 있는 코드를 분리해서,
controller-service-repository 이렇게 세 구조로 나뉘게 코드를
리팩토링 해볼거다.음 근데,, 애초에 테이블부터 새로 만들어야 해서 구현을 다시 해야될거 같다..ㅋㅎㅋ