search

DanH957 / ATM-Machine

ATM machine written in Java, with interface.
201 stars 150 forks source link

Posible refactorización Substitute algorithm #17

Open mbravop opened 1 year ago

mbravop commented 1 year ago

Hola, revisando el repositorio en la clase CashDispenser, el método dispenseCash, realiza ciertas operaciones para la entrega de dinero.

image

Considero que el método debe solo realizarse si el ATM cuenta con la cantidad necesaria, es por esto que he refactorizado el código aplicando Substitute Algorithm quedando así:

image
mbravop commented 1 year ago

Posible refactorización Hide Method

Continuando con la refactorización anterior, aplicando Substitute algorithm, es posible darse cuenta que el método que verifica la existencia de billetes disponibles solo será utilizado por el método dispenseCash, por lo que considero que aplicar Hide Method, para poner privado el método isSufficientCashAvailable es válido. Así se evitarán redundancias en el código principal.

image