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

Merge Gamble Feature #18

Merged
merged 8 commits into from
Dec 3, 2023
Merged

Merge Gamble Feature #18

merged 8 commits into from
Dec 3, 2023

Conversation

khj0998
Copy link
Owner

@khj0998 khj0998 commented Dec 3, 2023

겜블 스크린 제작

겜블이 끝나면 다시 겜블 선택화면으로 복귀
겜블 선택화면에서 ESC를 누르면 메인메뉴로 이동
GambleScreen이 호출 될 때 플레이어의 Currency를 불러옴
초기화 함수는 Ship, Bullet, Entity를 초기화
겜블 화면은 총 3개 겜블 선택화면, 파칭코, 가위바위보
이를 update(), draw()에서 case문으로 나눠서 사용.
Entity class에서 Input Parameter에 따라 다른 선언 추가

  • 겜블에서 Entity는 계속 Sprite가 바뀌는 데 이 때 사용하는 Sprite가 확정됐는지 확인하는 Boolean, Sprite의 번호를 확인하는 Int 추가
    베팅금은 좌우로 10씩, 상하로 100씩 오르내리고
    파칭코와 가위바위보 둘 중 하나의 겜블을 선택
    파칭코는 계속해서 바뀌는 Entity를 공격해 맞출 시 Sprite가 확정되고 Entity는 3개가 있음
    3개 모두 맞출 경우 베팅금의 3배, 특별한 한 경우는 7배
    2개 맞출 경우 베팅금의 1.2배
    가위바위보는 플레이어가 가위 바위 보 중에 하나를 고름
    고르는 순간 계속해서 바뀌던 컴퓨터의 가위바위보가 멈추고 결과를 계산한다.
    비기는 경우 잠깐 멈춘 뒤 다시 시작하고
    이기면 랜덤으로 상금의 배율이 정해진다.
    배율은 {1,1,1.2,1.5,1.5,2,2,3,5,7,} 중에 하나로 정해진다.
    겜블이 끝났는지 확인 후 플레이어의 Currency를 업데이트한다.
    겜블 도중에는 나갈 수 없고 겜블이 끝나면 ESC로 이번 겜블에서 나갈 수 있다.
    함수와 변수의 사용처는 코드에 주석으로 넣어둠

@khj0998 khj0998 requested a review from myunghong December 3, 2023 09:46
@myunghong
Copy link
Collaborator

  1. 도박의 단위가 100원, 10원이 아닌 10원, 1원이 되는게 더 좋아보임
  • 스킨 구매가 50원, 아이템이 10~20원인것에 비해 도박 배팅액 단위가 큼
  1. 가위바위보 도박에서 무승부가 됐을 때 커서 이동이 가능함

  2. 가위바위보 도박에서 무승부와 이겼지만 X1의 결과가 나왔을 때의 얻는 금액이 같음. 이기고 X1 돈을 받는 경우를 없애면 어떨까 함

@myunghong myunghong merged commit 765f96f into Develop Dec 3, 2023
@khj0998
Copy link
Owner Author

khj0998 commented Dec 4, 2023

  1. 수정하겠음
  2. 원래 의도가 맞음, 무승부 됐을 때 선택은 불가능
  3. 수정하겠음

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