2010-11-05 13 views
14

¿Cuáles son mis mejores opciones para realizar una revisión de código ya que soy un 'ejército de 1' en el trabajo? Lo mejor que he hecho hasta ahora es ejecutar Sonar en mis proyectos, pero estoy pensando que se podría hacer más.Revisión de código cuando se trabaja solo

me gustaría ir más allá de la publicación y la petición de evaluar en pequeños trozos de código. Gracias.

+0

echa un vistazo a http://area51.stackexchange.com/proposals/11464/code-review podemos ponernos en marcha si nos comprometemos lo suficiente. – greatwolf

Respuesta

9

Una técnica que puede probar es ejecutar un diff en cada uno de sus archivos fuente antes de registrarlos. Simplemente inicie su herramienta diff en cada archivo fuente y siga los cambios en su código. Tómese un momento para reflexionar sobre cada cambio tal como lo haría si estuviera revisando el código de otro desarrollador, y brevemente pregúntese:

  • ¿Qué significa este cambio?
  • ¿Cómo se relaciona con el error/mejora/solicitud original?
  • ¿El nuevo código realmente hace lo que se pretendía?
  • ¿Veo algún error?
  • etc ...

Después de un tiempo que se pondrá en el hábito de correr a través de estas revisiones de código rápido, y garantiza que al menos vistazo a cada uno de los cambios más de una vez.

De acuerdo, este método funciona mejor cuando se mantiene el código y se realizan pequeños cambios, es más difícil cuando se cambian bloques grandes de código, pero este nivel de disciplina sigue siendo valioso.

+1

Estoy de acuerdo en que seguir esa recomendación puede ser útil. Tal vez podría ser más de algo que haces como parte de la segunda naturaleza si tienes código bajo control de versiones y tienes la costumbre de ejecutar tus cambios a través de un programa de diferencias antes de comprometerlos. Sé que, incluso cuando sé qué cambios he realizado con precisión, también al menos una vez comprobé la salida de la diferencia antes de comprometer los cambios. . – ayaz

1

solución no técnica: Tal vez usted podría buscar otro 'ejército de 1' y el código de cambio?

Alternativamente, make partes generales de código abierto, súbelas y esperar a que los informes de fallos (que además probablemente dará mejores resultados).

+1

Dudo que "probablemente": su "parte general" necesita ser significativamente útil para otros codificadores, y ese no es el caso natural para la mayoría del código. – gimpf

+0

¿Lo haría si pudiera, alguna sugerencia? – vector

+0

Me cree que el enfoque de fuente abierta funcionaría mejor para proyectos independientes con algún valor para otros. En el trabajo, es un tipo de cosas 'ho-ham' :-( – vector

1

Podría utilizar una herramienta como Resharper de FxCop.

+0

... herramientas interesantes, para mí, aunque es Java o PHP, por el momento, sin embargo, gracias – vector

4

Existen comunidades de revisión de código, vea mi anterior question para algunos ejemplos.

+0

1 para guardar que la apertura de la 'question', sólo hay una respuesta con este sitio: http: //www.refactormycode .com. Se ve bien –

+0

@Chris Knight - Pero obviamente voté mi pregunta de todos modos :) – willcodejavaforfood

+0

... de hecho, refactormycode podría hacerlo :-), ¡definitivamente lo intentarás! – vector

1

No especificó en sus preguntas, pero si usa Java, entonces una configuración Checkstyle muy ajustada le ayudará a mantenerse en línea recta e identificar las áreas donde su diseño es débil y la refactorización sería beneficioso.

+0

... es Java y estoy usando NB, pero el plugin checkstyle es realmente antiguo, pensó que Sonar hace lo mismo. – vector

1

Solo revisando las diferencias, después de algún día como un día, usar una herramienta de revisión de código puede ayudarlo a identificar, rastrear y solucionar problemas.

Cuando esté revisando debe ponerse sombrero de "revisor" e intentar criticar sus cambios lo mejor que pueda.

En mi experiencia personal, la auto revisión de código puede ser tan eficaz como la unidad de pruebas de auto en la identificación de problemas.

Nota: estoy asociado con la empresa que construye el producto vinculado anteriormente.

+1

Revise las [preguntas frecuentes] (http://stackoverflow.com/faq#promotion). "Además, si un gran porcentaje de tus publicaciones incluye una mención de tu producto, claramente estás aquí por las razones equivocadas". 13 de 14 es bastante alto, especialmente para un nuevo usuario. Tus publicaciones ya están marcadas como spam, por lo que debes intentar contribuir de otras formas. –

Cuestiones relacionadas