views:

353

answers:

3

Hi, guys I know this question is very basic but I've met in few publications (websites, books) different style of override virtual function. What I mean is: if I have base class:

class Base
{
public:
virtual void f() = 0;
};

in some publications I saw that to override this some authors would just say:

void f();     

and some would still repeat the virtual keyword before void. Which form of overwriting is in good style? Thank you for your answers.

+8  A: 

This is purely a matter of taste. Some weak arguments can be made back and forth as to the self-documentation value of some styles versus the non-redundancy of others.

bmargulies
Nice to see the first answer is the best one. Note also the new standard adds some decorators to doc and check just why you are overloading,overriding,hiding,etc
Lance Diduck
+1  A: 

It is not necessary to add the virtual keyword to a method you override in a subclass as this qualifier can not be removed by omitting it in subclass declarations.

It is however good practise to repeat the virtual keyword as it documents which derived functions are virtual in the base class.

This is also recommended by the 'High Integrity C++ Coding Standard Manual'.

which is linked in the C++ FAQ.

MKroehnert
A: 

I personally use both styles: virtual means that the method can be overridden.

So we have two cases:

  • if you wish the user of your class to know that the method may be overridden, use the virtual keyword
  • if you would prefer to hide this fact (because you provide new methods for example following the Template Pattern), then do not use the virtual keyword

I guess it's yet another way of (ab?)using the keyword.

Matthieu M.