Preguntas etiquetadas con code-reviews

Nueva publicación solicitada de StackOverflow: Estoy trabajando en un pequeño tiempo de desarrollo con un tiempo muy limitado para el desarrollo. Desarrollamos una herramienta que es importante para el resultado de nuestro trabajo, pero que no se usa a diario. Soy la única persona en el equipo que tiene experiencia …

10 code-quality  collaboration  code-reviews 

Cerrado. Esta pregunta está fuera de tema . Actualmente no está aceptando respuestas. ¿Quieres mejorar esta pregunta? Actualice la pregunta para que sea sobre el tema de Software Engineering Stack Exchange. Cerrado hace 5 años . Al inspeccionar manualmente un código desconocido (para revisar o modificar), parece que tengo tres …

9 tools  code-reviews  source-code  maintenance 

Solo quería saber cuál es la diferencia entre el análisis de código estático y la revisión de código. ¿Cómo se hacen cada uno de estos dos? Más específicamente, ¿cuáles son las herramientas disponibles hoy para la revisión de código / análisis estático de PHP? También me gustaría saber sobre buenas …

9 code-quality  terminology  code-reviews  quality 

¿Cuáles son los problemas de diseño al devolver una tabla hash de un método público cuando desea devolver varios elementos en lugar de crear una clase y devolver un objeto de eso? Si tiene problemas, ¿en qué circunstancias tiene sentido hacerlo? ¿Cómo cambia la respuesta a esta pregunta dependiendo de …

9 design  object-oriented  code-reviews 

Me han pedido que cree pautas para revisar CSS, JS y HTML. Sé que hay pautas de codificación para JS, pero no sé nada sobre HTML y CSS. Para revisar JS, ciertamente seguiré esas pautas y las mencionaré. ¿Pero qué pasa con CSS y HTML? Además de los errores lógicos …

9 javascript  html  code-reviews  css 

Hoy, en el trabajo, uno de mis colegas revisó mi código y sugirió que eliminara una propiedad de solo conjunto y utilizara un método. Como ambos estábamos ocupados con otras cosas, me dijo que mirara la Property Designsección del libro "Pautas de diseño del marco". En el libro, el escritor …

9 design  code-quality  code-reviews  code-smell 

¿Cuál es el mejor proceso para la revisión de código cuando se usa GIT? Tenemos un proveedor externo de GIT (Unfuddle) y tenemos límites en el uso de recursos, por lo que no podemos tener repositorios remotos dedicados para cada desarrollador. Proceso actual: Tenemos un servidor GIT con una masterrama …

9 git  code-reviews 

Mi equipo rara vez revisa el código, principalmente porque no tenemos suficiente tiempo y las personas carecen de la energía y la voluntad para hacerlo. Pero realmente me gustaría saber qué piensan las personas sobre mi código cuando lo leen. De esta manera, entiendo mejor cómo piensan otras personas y …

8 code-quality  code-reviews 

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 …

8 coding-style  code-reviews  community  social  feedback 

En mi experiencia, la mayoría de los equipos de la compañía revisan el código del miembro del equipo con una pequeña cantidad de código, siempre menos de cientos de líneas. ¿Es apropiado revisar una gran cantidad de código, por ejemplo, un módulo, cuando el código está completo y listo para …

8 code-reviews 

Ahora estoy aprendiendo Gerrit (que es la primera herramienta de revisión de código que uso). Gerrit requiere un cambio revisado que consista en una sola confirmación. Mi rama de características tiene aproximadamente 10 confirmaciones. La forma preferida de gerrit es aplastar esos 10 commits en uno solo. Sin embargo, de …

8 git  code-reviews  gerrit 

¿Existe una herramienta que tome como entrada un parche propuesto y un repositorio git e identifique que los desarrolladores son los mejores candidatos para revisar el parche? Usaría el historial de git para identificar a los autores que tienen más experiencia con los archivos / secciones de código que se …

8 git  code-reviews 

Hace aproximadamente 3 meses, nuestro grupo de ingeniería lanzó la Junta de Revisión para ser utilizada en todas las revisiones de código de pares. Hoy, tuve una discusión con una de las personas involucradas en ese proceso y descubrí que ya estamos buscando un reemplazo (posiblemente algo comercial) debido a …

8 development-process  code-reviews 

Trabajo con muchos códigos heredados de Java y RPG en una aplicación interna de la compañía. Como es de esperar, gran parte del código está escrito en muchos estilos diferentes y, a menudo, es difícil de leer debido a las variables mal nombradas, el formato inconsistente y los comentarios contradictorios …

7 code-quality  coding-standards  code-reviews