C++实现8/16/24/32位交错音频解交织时非0通道输出异常

问题根源 #

代码中的索引计算没有考虑多字节样本的通道偏移，8位位深场景因为单样本长度为1字节，碰巧匹配了错误公式的计算结果；16/24/32位深场景下多字节通道偏移未按样本长度缩放，导致非0通道取数位置完全错位。

正确偏移推导 #

交错PCM音频的固定存储规则为：按采样帧顺序存储，单帧内按通道顺序排列所有通道的样本数据，单样本长度为byterate = bitrate / 8字节。基于此可推导任意数据点的索引计算逻辑：

• 单帧总长度：frame_size = channel_count * byterate 字节
• 第n个采样帧的起始索引：n * frame_size
• 第n帧内通道i的样本起始索引：帧起始索引 + i * byterate（通道i的样本需要跨i*byterate个字节的前置样本）
• 通道i样本内第z个字节的最终索引：帧起始索引 + i * byterate + z

核心公式修正 #

原代码中的错误公式为：

audio_ptr[(x * channel_count) + i + z]

其中x是单通道内累计的样本字节偏移（x = n * byterate），代入帧索引关系化简后的正确公式为：

audio_ptr[x * channel_count + i * byterate + z]

验证：当byterate=1（8位深）时，i * byterate等价于i，公式和原表达式完全一致，因此8位场景可正常运行，和观察到的现象完全匹配。

高效实现优化 #

逐字节push_back的实现效率较低，可改用memcpy按样本块拷贝，省去逐字节的内层循环：

void Demux(int bitrate) {
    int byterate = bitrate / 8;
    std::map<int, std::vector<uint8_t>> channel_audio;
    // 计算总帧数
    int total_frames = audio_size / (channel_count * byterate);

    for (int i = 0; i < channel_count; i++) {
        std::vector<uint8_t> audio;
        audio.reserve(bytes_per_channel);
        for (int n = 0; n < total_frames; n++) {
            // 计算当前通道单样本的起始位置
            const uint8_t* sample_pos = audio_ptr + n * channel_count * byterate + i * byterate;
            // 扩容并整块拷贝样本
            size_t old_len = audio.size();
            audio.resize(old_len + byterate);
            memcpy(audio.data() + old_len, sample_pos, byterate);
        }
        channel_audio.emplace(i, std::move(audio));
    }

    // 原有输出逻辑保持不变
    int remapsize = 0;
    std::cout << "\nRemapped  Audio";
    for (auto& [ch, v] : channel_audio) {
        std::cout << "\nChannel" << ch << " ";
        remapsize += v.size();
        for (size_t i = 0; i < v.size(); i++) {
            std::cout << "0x" << std::hex << std::setfill('0') << std::setw(2) << +v[i] << " ";
            if (i && (i + 1) % 32 == 0) {
                std::cout << std::endl;   
            }
        }
    }
    std::cout << "Total remapped audio size is " << std::dec << remapsize << std::endl;
}

注意事项 #

• 原有代码存在全局bitrate常量和Demux函数入参重名的问题，会触发作用域遮蔽，容易出现byterate计算错误，建议将全局常量改名为global_bitrate或删除全局常量，完全使用函数入参计算
• 生产环境使用时需要增加边界校验，确保传入的原始缓冲大小能被channel_count * byterate整除，避免内存越界

内容的提问来源于stack exchange，提问作者allthewayaround