Solución ejercicio Fibonacci - Historial de revisiones

Pneira: /* Identificación y Análisis del Problema */ mejoras

2013-11-05T20:54:28Z

‎Identificación y Análisis del Problema: mejoras

Pneira: /* Preparación */ mejoras

2013-11-05T20:52:26Z

‎Preparación: mejoras

Ferramgon1: /* Reporte en Mantis */ no se visualizaba el cambio en la edición de la imagen. Se a creado otra ruta.

2013-10-24T21:16:51Z

‎Reporte en Mantis: no se visualizaba el cambio en la edición de la imagen. Se a creado otra ruta.

Ferramgon1: Se ha añadido la discusión de la solución, falta añadir generar el path del cambio y subir la solución en mantis.

2013-10-24T21:14:24Z

Se ha añadido la discusión de la solución, falta añadir generar el path del cambio y subir la solución en mantis.

Ferramgon1: Identificación y reporte del problema. Falta la solución y la diferencia incremental usando git.

2013-10-24T02:05:35Z

Identificación y reporte del problema. Falta la solución y la diferencia incremental usando git.

Página nueva

===Preparación===
Empecemos primero haciendo una copia del proyecto: <code> git clone https://github.com/bitsandbooks/py-fibonacci.git </code>
Una vez ya clonado, vamos a la carpeta donde se ha descargado <code> cd py-fibonacci </code> y abrimos el archivo <code> gedit fibonacci.py </code> para analizarlo con detalle.

===Identificación y Análisis del Problema===
Hacemos unas cuantas pruebas para verlo funcionar:

<source lang="python">
>>python fibonacci.py
Traceback (most recent call last):
File "fibonacci.py", line 11, in <module>
sys.argv[1] # The first argument after the program name.
IndexError: list index out of range

>>python fibonacci.py 8
[0, 1, 1, 2, 3, 5, 8, 13]

>>python fibonacci.py 8 9 10
[0, 1, 1, 2, 3, 5, 8, 13]

</source>

Como podemos ver, el problema es que no se controla el número de parámentros de entrada, simplemente lee el primer parámetro que recibe.

<source lang="python">
times = int( # Force the parameter to be an integer.
sys.argv[1] # The first argument after the program name.
)
</source>

