[톰캣 구현하기 - 1,2단계] 모디(전제희) 미션 제출합니다. #327
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
jaehee329
requested review from
HubCreator
and removed request for
HubCreator
| private void validateRequestLineElementsCount(String[] elements) { | ||
| if (elements.length != 3) { | ||
| throw new HttpFormatException(); | ||
| } | ||
| } | ||
|
|
||
| private void validateUrlAndQueryStringLength(String[] urlAndQueryString) { | ||
| if (urlAndQueryString.length > 2) { | ||
| throw new HttpFormatException(); | ||
| } | ||
| } |
| for (Handler handler : HANDLERS) { | ||
| if (handler.supports(httpRequest)) { | ||
| return handler; | ||
| } | ||
| } |
There was a problem hiding this comment.
핸들러를 독립적으로 나눈 것이 인상깊네요! 응집도도 더 올라가고 좋은 것 같습니다
| public class MemberService { | ||
|
|
|
|
||
| @Override | ||
| public HttpResponse handle(HttpRequest request) { | ||
| if (doesRequestContainsCookie(request)) { |
There was a problem hiding this comment.
handler에서 쿠키가 있는지 확인하는 것도 좋지만, HttpRequest 객체에게 쿠키가 있는지 물어보는 것도 좋을 것 같습니다! 쿠키가 있는지 알려주는 것은 HttpRequest의 책임에 조금 더 가깝지 않을까 생각해봤어요 ㅎㅎㅎ
|
|
||
| public static HttpResponse register(HttpRequest httpRequest) { | ||
| String body = httpRequest.getBody(); | ||
| Map<String, String> requestParam = parseForm(body); |
There was a problem hiding this comment.
아마 중간에 GET 요청 방식에서 POST 요청 방식으로 바뀌었는데, 요거 변수명도 같이 안바뀐 것 같네요 ㅎㅎㅎ
이제 requestParam이 아니라 requestBody 어떠신가요!?
| addRequestLine(outputStream); | ||
| addHeaders(outputStream); | ||
| addBody(outputStream); |
There was a problem hiding this comment.
아아 outputStream을 넘겨서 쭉쭉 써내려갈 수 있군여... 아니 학습테스트를 했었는데도 새롭네요 ㅋㅋㅋㅋ 배워갑니다 모디 ㅎㅎ
|
|
||
| public static class Builder { |
| if (handler.supports(httpRequest)) { | ||
| return handler; | ||
| } | ||
| } | ||
| return HANDLER_NONE_MATCHED_DEFAULT; |
안녕하세요 헙크..!
하하 테스트 없이 굴러가는 불안한 코드로 일단 제출합니다!
아직 리팩터링할 부분이 당연히 많은데 3단계에서 추가적으로 고쳐보겠습니다!
큰 흐름에서 제안해주실 사항이나 하면서 좋다고 생각했던 아이디어들 말씀 주시면 수정에 참고하겠습니다~!