抽象基类未正确返回

Question

我还没有完全理解抽象基类。这是我为我的作业创建的，它似乎主要工作，除了它不会返回通过派生类添加的任何内容。每当我选择 hasDMA 或 lacksDMA 时，我都不会得到颜色或样式返回的结果。

Main.cpp

#include <iostream>
#include <conio.h>
#include "DMA.h"

using namespace std;

const int RECORDS = 1;
const int LEN = 40;

int main()
{
    ABC * p_records[RECORDS];

    int i;
    for (i = 0; i < RECORDS; i++)
    {
        char temp[LEN];
        int temprate;
        char choice;

        cout << "\nEnter label name: ";
        cin.getline(temp, LEN);
        cout << "Enter Rating: ";
        cin >> temprate;

        cout << "Enter 1 for lacksDMA or 2 for hasDMA: ";
        while (cin >> choice && (choice != '1' && choice != '2'))
            cout << "Enter 1, 2: ";
        if (choice == '1')
        {
            char tempcolor[LEN];
            cout << "Enter the color: ";
            cin.getline(tempcolor, LEN);
            p_records[i] = new lacksDMA(temp, temprate, tempcolor);         
        }
        else
        {
            char tempstyle[LEN];
            cout << "Enter the style: ";
            cin.getline(tempstyle, LEN);
            p_records[i] = new hasDMA(tempstyle, temp, temprate);           
        }
        while (cin.get() != '\n')
            continue;
    }
    cout << endl;
    for (i = 0; i < RECORDS; i++)
    {
        p_records[i]->View();
        cout << endl;
    }

    for (i = 0; i < RECORDS; i++)
    {
        delete p_records[i];
    }

    cout << "\nPress any key to continue...";
    cin.sync();
    _getch();

    return 0;
}

DMA.cpp

#include "DMA.h"

using namespace std;


ABC::ABC(const char * l, int r)
{
    label = new char [strlen(l) + 1];
    strcpy(label, l);
    rating = r;
}

ABC::ABC(const ABC & rs)
{
    label = new char[strlen(rs.label) + 1];
    strcpy(label, rs.label);
    rating = rs.rating;
}

ABC::~ABC()
{   
}

ABC & ABC::operator=(const ABC & rs)
{
    if (this == &rs)
        return *this;
    delete [] label;
    label = new char[strlen(rs.label) + 1];
    strcpy(label, rs.label);
    rating = rs.rating;
    return *this;
}

ostream & operator<<(ostream & os, const ABC & rs)
{
    rs.View();
    return os;
}

void ABC::View() const
{
    cout << "\nLabel: " << label << endl;
    cout << "Rating: " << rating << endl;
}

baseDMA::baseDMA(const char * l, int r) : ABC(l,r)
{
}

lacksDMA::lacksDMA(const char * l, int r, const char * c) : ABC(l,r)
{
    strncpy(color, c, 39);
    color[39] = '\0';
}

lacksDMA::lacksDMA(const ABC &rs, const char * c) : ABC(rs)
{
    strncpy(color, c, 39);
    color[39] = '\0';
}

void lacksDMA::View() const
{
    ABC::View();
    cout << "Color: " << color << endl; 
}

hasDMA::hasDMA(const char * s, const char * l, int r) : ABC(l,r)
{
    style = new char [strlen(s) + 1];
    strcpy(style, s);   
}

hasDMA::hasDMA(const char * s, const ABC & rs) : ABC(rs)
{
    style = new char [strlen(s) + 1];
    strcpy(style, s);
}

hasDMA::hasDMA(const hasDMA & hs) : ABC(hs)
{
    style = new char [strlen(hs.style) + 1];
    strcpy(style, hs.style);
}

hasDMA::~hasDMA()
{
    delete [] style;
}

void hasDMA::View() const
{
    ABC::View();
    cout << "Style: " << style << endl;
}

DMA.h

#ifndef DMA_H_
#define DMA_H_
#include <iostream>

using namespace std;

// Abstract Base Class
class ABC
{
private:
    char * label;
    int rating;
public:
    ABC(const char * l = "null", int r = 0);
    ABC(const ABC & rs);
    virtual ~ABC() = 0;
    virtual ABC & operator=(const ABC & rs);
    virtual void View() const;
    friend ostream & operator<<(ostream & os, const ABC & rs);
};

// Former Base Class Using DMA
class baseDMA: public ABC
{
private:

public:
    baseDMA(const char * l = "null", int r = 0);
};

// derived class without DMA
// no destructor needed
// uses implicit copy constructor
// uses implicit assignment operator
class lacksDMA : public ABC
{
private:
    char color[40];
public:
    lacksDMA(const char * l = "null", int r = 0, const char * c = "blank");     
    lacksDMA(const ABC & rs, const char * c);
    virtual void View() const;
};

// derived class with DMA
class hasDMA : public ABC
{
private:
    char * style;
public:
    hasDMA(const char * s = "none", const char * l = "null", int r = 0);
    hasDMA(const char * s, const ABC & rs);
    hasDMA(const hasDMA & hs);
    ~hasDMA();
    hasDMA & operator = (const hasDMA & rs);
    void View() const;
};

#endif

Answer 1

在您所说的 cmets 中，显示了字符串“Color:”，但不显示示例输入“red”。因为显示了“颜色：”，所以这不是代码的抽象/虚拟部分失败的问题（否则即使“颜色：”也不会显示）。

失败在输入中。当您读取“1”或“2”的选择时，您只读取了一个字符，但用户实际上也输入了额外的新行，因此下一个cin.getline 将使用这个额外的新行并返回一个空字符串。对于所有输入，您应该坚持使用cin.getline，即使输入只是一个字符，即使输入是一个数字。永远不要使用cin >>，自己处理输入中的错误，比将operator>> 和getline 混合在一起，然后在它们不能很好地配合使用时处理所有边界情况要容易。

虽然这不是答案的一部分，但这里有一些一般性建议：

1) 在 C++ 中，尽可能避免使用new char[x]。对字符串使用std::string，在std::string 不够好的极少数情况下使用std::vector<char>。

1b) 你忘了delete [] label 在ABC 的析构函数中。如果是std::string，您就不必担心这些事情。

2) 可以在源文件 (.cpp) 中使用 use namespace std，并且只在所有 #includes 之后，但不要在头文件 (.h) 中使用它。绝不！在头文件中使用全名（如std::string 而不是string）

编辑：因此，需要一些代码示例。就像我说的，坚持cin.getline，避免operator>>，即使输入只是一个字符，即使输入是整数：

将cin >> temprate; 行替换为：

    cin.getline(temp, LEN);
    temprate = strtol(temp, NULL, 10);

替换两行while循环

    while (cin >> choice && (choice != '1' && choice != '2'))
        cout << "Enter 1, 2: ";

这些for 行在中间有出口（我最喜欢的无限循环方式）：

    for (;;)
    {
        cin.getline(temp, LEN);
        choice = temp[0];

        if (choice == '1' || choice == '2')
            break;

        cout << "Enter 1, 2: ";
    }