Aquí el problema que se presenta al llamar a las funciones de forma no estática, es que en algún momento podría pasar que se llamase a un método que no exista.
No obstante el primer error es crear un array de esta forma:
$this->tables = ('a','b','n');
Lo ideal es hacerlo así:
$this->tables = array('a','b','n', 'z'); // <------- mejor así !
$this->tables = ['a','b','n', 'z']; // <------- o así !
La forma rápida:
class MyClass1
{
private $tables;
function __construct()
{
$this->tables = array('a', 'b', 'n', 'z');
}
/**
* Lo rápido
* Pero si no existe el método lanza un error:
* Fatal error: Call to undefined method MyClass::migration_z() in //...
*/
public function runMigrations1()
{
foreach($this->tables as $table)
{
$method = 'migration_' . $table;
$array[] = $this->$method();
}
return $array;
}
private function migration_a()
{
return 'a';
}
private function migration_b()
{
return 'b';
}
private function migration_n()
{
return 'n';
}
}
El problema es que si no existe el método, lanza un error. El cual podríamos controlar mediante:
class MyClass2
{
/**
* La segunda opción para evitar errores
* Comprobar que el método lamado existe mediante getMethod()
*/
public function runMigrations2()
{
foreach($this->tables as $table)
{
$method = 'migration_' . $table;
$array[] = $this->getMethod($method);
}
return $array;
}
private function getMethod($method)
{
if(method_exists($this, $method)):
return $this->$method();
else:
return null;
endif;
}
}
Si se decide seguir con la práctica de usar call_user_func
una forma correcta sería así:
class MyClass3
{
/**
* El problema es igual que runMigrations1()
* Si el método llamado no existe, lanza un error de tipo:
* Warning: call_user_func() expects parameter 1 to be a valid callback,
* class 'MyClass' does not have a method 'migration_z' in //...
*/
public function runMigrations3()
{
foreach($this->tables as $table)
{
$method = 'migration_' . $table;
$array[] = call_user_func(array($this, $method));
}
return $array;
}
}
Los errores, siempre que sean susceptibles de ocurrir, deben ser controlados para seguir con las buenas prácticas.
class MyClass4
{
/**
* Podemos solucionarlo contorlando el error
*/
public function runMigrations4()
{
foreach($this->tables as $table)
{
$method = 'migration_' . $table;
$array[] = $this->getMethodCallUserFunc($method);
}
return $array;
}
private function getMethodCallUserFunc($method)
{
if(method_exists($this, $method)):
return call_user_func(array($this, $method));
else:
return null;
endif;
}
}
Si haces un var_dump()
sobre cualquiera de los runMigrations
verás el resultado que lanza cada uno, a excepción de los que lanzan error por no encontrar el método z
creado a modo error.
$instance = new MyClass1(); // 2, 3, 4
var_dump($instance->runMigrations1()); // 2, 3, 4
Dará un resultado así con los que tiene el control de error
array (size=4)
0 => string 'a' (length=1)
1 => string 'b' (length=1)
2 => string 'n' (length=1)
3 => null