Nettoyage, commentaires.

Fermeture de la fenêtre n'attend plus la fin des animations

Author: GrillePainVert

SabaccModel.cpp

@@ -442,11 +442,6 @@ void JeuSabacc::initTour() {
 
     m_tour++;
     LOGDEBUG << u8"début tour: " << m_tour << std::endl;
-    //nouveau tour, aucun joueur n'a passé ni pioché
-    // todo test
-    //for (uint32_t i = 0; i < NB_POSITIONS; ++i) {
-    //    m_joueurs[i].m_aPasseCeTour = false;
-    //}
 }
 
 
@@ -480,7 +475,7 @@ void JeuSabacc::appliqueConnaissanceJoueur(Position pj, PCGAlea& rng)
     }
 }
 
-//TODO virer les vecteurs
+
 void JeuSabacc::listerActionsLegales(std::vector<Action>& actions) const
 {
     actions.clear();
@@ -1299,9 +1294,6 @@ bool JeuSabacc::finTourJoueurActuel(PCGAlea& rng)
                 initTour(); // MISE A JOUR: m_tour, reset m_aPasseCeTour
                 // MISE A JOUR: m_phaseActuelle (remis à JETON par initTour implicitement ou explicitement)
                 m_phaseActuelle = PHASE_JETON; // Assure qu'on recommence par la phase jeton
-                // Réinitialise aussi m_doitPasser pour le nouveau tour (sauf si un nouvel embargo est joué)
-                // TODO
-                //for (uint32_t i = 0; i < NB_POSITIONS; ++i) m_joueurs[i].m_doitPasser = false;            
             }
         }
         else {

SabaccUI.cpp

@@ -15,19 +15,19 @@ SabaccUI::SabaccUI() : m_jeu(), m_vue(m_jeu), m_rng(rand())
 	m_jeu.m_joueurs[Position::N].m_type = TYPE_JOUEUR::TJ_MONTE_CARLO_SIMPLE;
 	m_jeu.m_joueurs[Position::E].m_type = TYPE_JOUEUR::TJ_MONTE_CARLO_SIMPLE;
 
-	//m_jeu.initJeu(m_rng);
-    //ouvre la view et lance le thread
+    //lance le thread de la vue qui va la créer
     m_vue.demarre();
 
     LOGDEBUG.setActif(false);
 }
 
