通过基类指针数组修改类内对象的问题修复咨询

核心问题分析 #

计算结果不符合预期的原因有两点：

1. VectorNumericalParameter未持有原向量的引用：当前代码中VectorNumericalParameter的values_是std::vector<double>类型，构造时会拷贝传入的向量，而非持有原向量的引用，导致修改values_不会影响Model3内部的x_。
2. 范围for循环修改的是元素副本：tilt_up和tilt_down方法中使用for(auto value : values_)遍历，这里的value是向量元素的拷贝，修改它不会改变原向量中的元素值。

修正后的代码实现 #

1. 修改VectorNumericalParameter类 #

将values_改为引用类型，并修改循环方式：

class VectorNumericalParameter : public NumericalParameter
{
    std::vector<double> &values_; // 改为引用，持有原向量的引用
public:
    VectorNumericalParameter(std::vector<double> &values) : values_(values) {}
    void tilt_up(const double bumpSize = 1e-8) override
    {
        // 使用引用遍历，直接修改原元素
        for(auto &value : values_)
            value += bumpSize;
    }
    void tilt_down(const double bumpSize = 1e-8) override
    {
        // 使用引用遍历，直接修改原元素
        for(auto &value : values_)
            value -= bumpSize;
    }
};

2. 验证修正后的运行结果 #

修改后，Model3内部的x_会被正确修改，运行main函数后会输出预期的1.0。

额外优化建议 #

• 解决内存泄漏：当前Model类中使用new创建参数对象，但没有对应的delete操作，建议在Model的析构函数中释放这些动态分配的内存，或者改用std::unique_ptr等智能指针管理资源。
• 优化构造函数效率：Model3的构造函数接收const std::vector<double> x会触发一次向量拷贝，可改为std::vector<double> &&x（移动语义）或const std::vector<double> &x（引用传递）减少不必要的拷贝。

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