Mon, 28 Mar 2011
12:11 AM - Wow
So I'm reading this PHP code today and I can't believe my eyes. There's a Box class that tracks boxes in a warehouse. They have a concept of an open box and a closed box. The latter means it's full and taped and ready to ship. Instead of adding a close method to the Box class, they have a completely different (and somewhat unrelated) class that calls Box::update() after modifying two properties (not setter methods) on the class first to make the closed by (person) and closed date.
To make it more interesting, the "open" command is called insert.