[SFML] Tilemap aufbau

Werbeanzeige

moritz31

Treue Seele

Beiträge: 259

Wohnort: Hessen

Beruf: Student

02.08.2016, 17:39

also die Klasse sieht so aus:

C-/C++-Quelltext

class TileMap : public sf::Drawable
{
private:
    mutable sf::Sprite tile;
    sf::Texture tileTexture;
    virtual void draw(sf::RenderTarget& target, sf::RenderStates states) const;
    int field[10][10];

    int width;
    int height;

public:
    TileMap();
    ~TileMap();

    sf::Sprite* getTile();
    void loadTileTexture(std::string path);

    
};

Die Funktion draw um die es ja eigentlich geht so:

C-/C++-Quelltext

void TileMap::draw(sf::RenderTarget& target, sf::RenderStates states) const {

    tile.setTexture(this->tileTexture);
    tile.setTextureRect(sf::IntRect(0, 0, 95, 95));

    for (int x = 0; x < width; x++) {
        for (int y = 0; y < height; y++) {
            switch (this->field[x][y])
            {
            case 0:
                tile.setPosition(sf::Vector2f(x * 95, y * 95));
                target.draw(this->tile);
                break;
            default:
                break;
            }
        }
    }

}

Ist jetzt eigentlich nur ne einfache Funktion die das Tile überall hinsetzt und zeichnet.

Zum Seitenanfang

Koschi

Alter Hase

Beiträge: 657

02.08.2016, 18:20

Das const am Ende de Funktion draw sagt aus das Innerhalb der Methode nichts geändert werden darf (Klassenmember).
Du änderst ab fleißig Werte (der Klassenmember) innerhalb der Methode.

Dann ist klar das der Compiler mekert. Dafür ist aber auch die Update-Methode gedacht, da werden alle Berechnungen durchgeführt.

Edit: Ich würde Tile als Extraklasse machen, die dann auch ein sf:

prite haben. Auch würde ich von einem festen Array weggehen hin zu einem Vector.

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

moritz31

Treue Seele

Beiträge: 259

Wohnort: Hessen

Beruf: Student

03.08.2016, 21:45

Natürlich ist das Array nur ein Platzhalter zum schnellen testen, wird später ausgetauscht.
Warum genau würdest du Tile nochmal als Extraklasse machen ? Hätte das irgendwelche Vorteile?

Und klar verändere ich in Draw etwas. Der Vorteil der Tilemap ist ja das ich nur ein Tilehabe, dass ich an verschiedenen stellen zeichne, von daher kann man das doch auch gar nicht in die update packen oder ?
Der Vorgang ist ja im Prinzip Update -> Draw -> Update -> Draw -> Update -> Draw (wobei jedes Update hier das neue setzten des Tiles darstellt).

Gruß
;oritz

Zum Seitenanfang

Koschi

Alter Hase

Beiträge: 657

04.08.2016, 01:03

Zitat von »moritz31«

Warum genau würdest du Tile nochmal als Extraklasse machen ?

In der Regel möchtest du ein Tile nicht nur Zeichnen, sondern auch andere Information hinterlegen z.B. Tile ist Begehbar oder nicht und noch andere Sachen.

Zitat von »moritz31«

Der Vorteil der Tilemap ist ja das ich nur ein Tilehabe,

In wie weit ist das ein Vorteil? Stell dir mal vor wie deine Draw-Methode ausschaut, wenn wir hier über 100 oder mehr verscheiden aussehende Tiles sprechen.

Zitat von »moritz31«

Der Vorgang ist ja im Prinzip Update -> Draw -> Update -> Draw -> Update -> Draw (wobei jedes Update hier das neue setzten des Tiles darstellt).

Du hattest nach einer Sauberen Lösung gefragt, das ist eine. Zu dem muß die Position nicht immer neu gesetzt werden wenn es in einer Extraklasse ausgelagert ist.

Wer aufhört besser werden zu wollen hört auf gut zu sein!

aktuelles Projekt:Rickety Racquet

Zum Seitenanfang

moritz31

Treue Seele

Beiträge: 259

Wohnort: Hessen

Beruf: Student

04.08.2016, 08:55

also könnte ich quasi so etwas wie myTile.getTileAtPos(x,y) machen, dann in der Klasse die Position des Sprites setzen und das dann an meine Draw Methode returnen ?

Würde dann quasi in etwa so Aussehen,

	Quellcode
1 2	newTile = myTile.getTileAtPos(x,y); target.draw(newTile);

Zum Seitenanfang

Koschi

Alter Hase

Beiträge: 657

04.08.2016, 12:25

Wie wäre es mit sowas?

C-/C++-Quelltext

class Tile : public sf::Drawable
{
privat:
    sf::Sprite m_sprite;
    sf::Vector2i m_mapPosition;
...
public:
    Tile(const sf::Vector2i& mapPosition,
         const sf::Texture& texture,
         const sf::IntRect& textureRect);
    void update();
    void draw(sf::RenderTarget& target, sf::RenderStates states) const;
};

class TileMap
{
private:
    std::vector<Tile> m_tiles;
public:
    TileMap();
    void update();
    void draw(sf::RenderTarget& target, sf::RenderStates states) const
    {
         for(auto it = begin(m_tiles); it != end(m_tiles); ++it)
             it->draw(target);
    }

};