Bonjour,

Je suis débutant en programmation (~environ 2mois de c++)

Je lis actuellement un livre sur le développement de jeu video avec la SFML.

Je suis confronté a un problème.

ce qui est étrange c'est que le code fonctionne avec CodeBlocks mais pas avec Visual Studio.

l'erreur que me retourne Visual Studio, c'est que Game::GetWindow doit retourner une valeur. Pourtant sur CodeBlocks je l'ai defini de la même manière et ça marche

Main.cpp

#include <SFML/Graphics.hpp>
#include "Game.h"

using namespace sf;

int main()
{
        Game game;
        
        while (!game.GetWindow()->IsDone()) 
        {
            
        }
        return 0;
}

#include "Window.h"

class Game 
{
public:
	Window* GetWindow();

private:
	Window m_window;

);

Game.cpp

#include "Game.h"

Window* Game::GetWindow() {}

Window.h

#include <string>
#include <SFML/Graphics.hpp>


class Window 
{
public:
	Window();
	Window(const std::string& l_title, const sf::Vector2u& l_size);

	void Update();

	bool IsDone();
	
private:
	void Setup(const std::string& l_title, const sf::Vector2u& l_size);

	bool m_isDone;
};

Window.cpp

#include "Window.h"

Window::Window() 
{
	Setup("Window", sf::Vector2u(640, 480));
}

Window::Window(const std::string& l_title, const sf::Vector2u& l_size)
{
	Setup(l_title, l_size);
}

void Window::Update()
{
	sf::Event event;
	while (m_window.pollEvent(event))
	{
		if (event.type == sf::Event::Closed)
		{
			m_isDone = true;
		}
		else if (event.type == sf::Event::KeyPressed && event.key.code == sf::Keyboard::F5)
		{
			ToggleFullscreen();
		}
	}
}

bool Window::IsDone()
{
	return m_isDone;
}

void Window::Setup(const std::string& l_title, const sf::Vector2u& l_size)
{
	m_windowTitle = l_title;
	m_windowSize = l_size;
	m_isFullscreen = false;
	m_isDone = false;
	Create();
}

J'ai volontairement réduit le code pour pas que ça ressemble à un pavé.

En faite, je sais pas quoi retourner. Ce que je comprend c'est que c'est un pointeur de type Window qui a pour nom GetWindow. Il devrait accéder a l'attributs de Window m_isDone pour savoir si la réponse est true ou false, afin de continuer ou fermer la boucle.

Quelqu'un peut-il m'aider ?

tout conseils sont les bienvenus =)

Merci et salutation !

Salut,

Ce qui se passe sans doute, c'est que tu n'as pas activé les avertissements, car, sur ce coup, c'est visual studio qui a raison:  A partir du moment où tu indique qu'une fonction est censée renvoyer une valeur, ben, il n'y a rien à faire, il faut que tous les chemins d'exécution mènent effectivement ... au renvoi d'une valeur du type indiqué (un pointeur sur un objet de type Window, en l'occurrence)

Maintenant, il est particulièrement étonnant que ton compilateur ne te donne pas au minimum un avertissement sur ce point, car c'est typiquement le genre d'avertissement que Gcc a activé depuis longtemps.  Peut-être utilises tu -- tout simplement -- une version trop ancienne de Gcc (ou de MinGW, vu que tu as l'air d'être sous windows)?

Ou peut-être n'as tu simplement pas fait attention aux différents avertissement sous code::blocks?

Par contre, il y a de sérieux problèmes de "const correctness" dans ton code, dans le sens où toutes les fonctions qui ne modifient clairement pas l'état de tes classes -- les accesseur (getters) en grosse priorité -- devraient s'engager à effectivement ne pas modifier l'état de la classe, et qu'elles devraient dés lors être déclarées constantes.

MinGW émet un warning : warning: no return statement in function returning non-void [-Wreturn-type]| 

et même sur une des plus vielle version de minGW (2000): warning: control reaches end of non-void function `Game::GetWindow()'|

Au pif:

Window* Game::GetWindow() { return &m_window;}

Je vous remercie d'avoir pris le temps de me répondre

J'ai été regarder sur CodeBlocks et il y belle est bien un avertissement (celui de rouloude). En faite je suis passé outre l'avertissement, et le code fonctionne même sans préciser le return. (la fenetre s'ouvre et mes images s'animent)

J'ai pris note des "const correctness" et je vais tâcher d'être plus rigoureux de ce côté là.

Window* Game::GetWindow() { return &m_window;} fonctione ;P

Vous étes trop fort !!!