Question

PHP has _get and _set functions built in. Is it better to write my own get and set functions for each variable or use the built in functions with a ton of if else if? What are the pros and cons of each method?

Answer 1

__get and __set are magic methods that should usually be used to solve difficult problems rather than to be used as a design basis.

For instance, I found myself on a project where I had to analyze a site that used OOP with deep inheritance (> 2) and one of the important base classes had a public property called name. However, it also had getters and setters (getName, setName) that accessed that property for the mere purpose of getting and setting it. Many classes called getName and just as many accessed the name property directly! Not great design.

The magic methods allowed me to solve the problem by renaming the property to _name and making it private, forcing all requests to the property through the getters and setters.

This being said, there’s no need for getters and setters if you’re just treating a property like a variable. In that case, just make the property public!

In your case, since there is validating/sanitizing going on, you should employ getters and setters and code them directly as methods (rather than unnecessarily incurring the overhead from the magic methods).