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

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.

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