* Define 11 como una constante, no lo "hardcodees" tres veces.
* Usa { } incluso para los "one-liners"; es una buena practica de estilo y ademas te facilitará la vida cuando estés depurando y tengas que meter más instrucciones en el bloque.
* Usa nombres descriptivos para las variables. wl,c,i,wc son *MUY* malos nombres.
* No uses variables globales, pásalas como parámetros. El array tendría alguna excusa como global, pero el contador no.
* No reutilices variables. En lugar de resetear "counter" a cero, crea una nueva variable. La memoria es barata y así mejoras la legibilidad de tu código.
* No uses ramas condicionales vacís. Reescribe la función "checkout".
* Modulariza mejor. En lugar de contar e imprimir en la función principal, usa dos funciones, una para contar y otra para escribir en la pantalla. En general, una función debe hacer una sola cosa, no varias.
--
En España la mejor manera de guardar un secreto es escribir un libro.
Los cambios fueron: definir 11 como una constante, usar { } para los for, while e ifs de una línea, cambiar los nombres a algunas variables (aunque tanto c como i quedaron igual), definir otra variable para contar, modificar la funcion checkcount y contar e imprimir también en funciones separadas.
La costumbre de reutilizar variables la obtuve programando microcontroladores PIC 12F y 16F donde la memoria si cuesta:P
Revision de codigo muy rapida
(Puntos:3, Informativo)( http://mcpolu.blogspot.com/ | Última bitácora: Miércoles, 05 Marzo de 2014, 00:04h )
* Usa { } incluso para los "one-liners"; es una buena practica de estilo y ademas te facilitará la vida cuando estés depurando y tengas que meter más instrucciones en el bloque.
* Usa nombres descriptivos para las variables. wl,c,i,wc son *MUY* malos nombres.
* No uses variables globales, pásalas como parámetros. El array tendría alguna excusa como global, pero el contador no.
* No reutilices variables. En lugar de resetear "counter" a cero, crea una nueva variable. La memoria es barata y así mejoras la legibilidad de tu código.
* No uses ramas condicionales vacís. Reescribe la función "checkout".
* Modulariza mejor. En lugar de contar e imprimir en la función principal, usa dos funciones, una para contar y otra para escribir en la pantalla. En general, una función debe hacer una sola cosa, no varias.
En España la mejor manera de guardar un secreto es escribir un libro.
Re:Revision de codigo muy rapida
(Puntos:2)( Última bitácora: Martes, 12 Enero de 2010, 19:47h )
Gracias por la revisión. Hice los cambios que sugeriste y aqui está la nueva versión [somekindofpaper.info]. También subí la salida del programa para su propio código fuente [somekindofpaper.info].
Los cambios fueron: definir 11 como una constante, usar { } para los for, while e ifs de una línea, cambiar los nombres a algunas variables (aunque tanto c como i quedaron igual), definir otra variable para contar, modificar la funcion checkcount y contar e imprimir también en funciones separadas.
La costumbre de reutilizar variables la obtuve programando microcontroladores PIC 12F y 16F donde la memoria si cuesta :P
A ver ahora que dejé fuera :P