본문 바로가기

승인

은퇴로 인해 베팅 유산이 잘못 산정되는 문제

류화영2023.06.06 09:17조회 수 36댓글 2

    • 글자 크기

버그 제보에 가깝긴 한데.. 일단 제안 분류로 올려 봅니다.

 

은퇴 전후로 토너먼트 베팅 및 당첨이 진행될 때 유산 포인트 계산을 잘못하는 문제가 있습니다. 동일 베팅이 진행되던 중에 은퇴를 하면, 은퇴 시점에 투자금 내역이 초기화되어 은퇴 이후의 투자분만 가지고 수익률을 계산하는 문제입니다.

 

장수가 은퇴하면 RankColumn의 내용을 초기화하게 되어 있고, 따라서 베팅 투자금(betgold)도 0이 됩니다(General.php rebirth()). 베팅에 당첨되면 투자금(betgold)이 0이 아닐 때만 누적 수익(betwingold)에 이번 베팅의 수익(reward)을 합산하지만(Betting.php giveReward()), 은퇴 후 투자를 하면 betgold가 업데이트 됩니다(Betting.php bet()). 즉, 해당 베팅으로 거둔 수익 전부가 누적 수익에 잡히지만, 투자금은 은퇴 후의 투자금만 잡히게 됩니다. 이후 베팅 유산을 betWinRate = betwingold / max(1, betgold)에 따라 계산하면 문제가 될 거 같습니다(InheritancePointManager.php getInheritancePoint()).

 

제 생각으로는, 은퇴할 때 현재 열려있는 베팅의 기존 투자 내역을 DB에서 삭제하는 게 좋을 거 같습니다. gameStor에 LAST_BETTING_ID_KEY가 있으니 베팅 정보는 가져올 수 있을 거고, 해당 베팅이 열려있는지 여부는 별도로 기록해야 할 수도 있겠습니다. 투자 내역을 그냥 삭제하면 해당 자금은 증발하게 되니, 투자한 장수명만 더미 값으로 바꾸는 게 어떨까 싶습니다. 은퇴 시점에 기존 베팅 내역이 날아간다는 사실을 사람들이 인지만 하고 있으면 큰 문제는 아리리라 생각합니다.

 

여담으로 InheritancePointManager.php 는 왜 인덴트가 스페이스 두 칸일까요? .. 조금 무서웠습니다. 감사합니다.

    • 글자 크기

댓글 달기

댓글 2
번호 분류 제목 글쓴이 최근 수정일 날짜 조회 수
공지 토론중 갠톡, 단톡 등에서 설명하였던 내용을 모두 적어둡니다.18 Hide_D 2023.04.25 2023.02.05 312
공지 완료 시나리오 제안 시 구성 방법에 대해 안내합니다. (22-03-13) Hide_D 2022.03.13 2022.03.13 984
공지 완료 마이너 서버 요청 가이드 갓세정 2022.06.28 2018.12.03 2419
공지 완료 신규 시나리오 제안 가이드 갓세정 2018.07.16 2018.05.30 2898
321 제안 턴 시간 고정 단위를 느슨하게 하는게 어떨까요?6 Hide_D 2023.06.06 2023.06.05 60
320 반려 태현청생부(부적) 상향해주세요1 김나영 2022.07.06 2022.05.13 144
319 완료 크보시나리오 수정판 갓세정 2018.05.22 2018.05.21 898
318 부분완료 쿠킹덤 이미지 수정 네이미 2021.09.19 2021.04.30 551
317 제안 쿠키런_킹덤 시나리오 업데이트 네이미 2022.05.18 2022.05.18 115
316 부분완료 쿠키런_킹덤 시나리오2 네이미 2021.09.19 2021.04.30 680
315 승인 커맨드 로깅 Hide_D 2022.03.19 2022.03.19 93
314 승인 카카오 로그인 시 등록 절차 및 에러 메시지를 좀 더 친절하게. Hide_D 2019.11.20 2019.11.20 609
313 부분완료 치안 맥스치 변경. 기밀권한에 국방 추가.4 Tiasse 2018.09.19 2018.08.31 959
312 부분완료 충차/보병 개선 참고안10 SARS 2018.09.19 2018.08.13 933
311 제안 추방 전메 대사 출력 빈도 좀 줄여주세요 블랙죠 2024.02.11 2024.02.11 32
310 반영중 추가 버그 Hide_D 2021.10.21 2021.10.21 376
309 부분완료 초반 전쟁금지 풀리는 날짜 알려주면 좋겠어요.1 노도치 2018.10.07 2018.09.13 879
308 완료 초기 스텟이 150이 되어야만 생성이 가능하도록.6 Tiasse 2018.08.21 2018.08.14 838
307 승인 체섭 국가성향 개편안10 홈모바일 2018.09.06 2018.08.12 828
306 완료 첫 하야 했을때 패널티 없는거->멸망에 준하는 패널티로5 김나영 2022.09.21 2022.07.20 219
305 승인 천통 이후 천통군주에게 '이민족 모드' 발동 권한 부여 Hide_D 2021.11.16 2021.11.16 197
304 완료 천재가 안뜨는 버그 수정 Hide_D 2020.05.13 2020.02.03 560
303 부분완료 차병 밸런스 조정 관련 (벽력거) 사스케 2023.04.09 2023.04.09 56
302 완료 찐유저무쌍!2 찌질이 2019.10.04 2019.09.27 627
이전 1 2 3 4 5 6 7 8 9 10... 18다음
첨부 (0)