оператор виртуального присваивания производного класса не вызывается - PullRequest
Новая
       5

оператор виртуального присваивания производного класса не вызывается

3 голосов
/ 30 октября 2011

Я довольно новичок в C ++ и пытаюсь справиться с виртуальным заданием. Приведенная ниже программа состоит из абстрактного базового класса с двумя членами-данными и производного класса с одним. Когда я устанавливаю абстрактный указатель на производный объект, программа использует абстрактную версию operator =, а не производную версию, даже если они оба объявлены «виртуальными». Что я тут не так делаю?

Заранее спасибо,

Jay 

#include <iostream>
#include <cstring>

class Abstract
{
  protected:
        char * label;
        int rating;
    public:
        Abstract(const char * l = "null", int r = 0);
        virtual Abstract & operator=(const Abstract & rs);
        virtual ~Abstract() { delete [] label; }
        virtual void view() const = 0;

};

class Derived : public Abstract
{
    private:
        char * style;
    public:
        Derived(const char * s = "none", const char * l = "null",
                  int r = 0);
        ~Derived() { delete [] style; }
        virtual Derived & operator=(const Derived & rs);
        virtual void view() const;

};

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

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

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

Derived & Derived::operator=(const Derived & hs)
{
    if (this == &hs)
        return *this;
    Abstract::operator=(hs);
    style = new char[std::strlen(hs.style) + 1];
    std::strcpy(style, hs.style);
    return *this;
}

void Derived::view() const
{
    std::cout << "label: " << label << "\nrating: "
              << rating << "\nstyle: " << style;
}

int main ()
{
    using namespace std;
    char label[20], style[20];
    int rating;


    cout << "label? ";
    cin >> label;
    cout << "rating? ";
    cin >> rating;
    cout <<"style? ";
    cin >> style;

    Derived a;
    Abstract * ptr = &a;
    Derived b(style, label, rating);
    *ptr = b;
    ptr->view();

    return 0;
}

Ответы [ 2 ]

0 голосов
/ 11 октября 2013

Это немного устарело, но в случае, если кто-то еще наткнется на него:

Чтобы добавить ответ Марка, вы можете сделать это, введя 

Derived & operator=(const Abstract & rs);

В этом случае вы можетенужно использовать rs, разыграв его: dynamic_cast<const Derived &>(rs)
Конечно, это следует делать только осторожно.Полная реализация будет: 

Derived & Derived::operator=(const Abstract & hs)
{
    if (this == &hs)
        return *this;
    Abstract::operator=(hs);
    style = new char[std::strlen(dynamic_cast<const Derived &>(hs).style) + 1];
    std::strcpy(style, dynamic_cast<const Derived &>(hs).style);
    return *this;
}
0 голосов
/ 30 октября 2011

C ++ не позволяет переопределять виртуальные функции с ковариантными типами параметров.Ваш производный оператор вообще не переопределяет абстрактный оператор присваивания, он определяет полностью ортогональный оператор, связанный только тем, что это одно и то же имя оператора.

Вы должны быть осторожны при создании таких функций, потому что, если два фактических производныхтипы не согласны, почти наверняка назначение будет бессмысленным.Я бы пересмотрел вопрос о том, может ли ваш дизайн лучше обслуживаться альтернативным подходом.

...