Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4주차 미션 / 서버 2조 조성현 #41

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

csh-scl
Copy link

@csh-scl csh-scl commented Apr 12, 2024

미션 진행도(80%)

미션 수행 내용

ModelandView 객체 구현
String -> View -> ModelandView를 반환하도록 변경
Controller 구현체가 AbstractController를 상속받도록 변경 후 로직 구현
JSONView, JSPView를 통해 사용자에게 render

주안점

궁금점

댓글, 답변 등록 시 Error 문구가 떠서 AddAnswerController를 수정해 문제를 해결하였는데 이제는 {0}과 {2}로 보이고 새로고침하면 제대로 보이는 문제가 발생했습니다. 이를 어떻게 해결해야 할 지 잘 모르겠습니다.

배운점

처음 미션을 할 때 View만으로도 충분할 것 같아 ModelandView 필요성을 잘 느끼지 못했는데 스터디를 진행하면서 이 리팩토링의 최종 결과물과 목표를 (reponse 제거, 서비스 로직 처리) 알게 되었습니다.

트러블 슈팅

새롭게 4주차 내용부터 fork를 받아와 classes 파일 설정이 안되어 정상적으로 작동이 안되었던 문제를 해결했습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants