试图返回一个向量

Question

我是 C++ 编程新手，并试图让一些 boid 都移到中心，我有两种方法 updateboid 和 cohesion。在凝聚力中，我试图将标准化向量返回到 updateBoid 中，当我这样做时，boids 只是全部侧向移动，而不是向中心移动。我在这里做了一些愚蠢的事情，将不胜感激。

void Scene::updateBoid()
{
    for(size_t i=0; i<m_collection.size(); ++i)
    {
        for(auto &m :m_collection[i])
        {
            m->acc = cohesion();

            m->pos += m->acc * 0.05f ;
        }
    }
}


Vec3 Scene::cohesion()
{
    const float pi = 3.14f;

    Vec3 center;
    Vec3 test;

    for(size_t i=0; i<m_collection.size(); ++i)
    {
        for(auto _m :m_collection[i]) // all of the boids
        {
            center += _m->pos;
        }

        center /= m_collection[i].size(); // doing this gives the center

        /// Boids move to the center of their average positions
        for(auto &m :m_collection[i])
        {
            m->dir =center - m->pos; //vector between the two objects

            m->dir.normalize();

            return m->dir;
        }
    }

}

---

cohesion() 中的先前代码

        m->dir =center - m->pos;        //length between the two objects
        m->dir.normalize();
        m->pos+=m->dir * 0.25f; //speed

这行得通，但想要通过使用另一种方法来更新的另一种方法。

Answer 1

for(auto &m :m_collection[i])
{
    m->dir =center - m->pos; //vector between the two objects
    m->dir.normalize();
    return m->dir; // <- Always return on the first iteration
}

正如 Max Langhof 已经指出的那样，for 循环只会执行一次。 我称之为错误。从修改代码开始。

Answer 2

在每个cohesion 调用中，您总是（基本上）返回相同的方向，因此您将所有m->acc 设置为相同的值。那是因为你在cohesion 中的return 已经退出了第一个boid。

问题是你还没有决定cohesion 应该做什么。如果它应该返回 one boid 的目标方向，那么你必须告诉它 which boid 要考虑。您也可以直接在cohesion 中更新m->acc，然后离开updateBoid 以修改m->pos（并且只读取m->acc - 这可能是更好的解决方案。

在代码中：

---

选项 1：不返回任何东西。只修改每个 boid。不要在updateBoid 内调用cohesion，只调用一次。

void Scene::updateBoid()
{
    for(size_t i=0; i<m_collection.size(); ++i)
    {
        for(auto &m :m_collection[i])
        {
            float acc = m->dir;
            m->pos += acc * 0.05f;
        }
    }
}


void Scene::updateCohesion()
{
    for(size_t i=0; i<m_collection.size(); ++i)
    {
        // This must be in here, otherwise it will be slightly wrong later.
        Vec3 center;

        for(auto _m :m_collection[i]) // all of the boids
        {
            center += _m->pos;
        }

        center /= m_collection[i].size(); // doing this gives the center

        /// Boids move to the center of their average positions
        for(auto &m :m_collection[i])
        {
            m->dir =center - m->pos; //vector between the two objects

            m->dir.normalize();
        }
    }
}

选项 2：返回每个 boid 的方向。这是低效的，因为您每次都要重新计算中心。

void Scene::updateBoid()
{
    for(size_t i=0; i<m_collection.size(); ++i)
    {
        for(auto &m :m_collection[i])
        {
            float acc = cohesionForBoid(m_collection[i], m);
            m->pos += acc * 0.05f;
        }
    }
}

// What are these types? We don't know, that is missing from the question.
Vec3 Scene::cohesionForBoid(??? collection, ??? m)
{
    Vec3 center;

    for(auto _m : collection) // all of the boids
    {
        center += _m->pos;
    }

    center /= collection.size(); // doing this gives the center

    Vec3 dir = center - m->pos; //vector between the two objects
    dir.normalize();
    return dir;
}