Ho creato un metodo di convalida privato per una certa convalida che si verifica più volte nella mia classe (non posso archiviare i dati convalidati per vari motivi). Ora, ReSharper suggerisce che la funzione potrebbe essere resa statica. Sono un po 'riluttante a farlo a causa di problemi noti con metodi statici. Sarebbe un metodo statico privato . La mia domanda è: i metodi statici privati possono causare problemi simili di accoppiamento e test come i metodi statici pubblici? È una cattiva pratica? Immagino di no, ma non sono sicuro che ci sia una trappola qui.
10
Quali sono i "problemi noti" con i metodi statici?
—
Robert Harvey,
@Ed: giusto. I metodi statici scritti correttamente non devono toccare API esterne o dichiarare comunque. Manipolare lo stato interno incapsulato in una classe mi sembra perfettamente OK e il metodo non avrebbe bisogno di essere testato dall'unità, poiché il test unitario verifica il comportamento esterno della classe.
—
Robert Harvey,
I metodi statici sono inclini a modificare lo stato globale e uccidono anche l'ereditarietà (ogni volta che si desidera estendere la funzionalità, è necessario modificare il codice chiamante perché non è possibile ignorare il metodo in una classe derivata). Sei legato a quell'unica implementazione. I metodi statici non possono essere derisi, il che li rende molto difficili da testare. Nascondono la dipendenza . Sono sicuro che c'è di più. Non solo per evitarli alla cieca, sto chiedendo di prendere una decisione informata.
—
Tamás Szelei,
@ Tamás I metodi statici modificano lo stato globale solo se li scrivi in questo modo, cosa che non faccio mai. In generale, utilizzo solo metodi statici nelle classi di utilità, metodi che accettano uno o più oggetti e restituiscono un oggetto senza effetti collaterali. Questi tipi di metodi non hanno nessuno dei problemi che descrivi.
—
Robert Harvey,
@ TamásSzelei Come sono inclini a modificare lo stato globale? Non riescono nemmeno a trovare lo stato globale se non lo si passa in un parametro.
—
CodesInChaos,