Disclaimer: sono un nuovo arrivato (questo è il mio terzo giorno di lavoro) e la maggior parte dei miei compagni di squadra ha più esperienza di me.
Quando guardo il nostro codice, vedo alcuni odori di codice e cattive pratiche di progettazione, come le seguenti:
- Linee guida per la denominazione un po 'incoerenti
- Proprietà non contrassegnate come readonly quando possibile
- Grandi classi - Ho notato una classe di utilità che consisteva in centinaia di metodi di estensione (per molti tipi). Era lungo più di 2500 righe!
- Metodi grandi: sto cercando di ridefinire un metodo lungo 150 righe.
Gli ultimi due sembrano essere un vero problema. Voglio convincere i miei compagni di squadra a utilizzare classi e metodi più piccoli. Ma dovrei farlo? Se sì, allora come?
La mia squadra ha un mentore della squadra principale (siamo una squadra satellite). Dovrei andare da lui prima?
UPDATE : dal momento che alcune risposte hanno chiesto informazioni sul progetto, ti preghiamo di sapere che si tratta di un progetto funzionante. E IMHO, le grandi classi / metodi di quelle dimensioni sono sempre cattive.
Comunque, non voglio mai far incazzare la mia squadra. Ecco perché ho chiesto - Dovrei farlo, e se sì, allora come lo faccio con delicatezza?
UPDATE : Ho deciso di fare qualcosa in base alla risposta accettata: perché sono nuovo arrivato, quindi vedo tutto in "occhi nuovi" Prenderò nota di tutti gli odori di codice che ho trovato (posizione , perché male, come possiamo farlo meglio, ...), ma al momento, cerco solo di raccogliere i rispetti dalla mia squadra: scrivere "codice migliore", conoscere le persone, sapere perché lo abbiamo fatto .. Quando sarà il momento, cercherò di chiedere al mio team alcune nuove politiche del codice (linee guida per la denominazione, classi più piccole, metodi più piccoli, ...) e, se possibile, refactoring di un vecchio codice. Dovrebbe funzionare, IMHO.
Grazie.