Wann ist eine lange Schlange einfach zu lang?

Question

Dies funktioniert (Visual Basic .NET) aber scheint komisch lange mit allen LINQ to Object-Methoden und in-place-Dimensionierung.

For Each PNGFile As System.IO.FileInfo In New System.IO.DirectoryInfo(Server.MapPath(".\Archive")).GetFileSystemInfos("*.png", System.IO.SearchOption.AllDirectories).OrderByDescending(Function(f) f.LastWriteTimeUTC).Skip(PageSize * Page).Take(PageSize) 
    'Do stuff with PNGFile 
Next 

Ich liebe, dass es alles auf einer Linie ist und ich denke, dass es sogar logisch für mich liest. Aber mein Bauchgefühl sagt mir, dass die Lesbarkeit für die nächste arme Seele, die meinen Code interpretieren muss, nicht existent ist. Oder ist es? Wie entscheidest du dich? Lohnt es sich, diese Zeile in mehrere andere Dimensionierungs- und Zuweisungsanweisungen aufzuteilen? Wie würden Sie diese bestimmte Linie als Beispiel brechen?

Ich bin neu in .NET, aber ich habe Code seit über 10 Jahren geschrieben. Bis heute habe ich meine maximale Zeilenlänge im Code basierend auf der typischen Auflösung bestimmt, die ich beim Erstellen des Codes verwende. Dies kann nicht der beste Weg, um zu entscheiden, ...

Answer 1

ich in der Regel Trennlinien auf eine neue LINQ-Erweiterungsmethode, ich sieht Reiniger:

For Each PNGFile As System.IO.FileInfo In New System.IO.DirectoryInfo(Server.MapPath(".\Archive")) 
    .GetFileSystemInfos("*.png",System.IO.SearchOption.AllDirectories) 
    .OrderByDescending(Function(f) f.LastWriteTimeUTC) 
    .Skip(PageSize * Page) 
    .Take(PageSize) 
     'Do stuff with PNGFile 
Next 

Answer 2

Wie BrokenGlass sagte ich auf der LINQ brechen Erweiterungen:

For Each PNGFile As System.IO.FileInfo In New System.IO.DirectoryInfo(Server.MapPath(".\Archive")) 
    .GetFileSystemInfos("*.png",System.IO.SearchOption.AllDirectories) 
    .OrderByDescending(Function(f) f.LastWriteTimeUTC) 
    .Skip(PageSize * Page) 
    .Take(PageSize) 
     'Do stuff with PNGFile 
Next 

bei Ihrem nächsten Punkt zu erweitern, wie maximale Leitungslänge zu bestimmen: ich gehe von zwei Möglichkeiten. Wenn der Code, den ich schreibe, intern ist, oder ich weiß, wer es ist, dann setze ich eine maximale Zeilenlänge von 120 Zeichen. Wenn der Code, den ich schreibe, global werden könnte, wo ich den Status eines anderen Programmiercomputers nicht kenne, dann verwende ich eine maximale Zeilenlänge von 80 Zeichen.

Answer 3

Um es zu einer allgemeineren Ebene zu nehmen:

Ich halte (eine Zeile) Code korrekt formatiert, wenn es „einfach auf das Auge“ ist.

In Ihrem Beispiel muss ich zu viel scrollen, also würde ich es in mehr Zeilen aufteilen, mehr oder weniger ähnlich wie es BrokenGlass tat.

Es ist natürlich sehr subjektiv, aber im Allgemeinen bevorzuge ich Code nur mit meinen Augen zu lesen, nicht mit der Tastatur und/oder Maus.

Answer 4

Ehrlich gesagt, ich hasse es, diese Art von Code zu sehen ... und das nicht nur, weil es VB;)

Warum nicht Sie es in mehrere Aussagen geteilt? Es macht alles einfacher zu lesen, und es ist auch einfacher zu debuggen, weil Sie Zwischenvariablen betrachten können.

Hier ist, wie ich es schreiben würde:

Dim archivePath As String = Server.MapPath(".\Archive") 
Dim archiveDir As New System.IO.DirectoryInfo(archivePath) 
Dim allFiles = achiveDir.GetFileSystemInfos("*.png", System.IO.SearchOption.AllDirectories) 
Dim filesToDisplay = allFiles.OrderBy(Function(f) f.LastWriteTimeUTC) 
          .Skip(PageSize * Page) 
          .Take(PageSize) 

For Each PNGFile As System.IO.FileInfo in filesToDisplay 
    'Do stuff with PNGFile 
Next 

IMHO, sobald Sie mehr als 2 Sekunden zu denken brauchen eine einzige Zeile Code zu verstehen, kann man bedenkt, es ist zu lang ...

Answer 5

Ja, es gibt keinen schlimmeren Bug als den, den man nicht sehen kann, Code, der rechts vom Bildschirmrand liegt. Ich hatte einige echte Tricks, bevor ich das selbst entdeckte und eine steinerne harte Regel darüber machte.

Wenn Sie nicht VB haben 10 noch, dann können Sie Speicherplatz verwenden + unterstreichen die Linie zu brechen:

For Each PNGFile As System.IO.FileInfo _ 
     In New System.IO.DirectoryInfo("mumble") _ 
      .GetFileSystemInfos("*.foo") 
     '' etcetera 
    Next