[230705] 미니 프로젝트(230628) 코드 피드백 및 적용

피드백 받은 내용

🎈 첫 번째 피드백

❗아래 코드에서 조건문의 경우 정규식에 포함시킬 수 있지 않을까요?

 

private boolean validateUsername(String userName){
    return userName.matches("[a-z0-9]+") && userName.length() >= 4 && userName.length() <= 10;
}

private boolean validatePassword(String password){
    return password.matches("[a-zA-Z0-9]+") && password.length() >=8 && password.length() <=15;
}

 

✔ 첫 번째 피드백 적용

 

private boolean validateUsername(String userName){
    return userName.matches("[a-z0-9]{4,10}")
}

private boolean validatePassword(String password){
    return password.matches("[a-zA-Z0-9]{8,15}")
}

 

---

🎈 두 번째 피드백

❗ getUserNameOf메서드가 Service 부분이 아닌 Controller 부분에 들어간 이유가 무엇인가요?

💬 Service 부분에 들어가기에는 중요한 로직이 아니라고 생각이 들었습니다. Controller 부분에서만 사용되기 때문입니다. 피드백을 들어보니 Controller에서 가지고 있는 것보다 이후 다른 곳에서도 사용을 고려해 보는 것이 옳다는 생각이 들었습니다. 그래서 Util 클래스를 따로 둬서 사용하는 것으로 적용해 보기로 했습니다.

 

✔ 두 번째 피드백 적용

 

public class HttpServletUtil {
    public static String getUserNameOf(HttpServletRequest httpServletRequest){
        return (String) httpServletRequest.getAttribute("username");
    }
}

GIT 저장소

GitHub - ironprayer/post-backend

생각

확실히 다른 분들에게 코드 리뷰를 받으니 배울 점이 많다는 것을 느꼈다. 내가 혼자 짤 때에는 그것이 가장 최선인 것처럼 보인다. 하지만 코드 리뷰를 하다 보면 내 것이 현 상황에서 최선이 아님을 깨닫게 되는 것이다. 그리고 조금 더 나은 코드를 짜는 방식에 대해서 알게 된다. 이런 시간을 종종 가지면서 코드 짜는 방식에 대해서 성장할 수 있는 개발자가 되고 싶다.