VB.net 需要内存高效功能

Question

当 RowCollection 超过 50000 时，我从以下函数中得到内存异常，因此我需要提高内存效率。该函数只需要构造一个存储在 RowCollection 中的行索引的逗号分隔字符串。任何人都可以在下面发现任何明显的内存消耗操作吗？

NB RowCollection 仅包含存储为整数的行索引列表。

 Private Function GetCommaSeparatedString(ByRef RowIndexes As ArrayList) As String
        Dim RowString As String = String.Empty

        'Build a string of the row indexes 
        'Add one onto each index value so our indexes begin at 1 
        For Each Row In RowIndexes 
            RowString += CInt(Row.ToString) + 1 & ","
        Next

        'Remove the last comma
        If RowString.Length > 0 Then
            RowString = RowString.Substring(0, RowString.Length - 1)
        End If

        Return RowString
    End Function

提前致谢。

Answer 1

我不确定为什么会出现内存不足错误，除非行的字符串表示非常大，因为您永远不会有超过一两个非垃圾收集字符串。

但是，您的方法效率非常低，因为它花费了大量时间来复制半构建字符串的内容。 StringBuilder 更适合构建大型字符串，因为可以修改它而无需每次都重新创建内容。

然而，在这种情况下，即使使用 StringBuilder 也是一个坏主意，因为您正在连接字符串，并且已经有一个方法可以做到这一点：String.Join。只需使用 LINQ 查询来执行“向索引添加一”的操作，您就会得到一行：

Private Function GetCommaSeparatedString(ByVal RowIndexes As ArrayList) As String
    Return String.Join(",", From index In RowIndexes Select CInt(index) + 1)
End Function

我还建议不要通过引用传递，除非您确实需要它。您没有修改 RowIndexes，因此按值传递它。我也不确定为什么你要 ToString()-ing 索引然后立即解析它。它们不是已经是整数了吗？只需使用 CInt 即可。

Answer 2

更新：虽然这是使用 stringbuilder 的直接更改，但请通过 Strilanc 或 史蒂文Sudit

好吧，您可能仍然会耗尽内存（毕竟内存是有限的），但您应该使用 StringBuilder，而不是连接字符串。每次，您都创建一个新的字符串对象而不是更改它（因为字符串是不可变的）

Private Function GetCommaSeparatedString(ByRef RowIndexes As ArrayList) As String
    Dim RowString As New StringBuilder()

    'Build a string of the row indexes 
    'Add one onto each index value so our indexes begin at 1 
    For Each Row In RowIndexes 
        RowString.AppendFormat("{0},",  CInt(Row.ToString) + 1)
    Next

    'Remove the last comma
    If RowString.Length > 0 Then
        RowString.Append(RowString.Substring(0, RowString.Length - 1))
    End If

    Return RowString
End Function

Answer 3

StringBuilder 是个好主意，但为什么不直接通过流式输出来避免问题，而不是尝试将其全部保存在内存中呢？

Answer 4

这是因为在每次迭代中，您都会在幕后创建 2 个字符串，并且它们在接近结束时会变得越来越大。

"1,2,3,4,5,....499,500"
“1,2,3,4,5,....499,500”，

在仅 500 次迭代结束时，您创建了 2 个近 2000 个字符长的字符串，结果却在下一次迭代中将它们丢弃（但运行时可能会将它们留在身边）。

在最后一次迭代中，假设您的行索引是连续的，您的字符串（从 1 到 50000）将是 100,000 个字符长。这意味着您已经分配了约 10,000,000,000 个字符或（我相信 2 个字节/字符）20 GB 的字符串。

您可以首先在字符串 (RowString) 上使用 StringBuilder 而不是 +=。
例如

Dim RowString As StringBuilder = new StringBuilder( 100000 )

For Each Row In RowIndexes 
    RowString.Append( CInt(Row.ToString) + 1).Append( "," )
Next

'...'

Return RowString.ToString

您也可以尝试下一个，但您应该对这两个进行分析并选择最适合您的一个。

Private Function GetCommaSeperatedString(ByRef RowIndexes As ArrayList) As String
    Dim indexArray as String[] = RowIndexes
                                 .Select(Function(r)=> (CInt(r.ToString) + 1)ToString)
                                 .ToArray
    return String.Join( ',', indexArray)
End Function

* 注意：这些是我写过的 VB 的第一行，所以我可能犯了一个基本错误（特别是在 linq/lambda 方面），但重点就在那里。