具有声音缓冲区转换的AccessViolationException

时间:2014-05-24 10:51:58

标签: c# type-conversion access-violation naudio asio

我使用 Naudio AsioOut 对象将数据从输入缓冲区传递到我的delayProc()函数,然后传递到输出缓冲区。

delayProc()需要float[]缓冲区类型,这可以使用e.GetAsInterleavedSamples()。问题是我需要将其重新转换为多维IntPtr,为此我使用AsioSampleConvertor类。

当我尝试应用效果时,它会在AsioSampleConvertor类的代码上显示错误: AccessViolationException

所以我认为问题是由于从float[]转换为IntPtr[] ..

我给你一些代码:

OnAudioAvailable()

floatIn = new float[e.SamplesPerBuffer * e.InputBuffers.Length];//*2

e.GetAsInterleavedSamples(floatIn);

floatOut = delayProc(floatIn, e.SamplesPerBuffer * e.InputBuffers.Length, 1.5f);
//conversione da float[] a IntPtr[L][R]
Outp = Marshal.AllocHGlobal(sizeof(float)*floatOut.Length);

Marshal.Copy(floatOut, 0, Outp, floatOut.Length);
NAudio.Wave.Asio.ASIOSampleConvertor.ConvertorFloatToInt2Channels(Outp, e.OutputBuffers, e.InputBuffers.Length, floatOut.Length);

delayProc()

private float[] delayProc(float[] sourceBuffer, int sampleCount, float delay)
{
    if (OldBuf == null)
    {
        OldBuf = new float[sampleCount];
    }
    float[] BufDly = new float[(int)(sampleCount * delay)];
    int delayLength = (int)(BufDly.Length - (BufDly.Length / delay));
    for (int j = sampleCount - delayLength; j < sampleCount; j++)
        for (int i = 0; i < delayLength; i++)
            BufDly[i] = OldBuf[j];
    for (int j = 0; j < sampleCount; j++)
        for (int i = delayLength; i < BufDly.Length; i++)
            BufDly[i] = sourceBuffer[j];
    for (int i = 0; i < sampleCount; i++)
        OldBuf[i] = sourceBuffer[i];
    return BufDly;
}

AsioSampleConvertor

public static void ConvertorFloatToInt2Channels(IntPtr inputInterleavedBuffer, IntPtr[] asioOutputBuffers, int nbChannels, int nbSamples)
{
    unsafe
    {
        float* inputSamples = (float*)inputInterleavedBuffer;
        int* leftSamples = (int*)asioOutputBuffers[0];
        int* rightSamples = (int*)asioOutputBuffers[1];

        for (int i = 0; i < nbSamples; i++)
        {
            *leftSamples++ = clampToInt(inputSamples[0]);
            *rightSamples++ = clampToInt(inputSamples[1]);
            inputSamples += 2;
        }
    }
}

ClampToInt()

private static int clampToInt(double sampleValue)
{
    sampleValue = (sampleValue < -1.0) ? -1.0 : (sampleValue > 1.0) ? 1.0 : sampleValue;
    return (int)(sampleValue * 2147483647.0);
}

如果您需要其他代码,请问我。

1 个答案:

答案 0 :(得分:2)

当您致电ConvertorFloatToInt2Channels时,您传递的是所有频道的样本总数,然后尝试读取那么多样本。因此,您尝试从输入缓冲区中读取两倍于实际存在的样本。使用不安全的代码,您试图在分配的块结束后很好地解决,这会导致您获得访问冲突。

for方法中的ConvertorFloatToInt2Channels循环更改为:

for (int i = 0; i < nbSamples; i += 2)

这将阻止您的代码尝试读取源内存块中实际存在的项目数的两倍。


顺便说一下,为什么要搞乱分配全局内存并在这里使用不安全的代码呢?为什么不将它们作为托管数组处理?处理数据本身并不会慢得多,并且可以节省从非托管内存复制数据的所有开销。

试试这个:

public static void FloatMonoToIntStereo(float[] samples, float[] leftChannel, float[] rightChannel)
{
    for (int i = 0, j = 0; i < samples.Length; i += 2, j++)
    {
        leftChannel[j] = (int)(samples[i] * Int32.MaxValue);
        rightChannel[j] = (int)(samples[i + 1] * Int32.MaxValue);
    }
}

在我的机器上,每秒处理大约1200万个样本,将样本转换为整数并分割通道。如果我为每组结果分配缓冲区,大约是速度的一半。当我写这篇文章以使用不安全的代码AllocHGlobal

时,大约有一半

永远不要认为不安全的代码更快。