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

[Update] Reduce StoreActivity listening feature #34

Merged
merged 6 commits into from
Sep 6, 2022

Conversation

phj0407
Copy link
Collaborator

@phj0407 phj0407 commented Sep 5, 2022

#33 에 올렸던 것 처럼 EuPI 를 이용해서 메뉴/주문 요청 구분해서 메시지 띄우는 기능까지만 정리했습니다. 기기가 하나 뿐이라 테스트는 아직 못 해봤네요ㅠㅠ 시간 되실 때 확인 부탁드리겠습니다!

Copy link
Member

@SeonJK SeonJK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!! 머지되면 실기기로 테스트 해볼게요!

@SeonJK SeonJK assigned phj0407 and unassigned SeonJK and serim53 Sep 5, 2022
@designe
Copy link
Member

designe commented Sep 5, 2022

@phj0407 @SeonJK
여러분이 작성하신 코드를 보니 EuPI 쪽에 개선되야 할 사항이 매우 많다는 것을 느끼게 됩니다!!

사실 EuRxManager나 EuTxManager는 Singleton 처럼 사용되는 걸 의도하긴 했으나
이게 여러개의 오브젝트화가 열려 있어서 우선 Singleton 적용이 필요함을 느끼네요.

나중에 이러한 코드 방지를 위한 EuTxManager나 EuRxManager에 Singleton 적용을 여러분이 해주시면 좋겠다는 생각이 드네요.

SIngleton 적용도 Dashboard Core에 추가 완료하였습니다!

        mEuPIRxManagerMenu = EuRxManager(EuOption.ModeType.EUPI)
        mEuPIRxManagerMenu.setOnWaveKeyDown(RequestCodeEnum.MENU_REQUEST.code.toInt()) {
            Toast.makeText(this, "Menu request detected", Toast.LENGTH_SHORT).show()
            Log.d(TAG, "mEuPIRxManagerMenu - Receive menu request.")
        }
        mEuPIRxManagerOrder = EuRxManager(EuOption.ModeType.EUPI)
        mEuPIRxManagerOrder.setOnWaveKeyDown(RequestCodeEnum.ORDER_REQUEST.code.toInt()) {
            Toast.makeText(this, "Order request detected", Toast.LENGTH_SHORT).show()
            Log.d(TAG, "mEuPIRxManagerOrder - Receive order request.")
        }

위 코드는 아래처럼 쓰는걸 권장합니다 :)

        mEuPIRxManager = EuRxManager(EuOption.ModeType.EUPI)
        mEuPIRxManager.setOnWaveKeyDown(RequestCodeEnum.MENU_REQUEST.code.toInt()) {
            Toast.makeText(this, "Menu request detected", Toast.LENGTH_SHORT).show()
            Log.d(TAG, "mEuPIRxManagerMenu - Receive menu request.")
        }
        mEuPIRxManager.setOnWaveKeyDown(RequestCodeEnum.ORDER_REQUEST.code.toInt()) {
            Toast.makeText(this, "Order request detected", Toast.LENGTH_SHORT).show()
            Log.d(TAG, "mEuPIRxManagerOrder - Receive order request.")
        }

@phj0407
Copy link
Collaborator Author

phj0407 commented Sep 5, 2022

아하, 굳이 오브젝트를 두 개 생성하지 않아도 되는 거였군요ㅎㅎ! 수정했습니다~~ 나머지는 이대로 머지해도 괜찮을까요?

@designe
Copy link
Member

designe commented Sep 6, 2022

@phj0407 넵!
두분 덕분에 EuPI 쪽 API 디자인이 많이 개선될 것 같습니다.
간단한 디자인 패턴이라도 의도에 맞게 사용되기 위해 꼭 필요하다는 생각이 듭니다 :)

@phj0407 phj0407 merged commit b4740c5 into euphony-io:main Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants