My window doesnt have a color

[That Martin Guy]

I know i have posted VERY similar code before, but i accidently deleted the project i was working in, so i had to redo the tutorial. Also, the error is completely different from what i can tell.

Bascially, when i try to make the screen red, its white instead, like if i didnt give it a color.

Game.cpp

#include "stdafx.h"
#include "Game.h"

Game::Game()
{
        sf::RenderWindow _mainWindow(sf::VideoMode(1024, 768, 32), "Pang!");
        _gameState = uninitialized;
        start();
}
void Game::start(void)
{
        if(_gameState != uninitialized)
                return;

        _gameState = Game::playing;

        while(!isExiting())
        {
                gameLoop();
        }

        _mainWindow.close();
}
bool Game::isExiting()
{
        if(_gameState == exiting)
        {
                return true;
        }
        else
        {
                return false;
        }
}
void Game::gameLoop()
{
        sf::Event currentEvent;
        while(_mainWindow.pollEvent(currentEvent))
        {
                switch(_gameState)
                {
                case playing:
                        _mainWindow.clear(sf::Color::Red);
                        _mainWindow.display();

                        if(currentEvent.type == sf::Event::Closed)
                        {
                                _gameState = exiting;
                        }
                }
        }
}

Game.h

#include "stdafx.h"

class Game
{
public:
        Game();
        void start();
private:
        bool isExiting();
        void gameLoop();

        enum GameState{uninitialized, showingSplash, paused, showingMenu, playing, exiting};

        GameState _gameState;
        sf::RenderWindow _mainWindow;
};

When i debug, it stops at

while(_mainWindow.pollEvent(currentEvent))

[dabbertorres]

Looks like you never changed the value of _gameState to playing.

[Arcade]

I'm not sure if this is directly related to your problem, but your gameloop seems a bit weird to me. Right now you are only clearing and displaying the window if an event happens. Perhaps you want something like this instead?

Code: [Select]

while(_mainWindow.pollEvent(currentEvent))
{
   if(currentEvent.type == sf::Event::Closed)
   {
      _gameState = exiting;
   }
}

switch(_gameState)
{
   case playing:
      _mainWindow.clear(sf::Color::Red);
      _mainWindow.display();
}


[That Martin Guy]

I'm not sure if this is directly related to your problem, but your gameloop seems a bit weird to me. Right now you are only clearing and displaying the window if an event happens. Perhaps you want something like this instead?

Code: [Select]

while(_mainWindow.pollEvent(currentEvent))
{
   if(currentEvent.type == sf::Event::Closed)
   {
      _gameState = exiting;
   }
}

switch(_gameState)
{
   case playing:
      _mainWindow.clear(sf::Color::Red);
      _mainWindow.display();
}


Its supposed to be like that. Later, when i get further in the tutorial, i will do different stuff.

[eXpl0it3r]

It's not only weird or something one can change "later", but it's simply wrong. Rendering doesn't belong into the event loop, so don't do it.

[That Martin Guy]

It's not only weird or something one can change "later", but it's simply wrong. Rendering doesn't belong into the event loop, so don't do it.

Ok, thanks for telling me. However, it still doesnt change the color. Its as white as it always was.

[eXpl0it3r]

Looks like you never changed the value of _gameState to playing.

[That Martin Guy]

Looks like you never changed the value of _gameState to playing.

1. I never saw that post.
2.

void Game::start(void)
{
    if(_gameState != uninitialized)
        return;

    _gameState = Game::playing; //I did actually, and changing it to _gameState = playing didnt work either

    while(!isExiting())
    {
        gameLoop();
    }

    _mainWindow.close();
}

I have the debugger to back me up. When it comes to the switch statement, it tells me that gamestate indeed is playing.

[eXpl0it3r]

Since you have the debugger running, does it reach the clear statement?

Is your graphics driver installed/uptodate?

[That Martin Guy]

Since you have the debugger running, does it reach the clear statement?

Is your graphics driver installed/uptodate?

Yes, it reaches the clear statement AND the display.

Since ive had it show me a red screen before, i would assume that they are uptodate.

[eXpl0it3r]

Don't assume, check...

Also what did you change if it worked before?

[That Martin Guy]

Don't assume, check...

Also what did you change if it worked before?

Drives updated, still as white as ever. The only thing i recall changing is using the constructor to create the window instead of

_mainWindow.create(stuff);

[eXpl0it3r]

Provide a minimal and compilable example of your updated code.

[That Martin Guy]

You meant from the code that i posted above? I thought you meant the one i tried a week or so ago that i posted somewhere around this forum.
Gave comments to the lines that changed.

#include "stdafx.h"
#include "Game.h"

Game::Game()
{
        sf::RenderWindow _mainWindow(sf::VideoMode(1024, 768, 32), "Pang!");
        _gameState = uninitialized;
        start();
}
void Game::start(void)
{
        if(_gameState != uninitialized)
        {
                return;
        }

        _gameState = playing;

        while(!isExiting())
        {
                gameLoop();
        }

        _mainWindow.close();
}
bool Game::isExiting()
{
        if(_gameState == exiting)
        {
                return true;
        }
        else
        {
                return false;
        }
}
void Game::gameLoop() //Where the actual change happened
{
        sf::Event currentEvent;
        while(_mainWindow.pollEvent(currentEvent)) //No longer contains the rendering
        {
                if(currentEvent.type == sf::Event::Closed)
                        {
                                 _gameState = exiting;
                        }
        }
        switch(_gameState) //Note that i moved this from the while loop
        {
                case playing:
                _mainWindow.clear(sf::Color::Red);
                _mainWindow.display();
        }
}

[eXpl0it3r]

It's not compilable by others on this forum.
Alternatively you could just run an example from SFML. If that works, your code must be bugged.