[EAT-112] ServletInputStream 구현체 분리

[wjdwnsdnjs13]

상위 이슈

• EAT-70

  이슈 링크

• EAT-112

  이슈 내용

• 이미지
  • 전
  • 후
• 설명
  • 코드 리뷰 시 피드백 받았던 ServletInputStream 구현체를 분리했습니다!
  • 분리 후 정상작동 확인 완료했고, 테스트 완료했습니다

[jaewoong-gwon]

질문

• isReady() 의 경우 항상 true 만 리턴하는데, 해당 스트림을 읽을 수 있을때 true 를 리턴하는게 맞는거 아닌가요??
  • 사용해 본적이 없어서 어떻게 쓰는건지 진짜 모름

[jaewoong-gwon]

바꾸기 전의 이미지에서 확인했을 때, getInputStream() 에서 ServeltInputStream 을 리턴할 때 어떻게 바뀐건지 이미지가 있으면 좋을것 같습니다.

현재는 분리하여 작성한것은 알겠는데, 사용하는 부분에서 어떤 변경사항이 있는지 확인이 어려운것 같네요

[wjdwnsdnjs13]

바꾸기 전의 이미지에서 확인했을 때, getInputStream() 에서 ServeltInputStream 을 리턴할 때 어떻게 바뀐건지 이미지가 있으면 좋을것 같습니다.

현재는 분리하여 작성한것은 알겠는데, 사용하는 부분에서 어떤 변경사항이 있는지 확인이 어려운것 같네요

getInputStream()에서 변경된 코드는 다음과 같습니다.

[wjdwnsdnjs13]

질문

• isReady() 의 경우 항상 true 만 리턴하는데, 해당 스트림을 읽을 수 있을때 true 를 리턴하는게 맞는거 아닌가요??

  • 사용해 본적이 없어서 어떻게 쓰는건지 진짜 모름

다음 이미지와 같게 수정했습니다. 비동기 처리를 고려하지 않은 코드를 기준으로 작성했으며, 좀 더 명확하게 명시했습니다...!

[jaewoong-gwon]

바꾸기 전의 이미지에서 확인했을 때, getInputStream() 에서 ServeltInputStream 을 리턴할 때 어떻게 바뀐건지 이미지가 있으면 좋을것 같습니다. 현재는 분리하여 작성한것은 알겠는데, 사용하는 부분에서 어떤 변경사항이 있는지 확인이 어려운것 같네요

getInputStream()에서 변경된 코드는 다음과 같습니다.

• getInputServlet() 을 호출할 때 마다 새로운 InputStream 을 리턴하는거 같은데 저렇게 동작하는게 맞는건가요?
  • 저렇게 구현한 이유가 궁금합니다