+//boucle du thread de l'UI
 void SabaccUI::boucle()
 {    
     while (!m_fermeFenetreDemande) {
         //au tour d'une IA de jouer.
         if (m_jeu.m_joueurs[m_jeu.m_positionJActuel].m_type != TYPE_JOUEUR::TJ_HUMAIN &&
-            !m_jeu.m_jeuTermine && !m_vue.animationEnCours()) {
+            !m_jeu.m_jeuTermine && !m_vue.animationEnCours() && !m_fermeFenetreDemande) {
 
             //choisis action
             Action acChoisie;
@@ -44,15 +44,15 @@ void SabaccUI::boucle()
             m_vue.ajouteAnimationApresAction(acChoisie);
             Sleep(10);
             m_jeu.executeAction(acChoisie, m_rng);
-            while (m_vue.animationEnCours()) Sleep(10);
+            while (m_vue.animationEnCours() && !m_fermeFenetreDemande) Sleep(10);
         }
 
         //Tour d'un humain
-        else if (m_jeu.m_joueurs[m_jeu.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN && !m_jeu.m_jeuTermine && !m_vue.animationEnCours()) {
+        else if (m_jeu.m_joueurs[m_jeu.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN && !m_jeu.m_jeuTermine && !m_vue.animationEnCours() && !m_fermeFenetreDemande) {
             std::vector<Action> legalActions;            
             m_actionJeuChoisie = Action::A_INDEFINIE;
             m_jeu.listerActionsLegales(legalActions);
-            m_vue.definitActionsAChoisir(legalActions);
+            m_vue.definitActionsJeuAChoisir(legalActions);
 
             //attend d'avoir reçu une action choisie et que toutes les animations soient finies
             while (!m_redemarrageDemande && !m_fermeFenetreDemande &&( m_vue.animationEnCours()|| (m_actionJeuChoisie == Action::A_INDEFINIE && m_jeu.m_joueurs[m_jeu.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN && !m_jeu.m_jeuTermine)))
@@ -61,7 +61,7 @@ void SabaccUI::boucle()
             m_actionJeuChoisie = Action::A_INDEFINIE;
 
             if (aChoisie != Action::A_INDEFINIE) {
-                m_vue.videActionsAChoisir();
+                m_vue.videActionsJeuAChoisir();
 
                 m_vue.ajouteLogMessage(aChoisie);
                 m_vue.ajouteAnimationAvantAction(aChoisie);
@@ -194,7 +194,7 @@ void SabaccUI::surFinInitManche()
     m_vue.ajouteAnimationSon(Son::SON_WOOSH);
     m_vue.ajouteGrandMessage(ss.str());
 
-    while (m_vue.animationEnCours())
+    while (m_vue.animationEnCours() && !m_fermeFenetreDemande)
         Sleep(20);
 }
 
@@ -236,6 +236,7 @@ void SabaccUI::surMAJReserveJoueur(Position p, uint32_t reserveActuelle, int del
         m_vue.ajouteLogMessage(ss.str());
         m_vue.ajouteGrandMessage(ss.str());
     }
+    
     //LorexZen joue à Ghost Recon Breakpoint comme à CallOf.
 
     //calcul du score si on est perdant

SabaccUI.h

@@ -1,26 +1,37 @@
-#pragma once
+#pragma once
 #include "SabaccUI.h"
 #include "SabaccModel.h"
 #include "SabaccView.h"
 #include "windows.h"
 
 class SabaccUI : JeuSabaccListener, SabaccVueListener{
 public:
-    SabaccUI();    
+    SabaccUI();
+
+    //boucle du thread de l'UI
     void boucle();
 
 private:
+    //flags enregistrant les demandes faite via la vue
     bool m_redemarrageDemande = false;
     bool m_fermeFenetreDemande = false;
-
+    
+    //générateur de nombres aléatoires rapide
     PCGAlea m_rng;
+
+    //logique du jeu
     JeuSabacc m_jeu;
+
+    //vue
     SabaccVue m_vue;
 
+    //action choisie dans la vue
     Action m_actionJeuChoisie=Action::A_INDEFINIE;
+    
+    //stocke le score des 3 dernières parties
     std::deque<int> m_score3Parties;
 
-    // Inherited via JeuSabaccListener
+    // Héritées de JeuSabaccListener
     void surFinInitJeu() override;
     void surFinInitManche() override;
     void surJeuTermine() override;
@@ -30,7 +41,7 @@ class SabaccUI : JeuSabaccListener, SabaccVueListener{
     void surFinTourJoueur() override;
     void surTirageDesResultat() override;
 
-    // Inherited via SabaccViewListener
+    // Héritées de SabaccViewListener
     void surActionJeuChoisie(Action a) override;
     void surRedemarreJeu() override;                   
     void surFermeFenetre() override;

SabaccView.cpp

@@ -167,17 +167,17 @@ void SabaccVue::boucleThread() {
 
     m_fenetre.create(vm, "Sabacc de Kessel par GrillePainVert", sf::Style::Close | sf::Style::Titlebar, sf::State::Fullscreen, settings);
     m_fenetre.setVerticalSyncEnabled(true);
-    m_fenetre.setMinimumSize(sf::Vector2u(1400, 800));
+    m_fenetre.setFramerateLimit(60);    
     chargeElements();
 
     while (m_fenetre.isOpen() && m_keepRunning) {
         traiteEvenements();
         // update(); // If needed for animations or GUI state not tied to game state
         affiche();
-        Sleep(15);
+        //ici pas besoin de delai Sleep car j'ai fixé la MaxFrameRate à 60 en créant la fenetre
     }
 
-    //unload assets
+    //unload assets pas nécessaire ici
 
     //close window
     m_fenetre.close();
@@ -202,22 +202,13 @@ void SabaccVue::traiteEvenements() {
         // Mouse Button Pressed
         if (const auto* mouseButtonReleased = event->getIf<sf::Event::MouseButtonReleased>()) {
             if (mouseButtonReleased->button == sf::Mouse::Button::Left) {
-                //todo test
-                //if (m_jeuSabacc.m_jeuTermine || m_jeuSabacc.m_joueurs[m_jeuSabacc.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN || !m_jeuSabacc.m_joueurs[Position::S].m_enJeu) {
-                //if (m_jeuSabacc.m_joueurs[m_jeuSabacc.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN) {
-                    traiteClicJoueur(mouseButtonReleased->position);
-                //}
+                traiteClicJoueur(mouseButtonReleased->position);
             }
         }
-        // Press Enter Event
-        //if (const auto* keyPressed = event->getIf<sf::Event::KeyPressed>()) {
-        //    if (keyPressed->scancode == sf::Keyboard::Scan::Enter) {
-        //        // This logic needs to be more nuanced.
-        //        // Only advance if waiting for this specific input.
-        //        // For now, let's assume AI turns are fast.
-        //    }
-        //}
-        // catch the resize events
+        
+        //C'est important car le framework SFML affiche une View dont la taille est indépendante de la fenêtre
+        //Ca peut être très utile dans pas mal de cas, mais ici je veux que ma View ait la même résolution que la fenêtre
+        //A chaque resize la View doit être remplacée
         if (const auto* resizedEvent = event->getIf<sf::Event::Resized>())
         {            
             // update the view to the new size of the window
@@ -543,7 +534,7 @@ void SabaccVue::afficheActionsCliquables(const sf::FloatRect& area) {
 
         for (const auto& action : m_actionsAChoisir) {
 
-            ActionCliquable cad(m_policeTexte); // Assuming constructor takes font, or setFont is called
+            BoutonCliquable cad(m_policeTexte); // Assuming constructor takes font, or setFont is called
             cad.gameAction = action;
 
             std::ostringstream actionOss;
@@ -628,7 +619,7 @@ void SabaccVue::afficheActionsCliquables(const sf::FloatRect& area) {
     //Nouveau Jeu
     currentY = area.position.y + area.size.y - buttonHeight;
 
-    ActionCliquable actionNouveauJeu(m_policeTexte);
+    BoutonCliquable actionNouveauJeu(m_policeTexte);
     actionNouveauJeu.ViewAction = VA_NOUVELLE_PARTIE;
     std::string actionStr("Nouvelle Partie");
 
@@ -654,7 +645,7 @@ void SabaccVue::afficheActionsCliquables(const sf::FloatRect& area) {
     currentY -= 1.0f * buttonHeight + 1.0f * padding;
 
     //afficher-cacher cartes
-    ActionCliquable actionAfficherCacher(m_policeTexte);
+    BoutonCliquable actionAfficherCacher(m_policeTexte);
     actionAfficherCacher.ViewAction = VA_AFFICHER_CACHER_CARTES;
 
     actionStr;
@@ -684,7 +675,7 @@ void SabaccVue::afficheActionsCliquables(const sf::FloatRect& area) {
     m_actionsCliquables.push_back(actionAfficherCacher);
 
     //changer la vitesse
-    ActionCliquable actionVitesse(m_policeTexte);
+    BoutonCliquable actionVitesse(m_policeTexte);
     actionVitesse.ViewAction = VA_CHANGE_VITESSE;
 
     actionStr;
@@ -714,7 +705,7 @@ void SabaccVue::afficheActionsCliquables(const sf::FloatRect& area) {
     m_actionsCliquables.push_back(actionVitesse);
 
     //Ferme le jeu
-    ActionCliquable actionFerme(m_policeTexte);
+    BoutonCliquable actionFerme(m_policeTexte);
     actionFerme.ViewAction = VA_FERME;
 
     actionStr="X";
@@ -1066,8 +1057,8 @@ void SabaccVue::traiteClicJoueur(sf::Vector2i mousePos) {
         if (cad.bounds.contains(worldPos)) {
             joueSonBouton();
             Action actionJeu = cad.gameAction;
-            VueAction actionVue = cad.ViewAction;
-            if (actionVue == VueAction::VA_AFFICHER_CACHER_CARTES) {                
+            ActionBouton actionVue = cad.ViewAction;
+            if (actionVue == ActionBouton::VA_AFFICHER_CACHER_CARTES) {                
                 if (!m_afficheCartes && MessageBox(m_fenetre.getNativeHandle(), L"Attention: Afficher les cartes efface les scores. Voulez-vous continuer?", L"Afficher toutes les cartes", MB_YESNO) == IDYES)
                 {
                     m_afficheCartes = true;
@@ -1079,7 +1070,7 @@ void SabaccVue::traiteClicJoueur(sf::Vector2i mousePos) {
 				}
                 return;
             }
-            else if (actionVue == VueAction::VA_NOUVELLE_PARTIE) {
+            else if (actionVue == ActionBouton::VA_NOUVELLE_PARTIE) {
                 //appelle onGameRestart des listeners
 
                 if (m_jeuSabacc.m_jeuTermine || MessageBox(m_fenetre.getNativeHandle(), L"Voulez-vous vraiment recommencer la partie?", L"Nouvelle Partie", MB_YESNO) == IDYES) {
@@ -1094,7 +1085,7 @@ void SabaccVue::traiteClicJoueur(sf::Vector2i mousePos) {
                 }
                 return;
             }
-            else if (actionVue == VueAction::VA_CHANGE_VITESSE) {
+            else if (actionVue == ActionBouton::VA_CHANGE_VITESSE) {
                 m_rapide = !m_rapide;
                 if (m_rapide) {
                     m_carteAnimDuree = CARTE_ANIM_DUREE_RAPIDE;
@@ -1106,7 +1097,7 @@ void SabaccVue::traiteClicJoueur(sf::Vector2i mousePos) {
                 }                    
                 return;
             }
-            else if(actionVue ==VueAction::VA_FERME) {                
+            else if(actionVue ==ActionBouton::VA_FERME) {                
                 if (MessageBox(m_fenetre.getNativeHandle(), L"Voulez-vous vraiment quitter?", L"Quitter le Jeu", MB_YESNO) == IDYES) 
                     m_pListener->surFermeFenetre();
                 return;
@@ -1120,7 +1111,7 @@ void SabaccVue::traiteClicJoueur(sf::Vector2i mousePos) {
 }
 
 void SabaccVue::ajouteLogMessage(const std::string& message) {
-    if (m_texteMessages.size() >= MAX_MESSAGES) {
+    while (m_texteMessages.size() >= MAX_MESSAGES) {
         m_texteMessages.erase(m_texteMessages.begin());
     }
     sf::Text text(m_policeTexte);
@@ -1398,10 +1389,7 @@ void SabaccVue::affiche() {
 
     afficheCentreTable(boardArea);
 
-    //todo test
-    //if (m_jeuSabacc.m_jeuTermine || m_jeuSabacc.m_joueurs[m_jeuSabacc.m_positionJActuel].m_type == TYPE_JOUEUR::TJ_HUMAIN || !m_jeuSabacc.m_joueurs[Position::S].m_enJeu) {
-        afficheActionsCliquables(actionArea);
-    //}
+    afficheActionsCliquables(actionArea);
 
     afficheMessages(zoneBoiteMessages);