0

我正在写一个八叉树算法。在函数内部我遍历八叉树。我得到节点指针和球体作为输入。我检查节点是否应该保持球体然后我想将它添加到节点s object list and remove it from its parent的列表中。以下是代码

functionBody()
{
 .....

  if (!node->objectList.empty())
      node->objectList.erase(std::remove_if(node->objectList.begin(), node->objectList.end()-1 , [&t](auto& temp) { return temp == t; }));

...
}

typedef struct Sphere
{
    Sphere() = default;
    Sphere(const Vector3 centre_, const float radius_, const Material& material_) : centre(centre_), radius(radius_), material(material_)
    {
        assert(radius != 0);
        invRadius = 1.0 / radius;

        Vector3 radiusOffset = Vector3(radius);
        aabb.min = Vector3(centre - radiusOffset);
        aabb.max = Vector3(centre + radiusOffset);
    }

    bool operator==(const Sphere& rhs)
    {
        return (centre == rhs.centre) && (radius == rhs.radius);
    }

    Vector3 centre;
    float radius;
    float invRadius;
    Material material;
    AABB aabb;

}Sphere;

正如你所看到的，我已经operator==定义了 Sphere。

我看到remove_if即使谓词返回错误，它也会删除元素。

例如，第一次迭代它找到一个球体t并使用remove_if. 这t最后出现在向量中。现在考虑父母在其向量中仍然有3个球体，但是，当我现在去其他孩子时，我们仍然尝试t在父母中搜索并且remove_if仍然删除最后一个条目。我不明白为什么？

4

2 回答 2

1

std::remove_if返回end没有找到要删除的内容时提供的迭代器。您已经给出node->objectList.end()-1了结束迭代器，它是 . 中最后一个元素的迭代器node->objectList。这是您erase在找不到时传递给t的内容，因此最后一个元素被删除。

要解决此问题，请使用erase需要一系列元素的重载：

if (!node->objectList.empty())
{
    auto end_iter = node->objectList.end();
    auto to_remove = std::remove_if(
        node->objectList.begin(), end_iter, 
        [&t](auto& temp) { return temp == t; });

    node->objectList.erase(to_remove, end_iter);
}

现在 if tis not founderase根本不会做任何事情。在这种情况下，remove_if返回end_iter并erase尝试擦除由end_iter和自身定义的空范围内的元素。

我不确定你为什么使用node->objectList.end() - 1. 我假设这是一个错误或解决方法，否则您可能会在以前的代码中遇到这种情况。

于 2020-03-19T18:15:26.237 回答
0

当您调用仅接受一个迭代器（而不是迭代器范围）的方法擦除时，该算法std::remove_if使用指定的容器元素范围的第二个迭代器调用，例如

node->objectList.end()-1

那么即使在容器中找不到元素，算法也会返回指向容器中有效对象remove_if的迭代器。node->objectList.end()-1该对象将从容器中删除。

这是一个重现问题的演示程序。

#include <iostream>
#include <vector>
#include <iterator>
#include <algorithm>

int main() 
{
    std::vector<int> v = { 1, 3, 5, 7, 9 };

    for ( const auto &item : v ) std::cout << item << ' ';
    std::cout << '\n';

    while ( v.size() > 1 )
    {
        v.erase( std::remove_if( std::begin( v ), std::prev( std::end( v ) ),
                                 []( const auto &item )
                                 { 
                                    return item % 2 == 0; 

                                 } ) );
    }

    for ( const auto &item : v ) std::cout << item << ' ';
    std::cout << '\n';

    return 0;
}

它的输出是

1 3 5 7 9 
1

也就是说，向量中的任何元素都不是偶数。尽管如此，除了一个之外的所有元素都从向量中删除。

您似乎错误地指定了范围。它应该像一对一样指定

node->objectList.begin(), node->objectList.end()

或者在擦除元素之前，您应该检查返回的迭代器是否等于node->objectList.end() - 1（前提是您确实想要使用问题中显示的范围）。在这种情况下，不应调用方法擦除。或者您应该指定一个已擦除的迭代器范围，例如

if (!node->objectList.empty())
      node->objectList.erase(std::remove_if(node->objectList.begin(), node->objectList.end()-1 , [&t](auto& temp) { return temp == t; }).
                             node->objectList.end()-1);

再次假设您确实想使用范围的第二个迭代器，node->objectList.end()-1而不是node->objectList.end().

于 2020-03-19T18:11:07.707 回答