Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring #2

Open
AndresBorbor opened this issue Jan 14, 2023 · 0 comments
Open

Refactoring #2

AndresBorbor opened this issue Jan 14, 2023 · 0 comments

Comments

@AndresBorbor
Copy link

Se realizaron cambios en el código para mejorar la mantenibilidad.
Duplicate Code
En las siguientes líneas de código 84-95 notar que hay partes del código que se repite, por lo tanto nos podemos dar cuenta que este es un claro caso de duplicate code.

Solución
Extract method
Se realizó esta refactorización ya que se encontró código repetido, por lo tanto es mucho mas conveniente sacar ese código repetido y ya utilizarlo en cada caso en el que se necesite.

Hide method
Se procedió a hacer este Refractor ya que este método solo se utiliza en esta clase, por lo tanto lo hacemos privado evitando futuros conflictos, escondiendo este método por seguridad.

Long Method
Como nos podemos dar cuenta en las líneas de código 404-436, este método searchFlight contiene una gran cantidad de líneas de código, sin mencionar la complejidad que puede tener este código ya que a simple vista podemos ver varios condicionales anidados, lo cual ya es un mal síntoma. Por ende identificamos este code smell Long method, esto debido a la cantidad de líneas de código que tiene.

Solución
Utilizar las tecnicas de refactorizacion preserve whole object y rename method, se hicieron uso de estas dos técnicas para refactorizar lo que en un principio era un long method, y ya una vez refactorizado podemos notar la gran diferencia que existe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant