Question

我正在尝试将使用“mplayer -ao pcm：nowaveheader”生成的pcm音频编码为带有c程序的mp3。我不想把mp3写入文件，我想保留在一个数组中，直到我需要将它写入文件，我写了这个，它似乎工作在一个短的.9秒测试文件，但这很慢。究竟出了什么问题？

#include <stdio.h>
#include <stdlib.h>
#include <lame/lame.h>

lame_global_flags *gfp;
int loopcount;
int inputSize;
FILE *fp=NULL;
FILE *fpo=NULL;
char *mp3buffer;
int mp3buffersize;
int countsize;
int x=0;
int y=0;
short *pcmbuffer;
short *lpcmbuffer;
short *rpcmbuffer;

int parse()
{
    printf("loading PCM data...\n");
    pcmbuffer=malloc(inputSize);
    fread(pcmbuffer,2,(inputSize/2),fp);
    printf("data in buffer\n");
    printf("splitting left and right channels\n");
    lpcmbuffer=malloc(inputSize/2);
    countsize=((inputSize/4)-1);
    while (x<=countsize)
    {
        lpcmbuffer[x]=pcmbuffer[x*2];
        x++;
    }
    x=0;
    rpcmbuffer=malloc(inputSize/2);
    while (x<=countsize)
    {
        rpcmbuffer[x]=pcmbuffer[(x*2)+1];
        x++;
    }
    x=0;
    printf("starting lame\n");
    gfp=lame_init();
    lame_set_num_channels(gfp,2);
    lame_set_in_samplerate(gfp,44100);
    lame_set_brate(gfp,256);
    lame_set_mode(gfp,1);
    lame_set_quality(gfp,5);
    if (lame_init_params(gfp)<0)
    {
        return 1;
    }

}

encode()
{
    x=0;
    mp3buffersize=(1.25*countsize+7200);
    mp3buffer=malloc(mp3buffersize);
    while (x!=countsize)
    {
        lame_encode_buffer(gfp,lpcmbuffer,rpcmbuffer,x,mp3buffer,mp3buffersize);
        x++;
        y++;
        if(y==1000)
        {
            printf("%d     %d\n",countsize,x);
            y=0;
        }
    }
    x=0;
    lame_encode_flush(gfp,mp3buffer,mp3buffersize);
    fpo=fopen("test.mp3","w");
    fwrite(mp3buffer,1,countsize,fpo);
}

decode()
{
}

bounty()
{
    //the quicker picker upper
    printf("closing files\n");
    fclose(fpo);
    fclose(fp);
    printf("closing lame\n");
    lame_close(gfp);
    printf("freeing pcmbuffer\n");
    free(pcmbuffer);
    free(lpcmbuffer);
    free(rpcmbuffer);
    free(mp3buffer);
}

int main(int argc,char **argv)
{
    loopcount=atoi(argv[1]);
    fp=fopen(argv[2],"r");
    if (fp==NULL)
    {
        printf("File Read Error\n");
        return 0;
    }
    fseek(fp,0,SEEK_END);
    inputSize=ftell(fp);
    fseek(fp,0,SEEK_SET);
    printf("detected a %d byte(s) file\n",inputSize);
    printf("Proceeding with parsing and importing...\n");
    if (parse()==1)
    {
        printf("lame init error\n");
    }
    printf("loopcount is %d\n",loopcount);
    encode();
    //the Quicker Picker Upper
    bounty();
    return 0;
}

Answer 1

简短回答，将其作为编码功能：

void encode()
{
    mp3buffersize=(1.25*countsize+7200);
    mp3buffer=malloc(mp3buffersize);
    lame_encode_buffer(gfp, lpcmbuffer, rpcmbuffer, countsize, mp3buffer, mp3buffersize);
    lame_encode_flush(gfp,mp3buffer,mp3buffersize);
    fpo=fopen("test.mp3","w");
    fwrite(mp3buffer,1,countsize,fpo);
}

我从来没有使用过跛脚，但是，它看起来像你encode()函数一次又一次地调用lame_encode_buffer()，每次都覆盖结果，从0到countsize lame_encode_buffer_interleaved()作为每个通道的样本数（参数4）。

其他评论：

为什么不使用parse()？您encode()函数的大部分内容只是撤消文件的现有交错，看起来很浪费。

IMO，你正在使用的大量全局变量是UGLY。理想情况下，您的encode(lame_global_flags *gfp, const short * lpcmbuffer, const short * rpcmbuffer, const int countsize)看起来更像：const这样，通过读取参数列表变量的类型，以及它们必须来自/已由调用者设置，这一点很清楚。 encode()很高兴澄清他们只是为了阅读。

最后，你真的应该做一些分析，例如打印功能的进入和退出之间的时间差异，以确定您的时间接收器的位置，并发布您找到的内容。我冒昧地猜测你的循环，{{1}}函数有唯一的循环，里面有任何肉。我从来没有运行你的程序，也许我100％错了。

libmp3lame编码为char数组慢

1 个答案: