Sign in to follow this  

Best way to get width of object before creation

This topic is 1838 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

I'm currently working on a Fruit Ninja clone using SDL and I want to restrict my fruit to being created at a random x co-ordinate, but so that they fully appear within the bounds of the window (800x600). My current code (which is a work in progress) looks something like this:

 

 

 

fruit.push_back(std::unique_ptr<Fruit>(new Fruit(rand() % (mediaCache.scrWidth() - 50), mediaCache.scrHeight(), mediaCache.getImage("fruit.png"))));

 

 

Which creates a new Fruit at a random x position at the bottom of the window (so it can bounce up into it) and gives it an image.

 

50 is the width of the Fruit, so to ensure the whole Fruit starts inside the window I take it away from the screen width. I can't access the width through a get() method before the Fruit itself is actually created, so is the best way to do this to make the width a public const static member inside Fruit and use Fruit::width instead of 50 or is there a better way?

 

Thanks

Share this post


Link to post
Share on other sites
Since the fruit object will know it's own size, why not have the fruit constructor call rand()?

Also, since it looks like you're using a C++11 compiler, you can simplify that expression by using std::vector::emplace_back() rather than push_back(). So the call would look something like this instead:
fruit.emplace_back(new Fruit(mediaCache.scrWidth(), mediaCache.scrHeight(), mediaCache.getImage("fruit.png")));

Share this post


Link to post
Share on other sites

Or, since you may not want all Fruit to be randomly placed, you could create a second Constructor that only takes the Image, and then assign the Location via a SetLocation() method.

[code]Fruit *fruit = new Fruit(mediaCache.getImage("fruit.png")); fruit.SetLocation(rand() % (mediaCache.scrWidth() - fruit.GetWidth()), mediaCache.scrHeight()); // I'd suggest renaming the vector fruits (with an s) to denote it's not a single fruit fruits.push_back(fruit);[/code]

Share this post


Link to post
Share on other sites
Cheers guys, seems so obvious to do it that way SiCrane, thanks for that. I'm using vs2012 so I'll have to see if emplace_back is one of the c++11 features that made it in.

Share this post


Link to post
Share on other sites

This topic is 1838 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this