Как может производный класс C ++ клонировать себя через базовый указатель? - PullRequest
5 голосов
/ 29 июня 2010

Вот что я пытаюсь сделать (этот код не работает):

class Base
{
    virtual Base *clone() { return new Base(this); }
    virtual void ID() { printf("BASE");
};

class Derived : publc Base
{
    virtual Base *clone() { return new Derived(this); }
    virtual void ID() { printf("DERIVED"); }
}

.
.
Derived d;
Base *bp = &d;
Base *bp2 = bp->clone();

bp2->ID();

То, что я хотел бы , - это распечатка "DERIVED" ... то, что я получил, это "BASE". Я давний программист на C и довольно опытный в C ++ ... но я не добиваюсь успехов с этим ... любая помощь будет оценена.

Ответы [ 6 ]

7 голосов
/ 29 июня 2010

Как только все ошибки компиляции были исправлены, я получил следующее:

#include <cstdio>

class Base
{
  public:
    Base() {}
    Base(const Base&) {}
    virtual Base *clone() { return new Base(*this); }
    virtual void ID() { printf("BASE"); }
};

class Derived : public Base
{
  public:
    Derived() {}
    Derived(const Derived&) {}
    virtual Base *clone() { return new Derived(*this); }
    virtual void ID() { printf("DERIVED"); }
};


int main()
{
  Derived d;
  Base *bp = &d;
  Base *bp2 = bp->clone();

  bp2->ID();
}

Что дает вам то, что вы ищете - ПРОИЗВОД.

3 голосов
/ 29 июня 2010

Этот код пронизан синтаксическими ошибками.Возможно, что наиболее важно, Derived не наследуется от Base.Во-вторых, кроме синтаксических ошибок (возможно, простых опечаток), Base явно нуждается в виртуальном деструкторе.Метод clone в значительной степени требует, чтобы вы могли вызывать оператор delete для базового указателя (Base *).

class Base
{
public:
    virtual ~Base() {}
    virtual Base* clone() const { return new Base(*this); }
    virtual void ID() const { printf("BASE"); }
};

class Derived: public Base
{
public:
    // [Edit] Changed return type to Derived* instead of Base*.
    // Thanks to Matthieu for pointing this out. @see comments below.
    virtual Derived* clone() const { return new Derived(*this); }
    virtual void ID() const { printf("DERIVED"); }
};

int main()
{
    Derived d;
    Base* bp = &d;

    Base* bp2 = bp->clone();
    bp2->ID(); // outputs DERIVED as expected
    delete bp2;
}
1 голос
/ 29 июня 2010

С Base bp = &d;

Вы "нарезали" d, поэтому для компилятора bp действительно имеет тип Base, поэтому при вызове bp->clone()вызовы компилятора Base::clone(); и bp2->ID() print BASE.

Base& bp = d; будут делать то, что вы хотите.

0 голосов
/ 29 июня 2010

Код выглядит отлично, кроме глупых синтаксических опечаток и отсутствующих ctors.

0 голосов
/ 29 июня 2010

Ваш пример неверен и не скомпилируется. Конкретно эта строка:

Base bp = &d;

Это также может быть основной причиной вашей проблемы (вы можете нарезать свой объект), но я не могу сказать наверняка, не увидев рабочий код.

У вас также есть проблема, когда ваши два класса не связаны (вы хотели написать class Derived : public Base?)

0 голосов
/ 29 июня 2010

Вы нарезаете класс в Base bp = &d; (это создает новый базовый бит из производного-ptr.)

Попробуйте Base* bp = &d; вместо этого.(т.е. создать указатель базового типа на производный объект.)

...