CODEWITHSUNDEEP
phpdesign-patternssolid-principlessingle-responsibility-principle
Terry Djony
Terry Djony

Reputation: 2015

Single Responsibility Principle Implementation

I have a SQLStatement class which builds up sql query and supports bind parameters (which is stored to the property $params).
I have a PDOAdapter class which acts just like PDO but has additional features such as prepare_query(SQL Statement $sql). It prepares the sql statement and binds the parameter which is stored in $sql->params, then execute it.

And,

I think I want to add a QueryEngine class which can select, insert, update, delete which means it'll build up SQL statement using the SQLStatement class and make PDOAdapter prepare_query it.

I can say its responsibilites are to generate sqlstatement based on the method parameters using SQLStatement class AND make PDOAdapter process it. (Multiple Responsibility)
But, I can also say, its responsibility is to do basic CRUD operations. (Single Responsibility)

Does this QueryEngine class violate Single Responsibility Principle ?
Does this make SQLStatement and PDOAdapter tight-coupled ?
Actually, what is the exact definition of the word Single? I'm confused whether moving a table is single responsibility or multiple responsibility (lift the table up, move it, and put it down).

Upvotes: 0

Views: 226

Answers (1)

Yang
Yang

Reputation: 8701

Actually, what is the exact definition of the Single-Responsibility?

Imagine for a second, that you build computer's case. It contains things like power-supply, motherboard, HDD inside it. And in any programming language, that could be represented like so:

case = new Case(new Motherboard(), new PowerSupply(), new HDD() ...);

As you can see the parts of the case are completely separated from each other. That means they can be easily replaced without touching another parts. For example, if you need to replace HDD, you'd simply pass another instance of HDD to it. That's it. HDD (and similar parts) has one reason to change.

As for your question, what you want to do needs to be Done in so-called Data Mappers, which abstract a specific table with common abstractions on it.

First of all, I'll show you how it's typically done:

$queryBuilder = new QueryBuilder();
$pdo = new PDO(...)

$dbh = new DBH($pdo, $queryBuilder);

// ... Then somewhere in some mapper

public function fetchById($id)
{
   return $this->dbh->select('*')
             ->from('some table')
             ->where('id', '=', $id)
             ->query();    
}

This adheres to the SRP, because each dependency ($pdo or $queryBuilder) can be replaced easily, therefore they have one reason to change.

As for your original question,

QueryEngine (Correct name is QueryBuilder) will not violtate anything as long as it only knows how to build query strings.

Upvotes: 1

Related Questions