0
#include <iostream>
#include <string>
#include <map>
#include <vector>

class base {};
class derived1 : public base
{
    public:
        unsigned short n;
        derived1()
        {
            n = 2;
        }
};
class derived2 : public base {};

void main()
{
    // way 1
    {
        std::vector<derived1> a1;
        std::vector<derived2> a2;
        std::map<std::string, base*> b;
        a1.push_back(derived1());
        b["abc"] = &a1.at(0);
        std::cout<<(dynamic_cast<derived1*>(b.find("abc")->second))->n<<std::endl;
    }

    // way 2
    {
        std::map<std::string, base*> b;
        b["abc"] = new derived1();
        std::cout<<dynamic_cast<derived1*>(b.find("abc")->second)->n<<std::endl;
        delete dynamic_cast<derived1*>(b.find("abc")->second);
    }
}

错误是“'dynamic_cast':'base' 不是多态类型”。应该怎么做才能解决这个问题?方式 1 和方式 2 是否都已正确清理?

4

1 回答 1

7

要创建Base一个多态类型,你需要给它至少一个虚函数。在这种情况下最简单的是析构函数:

class Base {
public:
  virtual ~Base() { }
};

关于您关于清理的问题:
从技术上讲,两种方式都有一些未定义的行为,因为地图所指的对象在从地图中删除指针之前被销毁。这导致映射在被破坏时包含无效指针并导致未定义的行为。
出于实际目的,这不会对任何已知的编译器造成任何问题。

否则,您正在正确清理所有内容。
但是在方式2中,您可以进行简化。当Base有一个虚拟析构函数时,你可以这样做

delete b.find("abc")->second;

没有动态演员表。

于 2011-01-09T14:43:09.640 回答