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

[Bug] EuPI 통신 후 string데이터 통신에도 EuPIRxManager가 수신되는 현상 #31

Open
SeonJK opened this issue Aug 28, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@SeonJK
Copy link
Member

SeonJK commented Aug 28, 2022

#30 이슈 관련 수정으로 CustomerActivity에서 주문에 대한 전송 구현.

Issue

  • CustomerActivity에서 setCode()를 활용한 String데이터 통신 수신 시에 StoreActivity에서는 mEuPIRxManager가 해당 통신을 수신함...
  • EuPIRxManager를 제거하고 String데이터 통신만 수행해도 RxManager와 관련한 로그는 나오지 않고 있음.
  • CustomerActivity에서 전송에 대한 로그는 정상적으로 기록됨.
@SeonJK SeonJK added the bug Something isn't working label Aug 28, 2022
@phj0407
Copy link
Collaborator

phj0407 commented Aug 29, 2022

  • 첫번째 문제는 메뉴 데이터가 채워진 상태였나요? 현재 최종수정된 파일을 보니 메뉴 데이터가 없으면 string데이터 받는 부분은 작동하지 않고 바로 EuPI 리스너로 넘어가는 듯 합니다.
  • 로그가 아무것도 뜨지 않는다면 통신 타이밍 문제일 가능성도 있을 것 같아요. play() 의 count를 3 말고 -1 로 수정 후 확인해주실 수 있으실까요?
  • storeActivity 파일 로그 더 자세하게 추가 후 PR 올리겠습니다!

@SeonJK
Copy link
Member Author

SeonJK commented Aug 29, 2022

@phj0407

  1. 현재 Customer에서는 저희가 약속했던 #&만 담아서 전송하고 있습니다. 그래서 ORDER_REQUEST 케이스에 걸려서
    Log.i(TAG, "Receive Order Request.")

    해당 라인 로그가 출력될거라고 기대했는데 제가 잘못 이해한 부분이 있을까요?
  • 지금 회사라 다른 부분은 확인 어렵네요ㅠㅠ

@phj0407
Copy link
Collaborator

phj0407 commented Aug 29, 2022

에고 천천히 확인해주셔도 됩니다!!!!

일단 string 데이터 리슨 조건이

if (!allMenu.isNullOrEmpty()) {

여기에 걸려있어서요. 메뉴 디비가 비어있으면 아마 그냥 넘어가버릴 거예요. 그래도 메뉴가 없다는 로그가 뜨게 해뒀는데.... 이 부분도 아무 로그가 안 떴나요ㅠㅠ??
그리고 저희 메뉴 요청을 EuPI 방식으로만 받을거라면 구조가 좀 수정되어야할 것 같아요. 일단 제가 코드 짜고 저녁쯤 PR 올릴테니 리뷰해주시겠어요?? 😂😂

@SeonJK
Copy link
Member Author

SeonJK commented Aug 29, 2022

그 부분이 있었군요!! 네네 아무래도 EuPI 방식으로 받도록 수정을 좀 해야할 것 같습니다ㅜㅜ PR올려주시면 확인할게요!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants