nahuelDolian / FMI

0 stars 0 forks source link

El 20 no debería ser un 2? #4

Open natoszme opened 6 years ago

natoszme commented 6 years ago

https://github.com/nahuelDolian/FMI/blob/cd4a28b04b55bdc5d89223e8063c7f456db72017/FMI.hs#L31

O me estoy perdiendo de algo? Además: la función debería tener un nombre vinculado a que se reduce la deuda en 2 millones, no uno genérico. No es que la función reciba por parámetro cuánto se va a descontar

Una cosilla más: eso no está haciendo 20 - deuda, en vez de deuda - 20? (y además el 20, debería ser un 2)

nahuelDolian commented 6 years ago

si! ahi le cambié el nombre y el orden de los parametros! me parece que el tema de disminuir deuda en 2 millones es super específico, y por un tema de reutilizacion, estaría bueno que sea para disminuir en X cantidad siempre, como vos decis. en estos casos, que decis? hago una funcion genérica que reciba un monto a disminuir o dejo disminuirDeuda2EnMillones? (es algo subjetivo, a veces me han dicho que haga una generica a veces me han dicho que no me aleje demasiado de los req del enunciado

natoszme commented 6 years ago

Si vas a reutilizar, si hacela genética y pasale el 2 en este caso como parámetro. Pero sino, bastaba con cambiarle el nombre para que no tuviera el nombre de algo genérico cuando en realidad no lo era