C++中存储指向MonkeyModel的指针出现段错误，求助解决

问题根源 #

你的代码出现段错误的核心原因是悬空指针：
当你调用_collection.getModels()时，如果这个函数的返回值是std::vector<MonkeyModel>（值传递），每次调用都会创建一个临时的vector副本。你取&_collection.getModels().at(modelNumber)得到的是这个临时副本里元素的地址，但临时对象会在当前语句执行完毕后立即销毁，导致_selectedModel变成无效的悬空指针。后续访问这个指针时，就会触发内存访问错误，也就是段错误。

看你的debug代码就能验证这一点：

• MonkeyModel test = _collection.getModels().at(0); 是把临时vector里的元素拷贝到局部变量test中，所以test是有效的；
• test_p指向的是局部变量test，自然能正常访问；
• 但_selectedModel = &_collection.getModels().at(0);拿到的是临时vector元素的地址，临时对象销毁后这个指针直接失效，输出时访问无效内存导致崩溃。

解决方案 #

方案1：修改getModels()返回引用（最直接） #

修改MonkeyCollection类中getModels()的返回值为引用，让函数直接返回内部存储的vector，而不是副本：

// 在MonkeyCollection类中修改函数声明
std::vector<MonkeyModel>& getModels();

// 如果需要支持const上下文，补充const版本
const std::vector<MonkeyModel>& getModels() const;

这样调用_collection.getModels()时，你拿到的是MonkeyCollection内部真实存储的vector的引用，取元素地址后，指针指向的是持久存在的MonkeyModel实例，不会再出现悬空问题。

方案2：改用索引存储（更安全） #

如果你担心后续_collection的vector发生扩容、元素删除等操作导致指针失效，可以改用索引存储选中的模型，避免指针悬空风险：

// 在MonkeyShell类中替换指针成员
class MonkeyShell {
private:
    // ... 其他原有成员
    std::optional<int> _selectedModelIndex; // 用std::optional表示可能无选中模型
};

在editCommand()中赋值：

void MonkeyShell::editCommand() {
    std::string modelNumberString;
    int modelNumber = 0;

    std::cout << "Which model do you want to work on ?" << std::endl << _collection << std::endl;
    std::getline(std::cin, modelNumberString);
    modelNumber = std::stoi(modelNumberString) - 1;
    
    // 提前检查索引有效性，避免out_of_range异常
    if (modelNumber >= 0 && modelNumber < _collection.getModels().size()) {
        _selectedModelIndex = modelNumber;
    } else {
        std::cout << "Invalid model number!" << std::endl;
    }
}

访问选中模型时通过索引获取：

// 示例：操作选中模型
if (_selectedModelIndex.has_value()) {
    MonkeyModel& selected = _collection.getModels().at(_selectedModelIndex.value());
    // 对selected进行编辑操作
} else {
    std::cout << "No model selected!" << std::endl;
}

额外注意事项 #

• 无论用指针还是索引，都要提前检查输入的编号是否在有效范围内，避免at()抛出out_of_range异常；
• 如果MonkeyCollection内部的vector会被修改（比如删除元素、扩容），要同步更新_selectedModel指针或_selectedModelIndex，防止访问无效元素。

内容的提问来源于stack exchange，提问作者GAlbert