[Hamill&Dan][HTTP 웹서버 구현하기] Step1, 2 Submit #50

hanurii · 2020-03-20T08:19:59Z

HTTP 웹서버 구현하기

# Step 1 기능 구현 완료

# Step 2 리팩토링

메소드 분리 및 클래스 분리
- HttpRequest, HttpResponse 객체화를 통한 리팩토링 완료

# Ground Rule

https://github.com/Hamill210/java-was/wiki/Ground-Rule

- Ground Rule 추가 - Repository 관리 추가 - 커밋 메시지 가이드 추가

refactor : README.md update

- Handlebar 수정 중 associate with : #1

- 모든 Request Header 출력하기 - Request Line에서 path 분리하기 - path에 해당하는 파일 읽어 응답하기 associate with : #1

- index.html 완료 associate with : #1

- Ground Rule wiki 로 이동 - Day 의 회고는 issue 로 관리

- 상대 경로였던 프로그램 경로를 절대 경로로 수정

- UserController 클래스 추가 - WelcomeController 클래스 추가 - HttpRequestUtils 클래스에 parseUriString 메서드 추가 associate with : #2

- WelcomeController 추가 - UserController 추가 - User 생성자 추가 (매개변수 userParameterMap) associate with #2

- 코드 정리 - Test case 추가 associate with #2

- user/form.html 수정 * requestHeader, requestBody 메서드 추가 및 수정 중 associate with : #6

- requestBody 가져오기 associate with #6

- requestLine, requestHeader, requestBody 를 나누어 구현 - Pair 객체를 사용 - GET /user/form.html 구현 - POST /user/create 구현 - WebServer 에 userDB 를 두어 User 정보를 관리 associate with #7, #8

- requestLine, requestHeader, requestBody 를 가져올 수 있는 메소드를 나누어 구현 - statusLine, responseHeader, responseBody 를 가져올 수 있는 메소드를 나누어 구현

- PageController - dowork()의 리턴타입 변경 (String -> Map) - ReqestHandler - dowork() 리턴값 변경에 따른 변수명 변경 associate with : #7

- /index.html, /user/login.html 구현 - User create 연결 associate with #7

- login_failed는 GET에 추가 associate with #7

- /user/login 구현 - parseRequestBody 메서드 구현 associate with #7

- Post /user/login 구현 완료 associate with #7 #8

- RequestHandler.java : dowork() - requestHeader에서 cookie 값 추출 후 조건문 사용 associate with #1

- PageController.java : doWork() - css 처리를 위한 로직 추가 - RequestHandler.java : responseHeader() - Content-Type 추가 associate with #9 #10

[Hamill&Dan][HTTP 웹서버 구현하기] Step1 Submit

- HttpTemplate.java - HttpRequest 상속을 위한 Class 생성 - HttpRequest.java 추가 - request 생성 - RequestHandler.java : run() - 로직 변경 associate with #18

- HttpRequest.java : requestHeader() 추가 - HttpRequestUtils.java : getKeyValueMap() 추가 - HttpTemplate.java - requestHeader type Map<String, String> 으로 변경 associate with #18

- HttpRequest.java : requestBody() 추가 - RequestHandler.java 에 있었던 메서드를 추가하고 리팩토링 associate with #18

- PageController.java : dowork() - 메서드 파라미터 변경 (Map<String, String>) - 파라미터 변경에 따른 getValue() 삭제 - HttpRequest.java : getStartLine() 추가 associate with #18

- HttpResponse.java 구현 완료 - WebServer.java : getWebappPath() - WEB_PATH 위치 이동 및 static 선언 - HttpTemplate.java - 필드 변경 - PageController.java - protocol 누락 수정 associate with #18

- RequestHandler.java : sendResponse() - 코드 개선 associate with #18

[Hamill&Dan][HTTP 웹서버 구현하기] Step2 Submit

imjinbro · 2020-03-22T18:32:55Z

src/main/java/Controller/PageController.java

