C ++将父指针降级为用作功能参数的子指针

时间:2018-07-01 23:21:19

标签: c++ oop dynamic-cast downgrade

我想在某些函数中使用子指针代替父参数。 这是简化的代码

#include <iostream>
#include <string>
using namespace std;
class Parent
{
    private:
        string value;
    public:
        void SetValue(const string& in){ value = in; }
        const string& GetValue() const { return value; }
        virtual ~Parent() {};
};
class Child : public Parent
{
};
void Func(Parent* obj)
{
    Child* cobj = dynamic_cast<Child*>(obj);
    cobj->SetValue("new value");
    delete cobj;
}
int main(int argc, char** argv)
{
    Child* cobj = new Child();
    Func(cobj);
    cout<<"value: "<<cobj->GetValue();
    return 0;
}

但是它返回value:
Func未设置value属性。如何修复代码?
UPDATE1:
但是使用以下代码:

#include <iostream>
#include <string>
using namespace std;
class Parent
{
    private:
        string value;
    public:
        void SetValue(const string& in){ value = in; }
        const string& GetValue() const { return value; }
        virtual ~Parent() {};
};
class Child : public Parent
{
    private:
        int number;
    public:
        void SetNumber(int in){ number = in; }
        int GetNumber() const { return number; }
};
void Func(Parent* obj)
{
    Child* cobj = dynamic_cast<Child*>(obj);
    cobj->SetValue("new value");
    cobj->SetNumber(100);
    delete cobj;
}
int main(int argc, char** argv)
{
    Child* cobj = new Child();
    Func(cobj);
    cout<<"value: "<<cobj->GetValue()<<endl;
    cout<<"number: "<<cobj->GetNumber()<<endl;
    return 0;
}

我得到value: number: 100
为什么SetNumber有效但SetValue不起作用?

2 个答案:

答案 0 :(得分:1)

您的问题与继承或函数参数无关。有问题的代码可以简化为:

Child* cobj = new Child();
cobj->SetValue("new value");
delete cobj;
cobj->GetValue();

最后两行是问题所在。 delete cobj破坏了对象,但是cobj->GetValue()仍然尝试使用它。该代码具有未定义的行为。

要修复代码,您需要删除delete cobjGetValue()调用。无论如何,在对象被销毁后不要尝试使用它。

答案 1 :(得分:1)

您删除该对象,然后继续访问它。这将导致undefined behaviour,这意味着可能发生任何事情。

您可以通过删除行delete cobj;来修复代码。但是,在取消引用之前,还应检查dynamic_cast的结果是否不为空。

如果不清楚,则delete表达式将删除所指向的对象。没有“删除指针”这样的概念。