mettre une partie de buffer à 0

**madmax51** · 31/05/2010, 17h40

Bonjour,

j'ai un buffer que je rempli ainsi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
m_headerbufferlen = m_wfx.nAvgBytesPerSec/20 * 2;
for(int t=0; t<HDRCUNT; t++){
    CWaveHdr * phdr = new CWaveHdr();
    ZeroMemory(phdr, sizeof(CWaveHdr)); 
    phdr->lpData = new char[m_headerbufferlen];
    m_freeq.push(phdr);
}

et je souhaiterai forcer à 0 du début du buffer (pour l'instant) phdr juska la fin, 16bits à 0 et ainsi de suite tous les 16 bits.. coment faire cela?

pour info, il s'agit d'un fichier wave, stéréo,16bits, et je veux forcer un canal à 0 afin de pouvoir en faire un fichier mono en forcant ensuite le canal à 0.
Merci

**3DArchi** · 31/05/2010, 23h04

Salut et bienvenu,
A moins que CWaveHdr ne soit un type POD, j'ai peur que ce que tu demandes ne t'entraine vers quelque soucis. Une structure/classe C++ n'est pas une structure C.
Si tu veux initialiser les membres de CWaveHdr, utilises un constructeur adéquat.

**madmax51** · 01/06/2010, 09h08

Mais je ne veux pas faire ca à l'initialisation, je veux faire un traitement à un moment donné.. et je me retrouve avec ce buffer à qui je doit mettre en gros la moitié des bits à zéro.. c'est pas possible??

**3DArchi** · 01/06/2010, 13h36

quelle est la définition de CWaveHdr ?

**madmax51** · 01/06/2010, 14h35

la voila..

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class CWaveHdr:
public WAVEHDR{
 
};
 
 
/* wave data block header */
typedef struct wavehdr_tag {
    LPSTR       lpData;                 /* pointer to locked data buffer */
    DWORD       dwBufferLength;         /* length of data buffer */
    DWORD       dwBytesRecorded;        /* used for input only */
    DWORD_PTR   dwUser;                 /* for client's use */
    DWORD       dwFlags;                /* assorted flags (see defines) */
    DWORD       dwLoops;                /* loop control counter */
    struct wavehdr_tag FAR *lpNext;     /* reserved for driver */
    DWORD_PTR   reserved;               /* reserved for driver */
} WAVEHDR, *PWAVEHDR, NEAR *NPWAVEHDR, FAR *LPWAVEHDR;

**madmax51** · 01/06/2010, 17h14

ou bien copier tout ca dans un std::vector<int> buffer ? mais idem, comment faire cela? je vois pas comment acceder à mon buffer..

**madmax51** · 01/06/2010, 17h53

si je fait ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
char *pCurrentBuffer = phdr->lpData 
int nbSamples = m_headerbufferlen / 4; // 2 canaux, 2 octets par sample 
newBuffer.reserve( nbSamples );
for( int i = 0; i < nbSamples; ++i )
{
  newBuffer.push_back( pCurrentBuffer[ i * 2 ] );
}
phdr->lpData = &newBuffer[0]; 
m_freeq.push(phdr);

ca plante mon programme... une idée?

Invité · 01/06/2010, 18h04

Bonjour,
Si j'étais vous j'essayerais de faire une union. C'est assez rarement utilisé, mais c'est bien pratique pour accéder aux membres d'une structure indirectement.

**madmax51** · 01/06/2010, 18h10

comment ca? une union de quoi et quoi? le but ici serait juste d'extraire une zone qui m'interresse dans le buffer.

Invité · 01/06/2010, 18h35

Une union est assez comparable à une structure.
Par contre, ce qui est utile est que la même zone peut être connue sous 2 noms différente et donc 2 type différents.
Je n'ai pas utilisé cette méthode très souvent, mais une toute petite lecture permet de s'y retrouver.
Le principe est le suivant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
MonUnion
{
  double D;
  int i; 
} mu;
  mu.D=3.1416;
  mu.i=0;

Alors les octets correspondant à i (4 octets probablement) sont mis à 0
donc les 4 premiers octets de D seront à 0 et les 4 autres inchangés, ce qui fait naturellement que D n'aura plus rien à voir avec 3.1416.
Un short int mesure normalement 2 octets, soit 16 bits.
Donc, avec ce principe vous devriez y arriver.

Il y a aussi les champs de bits qui permettent une décomposition au nibeau des bits.

J'ai regardé un peu plus votre structure. Il s'agit d'une structure de liste chainée, je n'ai pas très bien vu dedans lesquels 16 bits parmi 16 vous voulez mettre à 0, c'est à dire je ne comprend pas à quoi servent les différents membres de cette structure.

Il y aurait peut-être le membre dwFlags qui pourrait apporter des informations intéressantes.

**3DArchi** · 01/06/2010, 20h09

Salut,
Je ne recommande pas l'usage de l'union qui embrouillerait le code avec un pauvre hack et introduirait probablement du risque

(sans compter qu'un int peut ne pas faire 4 octets ni un short 2).

En revanche, j'ai du mal à comprendre ce que tu veux initialiser à 0 : toute la structure pointée par phdr ou un champs ? Et pourquoi passer par un std::vector ?

**madmax51** · 02/06/2010, 09h12

Ces champs sont deux bandes son d'un wave, donc je veux mettre la voie de droite ou gauche à 0..pour n'avoir qu'un seul canal. Je suis en train d'essayer de faire ca avec un std::vector mais je galère un peu.. ca ne marche pas comme je le souhaite..

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
vector< signed short > newBuffer;
signed short *pCurrentBuffer = reinterpret_cast<signed short*>( phdr->lpData );
int nbSamples = m_headerbufferlen / 4; // 2 canaux, 2 octets par sample 
newBuffer.reserve( nbSamples );
for( int i = 0; i < nbSamples; ++i )
{
  newBuffer.push_back( pCurrentBuffer[ i * 2 ] );
}
phdr->lpData = &newBuffer[0];  
m_freeq.push(phdr);

**madmax51** · 02/06/2010, 10h31

et pourquoi pas passer par un std::vector? c'est génant?

**madmax51** · 02/06/2010, 10h44

en fait j'essaye soit de mettre une partie à 0, soit d'extraire l'autre partie et de travailler avec. D'ou le std::vector car je ne voit pas comment mettre à zéro simplement.
le wave est fait ainsi : 16bit voie G,16bits voie D et ainsi de suite.. donc je doit travailler sur une certaine partie de buffer..tous les 16bits en fait.

**3DArchi** · 03/06/2010, 07h19

Le buffer associé au std::vector sera détruit au moment où le std::vector va sortir de sa portée.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
{
vector< signed short > newBuffer;
signed short *pCurrentBuffer = reinterpret_cast<signed short*>( phdr->lpData );
int nbSamples = m_headerbufferlen / 4; // 2 canaux, 2 octets par sample 
newBuffer.reserve( nbSamples );
for( int i = 0; i < nbSamples; ++i )
{
  newBuffer.push_back( pCurrentBuffer[ i * 2 ] );
}
phdr->lpData = &newBuffer[0];  
m_freeq.push(phdr);
}
// ici newBuffer est détruit et m_free.back()->lpData pointe sur une zone invalide

Ce que je comprends à partir de ton code :
-> A l'entrée, phdr->lpData contient headerbufferlen octets définissant nbrCanaux interlacés sur des mots de nbrOctets.
Par exemple avec 2 canaux :
|C10|C20|C11|C21|C12|C22....|C1N|C2N
-> A la sortie phdr->lpData contient headerbufferlen les données d'un seul canal. Le reste a été supprimé.
|C10|C11|C12|....|C1N
Est-ce bien ça ? Ou veux tu :
|C10|0|C11|0|C12|0....|C1N|0
?

**madmax51** · 03/06/2010, 09h03

c'est bien la première solution, je veux obtenir |C10|C11|C12|....|C1N..
mais si tu me dit que m_free.back()->lpData pointe sur une zone invalide, c'est pour cela que ca plante au bout d'un court instant... une idée sur comment palier ce problème?

**3DArchi** · 03/06/2010, 12h49

En reprenant bourrin ton code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
// nombre de canaux
const size_t nbr_canaux = 2;
// canal que l'on souhaite garder >=0 et <nbr_canaux:
const size_t canal = 0;
// où on va écrire la valeur
signed short *pCurrentBuffer = reinterpret_cast<signed short*>( phdr->lpData );
// la valeur à lire :
signed short const *pNext = pCurrentBuffer + canal;
for(unsigned long i=0;i<m_headerbufferlen/nbr_canaux;++i)
{
    *pCurrentBuffer = *pNext;
    // on écrit au short suivant :
    ++pCurrentBuffer;
    // mais on saute le bon nombre de canaux :
    pNext+=nbr_canaux;
}
// phdr->lpData; inchange
phdr->dwBufferLength = m_headerbufferlen/nbr_canaux;
m_freeq.push(phdr);

**madmax51** · 03/06/2010, 14h06

quand tu mets : // phdr->lpData; inchange
je suppose quand même que je dois mettre du genre : phdr->lpData = reinterpret_cast<char*> (&pCurrentBuffer [0]); sinon j'ai rien dans mon lpData ..
mais ca plante dans la boucle, à priori, il faudrait faire un /4 au lieu de /2 car m_headerbufferlen = nb samples * nb canaux(2) * taille de sample en octet(2)..
mais la encore ca plante mais plus loin..

**madmax51** · 03/06/2010, 17h33

bon sinon autre technique, recopier une voie sur lautre..histoire d'avoir la même valeur sur mes 2 voie..
en gros je rajoute ca :
{
*pCurrentBuffer = *pNext;
// on écrit au short suivant :
++pCurrentBuffer;
*pCurrentBuffer = *pNext; //copie d'une voie sur l'autre..
// mais on saute le bon nombre de canaux :
pNext = pNext + nbr_canaux;
}

ca devrait le faire nan?

**3DArchi** · 03/06/2010, 20h11

A tout hasard, comment est alloué phdr->lpData ? Avant cette fonction, j'espère.

mettre une partie de buffer à 0

C++

Discussions similaires

Partager

Partager