프로젝트B. 방명록 만들기 리뷰

프로젝트A에 이어서 바로 프로젝트B를 진행하였다.
제출해놓고 여행가고 싶은 마음에 급하게 진행했는데 다행히 통과할 수 있었다.

 

방명록 만들기 프로젝트는 Servlet과 JSP를 사용하여 방명록 목록 조회와

방명록 작성 기능만 구현하면 되는 간단한 프로젝트이다.

Comment1
오버라이드한 메소드들에 @Override 어노테이션을 붙여주는 것이 좋다.

해당 메소드가 오버라이드 된 메소드라는 것을 명시적으로 알릴 수 있으며,
컴파일 시 상속한 부모 클래스에 해당 메소드가 있는지 여부등을 통해
예외를 발생할 수 있어 오류를 인지할 수 있기 때문이다.

@Override 
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
... } 

@Override 
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
... }

 

Comment2
사용하지 않는 import는 삭제해주는 것이 좋다.

컴파일 시점에 제거되긴 하지만 사용하지 않는 import를 삭제해주는 것이 더 좋을 것이다.
eclipse에서 사용하지 않는 import를 정리해주는 단축키가 있다. [Ctrl + Shift + O]
출처 : https://zzznara2.tistory.com/76

[이클립스] eclipse에서 불필요한 import문 정리하기 (단축키 Ctrl+Shift+O)

 

Comment3
지역변수를 선언하여 사용하는 것의 이점은
반복적인 값을 사용하거나 코드의 가독성을 높이자는 것이다.
지역변수를 한번밖에 사용하지 않고 사용하는 메소드의 의미가 명확하다면
지역변수를 사용하지 않아도 될 것이다.

//기존
List<User> list = dao.getUsers(); 
request.setAttribute("list", list); 

//수정 후 
request.setAttribute("list", dao.getUsers()); 
//또는
List<User> users = dao.getUsers(); request.setAttribute("list", users);

또는 지역변수명을 좀더 명확하게 하여 가독성을 높이는 것도 방법일 것이다.