Question

Do you think it’s a good idea?

Let’s say you have an application component that is used by other components to retrieve / update data in the db. It’s basically a class with get(), set(), update() methods.

Would it be a good idea for that component to update (or set) data only in it’s properties when called, and on __destruct to update the db as well? Or should it directly update the db on each set/ update call ?

Answer 1

Updating the database on object destruction smells to me a little bit like a software side effect. That is, an action that takes place in an unexpected and somewhat non-explicit place. It would not be obvious from viewing your code that a database action is happening when __destruct() is called, even if you call it explicitly. Future code maintainers (including yourself) could be easily confused when trying to hunt down a bug involving inconsistent data, but not seeing any calls to the database or method calls resembling data interactions when viewing the code.

I would advise against it.