RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 04:07:44 +00:00
Code Issues Activity Actions

Solitaire: Only update high score after a victorious game

Browse source 
Doesn't make much sense to update the high score on a lost game.
This commit is contained in:
Timothy Flynn 2021-05-25 11:44:25 -04:00 • committed by Andreas Kling
parent 95401d2ca2
commit 0b30a0febb
3 changed files with 14 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Userland/Games/Solitaire/Game.cpp
View file

@ -80,7 +80,7 @@ void Game::start_game_over_animation()
start_timer(s_timer_interval_ms);
if (on_game_end)
on_game_end();
on_game_end(GameOverReason::Victory, m_score);
}
void Game::stop_game_over_animation()
@ -103,7 +103,7 @@ void Game::setup(Mode mode)
m_mode = mode;
if (on_game_end)
on_game_end();
on_game_end(GameOverReason::NewGame, m_score);
for (auto& stack : m_stacks)
stack.clear();

7
Userland/Games/Solitaire/Game.h
View file

@ -21,6 +21,11 @@ enum class Mode : u8 {
__Count
};
enum class GameOverReason {
Victory,
NewGame,
};
class Game final : public GUI::Frame {
C_OBJECT(Game)
public:
@ -34,7 +39,7 @@ public:
Function<void(uint32_t)> on_score_update;
Function<void()> on_game_start;
Function<void()> on_game_end;
Function<void(GameOverReason, uint32_t)> on_game_end;
private:
Game();

12
Userland/Games/Solitaire/main.cpp
View file

@ -93,11 +93,6 @@ int main(int argc, char** argv)
game.on_score_update = [&](uint32_t score) {
statusbar.set_text(0, String::formatted("Score: {}", score));
if (score > high_score) {
update_high_score(score);
statusbar.set_text(1, String::formatted("High Score: {}", high_score));
}
};
uint64_t seconds_elapsed = 0;
@ -116,9 +111,14 @@ int main(int argc, char** argv)
seconds_elapsed = 0;
timer->start();
};
game.on_game_end = [&]() {
game.on_game_end = [&](Solitaire::GameOverReason reason, uint32_t score) {
if (timer->is_active())
timer->stop();
if ((reason == Solitaire::GameOverReason::Victory) && (score > high_score)) {
update_high_score(score);
statusbar.set_text(1, String::formatted("High Score: {}", high_score));
}
};
GUI::ActionGroup draw_setting_actions;