The Clean Architecture in php

class
CustomersController
{
public function
indexAction
() {
$db
=
Db
::
getInstance
();
$customers
=
$db
->
fetchAll
(
'SELECT * FROM customers ORDER BY name'
);
return
[
'customers'
=>
$customers
];
}
}
<
h2
>
Customers
h2
>
<
ul
>
php
foreach
(
$this
->
customers
as
$customer
)
:
?>

=
$customer
[
'name'
]
?>

endforeach
;
?>

This code looks better. We’re using controllers and views, so we’ve separated the presentation
logic from the control logic. The controller’s
indexAction()
grabs all the customers and then
returns them, which gets passed to the view so it can render the data. This should make the code
much easier to test and refactor. Except it doesn’t.

---

What is Architecture?
10
Still Hard-coding Queries
This is obvious. We still haven’t solved the problems of having hard-coded queries in layers
that don’t concern the database, such as controllers. See the comments above in A Refactoring
Nightmare for more details.