Skip to content
This repository has been archived by the owner on May 5, 2020. It is now read-only.

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

Open
1 of 3 tasks
anubhavpulkit opened this issue Feb 28, 2020 · 17 comments
Open
1 of 3 tasks

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

anubhavpulkit opened this issue Feb 28, 2020 · 17 comments

Comments

@anubhavpulkit
Copy link
Contributor

anubhavpulkit commented Feb 28, 2020

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]

  • [Changes in result code]

Definition of Done

  • All of the required items are completed.
  • Approval by 1 mentor.

Estimation

[6] hours

@anubhavpulkit
Copy link
Contributor Author

anubhavpulkit commented Feb 28, 2020

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

@sunjunkie
Copy link
Contributor

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

@anubhavpulkit
Copy link
Contributor Author

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

@sunjunkie
Copy link
Contributor

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

@anubhavpulkit
Copy link
Contributor Author

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
Copy link
Contributor Author

anubhavpulkit commented Mar 4, 2020 via email

@anubhavpulkit
Copy link
Contributor Author

@sunjunkie Are you available on this weekend for help?

@sunjunkie
Copy link
Contributor

@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
Copy link
Contributor Author

Ok, Thank You @sunjunkie

@anubhavpulkit
Copy link
Contributor Author

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
Copy link
Contributor Author

@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
Copy link
Contributor

ok. sounds good.

@anubhavpulkit
Copy link
Contributor Author

@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
Copy link
Contributor Author

ok. sounds good.

Thank You @sunjunkie

@anubhavpulkit
Copy link
Contributor Author

@sunjunkie Please have a look.

@sunjunkie
Copy link
Contributor

Have a look at what?

@anubhavpulkit
Copy link
Contributor Author

Sorry @sunjunkie 🙄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants