Ist Resharpers Empfehlung, meine private Methode statisch zu machen, eine gute Empfehlung?

Question

Ich habe kürzlich festgestellt, dass Resharper, wenn ich private Methoden erstelle, die einige Felder in den Objekten setzen, die an sie übergeben werden, einen Hinweis anzeigt, dass die Methode statisch gemacht werden kann.

Hier ist ein stark vereinfachtes Beispiel für die Art von Methode, die ich haben könnte.

private void MakeStatusTheSame(MyClass mc, MySecondClass msc) 
{ 
    mc.Status = msc.Status; 
} 

Wenn ich eine Methode wie diese habe, gibt Resharper eine Empfehlung, dass die Methode statisch gemacht werden kann.

Ich versuche zu vermeiden, öffentliche Methoden statisch zu machen, da sie Chaos auf Unit-Tests zerstören ... aber ich bin mir nicht sicher, dass das gleiche für private Methoden gilt.

Ist die Empfehlung von Resharper eine gültige Best Practice oder sollte ich sie einfach abschalten?

Answer 1

Ich denke, das ist definitiv ein idealer Kandidat für eine statische Methode. Es ist nicht der Klasse des Eigenschaften zu verändern, Felder etc.

Hier ein Beispiel:

class MyClass 
{ 
    public static void MakeStatusTheSame(MyClass mc, MySecondClass msc) 
    { 
    mc.status = msc.status; 
    } 

    private void MakeStatusTheSame(MySecondClass msc) 
    { 
    this.status = msc.status; 
    } 

    private int status; 
} 

Auch

, können Sie es eine Erweiterungsmethode machen (die auch statisch sein würde):

public static class Extensions 
{ 
    public static MyClass MakeStatusTheSame(this MyClass mc, MySecondClass msc) 
    { 
    mc.status = msc.status 
    return mc; /* make the method chainable */ 
    } 
} 

Answer 2

Ich denke schon; zu sehen, dass eine Methode statisch ist, ist ein klarer Hinweis darauf, dass die Methode mit allen Instanzenmembern interagieren soll.

Stellen Sie sich vor, Sie würden eine nicht statische Methode debuggen und feststellen, dass die Instanz nicht berührt wird. Sofortiger Geruch, und wenn es keinen Kommentar gibt, der erklärt, was die Funktion tut, könnten Sie vom wirklichen Problem abgelenkt werden.

Answer 3

Ich gehe normalerweise mit R # Empfehlung. Es ist eine private Methode, also schreiben Sie (hoffentlich) keine Komponententests dagegen. Wenn Sie es statisch machen, wird explizit angegeben, dass es keine Instanzelemente verwendet, wodurch die Überprüfung auf Nebenwirkungen erleichtert wird.

Answer 4

Auf die Gefahr, wie ein Contrarian klingen, muss ich zugeben, dass ich statische Methoden nicht gerne mit Instanzmethoden mischen; und ich mag statische Methoden im Allgemeinen nicht. Statische Methoden sind schwer zu testen, schwer zu überschreiben und schwer zu warten. Ich bevorzuge es, alle statischen Methoden für den Umgang mit Foo-Objekten in einer einzigen FooUtils-Klasse zu verankern - oder noch besser in einer Singleton-Instanz einer FooSomethingDoer-Klasse.

Natürlich sind statische Methoden in einigen Fällen sinnvoll - zum Beispiel beim Erstellen der oben erwähnten Singletons oder Fabriken usw. Ich sage nicht, dass alle statischen Methoden aus reinem Bösen bestehen; Ich mag es lieber, auf der Seite zu bleiben, wenn es möglich ist.