虚函数,为什么这里调用基类函数？

Question

有人可以帮我解决我做错的事吗？始终调用基类指针!我正在尝试制作自定义类对象的Map.通过直接查找和索引来尝试,但结果相同!

#include "stdafx.h"
#include <iostream>
#include <string>
#include <Map>
#include <algorithm>

class Command
{
public:
    virtual int execute(std::string *args) { std::cout << "Base called ! ERROR!\n"; return -1; }
};

class ShowNames : public Command
{
public:
    int execute(std::string names)
    {
        std::cout << names;
        return 0;
    }
};

class ShowNos : public Command
{
public:
    int execute(std::string Nos)
    {
        std::cout << Nos;
        return 0;
    }
};

typedef std::map<std::string, Command*> CmdList;

CmdList buildMaps()
{
    CmdList c1;
    ShowNames s1;
    ShowNos   n1;

    c1["names"] = new ShowNames();
    c1["nos"] = new ShowNos();

    //c1.find("names")

    return c1;
}

void testCommandList()
{
    CmdList commands;
    Command *c1;
    commands = buildMaps();

    std::string cmd,args;
    std::cout << "Enter your command: ";
    std::cin >> cmd;
    std::cout << "Enter args for the command: ";
    std::cin >> args;

     auto it = commands.find(cmd);
     if (it != commands.end())
     {
         it->second->execute(&args);
     }
     else
     {
         std::cout << "Command not found, try again\n";
     }

}

Answer 1

您没有覆盖派生类中的基本函数,而是声明了新函数.比较函数类型:

int   Command::execute(std::string *args)
int ShowNames::execute(std::string  names)
int   ShowNos::execute(std::string  Nos)

(对齐以使其更明显)

要覆盖基类函数,必须完全匹配签名(如果需要,除了协变返回类型).因此,将签名更改为相同.当然,哪一个是正确的取决于您的问题域.

这就是为什么C++ 11引入了保留字override,你可以把它放在一个你打算覆盖基类函数的虚函数上.如果不是这种情况,则会导致编译错误.如果您有权访问C++ 11,那么您应该这样做在使用它时始终使用它,如下所示:

class ShowNames : public Command
{
public:
    int execute(std::string names) override
    {
        std::cout << names;
        return 0;
    }
};

这会立即告诉您它不会覆盖任何基类函数,并且您可以更好地开始调查原因.