Beim Refactoring einiger C#-Klassen bin ich auf Klassen gestoßen, die IDisposable implementieren.
Ohne nachzudenken, habe ich für jede Klasse, die die Schnittstelle IDisposable implementiert, partielle Klassendateien erstellt.
Z.B.) For Stamper.cs -> Stamper.cs + Stamper.Dispose.cs wobei Stamper.cs enthält die eigentliche Logik für das Stempeln und Stamper.Dispose.cs die Dispose-Logik enthält
// Stamper.cs
public partial class Stamper
{
// actual logic
}
// Stamper.Dispose.cs
public partial class Stamper: IDisposable
{
// Implement IDisposable
}
Wenn ich mir den Code ansehe, sieht Stamper.cs jetzt viel sauberer und lesbarer aus (jetzt etwa 52 Zeilen statt 100 Zeilen, wobei etwa 50 Zeilen einfach ein aufgeräumter Dispose-Code waren)
Gehe ich damit zu weit?
*EDIT : Vielen Dank für Ihre Meinung - ich habe beschlossen, zwei Dateien zu einer einzigen zusammenzufassen. Das Problem, mit dem ich konfrontiert war, war, dass ich tatsächlich vergessen habe, die IDisposable-Implementierung nach der Aktualisierung der eigentlichen Logik zu aktualisieren.
Außerdem war es kein großes Problem, zwischen den Methoden im Quellcode zu navigieren. Der erste Grund scheint mehr als Grund genug zu sein, um in meinem speziellen Fall bei der Lösung mit einer Datei zu bleiben.