=== Reporte en Mantis ===
Ahora que ya ha sido identificado el problema es momento de reportarlo en [https://1984.lsi.us.es/mantis Mantis].
Vamos a la sección de '''Report Issues'''y seleccionamos el proyecto '''Fibonacci'''. Luego rellenamos la plantilla y para generar el reporte le damos a '''Submit Report'''.
[[Archivo:Reporting%20Issues.png]]

Como bien sabemos, mientras más información aportemos sobre un problema, será mucho más rápido poder encontrarle una solución. Pero tengamos en cuenta también que la información debe de ser clara, precisa y concisa. Una buena práctica es incluir capturas de pantalla en ella.

← Revisión anterior		Revisión del 21:14 24 oct 2013
Línea 35:		Línea 35:

	Como bien sabemos, mientras más información aportemos sobre un problema, será mucho más rápido poder encontrarle una solución. Pero tengamos en cuenta también que la información debe de ser clara, precisa y concisa. Una buena práctica es incluir capturas de pantalla en ella.		Como bien sabemos, mientras más información aportemos sobre un problema, será mucho más rápido poder encontrarle una solución. Pero tengamos en cuenta también que la información debe de ser clara, precisa y concisa. Una buena práctica es incluir capturas de pantalla en ella.
		+
		+	=== Solución del problema ===
		+	Una vez ya visto cuál es el problema, planteemos soluciones a este problema y busquemos cuál sería la más indicada.
		+
		+	Una posible e intuitiva solución sería añadir en un condicional al inicio, es decir, si se recibe un único parámetro, la ejecución continúe normalmente, en caso contrario, si no recibiera parámetros o si recibe más de uno, se mostraría un mensaje de error indicando que debe sólo funciona con un sólo parámetro.
		+	<source lang="python">
		+	if len(sys.argv)!=2 :
		+	print "Sorry, it works only with one paremeter."
		+	else :
		+	times = int( # Force the parameter to be an integer.
		+	sys.argv[1] # The first argument after the program name.
		+	)
		+
		+	# Define a function (fib) which takes one parameter (t).
		+	def fib(t):
		+	f = [ 0, 1 ]
		+
		+	for i in range(len(f),t):
		+	fn = f[len(f)-1] + f[len(f)-2]
		+	f.append(fn)
		+
		+	print f
		+
		+	fib(times) # Call function fib and pass the value of times as its parameter.
		+	</source>
		+
		+
		+	Otra posible solución sería incluir un condicional donde si no se cumple la condición de recibir un único parámetro, muestre un mensaje de error informando al usuario de las restricciones y luego abortar la ejecución.
		+	<source lang="python">
		+	if len(sys.argv)!=2 :
		+	print "Sorry, it works only with one parameter"
		+	sys.exit()
		+
		+	times = int( # Force the parameter to be an integer.
		+	sys.argv[1] # The first argument after the program name.
		+	)
		+
		+	# Define a function (fib) which takes one parameter (t).
		+	def fib(t):
		+	f = [ 0, 1 ]
		+
		+	for i in range(len(f),t):
		+	fn = f[len(f)-1] + f[len(f)-2]
		+	f.append(fn)
		+
		+	print f
		+
		+	fib(times) # Call function fib and pass the value of times as its parameter.
		+	</source>
		+
		+	Obteniendo en ambas la funcionalidad esperada:
		+	[[Archivo:ejecucion%20fibonnaci.png]]
		+
		+
		+	Para poder elegir qué solución es la más adecuada para tener un buen seguimiento, veremos el impacto que tiene cada una usando git.
		+
		+	Realizamos el commit de la primera solución con: <code>git commit fibonacci.py</code> y explicamos qué es lo que hicimos.
		+	[[Archivo:git%20commit.png]]
		+
		+
		+	Ahora veremos el registro de cambios en git, usando <code> git log </code>, ubicamos el identificador del commit que hicimos según el título que pusimos al realizar el commit. Luego usando <code> git show TU_IDENTIFICADOR </code> podemos ver qué cambios se han efectuado en el commit.
		+	[[Archivo:git-show-identificador1.png]]
		+
		+	Como podemos ver, hay demasiados cambios aunque a simple vista el código parecía haberse mantenido, esto es debido a que el lenguaje python exige tener un estructura de jerarquía en el código y para tenerlo había que modificar todas las líneas que tenían que ir dentro del <code> if </code>y <code> else </code>. Esto no quiere decir que la solución sea incorrecta, sino que al momento de revisar los cambios se vuelve demasiado engorroso para la vista.
		+
		+	Hacemos lo mismos pasos con la segunda solución y obtenemos:
		+	[[Archivo:git-show-id2.png]]
		+
		+	A esta solución es mucho más fácil seguirle sus cambios, ya que presenta menos cambios y sobre todo no tiene cambios redundantes como la primera solución.
		+
		+	Concluimos en que hay que buscar siempre una solución la cual involucre muy pocos cambios sintácticos en el código, los justos y necesarios, ya que de esta forma estaremos aprovechando mucho mejor el seguimiento de cambios usando git.

@@ Línea 7: / Línea 7: @@
 Una vez ya clonado, vamos a la carpeta donde se ha descargado <code> cd py-fibonacci </code> y abrimos el archivo <code> gedit fibonacci.py </code> para analizarlo con detalle.
-===Identificación y Análisis del Problema===
+===Identificación y análisis de los problemas en el software===
-Hacemos unas cuantas pruebas para verlo funcionar:
 <source lang="python">
@@ Línea 16: / Línea 17: @@
      sys.argv[1] # The first argument after the program name.
 IndexError: list index out of range
 >>python fibonacci.py 8
 [0, 1, 1, 2, 3, 5, 8, 13]
 >>python fibonacci.py 8 9 10
 [0, 1, 1, 2, 3, 5, 8, 13]
 </source>
-Como podemos ver, el problema es que no se controla el número de parámentros de entrada, simplemente lee el primer parámetro que recibe.
+Como podemos ver, el software no valida los parámetros de entrada adecuadamente:
 <source lang="python">

@@ Línea 1: / Línea 1: @@
-===Preparación===
+=== Obtención de clon del repositorio del ejercicio ===
-Empecemos primero haciendo una copia del proyecto: <code> git clone https://github.com/bitsandbooks/py-fibonacci.git </code>
 Una vez ya clonado, vamos a la carpeta donde se ha descargado <code> cd py-fibonacci </code> y abrimos el archivo <code> gedit fibonacci.py </code> para analizarlo con detalle.

@@ Línea 32: / Línea 32: @@
 Ahora que ya ha sido identificado el problema es momento de reportarlo en [https://1984.lsi.us.es/mantis Mantis].
 Vamos a la sección de '''Report Issues'''y seleccionamos el proyecto '''Fibonacci'''. Luego rellenamos la plantilla y para generar el reporte le damos a '''Submit Report'''.
-[[Archivo:Reporting%20Issues.png]]
+[[Archivo:reportingAbug.png]]
 Como bien sabemos, mientras más información aportemos sobre un problema, será mucho más rápido poder encontrarle una solución. Pero tengamos en cuenta también que la información debe de ser clara, precisa y concisa. Una buena práctica es incluir capturas de pantalla en ella.