[C++] puntatore a struttura dichiarato (privato in una classe

[gianvituzzi]

Salve,

ho un problema con il seguente codice (che ho sintetizzato molto):

class marameo
{
public:
	void start();
	void stop();
private:
	WAVEHDR *buff;
	HWAVEIN hwi;
}

void marameo::start()
{
	WAVEHDR *buff = new WAVEHDR[2];

	waveInPrepareHeader(hwi, &buff[0], sizeof(WAVEHDR));
	waveInPrepareHeader(hwi, &buff[1], sizeof(WAVEHDR));
}

void marameo::stop()
{
	waveInUnprepareHeader(hwi, &buff[0], sizeof(WAVEHDR));
	waveInUnprepareHeader(hwi, &buff[1], sizeof(WAVEHDR));
}

in pratica ho bisogno che il puntatore a WAVEHDR sia visibile e utilizabile da start() e stop(), quindi lo dichiaro nella classe e poi lo imposto in start(). Purtroppo stop() non lo vede e il codice mi riporta "unhanlde exception" forse l'ho definito male in start() ??

grazie

[oregon]

Perchè lo hai ridefinito localmente nella start?

Dato che gia' esiste, devi scrivere

buff = new WAVEHDR[2];

E ti consiglio di

1) inizializzare il puntatore a NULL nel costruttore

2) controllare che il puntatore sia valido prima di utilizzarlo

[shodan]

Io mi chiederei anche chi libera la memoria alla fine.
Il C++ non ha nessun Garbage Collector, quindi la memoria rimane allocata fino al termine del programma.

[oregon]

Naturalmente ... ci penserà gianvituzzi nel distruttore ...

[shodan]

Veramente la domanda era rivolta al gianvituzzi, che a volte li dimentica

[oregon]

Originariamente inviato da shodan
Veramente la domanda era rivolta al gianvituzzi, che a volte li dimentica

Lo so ... ma era per dargli una mano ...

[gianvituzzi]

grazie a tutti, si avevo dimenticato. quindi in pratica:

marameo::~marameo()
{
       delete[] buff;
}

vero?

[oregon]

E tutto quello che ti ho detto sull'inizializzazione e il controllo che il puntatore sia valido?

[gianvituzzi]

marameo::marameo() : buff(NULL) { }

può andare bene questo? o magari:

marameo::marameo() { buff = {0} }

grazie

[shodan]

Originariamente inviato da gianvituzzi

marameo::marameo() : buff(NULL) { }

Questo.