Sprite Container/ Combiner?

[SethGandy]

So I'm new to the wonderful world of SFML.
I'm making an isometric tile engine as my first project.
Problem I'm having is I want to combine all the tiles I draw into one object.
That way the map moves all as one piece once rendered.
How would I go about making this said "container"?

My current map rendering system looks like this:

Code: [Select]

// image already loaded in to textureVector[x], mapVector[x][y] is the map in zeros and ones
Sprite tileSprite;
for(int i = 0; i < MapVector.size(); i++)
{
for(int j = 0; j < MapVector[i].size(); j++)
{
tileSprite.setTexture(textureVector[MapVector[i][j]]);
tileSprite.setPosition((j * BLOCKSIZE / 2)-(i * BLOCKSIZE /2)+40, (j * BLOCKSIZE / 4)+(i * BLOCKSIZE /4)+40);
tileSprite.setRotation(45);
Window.draw(tileSprite);
}
}


[Nexus]

class Tilemap : public sf::Drawable, public sf::Transformable
{
private:
    virtual void draw(sf::RenderTarget& target, sf::RenderStates states)
    {
        states.transform *= getTransform(); // sf::Transformable::getTransform()
        target.draw(member, states);
    }

    sf::Sprite member; // can be a container of tiles or whatever
}

The class Map inherits all the public methods like setPosition() etc. If you call them from the outside, the transform changes, which influences sf::RenderStates.

Take a look at the corresponding parts of the SFML 2 documentation to understand the mechanics in detail.

[SethGandy]

Apologies, I'm confused as to how to implement this?
I'm pretty rusty on inheritance classes and virtuals.
I can't create a new class object such as Tilemap newOb; , so I'm kinda at a loss as how to use this lol 

[eXpl0it3r]

I'm pretty rusty on inheritance classes and virtuals.

Then you should read up on it, it's one of the most essential subjects in C++.

I can't create a new class object such as Tilemap newOb;

Why not?
Where are you exactly stuck? What don't you understand?

[SethGandy]

When I try to create the new object it errors out 'Tilemap': cannot instantiate abstract class.
pure virtual function "sF::Drawable::draw" has no overrider.

Also I'm actually reading up on it, from C++ PROGRAMMING: PROGRAM DESIGN INCLUDING DATA STRUCTURES, 5TH EDITION. It's for class.

This whole isometric thing is just my own infatuation getting the best of me.

[eXpl0it3r]

'Tilemap': cannot instantiate abstract class.
pure virtual function "sF::Drawable::draw" has no overrider.

This means you haven't defined the draw function for the class.
Maybe you forgot to add Tilemap:: infront of draw(), when defining it in the source file?

[FRex]

Nexus forgot const after his draw (which is/should be there because Evil Laurent™ makes things const correct).

[SethGandy]

Attached is my project in it's entirety.
So far, I've only written to the main.cpp, I haven't bothered splitting up my functions and creating headers and implementation files yet, not until I have the base of the program done... bad habit of mine.

But here's the code thus far:

#include<SFML\Graphics.hpp>
#include<iostream>
#include<fstream>
#include<string>
#include<vector>
#include<sstream>
using namespace std;
using namespace sf;

#define ScreenWidth 800
#define ScreenHeight 600

#define BLOCKSIZE 40
int currentMapX;
int currentMapY;

vector <vector <int> > MapVector;
vector <Texture> textureVector;

class Tilemap : public Drawable, public Transformable
{
private:
        virtual void draw(RenderTarget& target, RenderStates states)
        {
                states.transform *= getTransform(); // sf::Transformable::getTransform()
                target.draw(member, states);
        }
        Sprite member; // can be a container of tiles or whatever
};

