Nic się nie rysuje

Question

Dlaczego przy wywołaniu tej funkcji:

   Engine engine(window);

Nic się nie rusuje, a powienna się namalować mapa kafelkowa;

Funkcja ta wygląda tak:

Engine::Engine(RenderWindow &window)
{
    for(int i = 1; i < Map::COUNT; i++)
    {
        texture[i].loadFromFile("tileset.png", IntRect((i-1) * maps.tileWidth, 0, maps.tileWidth, maps.tileHeight));
    }

    maps.loadFromFile("map.txt");
    for(int i = 0; i < maps.height; i++)
    {
        for(int j = 0; j < maps.width; j++)
        {
            if(maps.level[i][j].type != Map::NONE)
            {
                sprite[i][j].setTexture(texture[maps.level[i][j].type]);
                sprite[i][j].setPosition(j * maps.tileWidth, i * maps.tileHeight);
            }
        }
    }
    runEngine(window);
}
void Engine::runEngine(RenderWindow &window)
{
        Event event;

        while(window.pollEvent(event))
        {

        }
    window.clear();
    for(int i = 0; i < maps.height; i++)
    {
        for(int j = 0; j < maps.width; j++)
        {
            if(maps.level[i][j].type != Map::NONE)
            {
                window.draw(sprite[i][j]);
            }
        }
    }
    window.display();
}

 

Answer 1

Dlaczego wywołujesz metodę runEngine w konstruktorze? Jest to naganny błąd - konstruktor powinien jedynie inicjalizować dany obiekt.

Comments

No, dobrze, ale tam będzie można obsługiwać kafle?

---

Eee, no. To jest ta sama klasa, tylko że bez żadnych błędów (jak już będą, to może w funkcjach innych klas). Można do kafelków dodać kolizje itp.

---

Ok, dzięki, bo mi właśnie chodziło o tą operacje na kafelkach, tak to bym pozostał przy starej mapie 

---

OK, to poczekaj do piątku, ja ci całą klasę (Level.h i Level.cpp + prezentacyjny main.cpp) wyślę na PW. Jak połączysz klasę Level do swojego projektu i nie będzie działać, to nia zakładaj kolejnych pytań, tylko mi odpisz, ale chyba nie powinno być żadnych problemów ;)

---

Dziwnie to zaimplementowałeś, bo masz wywoływane dwa razy metody clear oraz draw i masz dwie pętle zdarzeń - praktycznie powtórzona pętla gry! Powinieneś wszystko umieścić w jednym miejscu. Radzę ci skorzystać ze sprawdzonego schematu:

class Game
{
public:
	Game() :
	window(sf::VideoMode(1024, 768), "Moja Gra")
	{
		window.setFramerateLimit(100);
	}
	void events()
	{
		//...
	}
	void update()
	{
		//...
	}
	void runGame()
	{
		if(window.isOpen())
		{
			events();
            window.clear();
			update();
			window.display();
		}
	}
private:
	sf::RenderWindow window;
	sf::Event event;
}

Obiekty gry zazwyczaj rysują się same, czyli klasa, z której pochodzą, zawiera metodę do tego (a wszystkie dziedziczą z klasy abstrakcyjnej, która zawiera wspólny interfejs), ale nikt ci tak nie każe robić.

Answer 2

A jak wygląda plik map.txt? I przypominam ci, że jeśli nie chce ci się użerać z tymi błędami, to ja ci mogę (w piątek, ale lepiej późno niż wcale ;)) przesłać gotową klasę, lekko zedytowaną przeze mnie, bez żadnych błędów. Ale tylko tę klasę + jakiś main prezentujący tą tilemap'ę. Resztę chyba będziesz mógł sobie sam dołączyć ;)