The Clean Architecture in php

Strong Coupling to the
Db
Class
We’ve moved away from using the deprecated
mysql_
functions and instead have abstracted
away the database into some
Db
class, which is good. Except we still suffer the pitfalls of not
being able to refactor this code. Our same questions as above in A Refactoring nightmare still
apply. We can hardly change anything about our database layer without having to touch a large
amount of files that use that database layer to do so.
Still Hard to Test
We’ve made testing much easier now simply by extracting our code out of the HTML. We now
have a controller doing all the processing, and then passing that data off to the view. This is
much easier to test, as we can simply instantiate
CustomersController
, call the
indexAction()
method, and analyze the return value. But how many customers should we expect, and what are
their names? Again, we can’t know this unless we go the complicated route of setting up a test
database (a known state) before running our tests.
Since we are declaring our
Db
class right in the
indexAction()
method, there’s no way to mock
that. If there were, we could simply set it up to return a known set of customers, and then validate
that the
indexAction()
properly retrieved them.