3

我有 3 个频道：

byte[] Red;
byte[] Green;
byte[] Blue;

我需要将它们中的所有值复制到 abyte[Red.Length+Green.Length+Blue.Length] PA中，以便：

PA[0] = Red[0];
PA[1] = Green[0];
PA[2] = Blue[0];
PA[3] = Red[1];
/// and so on

以下是上述数组的示例：

byte[] Red = new byte[255];
byte[] Green = new byte[255];
byte[] Blue = new byte[255];
byte[] PA = new byte[Red.Length + Green.Length + Blue.Length];
for (int i = 0; i != 255; ++i)
{
    PA[i*3 + 0] = Red[i];
    PA[i*3 + 1] = Green[i];
    PA[i*3 + 2] = Blue[i];
}

我假设要合并的集合大小相同，并且它们之间确实有一些顺序，例如[0] = Red,[1]=Green等必须为“合并”集合中的项目保留。

最有效的方法是什么C#？集合不必是数组，也不必是项目字节（尽管接受字节的集合类型将不胜感激）。

4

3 回答 3

5

我会尽量避免3*i乘法：

byte[] Red = new byte[255];
byte[] Green = new byte[255];
byte[] Blue = new byte[255];
int newSize = Red.Length + Green.Length + Blue.Length;
byte[] PA = new byte[newSize];
for (int i = 0; i < newSize; i += 3)
{
    PA[i + 0] = Red[i];
    PA[i + 1] = Green[i];
    PA[i + 2] = Blue[i];
}

编辑

甚至是这样的：

for (int i = 0, j = 0; i < 255; i++)
{
    PA[j++] = Red[i];
    PA[j++] = Green[i];
    PA[j++] = Blue[i];
}

（由维克托建议）

于 2013-03-19T08:03:40.653 回答
5

我试图通过使用指针来做出更有效的方式：

unsafe {
  fixed (byte* red = Red, green = Green, blue = Blue, pa = PA2) {
    byte* r = red, g = green, b = blue, p = pa;
    for (int i = 0; i < 255; i++) {
      *p = *r; p++; r++;
      *p = *g; p++; g++;
      *p = *b; p++; b++;
    }
  }
}

在 x86 模式下，这大约是两倍快，但在 x64 模式下没有区别。

总之，您拥有的代码对于大多数应用程序来说已经足够快了。如果您需要它非常快，您可以对其进行一些优化，但不会太多。

于 2013-03-19T08:36:52.960 回答
2

效率是一个薄薄的决策层，但从性能的角度来看，我会说你已经以有效的方式做到了。

//allocate immediately memory need, so more shrinking of memory will happen 
byte[] PA = new byte[Red.Length + Green.Length + Blue.Length]; 

//use for loop, that normally equals to foreach in some cases is faster
for (int i = 0; i != 255; ++i)
{
    PA[i + 0] = Red[i];
    PA[i + 1] = Green[i];
    PA[i + 2] = Blue[i];
}
于 2013-03-19T08:02:55.280 回答