2

Having the following class hierarchy:

class TheParent{

    public function parse(){
        $this->validate();
    }

}

class TheChild extends TheParent{

    private function validate(){
        echo 'Valid!!';
    }
}

$child= new TheChild();
$child->parse();

What is the sequence of steps in which this is going to work?

The problem is when I ran that code it gave the following error:

Fatal error: Call to private method TheChild::validate() from context 'TheParent' on line 4

Since TheChild inherits from TheParent shouldn't $this called in parse() be referring to the instance of $child, so validate() will be visible to parse()?

Note:
After doing some research I found that the solution to this problem would either make the validate() function protected according to this comment in the PHP manual, although I don't fully understand why it is working in this case.

The second solution is to create an abstract protected method validate() in the parent and override it in the child (which will be redundant) to the first solution as protected methods of a child can be accessed from the parent?!!

Can someone please explain how the inheritance works in this case?

4 Answers 4

17

Other posters already pointed out that the mehods need to be protected in order to access them.

I think you should change one more thing in your code. Your base class parent relies on a method that is defined in a child class. That is bad programming. Change your code like this:

abstract class TheParent{

    public function parse(){
        $this->validate();
    }

    abstract function validate();

}

class TheChild extends TheParent{

    protected function validate(){
        echo 'Valid!!';
    }
}

$child= new TheChild();
$child->parse(); 

creating an abstract function ensures that the child class will definitely have the function validate because all abstract functions of an abstract class must be implemented for inheriting from such a class

2
  • pity the guy who actually understands OOP a level beyond the rest is overlooked like this. I would've given the abstract function answer aswell Nov 5, 2013 at 15:26
  • 1
    This is the correct answer. It is not enough to just implement the method in the child class, and crossing fingers in the hope that it will always be so. Ensuring that the child MUST define the method is the way to go! Oct 8, 2014 at 10:23
4

Your idea of inheritence is correct, just not the visibility.

Protected can be used by the class and inherited and parent classes, private can only be used in the actual class it was defined.

2
  • hmmm so when I call $child->parse() in my code it will run the function parse in the parent class and $this inside it will be referring to the parent class instance not the child?
    – Songo
    Oct 17, 2012 at 18:37
  • There is only one instance, but from the perspective of the instance.. private methods of parent classes are simply not allowed to be called. This is the exact purpose and difference between private, protected and public.
    – Evert
    Oct 17, 2012 at 18:43
2

Private can only be accessed by the class which defines, neither parent nor children classes.

Use protected instead:

class TheParent{

    public function parse(){
        $this->validate();
    }

}

class TheChild extends TheParent{

    protected function validate(){
        echo 'Valid!!';
    }
}

$child= new TheChild();
$child->parse();
0

FROM PHP DOC

Visibility from other objects

Objects of the same type will have access to each others private and protected members even though they are not the same instances. This is because the implementation specific details are already known when inside those objects.

Private can only be accessed by the class which defines or Same object type Example

class TheChild {
    public function parse(TheChild $new) {
        $this->validate();
        $new->validate(); // <------------ Calling Private Method of $new
    }
    private function validate() {
        echo 'Valid!!';
    }
}

$child = new TheChild();
$child->parse(new TheChild());

Output

Valid!!Valid!!

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.