将固定大小的数组添加到 IEnumerable

Question

这个方法得到：

IEnumerable<object[]> - 其中每个数组都是固定大小（它代表关系 数据结构）。

DataEnumerable.Column[] - 一些元数据列，大多数情况下，所有行的值都相同。

预期结果：

每个“行”都应该为这些列中的每一列获取值（因此数据结构保持相关性）。

    private IEnumerable<object[]> BindExtraColumns(IEnumerable<object[]> baseData, int dataSize, DataEnumerable.Column[] columnsToAdd)
    {
        int extraColumnsLength = columnsToAdd.Length;
        object[] row = new object[dataSize + extraColumnsLength];

        string columnName;
        int rowNumberColumnIndex = -1;

        for (int i = 0; i < extraColumnsLength; i++)
        {
            //Assign values that doesn't change between lines..
            // Assign rowNumberColumnIndex if row number column exists
        }

        //Assign values that change here, since we currently support only row number
        // i'ts not generic enough        
        if (rowNumberColumnIndex != -1)
        {
            int rowNumber = 1;

            foreach (var baseRow in baseData)
            {
                row[rowNumberColumnIndex] = rowNumber;

                Array.Copy(baseRow, 0, row, extraColumnsLength, dataSize);

                yield return row;

                rowNumber++;
            }
        }
        else
        {
            foreach (var baseRow in baseData)
            {
                Array.Copy(baseRow, 0, row, extraColumnsLength, dataSize);

                yield return row;
            }
        }
    }

这个方法可以从数百个具有相对大数据集的线程中调用，所以 这里的性能很关键，我尝试创建尽可能少的新对象。

请注意 - 这是一个私有方法，它使用 ONLY BY DataReader，它读取每一行，并立即将其传递给另一个数组在读取下一行之前 .

那么 - 在这里复制数组是否会以某种方式在这里进行优化，我应该（小心地）使用内存来提升这里的东西吗？

谢谢

Answer 1

您的代码从根本上被破坏了。您每次都只是返回对同一个数组的引用，这意味着除非调用者立即使用每个项目中的数据，否则它实际上会丢失。例如，假设我使用：

List<object[]> rows = BindExtraColumns(data, size, toAdd).ToList();

然后，当我遍历行时​​，我会在每一行中找到相同的数据。这真的不是一个很好的体验。

我认为为每次迭代创建一个新数组会让更加更有意义。是的，这占用了很多额外的内存 - 但调用者并没有那么惊讶。

如果您真的不想这样做，我建议您更改方法，以便调用者必须传入 Action<object[]> 才能在每一行上执行，并附带记录的附带条件，即如果调用者隐藏了引用到数组，他们可能会对结果感到惊讶。

您显然非常关心性能，但如果您的数据来自数据库，我希望数组创建/复制性能微不足道。您应该首先编写最简单（也是最可靠）的代码，然后对其进行基准测试以查看其性能是否足够好。除非你有证据表明你需要做出这个令人惊讶的设计选择，否则感觉你优化得太早了。

编辑：现在我们知道它是一个私有方法只在一个特定的地方使用，我会仍然避免这种重用。它简直是脆弱的。我真的会改为传入Action<object[]>，或者每次都将数据复制到一个新数组中。如果没有强有力的证据证明这是一个瓶颈，我当然不会保留当前的方法：正如我之前所说，我希望数据库通信更加重要。像这样在代码中留下定时炸弹很少有效果。

如果你真的，真的想要继续这样做，你应该非常强烈地记录它，并严重警告结果是非惯用的。

就您是否可以进行 更多 优化而言 - 嗯...一种替代方法是首先避免使用单个数组。您可以创建一个包含对 both 数组（当前基行和固定数据）的引用的类，并公开一个索引器，该索引器根据请求的索引从一个数组或另一个数组返回值。我们不知道您对数据做了什么（“将其传递给另一个数组”实际上并没有任何意义）所以我们不知道这是否可行，但它会很有效并且 可以在没有奇怪行为的情况下实现。