JArrighetti / PdepUTN

MIT License
0 stars 0 forks source link

Repetición de código al comparar usuarios #6

Closed sanuzzi closed 6 years ago

sanuzzi commented 6 years ago

¿No están poniendo lo mismo muchas veces? compararDosNombres (nombre usuarioAlQueSeLeDebeAplicarLaTransaccion) (nombre usuarioAlQueSeLeIntentaAplicarLaTransaccion)

compararDosNombres (nombre usuarioExtraccion) (nombre usuarioAlQueSeLeDebeAplicarLaTransaccion)

compararDosNombres (nombre usuarioRecibeDeposito) (nombre usuarioAlQueSeLeDebeAplicarLaTransaccion)

Y más allá de la repetición: ¿Por qué comparamos por nombre? Porque es la forma actual de saber si hablamos del mismo usuario. Es decir, no nos interesa saber si compara el nombre, el nickname, el dni, o qué. Queremos que diga si dos usuarios son iguales. Que desde afuera no se sepa cómo lo hace.

De esta forma, si el día de mañana se compara por DNI, sus guardas ni se enterarían del cambio.