displaying multiple sprites

[Wissam]

so eventually im going to get drawing a lot of sprites at once for example: the environment that the player will be in: the trees, the grass, flowers, etc etc. tilesets in general.

so I thought of going about that through making an std::vector dedicated for holding only sprites and during the main loop there will be a for loop weeding out the sprites and drawing them. But I can't seem to get it to work!

Here's what I've got so far.

defined the vector early on around where i declare the window

std::vector<sf::Sprite> object_list;

then in my main loop its going to call this void im currently working on under a class called Physics which was also declared earlier "myWindow" being the window name.

gamePhysics.draw_objects(&myWindow,&object_list);

and finally the void function itself

void draw_objects(sf::RenderWindow *gameWindow,std::vector<sf::Sprite> *objects){

    for(int i = 0;objects->size();i++){

        &gameWindow->draw(&objects[i]);
    }

};

which keeps throwing this error at me

include\physics.h|19|error: no matching function for call to 'sf::RenderWindow::draw(std::vector<sf::Sprite>*)'|

highlighting this line

&gameWindow->draw(&objects[i]);

and yes im a little knew to C++ programming but on it here and there trying my best lately.
any suggestions would be greatly appreciated!

[zsbzsb]

It is very imperative that you learn C++ to a decent level before trying to use SFML. Your code right now contains a very basic mistake and if you even properly knew how to use pointers it would stand out at you.

&gameWindow->draw(&objects);

You need to learn what the '&' symbol means, both of the times you use it here it is wrong. You should be dereferencing the pointer here instead of creating a pointer to a pointer.

[Hapax]

I'm no expert with pointers. In fact, I avoid them whenever possible, which is almost all of the time.
However, I did find that I needed to sort of know what they are 

This, I found quite descriptive and it might help you to resolve your issue.

[Wissam]

thanks for the answers and i took a look at the link you sent. I think i'm more familiar with them than I was before.

so here's an update on what I did.

 std::vector<sf::Sprite> object_list;


    sf::Sprite Box;

    Box.setTexture(myTexture);
    object_list.push_back(Box);

        for(int i = 0; i <= object_list.size(); ++i){
        myWindow.draw(object_list[i]);

        }

this does run without any errors but as soon as the application opens up it stays for about 2 seconds then crashes.

[dabbertorres]

object_list.size() is the number of items in the vector. So, by using <=, it tries to access an item outside of the vector's range. You need to use <.

[Xornand]

You could also write it this way instead:

std::for_each(object_list.cbegin(), object_list.cend(),
    [&](const sf::Sprite& sprite){ myWindow.draw(sprite); }
);

[Jesper Juhl]

Personally I'd probably use

for (const auto& sprite : object_list)
    myWindow.draw(sprite);

[Wissam]

unfortunately, none of the suggestions have worked.


all of them are throwing some sort of error

[Xornand]

The error must be caused by some other parts of your code. The iteration methods proposed here are correct.

[Hapax]

some sort of error

What sort of error?
Did just changing the <= to < also throw an error?