Ejemplo de Single Responsability Principle (SRP)

A continuación se muestra un sencillo snippet de código por el que se inserta un registro en la tabla de usuarios (he simplificado lo suficiente como para mostrar cómo aplicar el principio SRP, este ejemplo es de juguete):

var mysql = require('mysql');

function addUser( user, dbConfig ) {
   var dbConnection = mysql.createConnection( {
				host : dbConfig.dbHost,
				user : dbConfig.dbUser,
				password: dbConfig.dbPwd,
				database : dbConfig.dbName
			});

   var sqlQuery = mysql.format("INSERT INTO users (mail, password, alias) VALUES (?, ?, ?)",
		 	       [user.loginMail, user.loginUserPassword, user.loginAlias] );

   return dbConnection.query( sqlQuery );
};

De acuerdo, la función addUser() puede que funcione, pero está violando el principio SRP, porque se encarga de tres responsabilidades diferentes:

• Crear la conexión con la base de datos.
• Construir la consulta sql.
• Ejecutar la consulta.

De este modo, estamos dando pie a que cuando añadamos otra función de manipulación de la base de datos, repitamos en él la creación de la conexión de la base de datos, etc. O sea, que lo anterior nos deja poco margen para la reutilización y sin duda duplicaremos mucho código.

Una mejor más alineada con SRP sería la siguiente.

var util = require('util');

function getDBConnection( dbConfig ) {
   return mysql.createConnection( {
 			        host : dbConfig.dbHost,
				user : dbConfig.dbUser,
				password: dbConfig.dbPwd,
				database : dbConfig.dbName
			});
}

function execQuery( sqlQuery, dbConfig ) {
   var dbConnection = getDBConnection( dbConfig );

  return dbConnection.query( sqlQuery );
}

function buildInsertQuery( entity, fields, values ) {
   return util.format( "INSERT INTO %d (%s) values [%s]", entity, fields.join(','), values.join(',') );
}

function addUser( user, dbConfig ) {
   var sqlQuery = buildInsertQuery( 'users', 
  			            ['mail', 'password', 'alias']
				    [user.loginMail, user.loginUserPassword, user.loginAlias] );
   return execQuery( sqlQuery, dbConfig )
}

Se puede pensar que ahora el código es mayor, pero:

• la función addUser() es mucho más pequeña: en dos líneas inserta los datos del nuevo usuario en la base de datos. Deja a otras funciones la creación de la fontanería necesaria para ello.
• Se han creado algunas funciones reutilizables, como buildInsertQuery(), getDBConnection() y execQuery(), que, en caso de tratarse de un proyecto real, serían fácilmente reutilizables.
• Ahora este código es más fácil de probar en tests automatizados.
• Descoplamos más la solución de la infraestructura subyacente: si el proceso de creación de la conexión de la base de datos cambia, tan solo hay que modificar getDBConnection(), por poner un ejemplo.

Esto es solo un ejemplo para mostrar SRP en acción, por supuesto se echan en falta DIP y muchas otras cosillas...