Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EAT-112] ServletInputStream 구현체 분리 #31

Open
wjdwnsdnjs13 opened this issue May 6, 2024 · 5 comments
Open

[EAT-112] ServletInputStream 구현체 분리 #31

wjdwnsdnjs13 opened this issue May 6, 2024 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@wjdwnsdnjs13
Copy link
Member

wjdwnsdnjs13 commented May 6, 2024

상위 이슈

이슈 링크

이슈 내용

  • 이미지

    • image

    • image
  • 설명
    • 코드 리뷰 시 피드백 받았던 ServletInputStream 구현체를 분리했습니다!
    • 분리 후 정상작동 확인 완료했고, 테스트 완료했습니다
@wjdwnsdnjs13 wjdwnsdnjs13 self-assigned this May 6, 2024
@jaewoong-gwon jaewoong-gwon added the enhancement New feature or request label May 6, 2024
@jaewoong-gwon
Copy link
Contributor

질문

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

@jaewoong-gwon
Copy link
Contributor

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

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

@wjdwnsdnjs13
Copy link
Member Author

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

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

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

@wjdwnsdnjs13
Copy link
Member Author

질문

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

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

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

@jaewoong-gwon
Copy link
Contributor

jaewoong-gwon commented May 6, 2024

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants