기차 충돌(train wreck) 제거(디미터법칙 만족)

[Dae-Hwa]

RequestHandler

Request

• 노을 - header 클래스에서 content-length를 바로 가져올 수 있는 메서드 생성 #55
• 노을 - request의 확장자 판단하는 조건문 대체request.isResource 혹은 request.isHtml #54
• 노을 - request.getRequestMessage().getParameters(); #56
• 프레디 - Request.from(requestMessages.toString()); 리팩토링 #57

최종적으로 request.getMessage가 없어져야함.

[Dae-Hwa]

RequestHandler

Request

• getContentLength
• Request.from(requestMessages.toString());
  • toString으로 넘기는게 좀 어색해보임
  • RequestBuilder 생성
• request.isResource || request.isHtml
• request.getRequestMessage().getParameters();
  1. parameters를 바로 가져온다.
     • 이 것도 결국 내부의 데이터 가져와서 쓰는거라 별반 차이가 없음.
  2. UserBuilder랑 Request를 연결시켜주는 매개체를 만든다.
     1. RequestParser.parse(User.class)
        • 리플랙션을 써야돼서 구현이 오래 걸릴 수 있음
     2. 그냥 새로 만들어준다.
        • UserRequestMapper.map(Request, User)
        • LoginRequestMapper

최종적으로 request.getMessage가 없어져야함.

[sanhee]

기차 충돌을 중점으로 생각했는데, 아래 2가지가 나왔어요.
프레디가 도출해 낸 리팩토링 목록의 부분집합이네요ㅋㅋㅋㅋ..

1. header 클래스에서 content-length를 바로 가져올 수 있는 메서드가 있다면 깔끔할 것 같습니다.
   • requestHeader.getContentLenth();

requestHeader.getAttributes().getOrDefault("Content-Length", "0")

2. request.getParameters() 및 parameters를 일급 컬렉션으로 만들면 좋을 것 같습니다.
   • 나중에 파라미터를 조작하는 메서드가 필요하지 않을까? 생각해서 상태와 행위를 모아둔 일급 컬렉션을 생각 했습니다.

Map<String, String> parameters = request.getRequestMessage().getParameters();