void LoadMap(const char *filename)
{
        ifstream openfile(filename);
        vector <int> tempVector;

        string line;

        if(openfile.is_open())
        {
                while(!openfile.eof())
                {
                        getline(openfile, line);

                        string tempLine = line.erase(line.find_last_not_of(" ") + 1);
                        int space = tempLine.find_first_of(" ");
                        if(space == 0)
                                tempLine = tempLine.erase(0, space + 1);
                        stringstream str(tempLine);

                        while(!str.eof())
                        {
                                string value;
                                getline(str, value, ' ');
                                if(value.length() > 0)
                                        tempVector.push_back(atoi(value.c_str()));
                        }
                        MapVector.push_back(tempVector);
                        tempVector.clear();
                }
        }
        cout<<"[------- MAP -------]\n";

        //------Ouput Map---------
        int mapY=0;
        int mapXtemp=0;
        int mapX=0;
        for (int r=0; r < MapVector.size(); r++)
        {
                mapY++;
                for (int c=0; c < MapVector[r].size(); c++)
                {
                        cout << MapVector[r][c] << " ";
                        mapXtemp++;
                }
                if(mapXtemp > mapX)
                {
                        mapX=mapXtemp;
                }
                mapXtemp=0;
                cout << endl;
        }
        currentMapX = mapX;
        currentMapY = mapY;
        cout<<endl<<mapY<<"x"<<mapX<<endl;
}
void LoadTextures(const char *filename)
{
        cout << "#####################\n";
        string line;
        Texture inText;
        ifstream openfile(filename);

        if(openfile.is_open())
        {
                while(!openfile.eof())
                {
                        getline(openfile, line);
                        string tempLine = line.erase(line.find_last_not_of(" ") +1);
                        cout<<"Loaded : ["<<tempLine<<"]"<<endl;

                        if(tempLine.length() > 0)
                        {
                                if(!inText.loadFromFile(tempLine))
                                {
                                        cout << "Could not load" << line << endl;
                                }
                                textureVector.push_back(inText);
                        }
                }
        }
        cout << "#####################\n";
}
void DrawMap(RenderWindow &Window)
{
        RectangleShape mapCont;
        mapCont.setFillColor(Color::Green);

        mapCont.setSize(Vector2f((currentMapX)*BLOCKSIZE,(currentMapY)*BLOCKSIZE));
        mapCont.setPosition(20,40);
        mapCont.setRotation(45);
        mapCont.setPosition(mapCont.getPosition().x, mapCont.getPosition().y * 2);

        View v = Window.getDefaultView();
        v.setSize(v.getSize().x, v.getSize().y * 2);
        v.setCenter(v.getSize() *.5f);
        Window.setView(v);
        Window.draw(mapCont);
        Window.setView(Window.getDefaultView());
        //Window.setView(v);

        Sprite tileSprite;
        for(int i = 0; i < MapVector.size(); i++)
        {
                for(int j = 0; j < MapVector[i].size(); j++)
                {
                        tileSprite.setTexture(textureVector[MapVector[i][j]]);
                        tileSprite.setPosition((j * BLOCKSIZE)+40,(i * BLOCKSIZE)+40);
                        //tileSprite.setRotation(45);
                        Window.draw(tileSprite);
                }
        }
        Window.setView(Window.getDefaultView());
}

int main()
{
        RenderWindow Window(VideoMode(ScreenWidth,ScreenHeight,32), "isometric engine", Style::Close);
        LoadTextures("Textures.txt");
        LoadMap("Map1.txt");

        Font Font;
        if (!Font.loadFromFile("font.ttf"))
                return 1;

        Text Text("Seth Gandy | iso v0.01", Font);
        //Text.setStyle(Text::Style::Bold);
        Text.setCharacterSize(25U);
        Text.setPosition(510,565);

        while(Window.isOpen())
        {
                Event Event;
                while(Window.pollEvent(Event))
                {
                        if(Event.type == Event::Closed || Event.key.code == Keyboard::Escape)
                                Window.close();
                }

                Window.clear(Color(0, 125, 255));
                DrawMap(Window);
                Window.draw(Text);
                Window.display();
        }
        return 0;
}
 

[attachment deleted by admin]

[FRex]

virtual void draw(RenderTarget& target, RenderStates states)

Add const at the end of this line.
Also, globals, macros(#define xxx yyy) and using directives(using namespace xxx) are considered bad practice in c++, use / for paths, it works perfecly normal on windows and visual supports that in its advanced options for intelisense.

[Nexus]

So far, I've only written to the main.cpp, I haven't bothered splitting up my functions and creating headers and implementation files yet, not until I have the base of the program done... bad habit of mine.

There are a lot of bad habits in your code

Apart from the ones FRex mentioned, don't use using namespace everywhere. It defeats the purpose of namespaces, and leads to constructs like Font Font;. Don't put sf::Texture into std::vector (unless you pre-allocate).

By the way, you were not supposed to copy my code 1:1, it was meant as an example. I also wrote "Take a look at the corresponding parts of the SFML 2 documentation to understand the mechanics in detail." And if you have a problem with understanding of classes, you should really read about them first, because C++ basics are mandatory to use SFML.

[SethGandy]

Hitting the books, putting the project off for a bit.
Been reading for the past 5 hours, to help wash away the feeling of shame and incompetence.