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

explosion_count 추가 #102

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

byundojin
Copy link
Member

@byundojin byundojin commented Dec 30, 2024

지뢰 폭발 횟수 통계(event 사용 x, board로 찾음)
이유 ->
지뢰 폭발을 효율적으로 감지할 수 없습니다.
그러나 Section이 이를 자체적으로 감지하여 처리한다면 쉽고 빠르게 폭발 횟수를 구할 수 있습니다.
이는 다음과 같은 상황에도 적용시킬 수 있습니다(열린 타일 개수, 찾은 지뢰 개수)

아직 클라이언트에게 어떻게 전달할지 생각하지 않았습니다.
http로 통신을 계획하였으나, 이는 event로 동작하는 현재 서버 구조상 많은 어려움이 예상됩니다.
이에 대한 좋은 의견 있다면 제시해주세요.

- get_all 추가 -> section 전체 조회
- SectionHeader 추가
@byundojin byundojin requested a review from onee-only December 31, 2024 04:26
@onee-only
Copy link
Member

구현하는 기능이 효율 측면을 그렇게 따지지 않아도 된다고 생각했습니다.
(제가 잘 기억하고 있다면) API 호출을 통해 통계를 불러오는 게 아닌 나중에 따로 모아서 한번에 결과를 내기만 하는 것으로 결론이 났었으니까요.

그래서 저는 event 테이블을 이용하여 원하는 결과를 내는 스크립트 여러 개를 만드는 방법을 생각했습니다.

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.

2 participants