Fonctionnement incorrect de mon programme

Bonjour!

J'ai écrit un programme qui tourne en boucle en utilisant la structure suivante:

Code:

---

1 2 3 4 5 6 7 8 9

while (z != 100) { // Emplacement 1 switch (z) { case 0: // Emplacement 2 break; } }

---

Dans cet extrait, j'ai relevé deux emplacements. Au premier, j'instancie une classe, et au deuxième j'utilise cette instance de ma classe.

Le problème, c'est que je programme ne fonctionne pas correctement.

Dans la version précédente, l'instanciation de ma classe se trouvait en-dehors de la boucle "while", et ça marchait. Avez-vous une analyse plus précise pour moi, s'il-vous-plaît?

Merci d'avance!

Salut

C'est normal, si tu instancies ta classe en créant un objet (comment l'instancies-tu ? sur la pile je pense ?) dans Emplacement 1, cet instance sera détruite à la fin de l'itération du while, puis recréée au début de la suivante. Si tu n'as besoin de ton objet que pour ce while et qu'il doit rester le même pour toutes les itérations, tu peux utiliser un simple scope de cette manière :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

// Rien ici, ce n'est pas une boucle, c'est un simple scope { ClasseA objeta; while (z != 100){ ClasseB objetb; // Emplacement 1 switch (z) { case 0: // Emplacement 2 break; } } // objetb est détruit ici à chaque itération } // objeta est détruit ici

---

Oui, je comprends... mais je ne peux pas instancier mes classes en-dehors de la boucle "while", parce que ça ferait une liste de plusieurs milliers en fin de compte.
Qu'est-ce que je peux faire?

que signifie "pas correctement"?

montre nous ce que tu as fait concretement, et quel est le symptome de camarchepashitude?

Le compilateur passe dessus sans râler, ce qui signifie que le code est au-moins cohérent.
En gros, il y a un "RenderWindow" (SFML) qui est déclaré dans le programme principal, puis passé par adresse dans la classe que je dois instancier. Une des fonctions-membres réalise l'affichage en utilisant la méthode "draw" de "RenderWindow".
Le problème, c'est qu'à l'exécution, j'ai seulement un écran noir.

Mais la classe est faite correctement, puisqu'elle fonctionnait avant. Le dernier changement en date, c'est une restructuration du programme principal, et depuis je suis obligé de déclarer ma "RenderWindow" en-dehors de la classe. Du coup, ça n'affiche plus rien.

J'ai de nouvelles informations!

Ce code-ci fonctionne parfaitement:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45

#include <SFML/Graphics.hpp> #include <iostream> #include <windows.h> #include <string.h> #include <math.h> #include <stdio.h> #include <stdlib.h> #include <ctime>   #include "MenuPrincipal.h" #include "Texte1.h"   using namespace std; using namespace sf;   void MenuPrincipal() { int width(GetSystemMetrics(SM_CXSCREEN)); // Récupérer résolution horizontale de l'écran int height(GetSystemMetrics(SM_CYSCREEN)); // Récupérer résolution verticale de l'écran   int z(0);   Font fonte; // Fonte pour le texte des menus fonte.loadFromFile("Fontes/SuperMario256.ttf"); // Chargement d'un fichier font   Color couleur; // Couleur pour le texte des menus couleur.r=255; couleur.g=100; couleur.b=0; couleur.a=255; // Création d'une couleur (rouge-orangé opaque)   // Déclarer la fenêtre RenderWindow window;   // Afficher la fenêtre window.create(VideoMode(width, height), "Fenetre", Style::Fullscreen);   window.clear(); // Effacer l'écran   Texte1 A(width, height, fonte, couleur, window); // Constructeur   A.Set("Bonjour Monsieur"); // Constructeur   A.Anim(); // Animer Texte1   window.display(); // Afficher l'écran   Sleep(5000); // Ca c'est juste pour qu'on ait le temps de voir quelque chose =D }

---

Et celui-ci ne fonctionne pas (écran noir):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55

#include <SFML/Graphics.hpp> #include <iostream> #include <windows.h> #include <string.h> #include <math.h> #include <stdio.h> #include <stdlib.h> #include <ctime>   #include "MenuPrincipal.h" #include "Texte1.h"   using namespace std; using namespace sf;   void MenuPrincipal() { int width(GetSystemMetrics(SM_CXSCREEN)); // Récupérer résolution horizontale de l'écran int height(GetSystemMetrics(SM_CYSCREEN)); // Récupérer résolution verticale de l'écran   int z(0), y(0);   Font fonte; // Fonte pour le texte des menus fonte.loadFromFile("Fontes/SuperMario256.ttf"); // Chargement d'un fichier font   Color couleur; // Couleur pour le texte des menus couleur.r=255; couleur.g=100; couleur.b=0; couleur.a=255; // Création d'une couleur (rouge-orangé opaque)   // Déclarer la fenêtre RenderWindow window;   // Afficher la fenêtre window.create(VideoMode(width, height), "Fenetre", Style::Fullscreen);   while (z < 1000) { window.clear(); // Effacer l'écran   switch (z) { case 0: Texte1 A(width, height, fonte, couleur, window); // Constructeur   switch (y) { case 0: A.Set("Bonjour Monsieur"); // Constructeur y=1; break; case 1: A.Anim(); // Animer Texte1 break; } break; }   window.display(); // Afficher l'écran } }

