Класс, наследование

[nazavrik]

Добрый день!

Собственно вопрос: если в родительском классе определен и реализован виртуальный метод, обязательно переопределять его в дочернем?

Код:

class OneTwo
{
public:
	OneTwo(){}

	virtual void Print() = 0;
};

class One : virtual public OneTwo
{
public:
	One(){}
	virtual void Print() override 
	{
		std::cout<<"One!";
	}
};

class A : virtual public OneTwo
{
public:
	A(char *pS):pStr(pS){}

	char &GetString() {return *pStr;}
private:
	char *pStr;
};

class myClass1 : public One, public A
{
public:
	myClass1(char *pStr): A(pStr) {} 
};

Вылезает варнинг:

warning C4250: 'myClass1' : inherits 'One::One::Print' via dominance

В принципе не фолсит, но варнинг меня смущает. Как избавиться от него? ну и естественно не получается создать объект класса myClass1

cannot instantiate abstract class

[profi]

если в родительском классе определен и реализован виртуальный метод, обязательно переопределять его в дочернем?

Во-первых он у вас не реализован. У вас в родительском классе чисто виртуальная функция. И соответственно, в дочерних классах нужно её переопределять. Кстати ключевым словом override уже давно не пользуются. Кстати у вас ошибки в коде, слово virtual в методе Print в дочерних классах. Советую для начала почитать книгу Г. Шилдта. "Самоучитель С++" стр. 306

[nazavrik]

Сообщение от profi

Во-первых он у вас не реализован.

а это что:

	virtual void Print() override 
	{
		std::cout<<"One!";
	}

Кстати ключевым словом override уже давно не пользуются.

а чем пользуются?

[profi]

В классе OneTwo не реализован метод Print.

а чем пользуются?

Просто его не пишут. Я даже страницу книги указал. Почитайте. Лучше раз прочитать и спросить, чем по крупинкам на форуме спрашивать.

[pu4koff]

Сообщение от profi

Кстати у вас ошибки в коде, слово virtual в методе Print в дочерних классах.

Это не ошибка. Наличие/отсутствие virtual совершенно ничего не меняет. Тут исключительно вопрос предпочтений программиста в плане оформления кода.
Если из минусов тут пожалуй только необходимость синхронизировать виртуальность методов (сделали метод невиртуальным в базовом классе, значит нужно его делать невиртуальным во всех его потомках), то из плюсов:
1) не нужно смотреть базовый класс, чтобы узнать какие методы виртуальные, а какие нет;
2) Если в процессе рефакторинга "выкинуть" базовый класс, то не появятся непонятные баги из-за того, что виртуальные функции стали вдруг невиртуальными;
...

[profi]

Наличие/отсутствие virtual совершенно ничего не меняет.

Покажите мне хоть одну книгу где в дочерних классах используется слово virtual для виртуальных методов. Компилятор хоть это пропускает, но все равно лучше так не делать.

nazavrik вот пример:

#include <iostream.h>

class General
{
        public:
                int i;
                virtual void Print() = 0;
                virtual void SetVal(int x) = 0;
};

class A: virtual public General
{
        public:
                void SetVal(int x){ i = x; }
                void Print(){ cout << i << "\n"; }
};

class B: virtual public General
{
        public:
                void SetVal(int x){ i = x; }
                void Print(){ cout << i << "\n"; }
};

class C: public A, public B
{
        public:
                void SetVal(int x){ i = x; }
                void Print(){ cout << i << "\n"; }
};

int main()
{
        General *pG;

        A aaa;
        B bbb;
        C ccc;

        pG = &aaa;
        pG->SetVal(1);
        pG->Print();

        pG = &bbb;
        pG->SetVal(2);
        pG->Print();

        pG = &ccc;
        pG->SetVal(3);
        pG->Print();

        cin.get();

        return 0;
}

[nazavrik]

Покажите мне хоть одну книгу где в дочерних классах используется слово virtual для виртуальных методов.

Дж.Либерти "Освой самостоятельно С++ за 21 день"(2008), с.407-408 =)

Спасибо за пример! Как разберусь, отпишусь что да как.

[profi]

Освой самостоятельно С++ за 21 день

Я не читаю книги типа: "Самоучитель С++ для чайников" или "Освой С++ за неделю" и т.д. т.п и вам не советую. Лучше почитайте книги С. Прата и Г.Шилдта. Вот в них действительно хорошо объясняется все.

[pu4koff]

Сообщение от profi

Покажите мне хоть одну книгу где в дочерних классах используется слово virtual для виртуальных методов.

Покажите мне хоть одну книгу, в которой уделяется внимание оформлению кода. Я лично не видел ни одной книги, по которой можно учиться писать код не для университетских лабораторных. Ни необходимой обработки исключений, ни слежения за утечками памяти, а уж про наличие/отсутствие const, virtual и т.п. и говорить не приходится.
У обоих вариантов свои плюсы и минусы, а потому называть один из них ошибочным мне видится некорректным.

Сообщение от profi

Компилятор хоть это пропускает, но все равно лучше так не делать.

Почему? Что не так в этих "лишних" virtual?