anitab-org / powerup-iOS

PowerUp is an educational choose-your-own-adventure game that utilizes a users uploaded curriculum to empower pre-adolescents to take charge of their reproductive health. This is the iOS version of the game.
GNU General Public License v2.0
40 stars 108 forks source link

Score label is not functional when we replay the games #353

Open anubhavpulkit opened 4 years ago

anubhavpulkit commented 4 years ago

Description

When we replay Vocab Matching Game, Minesweeper Game and Sink To Swim Game final score is not updated in the upper left most side.

To update the value of score first we have to make a decision:- i) we update a new value in total score by removing the old value of that game. ii) we update the value if the new score is greater then the last score of that game.

Mocks

sys issue 789

Acceptance Criteria

Update [Required]

Definition of Done

Estimation

[6] hours

anubhavpulkit commented 4 years ago

@sunjunkie Please have a look can I work on this issue?

sunjunkie commented 4 years ago

Yes, you can work on this. First check to see what the Android app does about this score.

anubhavpulkit commented 4 years ago

@sunjunkie Is it important to use Android algorithm to count final score, Can I propose my own algorithm/idea to count final score?

sunjunkie commented 4 years ago

The apps should be consistent. Is there something wrong with the score calculation in the Android version?

anubhavpulkit commented 4 years ago

In Android version sequence of mini-games is different than iOS and calculation of score in Android is totally unpredictable I play 8-9 times a full game but I can't understand the logic of there Total score.

anubhavpulkit commented 4 years ago

Actually when I tap reply in android Powerup app it opens new mini games so but in some positions score label work fine.

On Thu, 5 Mar 2020, 03:03 Anubhav Singh, notifications@github.com wrote:

In Android version sequence of mini-games is different than iOS and calculation of score in Android is totally unpredictable I play 8-9 times a full game but I can't understand the logic of there Total score.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/anitab-org/powerup-iOS/issues/353?email_source=notifications&email_token=ALMYYFTXPQBZCLQG5NYJI7TRF3CLRA5CNFSM4K5TW6L2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN2NU7Q#issuecomment-594860670, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALMYYFX4TLXTLF5K7OJAXD3RF3CLRANCNFSM4K5TW6LQ .

anubhavpulkit commented 4 years ago

@sunjunkie Are you available on this weekend for help?

sunjunkie commented 4 years ago

@anubhavsingh16 Yes, I can check in periodically.

Go ahead and decide on an algorithm to update the score. Be sure to document that well in the code and in your PR description. We can always change this algorithm later if we need to.

anubhavpulkit commented 4 years ago

Ok, Thank You @sunjunkie

anubhavpulkit commented 4 years ago

In the current scenario, we have only one variable which store points is score which is not updating after the replay of the game. I want to propose 2 variables currentScore and maxScore. if the user plays the first time and gain "x" points then we store currentScore and maxScore is equal to x and we use currentScore for score view(which come after game is over) and maxScore for the total score. If user replay the game then, if(currentScore > maxScore) { then we update the maxScore is equal to currentScore } else maxScore is remain the last score and currentScore is update and show their current score in score view.

anubhavpulkit commented 4 years ago

@what you think about this?

In the current scenario, we have only one variable which store points is score which is not updating after the replay of the game. I want to propose 2 variables currentScore and maxScore. if the user plays the first time and gain "x" points then we store currentScore and maxScore is equal to x and we use currentScore for score view(which come after game is over) and maxScore for the total score. If user replay the game then, if(currentScore > maxScore) { then we update the maxScore is equal to currentScore } else maxScore is remain the last score and currentScore is update and show their current score in score view.

@sunjunkie any suggestions?

sunjunkie commented 4 years ago

ok. sounds good.

anubhavpulkit commented 4 years ago

@sunjunkie I want to add this issue in my proposal. And want to do it later after the proposal submission. Because I have an idea which is linked with this issue so can you give me permission to take some time and present my proposal, if you like my idea then we work on new method to calculate and use scores otherwise I solve this issue with the simple method which is explain above.

anubhavpulkit commented 4 years ago

ok. sounds good.

Thank You @sunjunkie

anubhavpulkit commented 4 years ago

@sunjunkie Please have a look.

sunjunkie commented 4 years ago

Have a look at what?

anubhavpulkit commented 4 years ago

Sorry @sunjunkie 🙄