Descargo de responsabilidad: soy un recién llegado (este es mi tercer día de trabajo), y la mayoría de mis compañeros de equipo tienen más experiencia que yo.
Cuando miro nuestro código, veo algunos olores de código y malas prácticas de ingeniería, como las siguientes:
- Pautas de nomenclatura algo inconsistentes
- Propiedades no marcadas como de solo lectura cuando sea posible
- Clases grandes: noté una clase de utilidad que constaba de cientos de métodos de extensión (para muchos tipos). ¡Tenía más de 2500 líneas de largo!
- Métodos grandes: estoy tratando de refactorizar un método que tiene 150 líneas de largo.
Los dos últimos parecen ser un problema real. Quiero convencer a mis compañeros de equipo para que usen clases y métodos más pequeños. ¿Pero debería hacer eso? ¿Si es así, entonces cómo?
Mi equipo obtuvo un mentor del equipo principal (somos un equipo satélite). ¿Debo ir a él primero?
ACTUALIZACIÓN : Dado que algunas respuestas preguntaron sobre el proyecto, sepa que es un proyecto que funciona. Y en mi humilde opinión, las clases / métodos enormes de ese tamaño siempre son malos.
De todos modos, nunca quiero molestar a mi equipo. Por eso pregunté: ¿debería hacerlo, y si es así, cómo lo hago con suavidad?
ACTUALIZACIÓN : decidí hacer algo basado en la respuesta aceptada: porque soy un recién llegado, así que veo todo en "nuevos ojos". Tomaré nota de todos los olores de código que encontré (posición, por qué es malo, ¿cómo podemos hacerlo? mejor, ...), pero en este momento, solo trato de reunir los respetos de mi equipo: escribir "mejor código", conocer gente, saber por qué lo hicimos ... Cuando sea el momento adecuado, lo intentaré para preguntarle a mi equipo sobre nuevas políticas de código (pautas de nomenclatura, clases más pequeñas, métodos más pequeños, ...) y, si es posible, refactorizar algún código antiguo. Debería funcionar, en mi humilde opinión.
Gracias.