Когда длинная линия слишком длинная?

Question

Это работает (Visual Basic .NET), но кажется комически длинным со всеми методами LINQ to Object и размером с размерами.

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 

Мне очень нравится, что все это на одной линии, и я думаю, что он даже читает логически для меня. Но моя кишка говорит мне, что читаемость для следующей бедной души, которая должна интерпретировать мой код, не существует. Или это? Как вы решаете? Стоит ли разбить эту строку на несколько других операторов определения размеров и присваивания? Как вы могли бы разбить эту конкретную линию в качестве примера?

Я новичок в .NET, но я пишу код уже более 10 лет. На сегодняшний день я определил максимальную длину строки в коде на основе типичного разрешения, которое я использую при создании кода. Это не может быть лучшим способом решить ...

Answer 1

Я обычно расщепленные линии на новый метод расширения LINQ, выглядит чище меня:

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

Как сказал BrokenGlass, я бы разорвать на LINQ расширения:

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 

для расширения на следующей точке, как определить максимальную длину строки: Я иду двумя вариантами. Если код, который я пишу, является внутренним, или я знаю, кто это собирается, тогда я устанавливаю максимальную длину строки 120 символов. Если код, который я пишу, может быть глобальным, где я не знаю состояние другого компьютера программистов, тогда я использую максимальную длину строки 80 символов.

Answer 3

Чтобы принять его на более общем уровне:

Я считаю (строка) код отформатирован, когда это «легко на глаз».

В вашем примере мне нужно прокрутить слишком много, поэтому я разделил бы его на несколько строк, более или менее похожих на то, как это сделал BrokenGlass.

Это очень субъективно, но в целом я предпочитаю читать код, используя только мои глаза, а не используя клавиатуру и/или мышь.

Answer 4

Честно говоря, я действительно ненавижу видеть этот вид кода ... и не только потому, что VB;)

Почему вы не разбить его на несколько утверждений? Это упрощает чтение, а также легче отлаживать, потому что вы можете просматривать промежуточные переменные.

Вот как я бы писать:

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 

ИМХО, как только вы должны думать более 2 секунд, чтобы понять строку кода, вы можете считать, что это слишком долго ...

Answer 5

Да, нет худшей ошибки, кроме той, которую вы не видите, кода, который находится справа от края экрана. У меня были какие-то настоящие doozies, прежде чем я узнал об этом и сделаю Stone Hard Hard Rule об этом.

Если вы не VB 10 еще, то вы можете использовать пространство + подчеркивания, чтобы разорвать строку:

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