override a public function to protected

  • Thread starter Thread starter Guest
  • Start date Start date
G

Guest

If I have a class A that has a virtual public function DoSomething()
Class A
{
public:
virtual void DoSomething();
}

then I override this function to protected is class B
Class B: public Class A
{
protected:
void DoSomething();
}

Is this valid? Thanks!

WJ
 
WJ said:
If I have a class A that has a virtual public function DoSomething()
Class A
{
public:
virtual void DoSomething();
}

then I override this function to protected is class B
Class B: public Class A
{
protected:
void DoSomething();
}

Is this valid? Thanks!

WJ
Click to expand...

Ignoring the syntax errors in your code, if your question is whether or not
it's syntactically legal in C++ (VC 8) to make the override protected, the
answer is yes. I'm interested to know what you want to achieve by doing
this, as if there is no class derived from B, it simply makes the override
version unusable.
 
WJ said:
If I have a class A that has a virtual public function DoSomething()
Class A
{
public:
virtual void DoSomething();
}

then I override this function to protected is class B
Class B: public Class A
{
protected:
void DoSomething();
}

Is this valid? Thanks!

WJ
Click to expand...

Should have said "override is unusable outside B if no class is derived from
B".
 
OK, I'll keep on using the example.
in class A, DoSomething() is declared as public, I thought the funciton
should be protected, since it is not used outside.

Class A is in a library, which I can't change, so I wanted to change
DoSomething() to protected in class B.

Thanks for answering the question!

WJ
 
WJ said:
OK, I'll keep on using the example.
in class A, DoSomething() is declared as public, I thought the funciton
should be protected, since it is not used outside.

Class A is in a library, which I can't change, so I wanted to change
DoSomething() to protected in class B.

Thanks for answering the question!
Click to expand...
Well, that's certainly a valid case I hadn't thought of! Thanks for telling
me!
 
pvdg42 said:
Should have said "override is unusable outside B if no class is derived
from B".
Click to expand...

In any case, it's pointless to try to reduce the visibility like that...

B* someB;
static_cast<A*>(someB)->DoSomething();

is perfectly legal.

If you want to hide access to A's members from users of B, then you have to
use non-public inheritance.
 
Back
Top