PHP - Object Oriented Practices and Inheritance

Question

This is the basic class design

class CustomModule {

    public __construct() {  }

    //Run me first automaticly  
    public function exec($str) {  } 

}

class Randomizer extends CustomModule {

    public __construct() {  }

    //Call me
    public function exec($str) {  } 

}

As I am designing a plugin/module system for extern developers I need the CustomModule->exec() to run first, I do not want to leave it up to the devs to have to call base->exec($str).

I want CustomModule::exec() called automaticly before Randomizer::exec() is called without having to put code in Randomizer::exec(). Is This Possible perhaps with magic function?

Tobi · Accepted Answer · 2012-04-25 06:05:27Z

3

In my opinion, i would use this way: Instead of calling _construct in exec of Randomizer, you can define a constructor in Randomizer and call parent::_construct

class CustomModule {

//Run me first automaticly  
public function exec($str) {  } 

public __construct($str) {
   $this->exec($str);
}

}

class Randomizer extends CustomModule {

//Call me
public function exec($str) {  
   parent::__construct($str);
}

}

edited Apr 25, 2012 at 6:05

answered Apr 25, 2012 at 6:00

Tobi

1,4381 gold badge13 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

IEnumerable Over a year ago

"Instead of calling _construct in exec of Randomizer.." I didnt code in a __construct() for the sample. But thanks for the tip. Ill fix my sample. cheers

tereško · Accepted Answer · 2012-04-25 06:10:36Z

2

If your object requires some initialization before you can "release* it into the rest of application, then it means that you need a factory for this. This is how you should be solving it, if you require to call some method only once.

If such execution happens each time you call exec() method, then instead you should be using some sort of containment (in a form of decorator or just you standard composition of objects).

Basically, you need to restructure your code.

answered Apr 25, 2012 at 6:10

tereško

58.5k26 gold badges100 silver badges151 bronze badges

1 Comment

IEnumerable Over a year ago

Yes I know, as per my last post. Was messy and a bit crazy. A sign of needing some sleep. Thankyou

IEnumerable · Accepted Answer · 2012-04-25 06:09:25Z

2

After a bit more thought I realized this is BAD design. I have to keep the code base simple and scaleable and this would only make a mess in large projects.

As the only program calling the Randomizer::exec() is my inhouse built program I can just call CustomModule::exec() on the previous line and get a boolean response to see if it should continue to the next line.

Sorry to have to end this Q' short

answered Apr 25, 2012 at 6:09

IEnumerable

3,85015 gold badges52 silver badges80 bronze badges

Collectives™ on Stack Overflow

PHP - Object Oriented Practices and Inheritance

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related