Escuela de Ingeniería en Computación Smell Code

Instituto Tecnológico De Costa Rica

Escuela de Ingeniería en Computación

Smell Code

Alexander Castillo Bruno

Carnet: 2013013677

Profesora: Ericka Solano

Diseño de Software

03/16

1. Opacidad

[pic 1]

El olor que se presenta en el siguiente código es que el mismo no se entiende para que sirve, es difícil de e interpretar el problema que soluciona.

Una forma para solucionar el olor es poner nombres significativos a las variables, ya que esta muy difícil interpretar que hace este código.

Primer examen/segundo semestre/ Intro

[pic 2]

Al igual que en el anterior caso, se da una ambigüedad de lo que realmente realiza el código, ya que no cuenta con nombres de variables significativos, al igual que el nombre de la función es ambiguo.

Al igual que en el caso anterior,  se deben renombrar las variables con nombres significativos y también renombrar la función con un nombre que describa de mejor manera lo que realiza el código.

Quiz 2/Segundo semestre/ Taller

2. Repetición Innecesaria

[pic 3]

El olor esta presente en la repetición innecesaria de código dentro del “if” y del “else”.

Lo que se puede hacer para solucionar este problema es sacar el código repetido de los “if” y “else”, ya que este código no pertenece a los “if”.

Tarea 3/ Segundo Semestre / Intro

[pic 4]

Este código ejemplifica perfectamente la repetición innecesaria ya que se dan muchas comparaciones innecesarias que entorpecen el leer y entender este código.

Este código se podría solucionar utilizando una lista de listas la cual se recorre y se van realizando las comparaciones dentro de la misma lista de listas.  

Examen 2/ Segundo Semestre / Intro

3. Complejidad Innecesaria

[pic 5]

El olor que presenta este código es que se le agrega un else al código el cual es totalmente innecesario.

El olor se soluciona fácilmente tratable, lo único que se debe hacer para eliminarlo es quitar el else, este es completamente innecesario.  

Tarea 11/ Segundo Semestre / Taller

[pic 6]

Este código representa perfectamente lo que es complejidad innecesaria ya que se crea una nueva lista de Individuos pero la lista nunca es utilizada.

Este problema se soluciona simplemente borrando esa linea de código, ya que al no utilizarse es lo mismo dejarla que quitarla. Pero sin duda esto ayudaría a no tener complejidad innecesaria.  

Progra 1/ Tercer Semestre / POO

4. Rigidez

[pic 7]

Este código presenta el olor de la rigidez, este no es tan notorio a simple vista, pero si nos ponemos a ver el código podría ser mejorado declarando todo como object, para abstraer aun mas el modelo.

Como dije anteriormente, la solución a este problema es declarar todos los atributos que recibe que función como objects.

Primer Proyecto/ Tercer Semestre / POO

[pic 8]

Este código presenta mas fuertemente el olor de la rigidez, ya que esta completamente alambrado a cierto formato del texto, si por ejemplo el archivo viniera con una coma de mas u otra cosa este código empezaría a cometer errores serios, por lo que abría que encontrar y corregir el archivo manualmente.

La solución a este olor es simplemente hacerlo mas robusto, ponerle mas  “else” . Pensar todas las posibles fallas que podría tener el archivo, para que el programa no genere errores.

...