0

我有一个 WPF 程序，它可以抓取某个网络目录中的所有目录并将它们列在列表视图中。

问题是目录太多，加载列表最多可能需要 5 秒。

我想知道是否有一种方法可以使用更高效的代码来加快速度。或者甚至可能是一种将列表存储在数组中并在第一次之后每次都查找更改的方法？

For Each i As String In Directory.GetDirectories(dir)
    If File.Exists(Path.GetFullPath(i) & "\cover.jpg") Then
        If (File.GetAttributes(Path.GetFullPath(i)) And FileAttributes.Hidden) <> FileAttributes.Hidden Then
            Dim foldername As String = Path.GetFileName(i)
            Dim moviename As String = foldername
            If moviename.Length > 4 Then
                If moviename.Substring(0, 4) = "The " Then
                    moviename = moviename.Remove(0, 4)
                End If
            End If
            Dim display As Boolean = True
            If IO.Directory.GetDirectories(Path.GetFullPath(i)).Length < 1 Then
                If IO.Directory.GetFiles(Path.GetFullPath(i), "*.avi").Length < 1 Then
                    If IO.Directory.GetFiles(Path.GetFullPath(i), "*.mp4").Length < 1 Then
                        If IO.Directory.GetFiles(Path.GetFullPath(i), "*.mkv").Length < 1 Then
                            display = False
                        End If
                    End If
                End If
            End If
            If display = True Then
                Showslist.Items.Add(New With {Key .img = Path.GetFullPath(i) & "\cover.jpg", .name = foldername, .path = Path.GetFullPath(i), .created = Directory.GetCreationTime(Path.GetFullPath(i)), .moviename = moviename})
            End If
        End If
    End If
Next
4

1 回答 1

2

1. 不要多次阅读目录的内容。相反，一次读取所有必需的内容并将其缓存在（内存中）变量中。这将有助于提高性能，因为访问内存比进行 I/O 快得多（这里：访问文件系统）。例如：

If IO.Directory.GetFiles(…, "*.avi").Length < 1 Then
    If IO.Directory.GetFiles(…, "*.mp4").Length < 1 Then
        If IO.Directory.GetFiles(…, "*.mkv").Length < 1 Then
            …

您刚刚查询了同一个目录 3 次。您可以将其更改为仅读取一次内容（从而可能将您的代码加快三倍），然后在内存中对其进行过滤：

'Imports System.IO
'Imports System.Linq

' access the file system only once and store the results in-memory…
Dim filePaths As String() = Directory.GetFiles(…, "*")

' … and perform everything else on that in-memory cache:
If Not filePaths.Any(Function(fp) Path.GetExtension(fp) = ".avi") Then
    If Not filePaths.Any(…) Then
        If Not filePaths.Any(…) Then
            …

PS：也许值得指出的是，与 不同的是Directory.GetFiles，这些System.IO.Path方法不会导致昂贵的文件系统命中：它们只是对已知包含文件系统路径的字符串进行操作。

2.考虑递归读取根目录的完整内容。该Directory.GetFiles方法有一个重载Directory.GetFiles(String, String, SearchOption)，其SearchOption参数可以设置为SearchOption.AllDirectories。在这种情况下，您不仅会从指定目录本身获取内容，还会从其所有子目录中获取内容。

这意味着，您将需要一次调用Directory.GetFiles（针对根目录）而不是多次调用，这意味着您再次减少了昂贵的 I/O 调用的数量。将结果存储在一个数组中，然后从那里开始构建您的 WPF 列表。

' read the contents of the network root directory, including all of its sub-directories…
Dim filePaths As String() = Directory.GetFiles(…, "*", SearchOption.AllDirectories)

' …and do everything else using the in-memory cache built above:
For Each filePath As String in filePaths
    …
    Showslist.Items.Add(…)
Next
于 2014-08-07T06:36:24.020 回答