2

我有一段代码在运行时因各种奇怪的内存损坏而失败。我已将其缩小到这部分代码：

List<CollisionBlock> WorldClient::getCollisionBlocks(RectF const& boundBox, bool doSort, Vec2F sortCenter) const {
  auto res = m_collisionGenerator.getPolys(boundBox);

  if (doSort) {
    sort(res, [=](CollisionBlock const& block1, CollisionBlock const& block2) {
    return magSquared(sortCenter - block1.poly.center()) < magSquared(sortCenter - block2.poly.center());
      });
  }

  return res;
}

如果我const&从 lambda 中删除，代码可以正常工作。我不知道为什么。我想知道我是否遇到了编译器错误，或者是否有一些明显的东西我忽略了。

这是 CollisionBlock 的定义：

struct CollisionBlock {
  PolyF poly;
  // Will never be None
  CollisionKind kind;
  // Normalzied vector encoding the slope of the block we collided with.
  // Always faces right, y component can be positive or negative.
  Vec2F slope;
};

我可以在 Linux 32 位（g++ 版本 4.7.0 和 4.6.3）、MacOSX（不确定字长和 g++ 版本）、Windows 7 64 位（g++ 版本 4.6.3）、Windows 7 32 位（g++版本 4.6.2 和 4.6.3），但不是 Linux 64 位（g++ 版本 4.6.1）。

我使用的是 C++11，而不是 boost。

Poly::center()

Coord center() const {
  return sum(m_vertexes) / (DataType)m_vertexes.size();
}

sum

template<typename Container>
typename Container::value_type sum(Container const& cont) {
  return reduce(cont, std::plus<typename Container::value_type>());
}

reduce

// Somewhat nicer form of std::accumulate
template<typename Container, typename Function>
typename Container::value_type reduce(Container const& l, Function f) {
  typename Container::const_iterator i = l.begin();
  typename Container::value_type res{};

  if (i == l.end())
    return res;

  res = *i++;
  while (i != l.end())
    res = f(res, *i++);
  return res;
}

sort 

template<typename Container, typename Compare>
void sort(Container& c, Compare comp) {
  std::sort(c.begin(), c.end(), comp);
}

这个问题有很多事情要做。抱歉，我会尝试制定一个较小的测试用例。

更新：

替换对suminPoly::center的调用std::accumulate没有帮助。

4

4 回答 4

1

我相当确定错误不在您发布的代码范围内。您可以在下面使用我的虚拟代码并慢慢将其转变为您的真实代码，直到它开始出现问题。如果你找到了请告诉我们，我很好奇。

仅供参考，我必须在几个地方编写代码才能获得接近你正在做的事情。所有这一切实际上就是锻炼您发布的一小段代码。

#include <list>
#include <deque>
#include <vector>
#include <algorithm>

typedef double mocish;

typedef int CollisionKind; //is actually enum class
typedef mocish RectF;

class Vec2F {
public:
  Vec2F() {
    vertexes.push_back(0);
    vertexes.push_back(0);
  }
  Vec2F(float a, float b) {
    vertexes.push_back(a);
    vertexes.push_back(b);
  }

  float operator[](unsigned index) const {
    return vertexes[index];
  }

  float operator[](unsigned index) {
    return vertexes[index];
  }

  Vec2F operator+(Vec2F const& other) const {
    return Vec2F(vertexes[0]+other[0], vertexes[1]+other[1]);
  }

  Vec2F operator-(Vec2F const& other) const {
    return Vec2F(vertexes[0]-other[0], vertexes[1]-other[1]);
  }

  Vec2F operator*(float other) const {
    return Vec2F(vertexes[0]*other, vertexes[1]*other);
  }

  Vec2F operator/(float other) const {
    return Vec2F(vertexes[0]/other, vertexes[1]/other);
  }

  Vec2F operator=(Vec2F const& other) {
    vertexes[0] = other[0];
    vertexes[1] = other[1];
    return *this;
  }

private:
  std::deque<float> vertexes;
};

float magSquared(Vec2F const& a) {
  return a[0]*a[0]+a[1]*a[1];
}

typedef Vec2F Coord;

// Somewhat nicer form of std::accumulate
template<typename Container, typename Function>
typename Container::value_type reduce(Container const& l, Function f) {
  typename Container::const_iterator i = l.begin();
  typename Container::value_type res{};

  if (i == l.end())
    return res;

  res = *i++;
  while (i != l.end())
    res = f(res, *i++);
  return res;
}     


template<typename Container>
typename Container::value_type sum(Container const& cont) {
  return reduce(cont, std::plus<typename Container::value_type>());
}


struct PolyF
{
    PolyF()
    {
        m_vertexes.resize(4);
        std::generate( m_vertexes.begin(), m_vertexes.end(), [](){ return Vec2F(std::rand(), std::rand());} );
    }

    std::vector<Coord> m_vertexes;

    Coord center() const 
    {
      return sum(m_vertexes) / (float)m_vertexes.size();
    }
};

struct CollisionBlock 
{
  PolyF poly;
  // Will never be None
  CollisionKind kind;
  // Normalzied vector encoding the slope of the block we collided with.
  // Always faces right, y component can be positive or negative.
  Vec2F slope;
};


template<typename Container, typename Compare>
void sort(Container& c, Compare comp) {
  std::sort(c.begin(), c.end(), comp);
} 

struct CollisionGen
{
    std::deque<CollisionBlock> getPolys( RectF const& ) const
    {
        std::deque<CollisionBlock> collision_block_moc(50);
        return collision_block_moc;
    }
};

struct WorldClient
{
    CollisionGen m_collisionGenerator;


    std::deque<CollisionBlock> getCollisionBlocks(RectF const& boundBox, bool doSort, Vec2F sortCenter) const 
    {
      auto res = m_collisionGenerator.getPolys(boundBox);

    //auto test = magSquared(sortCenter - res.front().poly.center()) < magSquared(sortCenter - res.front().poly.center());

      if (doSort) {
        sort(res, [=](CollisionBlock const& block1, CollisionBlock const& block2) {
        return magSquared(sortCenter - block1.poly.center()) < magSquared(sortCenter - block2.poly.center());
          });
      }

      return res;
    }
};

    int main() 
    {
        WorldClient wc;
        while (true) {
          wc.getCollisionBlocks( 42.0, true, {0,0} );
        }
    }
于 2012-06-29T06:58:05.580 回答
0

我的猜测是模板 List<> 上的迭代器不是随机访问迭代器，这是 std::sort 需要的，请注意 std::list 仅提供双向迭代。

如果您已经实现了自己的列表，我建议您仔细检查您的迭代器实现，以确保它是正确的，并提供了随机访问迭代器概念的完整和正确的实现。

于 2012-06-29T06:10:30.983 回答
0

我注意到您通过说 [=] 但使用引用将值传递给 lambda。您应该更改它，或者通过使用 std::ref 传递值以真正使用对实例本身的预期引用。

于 2012-06-29T06:21:12.233 回答
0

您的比较器是严格的弱顺序吗？它是否始终如一地对其输入进行排序？我怀疑您实际上并没有在 中进行零初始化res，reduce因此center()最终会返回垃圾。这应该很容易检查一般性：assert(f(res,res) == res)在 res 的声明之后添加一个权利。

如果比较器不一致，则使用 会遇到未定义的行为std::sort，这很容易导致崩溃。并对平台、优化等敏感。

于 2012-06-29T06:59:24.080 回答