StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
text
Body
copied!<p>Here, you pass a pointer to an object owned by the list to the constructor of <code>WorldObject</code>:</p> <pre><code>for(std::vector<Point>&pointObject : *inputList) { WorldObject worldObject(&pointObject); tempObjects->push_back(worldObject); } </code></pre> <p>In <code>WorldObject</code> you store the pointer:</p> <pre><code>//Default Constructor WorldObject::WorldObject(std::vector<Point>* pointList) { float fGeneratedY = (rand() % 20 + 2) / 2.0f;*/ cdt = nullptr; for (Point &point : *pointList) point.setY(HEIGHT); this->pointList = pointList; } </code></pre> <p>Which means <code>WorldObject::pointList</code> is only valid so long as the <code>std::list</code> which you constructed your <code>WorldObject</code>s from is still around. (After that, the result is undefined -- it could work, it could crash, it could format your hard drive and leak your identity to Texas).</p> <p>If you insist on working with raw pointers, you as programmer are responsible for checking and keeping track of the lifetime of every single pointer. This is error prone and will cause random crashes that you will find difficult to track down.</p> <p>Stop using raw pointers. Instead, if an object owns a resource, store it in a <code>std::unique_ptr<></code>. If you want the same resource to be shared by multiple objects, use <code>std::shared_ptr</code> and <code>std::weak_ptr</code>, unless the lifetime of all but one of these objects is much, much shorter than the others in a guaranteed way.</p>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload