Comments on: Lazy loading of object variables in php using __get()

By: Newbie

Newbie — Thu, 06 Aug 2009 08:47:24 +0000

Great article, and informative comments to go with it.

JT — Thu, 25 Dec 2008 20:29:11 +0000

“Whoa! I didn’t realise anyone read this blog”

You’re 10th hit on google for “php lazy load”

cormacscode — Mon, 11 Aug 2008 13:03:18 +0000

Whoa! I didn’t realise anyone read this blog

Anyway, ok, I shouldn’t have used eval(). My bad.

Scott Mattocks — Mon, 11 Aug 2008 12:28:16 +0000

Don’t use eval. You are just asking for trouble. Instead you should first check that the method exists and then call it using call_user_func().

if (is_callable(array($this, ‘_load’ . $var)))
{
call_user_func(array($this, ‘_load’ . $var));
}
return $this->$var;

Trii — Sun, 10 Aug 2008 17:41:10 +0000

Yeah your blog killed my opening ?php,

$method = ‘_load’ . $var;
$this->$method();

Trii — Sun, 10 Aug 2008 17:40:15 +0000

You know you can remove the eval() call and replace it with:

$method();
?>

iongion — Sat, 09 Aug 2008 19:04:43 +0000

Why don’ you just use

public function __get($var)
{
$loader = ‘_load’.$var;
$this->$loader();
return $this->$var;
}

it should be much faster than using eval and much portable without the placeholders {}

Matthew Turland — Sat, 09 Aug 2008 16:29:51 +0000

eval() is evil. This is a better way:
$this->{‘_load’ . $var}();
return $this->$var;

Why eval() ? « Sticky Tape — Sat, 09 Aug 2008 08:14:14 +0000

[...] In response to Lazy loading of object variables in php using __get() [...]

Fri, 08 Aug 2008 21:24:17 +0000

[...] Cormac posted this look at a method for lazy loading on variables in an object with the magic __get method. I used the magic [...]