+    String protocol = requestLine.get("protocol");
+    String contentType = requestHeader.get("Accept");
+
+    switch (method) {


요청 url이 추가될 때마다 if ~ else if ~ else 문이 길어지겠군요
Spring mvc 처리 흐름에서 HandlerMapping / Controller 역할 구분에 대하여 학습해보시고 리팩토링 해보시길 권해드려요

https://adora-y.tistory.com/entry/MEMO-Spring-MVC-%EA%B5%AC%EC%A1%B0-%EC%B2%98%EB%A6%AC-%EA%B3%BC%EC%A0%95

imjinbro · 2020-03-22T18:34:20Z

src/main/java/db/DataBase.java

-        users.put(user.getUserId(), user);
-    }
+  public static void addUser(User user) {
+    users.put(user.getUserId(), user);


User null 케이스에 대한 예외처리가 필요해보입니다.
항상 워스트 케이스를 생각하시면서 그에 대한 처리를 하시길 권해드려요

imjinbro · 2020-03-22T18:36:01Z

src/main/java/model/HttpRequest.java

+public class HttpRequest extends HttpTemplate {
+  private BufferedReader br;
+
+  public HttpRequest(BufferedReader br) throws Exception {


요청 데이터 읽는 방식이 파일 읽기 혹은 다른 방식으로 변경될 경우 HttpRequest 객체 코드에도 영향이 있지 않을까요?
BufferedReader를 생성자로 가지고 있다면 입력값을 읽는 방법 변경이 해당 클래스 코드에도 영향이 있을 것 입니다.
HttpRequest는 말그대로 Request에 대한 정보를 가지고 있는 객체로 한정 짓는 것이 필요합니다.

imjinbro · 2020-03-22T18:36:42Z

src/main/java/model/HttpRequest.java

+  }
+
+  public String getProtocol() {
+    return this.startLine.get("protocol");


생성자에서 Request를 받아서 모두 쪼개어 놓으면 되지 않을까요?
그렇게 하면 예외처리도 쉬울 것 같습니다.

hanurii and others added 29 commits March 16, 2020 15:38

refactor : README.md update

e3d8116

- Ground Rule 추가 - Repository 관리 추가 - 커밋 메시지 가이드 추가

Merge pull request #3 from Hamill210/Hamill210

040732b

refactor : README.md update

refactor : README.md 업데이트, Handlebar 수정

5543bbd

- Handlebar 수정 중 associate with : #1

refactor : Handlebar 수정

cac0a79

- 모든 Request Header 출력하기 - Request Line에서 path 분리하기 - path에 해당하는 파일 읽어 응답하기 associate with : #1

refactor : Day1 추가, Step1.요구사항1 완료

4116dd6

- index.html 완료 associate with : #1

refactor : 문서를 wiki 로 이동

84abfbb

- Ground Rule wiki 로 이동 - Day 의 회고는 issue 로 관리

refactor : RequestHandler 리팩토링

200f7e6

- 상대 경로였던 프로그램 경로를 절대 경로로 수정

feat : 회원가입 기능 구현중

d48b5d3

- UserController 클래스 추가 - WelcomeController 클래스 추가 - HttpRequestUtils 클래스에 parseUriString 메서드 추가 associate with : #2

feat : 회원가입 구현

a46ac31

- WelcomeController 추가 - UserController 추가 - User 생성자 추가 (매개변수 userParameterMap) associate with #2

feat : 회원가입 구현

f0511ee

- 코드 정리 - Test case 추가 associate with #2

feat : 회원 가입 구현

82dce59

- user/form.html 수정 * requestHeader, requestBody 메서드 추가 및 수정 중 associate with : #6

feat : 회원가입 구현 2

809b980

- requestBody 가져오기 associate with #6

feat : 요구사항 2, 3 구현

6365a0b

- requestLine, requestHeader, requestBody 를 나누어 구현 - Pair 객체를 사용 - GET /user/form.html 구현 - POST /user/create 구현 - WebServer 에 userDB 를 두어 User 정보를 관리 associate with #7, #8

refactor : RequestHandler 개선

35daff4

- requestLine, requestHeader, requestBody 를 가져올 수 있는 메소드를 나누어 구현 - statusLine, responseHeader, responseBody 를 가져올 수 있는 메소드를 나누어 구현

refactor : 요구사항 4 구현 성공

ce94f26

- PageController - dowork()의 리턴타입 변경 (String -> Map) - ReqestHandler - dowork() 리턴값 변경에 따른 변수명 변경 associate with : #7

feat : PageController 추가 구현, inmemory DB 연결

0e71953

- /index.html, /user/login.html 구현 - User create 연결 associate with #7

feat : login_failed 추가, User 테스트 데이터 추가

821df0f

- login_failed는 GET에 추가 associate with #7

feat : PageController 추가 구현, HttpRequestUtils 메서드 추가

7fa68b5

- /user/login 구현 - parseRequestBody 메서드 구현 associate with #7

feat : 요구사항 5 구현

cd6f632

- Post /user/login 구현 완료 associate with #7 #8

feat : 로그인 성공했을 때만 유저 목록 출력 구현

6b2ce2b

- RequestHandler.java : dowork() - requestHeader에서 cookie 값 추출 후 조건문 사용 associate with #1

feat : 요구사항 6, 7 구현

4360a92

- PageController.java : doWork() - css 처리를 위한 로직 추가 - RequestHandler.java : responseHeader() - Content-Type 추가 associate with #9 #10

Merge pull request #17 from Hamill210/Dan

3ab72c8

[Hamill&Dan][HTTP 웹서버 구현하기] Step1 Submit

refactor : request 파싱 로직 변경

4821d2a

- HttpTemplate.java - HttpRequest 상속을 위한 Class 생성 - HttpRequest.java 추가 - request 생성 - RequestHandler.java : run() - 로직 변경 associate with #18

feat : requestHeader 파싱 로직 추가

ec17872

- HttpRequest.java : requestHeader() 추가 - HttpRequestUtils.java : getKeyValueMap() 추가 - HttpTemplate.java - requestHeader type Map<String, String> 으로 변경 associate with #18

refactor : requestBody 파싱 로직 추가

b4e698b

- HttpRequest.java : requestBody() 추가 - RequestHandler.java 에 있었던 메서드를 추가하고 리팩토링 associate with #18

refactor : requestBody 파싱 로직 추가

a30861a

- PageController.java : dowork() - 메서드 파라미터 변경 (Map<String, String>) - 파라미터 변경에 따른 getValue() 삭제 - HttpRequest.java : getStartLine() 추가 associate with #18

feat : response 구현 완료

758b519

- HttpResponse.java 구현 완료 - WebServer.java : getWebappPath() - WEB_PATH 위치 이동 및 static 선언 - HttpTemplate.java - 필드 변경 - PageController.java - protocol 누락 수정 associate with #18

refactor : 새로운 로직에서의 Client에 응답 구현

a9daa61

- RequestHandler.java : sendResponse() - 코드 개선 associate with #18

Merge pull request #19 from Hamill210/Hamill210

4179774

[Hamill&Dan][HTTP 웹서버 구현하기] Step2 Submit

imjinbro reviewed Mar 22, 2020

View reviewed changes

honux77 merged commit 9ceda95 into code-squad:Hamill210 Mar 24, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Hamill&Dan][HTTP 웹서버 구현하기] Step1, 2 Submit #50

[Hamill&Dan][HTTP 웹서버 구현하기] Step1, 2 Submit #50

Uh oh!

hanurii commented Mar 20, 2020

Uh oh!

imjinbro Mar 22, 2020 •

edited

Loading

Uh oh!

imjinbro Mar 22, 2020

Uh oh!

imjinbro Mar 22, 2020

Uh oh!

imjinbro Mar 22, 2020

Uh oh!

Uh oh!

[Hamill&Dan][HTTP 웹서버 구현하기] Step1, 2 Submit #50

[Hamill&Dan][HTTP 웹서버 구현하기] Step1, 2 Submit #50

Uh oh!

Conversation

hanurii commented Mar 20, 2020

HTTP 웹서버 구현하기

# Step 1 기능 구현 완료

# Step 2 리팩토링

# Ground Rule

Uh oh!

imjinbro Mar 22, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

imjinbro Mar 22, 2020

Choose a reason for hiding this comment

Uh oh!

imjinbro Mar 22, 2020

Choose a reason for hiding this comment

Uh oh!

imjinbro Mar 22, 2020

Choose a reason for hiding this comment

Uh oh!

Uh oh!

imjinbro Mar 22, 2020 •

edited

Loading