¿Cómo obtener comentarios de la comunidad sobre grandes porciones de código?

8

Revisión de código: SE es excelente cuando necesita comentarios sobre un código corto y preciso.

Pero dónde obtener comentarios similares sobre el código en sí mismo cuando:

  • tienes miles de LOC,

  • no tiene colegas en su lugar de trabajo listos o dispuestos a revisar el código¹,

  • ¿No tiene miles de dólares para gastar en una revisión profesional de un desarrollador externo? ²

Lugares como CodePlex son una buena idea para dar a conocer su proyecto³, pero por lo que he visto, los comentarios que recibe sobre los proyectos conocidos son comentarios de los consumidores, es decir, se refieren a errores y solicitudes de características, no a la calidad del código fuente en sí.

¿Cuáles son las formas sociales de involucrar a la comunidad en la revisión del código de la base de código de cierto tamaño para un proyecto de código abierto que no tiene la escala de Firefox o productos similares?


¹ Cuál es el caso para la mayoría de los proyectos personales y de código abierto, o proyectos realizados en compañías donde la práctica de revisión de código regular y completa no existe.

² Lo cual es, nuevamente, el caso para la mayoría de los proyectos personales y de código abierto.

³ Incluso si nunca se conocen demasiados proyectos publicados en CodePlex, ya sea porque a nadie le importa o porque no se presentan muy bien.

Arseni Mourzenko
fuente
3
letra pequeña me pone nervioso!
AnthonyBlake
2
¿Qué hace que su proyecto sea lo suficientemente interesante como para que otros quieran verlo?
@ Thorbjørn Ravn Andersen: incluso con proyectos lo suficientemente interesantes, el problema persiste. StyleCop en CodePlex, por ejemplo, es indudablemente interesante, pero aún así el rastreador de problemas está lleno de "comentarios de los consumidores": informes de errores y solicitudes de funciones, nada relacionado con la revisión del código.
Arseni Mourzenko

Respuestas:

6

No hay una forma en las redes sociales para hacer que las personas dediquen horas de su tiempo a revisar el código de algún proyecto aleatorio ... o hacer cualquier otra cosa, de hecho. Minutos que puede tener, horas no tanto.

Entonces, tienes dos posibilidades:

1) Divídalo en trozos de minutos. NO necesita revisar TODAS sus miles de líneas de código. Extraiga los bits más importantes y solicite una revisión de uno o dos de ellos. Probablemente obtendrá algún tipo de comentarios generales de estilo / no-hacer-eso / nombrar-cosas-como-esto, que luego puede aplicar a toda la base de código. Si tiene MUCHOS bits ... bueno, pode más, y tal vez solicite una revisión de uno o dos por semana, y repita en función del comentario que reciba. Cuando todos los comentarios son cosas que ha visto antes y decidió no cambiar de todos modos, ya está.

2) Hacer que el proyecto sea importante para las personas. No creo que puedas hacer esto realmente, pero si tienes un código que realmente resuelva la crisis financiera griega, ganas.

mjfgates
fuente
3

Pregunta difícil; No estoy seguro de que haya una respuesta sólida.

Creo que su pregunta tiene dos partes:
primero, ¿dónde puede publicar el anuncio solicitando la revisión?
En segundo lugar, cómo encontrar personas que estén dispuestas a dedicar el tiempo necesario para realizar la revisión.
Están relacionados, pero son sutilmente diferentes.

El segundo podría ser más fácil que el primero para responder: tanto las comunidades de P.SE como las de CR.SE deberían tener personas que estarían dispuestas a dar un paso adelante y ayudar con eso. Personalmente, encuentro que casi siempre aprendo algo de una revisión, así que los encuentro gratificante. Las motivaciones varían, por lo que si se trata de un área interesante de código que puede ayudar a identificar a alguien o varias personas.

El primero no es tan fácil de responder. P.SE está fuera ya que la solicitud no cumple con el formato de preguntas y respuestas. CR.SE tampoco parece adecuado para las cosas a menos que las hayas dividido en fragmentos más pequeños (módulos / funciones) y tuvieras muchísimas publicaciones. Por otro lado, no conozco ningún sitio en el que pueda publicar ese tipo de solicitud. Si el sistema SE tuviera un sistema de correo de persona a persona, esa podría ser una forma de reclutamiento, pero estoy hablando de eso.

Dado que una revisión fuera de banda de varios miles de locs tomará un tiempo, alentaría la paciencia en el proceso.

En resumen, solo tiene que encontrar un desarrollador desarrollador que esté dispuesto a realizar la revisión.


fuente
FWIW, agradezco sus contribuciones a P.SE y estaría encantado de darle algunas horas de revisión a cambio. La única advertencia es si conozco el idioma lo suficientemente bien.
1

Tomaría una primera puñalada usando algo como stylecop y fxcop o equivalentes para su idioma. A continuación, puede documentar la arquitectura y obtener comentarios al respecto muy rápidamente. Finalmente, podría tomar algunos fragmentos problemáticos o algunas clases y pedirle a un desarrollador que lo revise con usted en lugar de toda la base de código. Siempre que no haya cambiado el estilo del código por completo, algunas clases deberían dar una idea de la base del código en su conjunto.

Peter Smith
fuente