Are Magic Methods Best practice in PHP?
4 Answers
cons
Text searches don't find the functions
System is harder to understand, especially for newcomers
Refactoring tools might fail more often
Generally, the magic methods do things behind the scenes and the programmer might not realize it's happening which makes debugging harder.
When searching for the functions (or other symbols) can't find all the matches it becomes a nightmare to remove old code and this fear can cause dead code to pile up in the codebase. If the dead code is removed, it can cause breakage in unknown places.
I don't think magic methods are best or worst practice: depending on what you want to achieve you can use them or not... What I mean is that you don't have to tweak your code as possible to use them, but if you have to there is no problem at all.
If you have an object with 3 and only 3 attributes you don't need to use magic setters/getters, but in some advanced cases they are a great way to do very complex things (ORM systems etc...)
Maybe some of them are deprecated, I don't know, but most of them are not.
At least, some of these magic functions are recommended by Google:
Avoid writing naive setters and getters
When writing classes in PHP, you can save time and speed up your scripts by working with object properties directly, rather than writing naive setters and getters. In the following example, the dog class uses the setName() and getName() methods for accessing the name property.
class dog { public $name = ''; public function setName($name) { $this->name = $name; } public function getName() { return $this->name; } }
Notice that setName() and getName() do nothing more than store and return the name property, respectively.
$rover = new dog(); $rover->setName('rover'); echo $rover->getName();
Setting and calling the name property directly can run up to 100% faster, as well as cutting down on development time.
$rover = new dog(); $rover->name = 'rover'; echo $rover->name;
Original link: http://code.google.com/speed/articles/optimizing-php.html
Archived version: https://web.archive.org/web/20120208060457/http://code.google.com/speed/articles/optimizing-php.html
Anyway, these methods might not be performant, but they ain't deprecated at all.
-
2and here, another opinion: garfieldtech.com/sites/default/files/benchmarks.png :D (see garfieldtech.com/blog/magic-benchmarks)– fbivilleApr 23, 2010 at 12:01
-
2This answer is completely off-topic. "Google" does not go against using native getters/setters vs. magic functions. They advise against using native getters/setters if they only directly modify the property. This is not the same thing. Aug 26, 2013 at 10:28
-
They go against using "naive" getters/setters. The popular answers to the initial question at that time (3 years ago, btw, here is my present to you: xkcd.com/386) were suggesting magic methods were deprecated. My answer was just making a point that this was not necessarily an absolute truth.– fbivilleAug 27, 2013 at 9:05
-
IDE support should be improved to prevent warnings telling method 'x' doesn't exists when a magic function exists ... Mar 2, 2018 at 8:12
I don't think so. My IDE is not able to show me "hints" for magic setter and getters. Altough the code is harder to debug sometimes.
I prefer not using them, better generate needed methods (like many setters and getters) by my ide.
-
2That may apply to getters and setters, but aren't
__construct
,__clone
etc considered "magical methods", too? php.net/manual/en/language.oop5.magic.php How would one work without them?– PekkaApr 23, 2010 at 11:02 -
i++
andgoto
or he's worried they are overused or unreliable. y'know, like using tables for styling.