5

void配列をtypedefにキャストするベストプラクティスは何ですか?

これは私が現在やっている方法です:

typedef struct {
    int16_t left,right;
} SampleStereo16;

void *buffer[100000]; // data buffer
SampleStereo16* sample;

// insert something here that writes to the buffer

sample = (SampleStereo16*) buffer;

これはうまくいくようですが、どういうわけかもっと良い方法があると感じています。また、サンプル配列とバッファが同じメモリを共有する方法があるかどうかも疑問に思っていました (現在、それぞれが独自のメモリを使用しています)。

ここに私が現在取り組んでいる完全なコードがあります:

#include "fmod.h"
#include "fmod.hpp"
#include "fmod_errors.h"
#include "wincompat.h"
#include <stdio.h>

typedef struct {
    int16_t left,right;
} SampleStereo16;

void ERRCHECK(FMOD_RESULT result)
{
    if (result != FMOD_OK)
    {
        printf("\nFMOD error! (%d) %s\n", result, FMOD_ErrorString(result));
        if (result != FMOD_ERR_FILE_EOF)
            exit(-1);
    }
}


int main(int argc, char *argv[])
{
    FMOD::System     *system;
    FMOD::Sound      *sound;
    FMOD_RESULT       result;
    unsigned int      version;

    int channels;
    int bits;

    unsigned int         lenbytes; // length in bytes read
    void                 *buffer[1000000]; // data buffer
    unsigned int         *read; // number of bytes actually read
    unsigned int        position; // offset in PCM samples
    unsigned int         samplesread; // number of PCM samples read
    unsigned int        samplesbuffer; //
    unsigned int        cueposition;

    SampleStereo16*     sample;


    /*
        Create a System object and initialize.
    */

    result = FMOD::System_Create(&system);
    ERRCHECK(result);

    system->getVersion(&version);

    result = system->getVersion(&version);
    ERRCHECK(result);

    if (version < FMOD_VERSION)
    {
        printf("Error!  You are using an old version of FMOD %08x.  This program requires %08x\n", version, FMOD_VERSION);
        getch();
        return 0;
    }

    result = system->setOutput(FMOD_OUTPUTTYPE_ALSA);
    ERRCHECK(result);

    result = system->init(32, FMOD_INIT_NORMAL, 0);
    ERRCHECK(result);

    result = system->createStream("/home/dpk/Dropbox/Music/Will Smith - Miami.mp3", FMOD_SOFTWARE, 0, &sound);


    result = sound->getFormat(0, 0, &channels, &bits);
    ERRCHECK(result);

    printf("channels : %d   bits : %d \n", channels, bits);

    if (channels!=2 and bits!=16)
    {
        printf("File must be stereo (2 channels) 16 bits \n");
        exit(-1);
    }

    lenbytes = sizeof(buffer);
    samplesbuffer = lenbytes / channels / ( bits / 8 );
    position = 0;
    cueposition = 0;

    do
    {
        result = sound->seekData(position);
        ERRCHECK(result);

        printf("Reading block : %u  ",position);

        result = sound->readData(&buffer, lenbytes, read);
        ERRCHECK(result);

        samplesread = *read / channels / ( bits / 8 );

        sample = (SampleStereo16*) buffer;

        printf("number of PCM samples read : %u \n", samplesread);


        for(unsigned int i=0; i<samplesread; i++)
        {
            if (cueposition==0 && ( abs(sample[i].left)>500 || abs(sample[i].right)>500 ) )
            {
                cueposition = position+i;
                printf("cue point : %u \n", cueposition);
            }
        }

        position += lenbytes / channels / ( bits / 8 );

    } while(samplesread==samplesbuffer);

    printf("\nExit\n");

    /*
        Shut down
    */
    result = sound->release();
    ERRCHECK(result);
    result = system->close();
    ERRCHECK(result);
    result = system->release();
    ERRCHECK(result);

    return 0;
}

バッファを増やしすぎるとセグメンテーション違反も発生しますが、これまでに見つけたものからは、スタック サイズの制限のようです。

また、私が間違っていることについて自由にコメントしてください。私はつい最近 (先週) C++ を始めたばかりなので、私のコードにはいくつかの見栄えが悪いと確信しています。

4

2 に答える 2

0

読み取られたサンプル数を把握するには、 sizeof 演算子を使用する必要があります。つまり、バイト配列はサイズ (SampleStereo16) の偶数倍である必要があります。

次に、通常、バイト配列をそのサイズのチャンクでウォークします。malloc(sizeof(SampleStereo16)) して各 SampleStereo16* ポインターを取得し、バイト配列からその量のデータを memcpy して、ベクターまたは配列に追加します。

構造体の各配列を一貫してそのようにパックおよびアンパックすれば、問題はないはずです。

また、後で構造体にフィールドを追加することにした場合に備えて、シリアル化されたファイルの最初の数バイトを何らかのバージョン番号に割り当てることも非常に一般的ですが、ここでのユースケースではないようです。コードサンプル全体。

于 2013-05-01T15:42:49.573 に答える