Vector for() iterator returning a null (or zero) object for each iteration (solved)

#cpp #help

Update, I think I have permanent brain damage.

I have a vector of tiles for a 2d game, each tile has an x and y and a get function. If I iterate over this vector using for (Tile t : tiles) and call t.getX() it returns 0 for every tile. Why is this? Should I be using a pointer instead? More verbose code below.

for (Tile t : tiles)
{
    if (sf::IntRect(t.getX(), t.getY(), t.getX() + 32, t.getY() + 32).contains(sf::Vector2i(x, y)))
    {
        return t;
    }
    return tiles.front();
}

I later found out that the return tiles.front(); is returning always if the first tile scanned isn't solid. Therefore moving it one line down will work. I found this because visual studio said not all control paths return. Of course.

Top comments (1)

Lesley Lai • Jan 14 '20 • Edited

You can try to use the STL algorithms for this kind of task and they avoid common mistakes when using loops. They should have almost the same code generated as your loop in your scenario. For example:

const auto result = std::find_if(std::begin(tiles), std::end(tiles), [](const Tile& t){
  sf::IntRect(t.getX(), t.getY(), t.getX() + 32, t.getY() + 32).contains(sf::Vector2i(x, y));
});
return (result != std::end(tiles)) ? *result : tiles.front();

You can watch the Sean Parent's C++ Seasoning video (a must watch to me) for more discussion on this topic.

Also, C++20 ranges would finally remove the stupid begin and end boilerplate. Though no standard library implementation ship that yet :-(.

DEV Community

Vector for() iterator returning a null (or zero) object for each iteration (solved)

Top comments (1)

Read next

𝗛𝗼𝘄 𝗕𝗮𝗰𝗸𝗲𝗻𝗱 𝗔𝗰𝗰𝗲𝗽𝘁 𝗖𝗼𝗻𝗻𝗲𝗰𝘁𝗶𝗼𝗻𝘀.

How to Expose C++ Qt WebView and Window Functions to Python?

𝗖𝗣𝗨 𝗧𝗵𝗿𝗼𝘁𝘁𝗹𝗶𝗻𝗴 𝗶𝗻 𝗞𝘂𝗯𝗲𝗿𝗻𝗲𝘁𝗲𝘀 ☸️

LeetCode in Swift - 1957. Delete Characters to Make Fancy String