---

Une idée?

Rien que le switch est un contexte si je ne m'abuse, l'objet A est détruit dès qu'on en sort, et jamais reconstruit, non? A mon sens tous les objets persistants devraient être instanciés hors de la boucle et utilisés ensuite dedans. Sinon, y'a t'il une quelconque synchro pour le clear ne suive pas immédiatement le display ?

En fait, chronologiquement on appelle "clear" et "display" alternativement et en boucle. Mais il y a une attente de 1/30 secondes après display (c'est de l'animation image par image).

Je peux instancier mes classes hors de la boucle, mais ça me fera une liste de plusieurs centaines d'objets (et c'est pas très utile de préparer le niveau 100 alors que le deuxième n'est même pas encore débloqué, non? =)

Tout dépend de ton modèle de transition entre niveaux...
Si c'est discontinu, à mon avis tu devrais avoir deux niveaux de boucle:

Code:

---

1 2 3 4 5 6

while(jeuEnCours){ //initialisation des objets du niveau (idéalement avec groupement sous la houlette d'une instance représentant le niveau et gérant donc sa durée de vie) while(niveau en cours){ //animation utilisant les objets du niveau } }

---

Par contre, si tu as une transition progressive sans frontière de niveaux définie, tu vas devoir avoir un objet (instancié hors de la boucle) qui gère la durée de vie de tes objets (lesquels devront être référencés par des (smart)pointers puisque leur durée de vie est définie par quelque chose de plus compliqué (et imprévisible) qu'un scope.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

Monde m; while(jeuEnCours){ if(elementAdoitNaitre){ w.creeElement("elementA",new elementA); } if(elementAdoitMourir){ w.detruireElement("elementA"); } if(elementAEvenementA){ w.retrouverObjet("elementA").evenementA(); } }

---

Edit: ces deux modèles ne sont pas mutuellement exclusifs, si certains objets suivent le premier, d'autres le deuxième tu peux les combiner. Par ailleurs, le second système étant plus complexe autant recourir au maximum au premier. Ceci dit, je dirais qu'il te faudra bien un objet

Ah! J'avais pas pensé à mettre ce qui fait l'affichage à 30 images par seconde dans la boucle while. Mais c'est un peu con de la mettre à chaque "case" du "switch (z)", non?

Citation:

---

Envoyé par Armulis

En fait, chronologiquement on appelle "clear" et "display" alternativement et en boucle. Mais il y a une attente de 1/30 secondes après display (c'est de l'animation image par image).

Je peux instancier mes classes hors de la boucle, mais ça me fera une liste de plusieurs centaines d'objets (et c'est pas très utile de préparer le niveau 100 alors que le deuxième n'est même pas encore débloqué, non? =)

---

En fait ce que leternel et therwald est de faire quelque chose dans ce genre :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

Texte1 A(width, height, fonte, couleur, window); while (z < 1000) { window.clear(); // Effacer l'écran   switch (z) { case 0: //1ere solution A = Text1(width, height, fonte, couleur, window);   //ou mieux encore afin d'éviter un appel au constructeur et une affectation, définir un setter sur window puisque c'est le seul paramètre a changer //2e solution A.setWindow(window)   switch (y) { case 0: A.Set("Bonjour Monsieur"); // Constructeur y=1; break; case 1: A.Anim(); // Animer Texte1 break; } break; }   window.display(); // Afficher l'écran } }

---

Les objets sont mis à jour a chaque tour de boucle (pas d'instanciation de 1000 objets), par contre autre point que j'ai remarqué dans ta boucle c'est que tu n'incrémentes pas ton z du coup tu fais une boucle infinie là.

Citation:

---

Envoyé par Armulis

Ah! J'avais pas pensé à mettre ce qui fait l'affichage à 30 images par seconde dans la boucle while. Mais c'est un peu con de la mettre à chaque "case" du "switch (z)", non?

---

Pourquoi à chaque case? La synchro survient à priori tout le temps (tu veux un FPS stabilisé, non?), donc autant synchroniser juste avant display, hors du switch (mais dans le while).

Ben je synchronise déjà comme ça. Je peux pas faire autrement, si?

Code:

---

window.setVerticalSyncEnabled(true);

---

ou

Code:

---

window.setFramerateLimit(30);

---

http://www.sfml-dev.org/tutorials/2....r-le-framerate

Citation:

---

Envoyé par Armulis

Ben je synchronise déjà comme ça. Je peux pas faire autrement, si?

---

Je posais juste la question pour m'assurer que ce n'était pas le problème (on ne sait jamais, les oublis les plus gros sont parfois les plus difficiles à repérer...justement parce que c'est si gros qu'on ne pense pas qu'on a pu faire l'erreur...)

Ton problème c'est la durée de vie des objets. Réfléchis à ce qu'on vient de te dire sur les scopes et les durées de vie, et tu devrais trouver la solution à ton problème.

Oui, merci Ehonn, mais j'ai l'impression que c'est un peu trop haut-niveau pour mon programme. En fait, je fais déjà ça avec un bloc d'instructions qui est parfaitement défini, tandis que je ne sais pas comment fonctionnent les méthodes qui tournent en arrière-plan... mais je sais pas, tu peux peut-être me fournir plus d'informations? C'est que je comprends pas ce que ça va m'apporter?

La SFML fournit ce qu'il faut pour limiter les FPS. Utiliser ces fonctions permet d'alléger le code, augmenter sa lisibilité et sa fiabilité :)
Si tu veux savoir comment elles fonctionnent, regarde le code source de la SFML (il est libre) ;)

OK, merci.
Sinon, par-rapport au problème que j'ai montré tout au début, vous avez quelque chose? =)

Citation:

---

Envoyé par Danny-k

En fait ce que leternel et therwald est de faire quelque chose dans ce genre :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

Texte1 A(width, height, fonte, couleur, window); while (z < 1000) { window.clear(); // Effacer l'écran   switch (z) { case 0: //1ere solution A = Text1(width, height, fonte, couleur, window);   //ou mieux encore afin d'éviter un appel au constructeur et une affectation, définir un setter sur window puisque c'est le seul paramètre a changer //2e solution A.setWindow(window)   switch (y) { case 0: A.Set("Bonjour Monsieur"); // Constructeur y=1; break; case 1: A.Anim(); // Animer Texte1 break; } break; }   window.display(); // Afficher l'écran } }

---

Les objets sont mis à jour a chaque tour de boucle (pas d'instanciation de 1000 objets), par contre autre point que j'ai remarqué dans ta boucle c'est que tu n'incrémentes pas ton z du coup tu fais une boucle infinie là.

---

Oooh! Je viens de voir un truc!
Tu veux dire quoi par "le seul paramètre a changer"??? Je dois changer window? Pourquoi?

Ton "problème" initial me parait assez "simple" : relis ton cours sur les structure de controle, switch et while, et la notion de portée d'une variable

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

while (z < 1000) { window.clear(); // Effacer l'écran   switch (z) { case 0: Texte1 A(width, height, fonte, couleur, window); // Constructeur   switch (y) { case 0: A.Set("Bonjour Monsieur"); // Constructeur y=1; break; case 1: A.Anim(); // Animer Texte1 break; } break; }   window.display(); // Afficher l'écran }

---

Je suppose que c'est la méthode Anim qui affiche le texte ?

Prenons 30s (pas besoin de plus et les connaissances basiques suffisent) pour analyser ce que fait le code
tant que z est < 1000, et z ne semble pas évoluer, donc boucle infinie
tour 1
je crée un texte vide
si y vaut 0, je mets "Bonjour Monsieur" dans mon texte, puis je passe y à 1
je dessine mon buffer sur l'écran
là le texte est supprimé

tour 2
je crée un texte vide
je dessine mon texte dans le buffer
je dessine mon buffer sur l'écran
là le texte est supprimé


ha ouais... il manquerait pas un truc ?
ton texte, soit il est vide et affiché, soit il contient quelque chose, mais n'est pas affiché :weird:

Citation:

---

Envoyé par Armulis

Oooh! Je viens de voir un truc!
Tu veux dire quoi par "le seul paramètre a changer"??? Je dois changer window? Pourquoi?

---

Ce que je veux dire par là c'est que dans le code que tu montres la seule variable qui est modifiée dans ton switch c'est la variable window (à première vue) que tu passes en paramètre de ton constructeur, le reste n'a pas l'air d'être modifié (height, width, font et couleur).
Donc la seule variable que tu modifie dans ta boucle c'est window (puisque tu fais window.clear() et window.display()) donc je disais juste de faire un setter dessus si toi dans ton programmes tu fais des modifications dessus dans ton constructeur ou le reste de ton code.
Mais si c'est pas le cas tu peux oublier ma solution et juste sortir cette ligne :

Code:

---

Texte1 A(width, height, fonte, couleur, window);

---

du while, et la mettre juste au-dessus.

Sinon regardes le post de bousk, il est très instructif et va t'aider à comprendre tes erreurs.

Aha... je vois. La solution est simple: il suffit d'instancier mes classes avant le lancement de la boucle Z (while-switch)
Est-ce que, quand même, il existe un moyen d'empêcher les objets d'être détruits à la fin d'un cycle? (donc juste avant de remonter au début de la boucle "while"?

non, précisément pour la raison que tu quitte le bloc ou il est défini.

En effet, tu sors de la paire { }, pour revenir à la condition de boucle. Ce faisant, tout ce qui est déclaré dans le bloc est détruit.

Il suffit de déclarer la variable dans le scope qui englobe la boucle.
Quitte à en créer un artificiellement:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

int main() { //du code   { int var =0; while (...) { //utiliser var; } }   //encore du code return 0; }

---

En tout cas, sortir simplement l'instanciation de la boucle "while" ne suffit pas à faire fonctionner le programme correctement. J'essaie avec une accolade "inutile"...

EDIT: fausse piste: ça ne donne aucun résultat

Citation:

---

Envoyé par Armulis

En tout cas, sortir simplement l'instanciation de la boucle "while" ne suffit pas à faire fonctionner le programme correctement. J'essaie avec une accolade "inutile"...

EDIT: fausse piste: ça ne donne aucun résultat

---

Est-ce aue tu as réglé déjà ton problème d'incrément dans ta boucle car aux dernières nouvelles tu es en boucle infinie.
Après avec tout ce qui a été dit je ne pense pas que nous puissions faire plus, pour voir ce qui se passe regarde l'avancement de ton programme avec un debbuger, peut-être que tu n'entres jamais dans le case y = 0 (enfin selon le code que tu as posté normalement si, tu y entres au moins une fois.)

C'est pas une boucle infinie. Dans le programme complet, on finit par tomber sur z=1000, et on sort =)
Par-contre, la machine circule correctement dans la boucle. Il suffit d'écrire un petit cout pour le vérifier, ça se voit tout de suite

Tu as donc bien un code du style:

Code:

---

1 2 3 4 5 6 7 8 9 10

RenderWindow w; Texte1 A(width, height, fonte, couleur, window); A.Set("D'ya get me, dude?"); //fait une seule fois aussi, alors pourquoi se prendre la tête avec un switch? for(int z=0;z<1000;z++){ w.clear(); A.Anime(); w.display(); //synchro (sinon clear survient immédiatement après display, donc on ne voit rien...) } //nettoyage si nécessaire

---

?

Exact, mais je tiens à utiliser une structure te type "while-switch", parce que le "for" me permet seulement d'incrémenter bêtement z. Je n'incrémente jamais z; je lui donne la valeur qui'il doit prendre.
Par-exemple, dans le menu principal où z vaut 100, je peux aller à "jeu" (z=110), "options" (z=120) ou "quitter" (z=1000). Tu comprends comment ça marche?

Je comprends que tu n'as jamais entendu parler de state-machine, du pattern State ou d'enum.

En gardant ton style:

• Ton z devrait s'appeler status ou currentStep
• les valeurs devraient être définie dans une énum

Le pattern State propose beaucoup mieux:
une classe State avec une méthode virtuelle affiche(...), et une autre next() qui dit quel est l'état suivant.
des classes qui en hérite pour l'implémenter: Menu, Jeu, Credits, Chargement...
Et une variable qui indique quel est l'état courant.

Nous avons des cours dédiés, très bien fait. Je n'ai juste pas le temps de le chercher.
Après tout, nos cours sont riches, tu feras probablement des détours en voyant les autres cours

OK, donc en gros z n'est pas un compteur mais un multi-state, au temps pour moi...
PS: j'espère que dans le vrai code tu travailles avec des constantes nommées, parce que tu n'as pas idée de la vitesse à la quelle on oublie que 235 veut dire menu et 3.14116[...] veut dire quitter... :zoubi:
A vrai dire, séparer la condition de fin dans un indicateur booléen sortir, initialement à false, serait sans doute bien plus facile à lire...

EDIT: grillé. Et la suggestion plus complète émise par leternel te permettrait de clarifier encore plus ton code...

Je me doutais bien qu'il devait exister des outils plus complets que ma seule imagination. Je prends note et je me réserve ça pour le prochain jeu. =)

Oh zut, je viens de trouver la solution! =D
Tout se passait très bien, en fait, mais une bête faute de signe a décalé tout l'affichage en-dehors de l'écran. C'est pour ça que le debuggage ne donnait aucun résultat! MDR
Désolé pour le dérangement, c'est vraiment idiot de ma part de ne pas avoir vérifié ces pistes d'abord! =D