Question

I’m currently doing it this way, but seems to not be the proper way:

class Name
{
    protected $jobs2do;

    public function __construct($string) {
        $this->jobs2do[] = $this->do;
    }

        public function do() {
        ...
    }
}

Because directly assign a function will cause warning, should do something like:

function func()
{
...
}

$func_array[] = 'func';

say, put it into a string, but I don’t know how to do it when it’s a member function.

Is the following version OK?:

class Name
{
    public $jobs2do;

    public function __construct($string) {
        $this->jobs2do[] = array($this,'do');
    }

        public function do() {
        ...
    }
}

when call it, just:

$instance = new Name();
foreach ($instance->jobs2do as $job)
{
    call_user_func($job);
}

Answer 1

Write a class for Job and use factory pattern to create them. Then Write a class JobManager wihich should maintain a list ob Job instances.

interface iCallableJob
{
    public function run();
}

class Job implements iCallableJob
{
    // The parameterized factory method
    public static function factory($type)
    {
        $classname = 'Job_' . $type;
        if (class_exists($classname) && in_array('iCallableJob', class_implements('Job')))
        {
            return new $classname();
        }
        return null;
    }
    public function run() { return null; }
}

class Job_type1 extends Job 
{
    public function run() 
    {
        echo 'Job 1';
    }
}

class JobManager
{
    protected $jobs2do = array();

    public function addJob($type)
    {
        if ($job = Job::factory($type))
        {
            $this->jobs2do[] = $job;
            return $job;
        }
        return null;
    }
    public function runAll()
    {
        foreach ($this->jobs2do AS $job)
        {
            $job->run();
        }
    }
}