C++ 我是否正确添加和访问我的向量？

Question

我正在创建一个对象并将其添加到 std::vector，然后访问 std::vector 以使用我放置在其中的对象的成员变量。

.h 包含std::vector

std::vector<Field *> vFields;
inline std::vector<Field *> getFields() { return vFields; };

在 .cpp 我这样做：

Field* f1 = Field::createWithLocation(ccp( p.x, p.y));
addChild(f1->getFieldSprite(), 2);
getFields().push_back(f1); // add the new field to a container
//vFields.push_back(f1); // add the new field to a container

std::cout << "add - # of Fields: " << getFields().size() << std::endl;

当实例化上面的Field* f1 时，会发生这种情况：

Field* Field::createWithLocation(cocos2d::CCPoint p)
{
    Field* f = new Field();
    //f->autorelease();
    f->initWithLocation(p);
    return f;
}

void Field::initWithLocation(cocos2d::CCPoint p)
{
    setFieldCenterPoint(p);
    setFieldGraphicName(FIELD::fieldIconFileName);

    setFieldSprite(cocos2d::CCSprite::create(getFieldGraphicName().c_str()));
     getFieldSprite()->setPosition(ccp(getFieldCenterPoint().x, getFieldCenterPoint().y));

    setFieldSize(getFieldSprite()->getContentSize());

    setFieldNumber(7159);

    setFieldTag(7159);

    std::cout << "When Field Created #: " << getFieldTag() << std::endl;
}

这很好用，当创建 Field 对象时，std::vector 在 size() 中显示 1 并且 getFieldTag() 像设置一样返回 7159。

问题是当我从向量访问Field 时，发生了一些事情并且我崩溃了。我发现，如果我输出 getTagNumber() ，那就大不相同了。

访问向量的例子：

else if (getFieldLayerState() == kActive)
{
    // so did they click on a field?
    std::cout << "Field Layer Status Is Active" << std::endl;
    std::cout << "touch - # of Fields: " << vFields.size() << std::endl;

    // Loop through the field vector and get the size of each field and see if it was tapped    
    for (int f=0; f<vFields.size(); f++)
    {
        //field = (Field*) getFields().at(f);
        Field* field = (Field*) vFields.at(f);

        std::cout << "field #: "<< field->getFieldNumber() << std::endl;
        std::cout << "field tag: "<< field->getFieldTag() << std::endl;

        if (field->getFieldSprite()->boundingBox().containsPoint(location))
        {
            std::cout << "touched field #: "<< field->getFieldNumber() << std::endl;
            std::cout << "touched field tag: "<< field->getFieldTag() << std::endl;
            _dir->Instance()->getAudioEngine()->playBackgroundMusic("tap.mp3", false);    
        }
     }
}

cout 语句的示例输出：

When Field Created #: 7159
add - # of Fields: 1
Field Layer Status Is Active

touch - # of Fields: 1
field #: 1769234804
field tag: 353394533`

我在上面的if (field->getFieldSprite()->boundingBox().containsPoint(location)) 行崩溃了EXEC_BAD_ACCESS，很明显，向量中的内容与我放入的内容有所不同，或者看起来是这样。

谁能帮我理解我做错了什么？

Answer 1

您的getFields 函数返回向量的副本。因此，对返回向量的任何更改都只会发生在副本中，而不会发生在原始向量中。

你想返回一个引用。

std::vector<Field*>& getFields() { return vFields; }
//                 ^
// Note the ampersand

您可能还想添加函数的 const 重载：

const std::vector<Field*>& getFields() const { return vFields; }
// ^                                      ^
// Note the two `const` modifiers, the last one is important

然后，如果您修改向量，例如通过添加它，编译器将选择第一个非常量重载。如果你不修改向量，例如当您在向量上调用size 时，编译器将选择第二个 const 重载。