[럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. #101

ksy90101 · 2020-09-11T10:54:31Z

안녕하세요!
한가지 궁금한 사항이 있습니다.

Body가 없을때 (예를 들면 get요청)같은 경우에는 null로 처리하는것이 좋을까요? 아니면 빈 맵으로 처리하는게 좋을까요?

답변이 궁금합니다 ㅠㅠ 잘 부탁드립니다 👍

hongsii

안녕하세요~ 럿고!
1단계 요구사항을 잘 구현해주셨네요! 👍
몇 가지 피드백을 남겼으니, 다음 단계를 진행하시면서 같이 반영해주세요~!

hongsii · 2020-09-12T11:05:37Z

src/main/java/controller/StaticController.java

+public class StaticController implements Controller {
+    @Override
+    public void service(Request request, DataOutputStream dos) throws IOException, URISyntaxException {
+        String requestUrl = request.getRequestLine().getUrl();


규칙 4. 한 줄에 한 점만 사용 을 지키도록 수정해보면 어떨까요?

hongsii · 2020-09-12T11:07:50Z

src/main/java/controller/UserCreateController.java

+    public void service(Request request, DataOutputStream dos) {
+        if (request.isMethod(GET)) {
+            doGet(request, dos);
+        } else if (request.isMethod(POST)) {
+            doPost(request, dos);
+        }
+    }


새로운 API가 추가된다면 해당 로직이 중복해서 생길 것 같습니다. 추상화를 통해 중복이 생기지 않도록 리팩토링해보면 어떨까요?

hongsii · 2020-09-12T11:09:52Z

src/main/java/controller/UserCreateController.java

+        QueryParams queryParams = new QueryParams(requestUrl);
+        Map<String, String> queryParamsMap = queryParams.getQueryParams();


객체에서 값을 꺼내서 처리하지 않고, 일급 컬렉션 객체인 QueryParams 에게 물어보도록 바꿔보면 어떨까요?

추가로 현재 QueryParams 가 필요하면 매번 사용하는 곳에서 직접 객체를 생성해야 합니다.
Request 가 들어오면 쿼리 여부에 따라 자동으로 생성하도록 바꿔보면 어떨까요?

hongsii · 2020-09-12T11:14:28Z

src/main/java/controller/Controller.java

+
+import http.Request;
+
+public interface Controller {


인터페이스 설계 👍

hongsii · 2020-09-12T11:18:36Z

src/main/java/http/RequestBody.java

+        Map<String, String> result = new HashMap<>();
+        for (String token : tokens) {
+            String[] value = token.split(BODY_DELIMITER);
+            result.put(value[0], value[1]);


key= 와 같이 값을 비워서 요청하면 어떻게 될까요?
예외 케이스에 대해서도 적절히 고민해보시면 좋을 것 같아요~!

hongsii · 2020-09-12T11:34:38Z

src/main/java/controller/Controller.java

+import http.Request;
+
+public interface Controller {
+    void service(Request request, DataOutputStream dos) throws IOException, URISyntaxException;


응답 객체도 만들어 보면 어떨까요?

hongsii · 2020-09-12T11:38:21Z

src/main/java/http/RequestLine.java

+    private RequestMethod method;
+
+    private String url;
+
+    private String protocol;


Request-Line 의 각 요소는 아래와 같이 부릅니다~!

Request-Line = Method SP Request-URI SP HTTP-Version CRLF

RFC 2616 - Request-Line

hongsii · 2020-09-12T11:41:02Z

src/test/java/QueryParamsTest.java

+class QueryParamsTest {
+
+    @Test
+    void name() {


테스트 메서드명을 바꾸시는 것을 깜빡하셨네요~!

hongsii · 2020-09-12T11:43:45Z

src/main/java/controller/TemplatesController.java

+        String requestUrl = request.getRequestLine().getUrl();
+
+        byte[] body = FileIoUtils.loadFileFromClasspath(Directory.TEMPLATES.getDirectory() + requestUrl);
+        ContentType contentType = ContentType.of(requestUrl);


ContentType 이 필요한 곳 마다 같은 로직을 따로 처리하고 있습니다. Request 객체로 메시지를 보내도록 바꿔보면 어떨까요?

hongsii · 2020-09-12T11:54:39Z

src/main/java/webserver/RequestMapping.java

+import controller.UserCreateController;
+import http.ContentType;
+
+public class RequestMapping {


RequestMapping, Controller 등 테스트 코드가 작성되지 않은 클래스가 있던데 테스트 코드를 작성해보면 어떨까요?

* [럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. (#101) * docs: README.md 작성 * feat: URL추출 메서드 및 request header 출력 메서드 작성 * feat: /index.html로 접속했을 때 index.html 파일 응답 구현 * feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현 * feat: post 처리 및 요청 값 객체 생성 * feat: 요구사항3 완료 * feat: 302 response 구현 * refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정 Co-authored-by: 황준호 <42054054+hwanghe159@users.noreply.github.com> * refactor: response 객체 생성 * refactor: header line 추가 * refactor: 필드명 변경, 디미터법칙 따르도록 수정 * refactor: 필드명 변경, RequestBody에 대한 예외처리, stringBuilder에서 append사용 * refactor: service 메서드 추상화 * refactor: QueryParams를 Request 내부로 이동, RequestUri, Path 생성 * feat: 쓰레드풀 적용 * refactor: / 경로 index.html 이동 controller 생성 및 전체 리팩토링 * test: Response에 대한 test작성 * test: controller 테스트 추가 * refactor: 파일 대문자 소문자로 변경 * test: 테스트 fail 해결, request, response파일 추가 * refactor: 지원하는 않는 메서드 예외 처리 리팩토링 * refactor: 리팩토링 - Header enum 삭제 - 컨트롤러 접근제어자 변경 - IllegalRequestException 메시지 변경 - 상수 이름 변경 * refactor: response test를 ByteArrayOutputStream으로 변경 Co-authored-by: SeYun <53366407+ksy90101@users.noreply.github.com> Co-authored-by: SeYun <ksy90101@naver.com>

* [그래] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. (#102) * docs: README.md 작성 * feat: URL추출 메서드 및 request header 출력 메서드 작성 * feat: /index.html로 접속했을 때 index.html 파일 응답 구현 * feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현 * feat: post 처리 및 요청 값 객체 생성 * feat: 요구사항3 완료 * feat: 302 response 구현 * refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정 Co-authored-by: SeYun <ksy90101@naver.com> * [그래] 2단계 - HTTP 웹 서버 리팩토링 미션 제출합니다. (#150) * [럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. (#101) * docs: README.md 작성 * feat: URL추출 메서드 및 request header 출력 메서드 작성 * feat: /index.html로 접속했을 때 index.html 파일 응답 구현 * feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현 * feat: post 처리 및 요청 값 객체 생성 * feat: 요구사항3 완료 * feat: 302 response 구현 * refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정 Co-authored-by: 황준호 <42054054+hwanghe159@users.noreply.github.com> * refactor: response 객체 생성 * refactor: header line 추가 * refactor: 필드명 변경, 디미터법칙 따르도록 수정 * refactor: 필드명 변경, RequestBody에 대한 예외처리, stringBuilder에서 append사용 * refactor: service 메서드 추상화 * refactor: QueryParams를 Request 내부로 이동, RequestUri, Path 생성 * feat: 쓰레드풀 적용 * refactor: / 경로 index.html 이동 controller 생성 및 전체 리팩토링 * test: Response에 대한 test작성 * test: controller 테스트 추가 * refactor: 파일 대문자 소문자로 변경 * test: 테스트 fail 해결, request, response파일 추가 * refactor: 지원하는 않는 메서드 예외 처리 리팩토링 * refactor: 리팩토링 - Header enum 삭제 - 컨트롤러 접근제어자 변경 - IllegalRequestException 메시지 변경 - 상수 이름 변경 * refactor: response test를 ByteArrayOutputStream으로 변경 Co-authored-by: SeYun <53366407+ksy90101@users.noreply.github.com> Co-authored-by: SeYun <ksy90101@naver.com> * docs: 2단계, 3단계 요구사항 작성 * feat: 로그인 기능 구현 * feat: 로그인 시 user/list를 접속할 수 있게 기능 구현 * feat: 로그아웃, 세션 기능 구현 * feat: 피드백 반영 * refactor: request에 session 추가 * refactor: response에 cookies 추가 * refactor: session의 attribute 값으로 로그인 검증 Co-authored-by: 황준호 <42054054+hwanghe159@users.noreply.github.com>

* [럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. (#101) * docs: README.md 작성 * feat: URL추출 메서드 및 request header 출력 메서드 작성 * feat: /index.html로 접속했을 때 index.html 파일 응답 구현 * feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현 * feat: post 처리 및 요청 값 객체 생성 * feat: 요구사항3 완료 * feat: 302 response 구현 * refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정 Co-authored-by: 황준호 <42054054+hwanghe159@users.noreply.github.com> * [럿고] 2단계 - HTTP 웹 서버 리팩토링 미션 제출합니다. (#149) * [그래] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. (#102) * docs: README.md 작성 * feat: URL추출 메서드 및 request header 출력 메서드 작성 * feat: /index.html로 접속했을 때 index.html 파일 응답 구현 * feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현 * feat: post 처리 및 요청 값 객체 생성 * feat: 요구사항3 완료 * feat: 302 response 구현 * refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정 Co-authored-by: SeYun <ksy90101@naver.com> * refactor: response 객체 생성 * refactor: header line 추가 * refactor: 필드명 변경, 디미터법칙 따르도록 수정 * refactor: 필드명 변경, RequestBody에 대한 예외처리, stringBuilder에서 append사용 * refactor: service 메서드 추상화 * refactor: QueryParams를 Request 내부로 이동, RequestUri, Path 생성 * feat: 쓰레드풀 적용 * refactor: / 경로 index.html 이동 controller 생성 및 전체 리팩토링 * test: Response에 대한 test작성 * test: controller 테스트 추가 * refactor: 파일 대문자 소문자로 변경 * test: 테스트 fail 해결, request, response파일 추가 * refactor: 지원하는 않는 메서드 예외 처리 리팩토링 * refactor: 리팩토링 - Header enum 삭제 - 컨트롤러 접근제어자 변경 - IllegalRequestException 메시지 변경 - 상수 이름 변경 * refactor: response test를 ByteArrayOutputStream으로 변경 Co-authored-by: 황준호 <42054054+hwanghe159@users.noreply.github.com> * docs: 2단계, 3단계 요구사항 작성 * feat: 로그인 기능 구현 * feat: 로그인 시 user/list를 접속할 수 있게 기능 구현 * feat: 로그아웃, 세션 기능 구현 * feat: 피드백 반영 * refactor: 필요없는 파일 제거 * refactor: Handlebars 한번 초기화해서 재사용 * refactor: static변수 이름 변경, 인스턴스 변수 이름 변경 * refactor: 리팩토링 - 리퀘스트 바디 String -> Map 파싱 로직 캡슐화 - Database -> UserRepository 이름변경 * refactor: HttpHeaders 구성 로직 이동 * refactor: Cookie 생성 Co-authored-by: SeYun <53366407+ksy90101@users.noreply.github.com> Co-authored-by: SeYun <ksy90101@naver.com>

hwanghe159 and others added 8 commits September 9, 2020 10:19

docs: README.md 작성

3ade990

feat: URL추출 메서드 및 request header 출력 메서드 작성

25e57c4

feat: /index.html로 접속했을 때 index.html 파일 응답 구현

6ec44c9

feat: 회원가입 시 회원에 대한 정보를 서버에 저장하는 로직 구현

b81fba8

feat: post 처리 및 요청 값 객체 생성

f5b3356

feat: 요구사항3 완료

30ac225

feat: 302 response 구현

4b75097

refactor: 요청에 따른 응답을 컨트롤러로 처리하도록 수정

6ab8414

ksy90101 changed the base branch from master to ksy90101 September 11, 2020 10:54

hongsii approved these changes Sep 12, 2020

View reviewed changes

hongsii merged commit 9cd5039 into woowacourse:ksy90101 Sep 12, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. #101

[럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. #101

ksy90101 commented Sep 11, 2020

hongsii left a comment •

edited

Loading

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020

hongsii Sep 12, 2020 •

edited

Loading

hongsii Sep 12, 2020

		QueryParams queryParams = new QueryParams(requestUrl);
		Map<String, String> queryParamsMap = queryParams.getQueryParams();

[럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. #101

[럿고] 1단계 - HTTP 웹 서버 구현 미션 제출합니다. #101

Conversation

ksy90101 commented Sep 11, 2020

hongsii left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hongsii Sep 12, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hongsii left a comment •

edited

Loading

hongsii Sep 12, 2020 •

edited

Loading