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

Tak było w poradniku Pana Szymona Siarkiewicza, on tak zrobił i mu działało

---

Kto by tego nie zrobił, to ten sposób mi się osobiście nie podoba. A gdzie masz pętlę gry?

---

Serio? Naganny błąd? Potwierdzam, że u mnie działa super (po paru przeróbkach, ale działa).

---

@filipekczek7 Głównie chodzi o czytelność kodu. Zauważ, że użytkownik, który używa takiego konstruktora, nie ma pojęcia, że wywołuje on pętle gry. Odpowiednia metoda powinna się znajdować poza konstruktorem.

---

Funkcja główna wygląda tak:

Time time;
    Clock clock;
    Time time2;
    Clock clock2;
    ValuesPlayer vp;

    RenderWindow window(VideoMode(1280, 720), "", Style::Fullscreen);

    Event event;
    Eny eny;
    eny.LoadValuesGroup();

    eny.ValuesGroup();
    Engine engine(window);

    while(window.isOpen())
    {
        eny.CollisionsGroup();
        eny.walk(player, window, time);
        eny.EnemyWalkGroup(time);
        while(window.pollEvent(event))
        {
            if(Keyboard::isKeyPressed(Keyboard::Escape))
            {
                window.close();
            }
        }
        window.clear();
        window.draw(player.player);
        window.draw(enemy.enemy);
        window.display();

        time = clock.getElapsedTime();
        clock.restart();
    }

Jeżeli chcecie, to mogę dać jeszcze klasę z tworzeniem, i wczytywaniem plików

Jak dam w pętli runEngine(), to zaczyna mrugać

---

Aha, czyli jednak chodzi o zasadę. Ale jeśli nikt poza mną nie będzie oglądał tego kodu, to chyba nic się nie stało :) A jak już będzie oglądał, to można nad tą linijką dać komentarz, pokazujący o co chodzi. Ale rzeczywiście, nie lepiej po prostu stworzyć obiekt Engine, a linijkę niżej wywołać funkcje engine.runEngine()? Teraz całego kodu nie pamiętam, ale obiecuję go poprawić i więcej nie grzeszyć ;D

---

@Avernis - Eny? Hm, strasznie dziwna nazwa dla klasy. Ale dziwniejszą sprawą są nazwy metod zaczynające się z dużej litery. Okropnie to wygląda! Weź to popraw ;)

---

Nadal nic, nie ważne czy jest w, czy poza konstruktorem

---

No tak, tak, w tym przypadku to akurat nie ma różnicy gdzie to jest, chodzi o zasadę i przejrzystość kodu. Ale jeśli chodzi o błąd, to patrz na moją odpowiedź - wolę ci dać gotową klasę super działającą, niż doszukiwać się błędu (w sumie to na blogu Szymona jest trochę tych błędów w kodzie, a przynajmniej u mnie były (no wiesz, inne środowisko i pewnie jeszcze jakieś inne czynniki)).

---

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ć ;)