[부스트코스] 웹 프로그래밍 코드리뷰 후기: 프로젝트A-1. 홈페이지 BE

안녕하세요 오늘은 코드리뷰 후기를 작성해보려고합니다.

BOOSTER 2기 활동을 하면서 무료 코드리뷰권을 받아서 코드리뷰를 받아보아서 그 후기를 작성해보려고 합니다. 저는 '1.웹 프로그래밍 기초' 홈페이지 만드는 프로젝트의 백엔드 부분을 코드리뷰받았습니다.

 

https://www.edwith.org/boostcourse-web/project/4/content/3#summary

 

기술요구사항

👉웹프론트엔드

• html layout tag를 사용합니다.
• classname은 일정한 컨벤션을 유지합니다.
• 의미에 맞는 tag를 최대한 사용합니다. (div 사용은 최대한 자제)
• position속성과 float를 사용해서 element를 배치합니다.
• 라이브러리를 사용한 레이아웃은 지양합니다. (부트스트랩 등)
• id와 class등의 다양한 selector문법을 잘 활용해야 합니다.

👉웹백엔드

• 톰캣서버를 통해서 자기소개 페이지가 동작되야 합니다. (ex http://localhost:8080/aboutme/index.html 에서 노출)
• 서블릿페이지하나를 생성해서, url을 입력했을 때 시간데이터가 화면에 노출돼야 합니다.

 

화면 구성

index.html

aboutme.html

photo.html

 

TodayServlet.java

 

 

코드 리뷰

리뷰 결과

백엔드 코드리뷰를 신청했기때문에 웹 백엔드 기술요구사항이 중요했는데 결론부터 말하자면 Fail 받았습니다ㅜ. 그 이유도 친절히 알려주셨는데, 시간데이터를 화면에 띄울 때 Formatter를 사용하지 않았기 때문입니다. 이는 평가기준표를 대충봤기때문에 이런 불상사가 일어났는데요... 반성하고 다음 부터는 평가기준표를 꼼꼼히보고 코드를 작성해야겠습니다! 

리뷰를 맡아주신 moons님께서 생각지도 못했던 부분까지 세심하게 리뷰를 남겨주시고 참고 자료까지 주셔서 정말 감사드리고 감동받았습니다. 리뷰어분께서 정성스레 리뷰를 달아주시니 프로젝트도 더 열심히 해야겠다고 생각이 드네요 ㅎㅎ.

그리고 이제 본격적으로 어떤 평가기준에서 어떤 코멘트를 받았는지 살펴보도록 하겠습니다.

 

기능 1, 페이지 네비게이션 구현

 

홈페이지 링크태그를 제대로 구현했는지 알아보는 항목이었습니다. 3개의 페이지에 동일한 네비게이션바를 구성하고 현재시간을 출력하는 TodayServlet에서는 메인화면으로가는 링크만 하나 추가했습니다. 감사하게도 큰 탈없이 통과했습니다 ㅎㅎ

🙏지금보니 화면 중앙의 자기소개, 내사진 등 클릭했을 때 해당 페이지로 이동해야했는데 그 부분 구현을 안했었네요;; 제대로 고쳐서 다시 한 번 리뷰 받도록 해보겠습니다ㅜ

기능 2, 현재시간노출 구현

TodayServlet에서 현재시간을 잘 노출했는지 알아보는 항목입니다. 일단 노출은 했기 때문에 통과했습니다.

 

소스코드 1, Java 코드 스타일

자바 코드스타일을 확인하는 항목입니다. 없는 장점을 어떻게 잘 만들어주셔서 네 개의 댓글을 남겨주셨습니다. 감사합니다 ^^.

 

소스코드 2, LocalDateTime, Formatter 이용

대망의 실패한 부분입니다. 평가 기준으로 TodayServlet을 띄울 때, LocalDateTime과 Formatter를 이용하라고 되어있는데 저의 기적의 시야로 보지 못하고 제출했기 때문에 Fail을 받게 되었습니다... 흑 얼른 다시 고쳐서 제출해보고 다시 후기 남겨보도록 하겠습니다!

 

추가 참고링크

그 외에 저는 제출을 GitLap pullrequest주소로 했는데, 해당 풀리퀘에 댓글로 많은 의견과 참고문서를 상세하게 남겨주셔서 같이 공유하고 마무리하도록하겠습니다. 친절한 리뷰어님께 다시 한 번 감사드립니다.

1.  gitignore

[GitHub] .gitignore 설정하기 - Heee's Development Blog

 

gitignore.io

 

2. 코드 스타일 포멧 설정(들여쓰기)

[Code Style Formatter] 이클립스 자바 코드 스타일 적용 방법

 

3. @Override 어노테이션

 

[Java] Annotation(Override, Deprecated, SuppressWarnings) - Onsil's blog

 

 

4. 날짜 포맷터

 

Guide to DateTimeFormatter | Baeldung

 

 

 

 

마치며

이렇게 비록 fail을 받았지만 코드리뷰 후기를 남겨보았습니다. 다음에도 좋은 글로 찾아뵙겠습니다.

이상 BOOSTER 2기 김다롬이었습니다. 감사합니다~!