Historias
Slashboxes
Comentarios
 
Este hilo ha sido archivado. No pueden publicarse nuevos comentarios.
Mostrar opciones Umbral:
Y recuerda: Los comentarios que siguen pertenecen a las personas que los han enviado. No somos responsables de los mismos.
  • Revision de codigo muy rapida

    (Puntos:3, Informativo)
    por McPolu (19560) <McPolu@gmail.com> el Viernes, 11 Diciembre de 2009, 23:01h (#1190447)
    ( http://mcpolu.blogspot.com/ | Última bitácora: Miércoles, 05 Marzo de 2014, 00:04h )
    * 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.

    Puntos de inicio:    1  punto
    Moderación   +1  
    Modificador extra 'Informativo'   0  
    Modificador por Bonus-Karma   +1  

    Total marcador:   3  
  • por perceptronico (40832) el Sábado, 12 Diciembre de 2009, 00:39h (#1190462)
    ( Ú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

    [ Padre ]