0

我正在编写一个函数来加载波形文件，并在此过程中将数据拆分为 2 个单独的缓冲区（如果它是立体声的）。程序到达 i = 18 并在左通道 fread pass 期间崩溃。（您可以忽略 couts，它们只是用于调试。）也许我应该一次性加载文件并使用 memmove 填充缓冲区？

if(params.channels == 2){
    params.leftChannelData = new unsigned char[params.dataSize/2];
    params.rightChannelData = new unsigned char[params.dataSize/2];

    bool isLeft = true;

    int offset = 0;
    const int stride = sizeof(BYTE) * (params.bitsPerSample/8);

    for(int i = 0; i < params.dataSize; i += stride)
    {
        std::cout << "i = " << i << " ";
        if(isLeft){
            std::cout << "Before Left Channel, ";
            fread(params.leftChannelData+offset, sizeof(BYTE), stride, file + i);
            std::cout << "After Left Channel, ";
        }
        else{
            std::cout << "Before Right Channel, ";
            fread(params.rightChannelData+offset, sizeof(BYTE), stride, file + i);
            std::cout << "After Right Channel, ";
            offset += stride;
            std::cout << "After offset incr.\n";
        }

        isLeft != isLeft;
    }

} else {
    params.leftChannelData = new unsigned char[params.dataSize];
    fread(params.leftChannelData, sizeof(BYTE), params.dataSize, file);
}
4

2 回答 2

3

这在我看来是错误的：file + i

你只有一个文件句柄，对吧？不要添加i它！

file 是一个 FILE* 我的假设是，像大多数指针一样，向指针添加一个整数将从文件中的偏移量读取。

FILE* 是一个指向 FILE 结构的指针，该结构存储有关您打开的文件的信息。您总是将该指针传递给 stdio 函数。如果要查找文件中的特定位置，请使用fseek. 如果您只是从头到尾顺序读取文件，则无需查找。

于 2012-09-13T03:35:28.067 回答
0

对于任何想要我的完整解决方案的人，我正在发布这个。事实证明，在 fread 调用之间循环比在 memcpy 调用之间循环一个 fread 调用要慢得多（尽管它需要两倍的内存）。我发布的代码中的另一个问题是我说：

isLeft != isLeft;

代替

isLeft = !isLeft;

解决方案：

if(params.channels == 2){
    params.leftChannelData = new unsigned char[params.dataSize/2];
    params.rightChannelData = new unsigned char[params.dataSize/2];
    unsigned char * buf = new unsigned char[params.dataSize];

    fread(buf, sizeof(BYTE), params.dataSize, file);

    bool isLeft = true;

    int offset = 0;
    const int stride = params.bitsPerSample/8;

    for(int i = 0; i < params.dataSize; i += stride)
    {

        if(isLeft)
            memcpy(&params.leftChannelData[offset], &buf[i], stride * sizeof(BYTE));
        else{
            memcpy(&params.rightChannelData[offset], &buf[i], stride * sizeof(BYTE));
            offset += stride;
        }
        isLeft = !isLeft;
    }

    params.dataSize = params.dataSize/2;

    delete(buf);

} else {
    params.leftChannelData = new unsigned char[params.dataSize];
    params.rightChannelData = NULL;
    fread(params.leftChannelData, sizeof(BYTE), params.dataSize, file);
}
于 2012-09-13T05:36:36.377 回答