Check if a function exists in a class before calling call_user_func()

You're looking for method_exists for starters. But what you should check, too is whether or not the method is callable. This is done by the helpfully named is_callable function:

if (method_exists($this->controller, $this->view)
    && is_callable(array($this->controller, $this->view)))
{
    call_user_func(
        array($this->controller, $this->view)
    );
}

But that's just the start of things. Your snippet contains explicit require calls, which suggests you're not using an autoloader.
What's more: all you're doing is check file_exists, not if the class was already loaded. Your code, then, will generate a fatal error if, per chance your snippet gets executed twice with the same values for $this->controller.
Begin fixing this by, in the very least, changing your require to require_once...


You can use the PHP function method_exists():

if (method_exists('ClassName', 'method_name'))
call_user_func(etc...);

or also:

if (method_exists($class_instance, 'method_name'))
call_user_func(etc...);

Tags:

Php