[C++/CLI]Private var wordt in eigen klasse niet herkend *

[BlackBurn]

Ik ben bezig met een avondcursus HBO TI, nu zijn we net begonnen met C++(.net) en over het algemeen lukt het vrij aardig, echter loop ik nu een beetje vast.

Ik heb een klasse TuneGenerator, die heeft de volgende header file.

C++:

using namespace Microsoft::DirectX::DirectSound;
using namespace System; //Nodig om Math te kunnen gebruiken.

public ref class TuneGenerator{
    public:
        TuneGenerator(WaveFormat waveFormat); //Constructor
        void geneerToon(); //Genereert een PiepToon
        void geneerDoom(); //Genereert een DoomToon
        array<unsigned char> ^getBuffer();
    private:
        array <unsigned char> ^mixBuffer;  //Array die de Buffer bevat.
        WaveFormat waveFormat; //Beschrijft de "wave file"

De constructor in de CPP file ziet er als volgt uit.

C++:

TuneGenerator::TuneGenerator(WaveFormat wFormat){
    //Constructor
     
    //Stel waveformat in, om de tonen juist te kunnen genereren.
    waveFormat = wFormat;
    Console::WriteLine("Samples per second: "+waveFormat.SamplesPerSecond); //Debug
    //waveFormat.SamplesPerSecond=22050;

    //Vul de array
    mixBuffer = gcnew array <unsigned char> (waveFormat.SamplesPerSecond);
};

Vanuit een andere klasse wordt de waveFormat gedefinieerd, en wordt er een object aangemaakt van de TuneGenerator, waar de waveFormat aan meegegeven wordt. Nu zou naar mijn inziens waveFormat bekend moeten zijn binnen het aangemaakte object TuneGenerator, in de constructor, schrijft deze ook netjes naar de console de waarde "22050".

Echter nu wordt er vanuit mijn TuneBeheer een methode aangeroepen in de klasse TuneGenerator.

C++:

void TuneGenerator::geneerToon(){
    int intVolume = 100;
    int intFrequentie = 2000;

    Console::WriteLine("Samples per second: "+this->waveFormat.SamplesPerSecond);
    for(int i=0;i< waveFormat.SamplesPerSecond-1;i++){
        mixBuffer[i]=(unsigned char)
            (intVolume*Math::Sin((double(i*2*Math::PI*intFrequentie/(waveFormat.SamplesPerSecond)))));
    }
};

Echter tijdens het draaien klapt deze er hier uit met een "System.NullReferenceException" error. m.b.v. een breakpoint, kwam ik erachter dat het gebeurde op this->waveFormat.SamplesPerSecond.
Als ik tijdens het debuggen ook de waveFormat controleer, staat er ook idd achter SamplesPerSecond, "error: cannot obtain value".

Iemand die mij hier wat licht in de duisternis kan brengen, waardoor de waarde van de waveFormat niet netjes doorgegeven worden?

[whoami]

dit duidt gewoon dat je ergens naar een variable verwijst die niet gealloceerd is. Hoe creeër je een instance van die class ? Je geeft wel degelijk een WaveFormat mee ?
Hoe wordt die waveFormat gecreeërd / gedefinieerd, en geeft SamplesPerSecond wel een waarde ?

[BlackBurn]

whoami schreef op vrijdag 02 maart 2007 @ 11:52:
dit duidt gewoon dat je ergens naar een variable verwijst die niet gealloceerd is. Hoe creeër je een instance van die class ? Je geeft wel degelijk een WaveFormat mee ?
Hoe wordt die waveFormat gecreeërd / gedefinieerd, en geeft SamplesPerSecond wel een waarde ?

In de constructor van de TuneBeheer klasse, staat dit.

C++:

TuneBeheer::TuneBeheer(Device ^sound){
    //SoundDevice toekennen
    devSound = sound;

    //WaveFormat instellen
    waveFormat.FormatTag = WaveFormatTag::Pcm;
    waveFormat.SamplesPerSecond=22050;
    waveFormat.BitsPerSample=16;
    waveFormat.Channels=1;
    waveFormat.BlockAlign=(short)(waveFormat.Channels * (waveFormat.BitsPerSample/8));
    waveFormat.AverageBytesPerSecond= waveFormat.SamplesPerSecond*waveFormat.BlockAlign; 
    
    //Bufferdescriptie instellen
    bufferDescription= gcnew BufferDescription(waveFormat);
    bufferDescription->BufferBytes=waveFormat.AverageBytesPerSecond;
    
    //Buffer instellen
    tuneBuffer = gcnew SecondaryBuffer(bufferDescription, devSound);
    
    //TuneGenerator aanmaken
    TuneGenerator ^objTuneGenerator = gcnew TuneGenerator(waveFormat); //Maak een object TuneGenerator aan.
};

Onderaan wordt de TuneGenerator aangemaakt, met de waveFormat die daarboven is gedefinieerd.
Het gekke is ook dat op het moment dat de constructor van de tunegenerator wordt uitgevoerd, de waarde netjes op het console geprint wordt.

[whoami]

En tussen het creeëren van je tune-generator en het aanroepen van die bewuste method wordt die waveFormat nergens op null gezet ofzo ?
Heb je al eens stap voor stap door je code gelopen ?

[BlackBurn]

Ik heb de code inderdaad al een keer stap voor stap doorlopen, maar alle code die ook maar iets doet met waveFormat staat hier.

Na de initialisatie wordt ook direct genereerToon aangeropen.

[BlackBurn]

whoami schreef op vrijdag 02 maart 2007 @ 12:05:
En tussen het creeëren van je tune-generator en het aanroepen van die bewuste method wordt die waveFormat nergens op null gezet ofzo ?
Heb je al eens stap voor stap door je code gelopen ?

Hmm, het lijkt erop dat er inderdaad wel iets mee gebeurd, als ik onder de regel, waar ik objTuneGenerator aanmaak, de genereertoon aanroep, gaat het wel goed.

Maar als ik hem direct daarna vanuit een andere methode aanroep gaat het mis, ik ga even verder puzzelen.

[.oisyn]

whoami schreef op vrijdag 02 maart 2007 @ 12:05:
En tussen het creeëren van je tune-generator en het aanroepen van die bewuste method wordt die waveFormat nergens op null gezet ofzo ?
Heb je al eens stap voor stap door je code gelopen ?

waveFormat kan niet op null worden gezet - het is immers geen (managed) pointer. Wat ik me echter afvraag is waar je de referentie naar de instance van je TuneGenerator opslaat. Want in TuneBeheer::TuneBeheer is dit een lokale variabele. Mijn gok is dus dat je ergens anders tuneGen->genereerToon() aanroept, terwijl tuneGen null is.

Dat verklaart ook die NullReferenceException - this is null in genereerToon().

.edit: deze taal heet trouwens C++/CLI, niet C++

[ Voor 8% gewijzigd door .oisyn op 02-03-2007 13:02 ]

[BlackBurn]

Het verhaal klinkt logisch en het lijkt er inderdaad op dat het zoiets dergelijks is, maar volgens mij sla ik het goed op, althans ik kan het niet vinden.

C++:

#include "TuneGenerator.h"

using namespace Microsoft::DirectX;
using namespace Microsoft::DirectX::DirectSound;
using namespace System;
using namespace System::IO;

public ref class TuneBeheer{
    public:
        TuneBeheer(Device ^sound);
        //void speelToon(String ^toon);
        void speelToon();
    private:
        Device ^devSound;
        SecondaryBuffer ^tuneBuffer;
        BufferDescription ^bufferDescription;
        TuneGenerator ^objTuneGenerator;
        array <unsigned char> ^mixBuffer; 
        WaveFormat waveFormat;
        MemoryStream ^mStream;
};

En in de CPP van TuneBeheer, staat ook deze methode, die de methode genereertoon aanroept.

C++:

void TuneBeheer::speelToon(){
    //Afspelen van de toon
    //Toon kiezen
    objTuneGenerator->geneerToon();

    //Buffer vullen
    mixBuffer = objTuneGenerator->getBuffer();

    mStream = gcnew MemoryStream(mixBuffer);
    tuneBuffer->Write(0,mStream,tuneBuffer->Caps.BufferBytes,LockFlag::None);
    tuneBuffer->Play(0,BufferPlayFlags::Default);
}

Het gaat dus mis bij de eerste regel van deze methode.
Deze methode, wordt wel aangeroepen vanuit een andere klasse, de complete werking is als volgt.

TuneMaker klasse -> in de constructor wordt een TuneBeheer object aangemaakt.
TuneBeheer object wordt aangemaakt vanuit TuneMaker en maakt een TuneGenerator object

Vanuit het object TuneMaker, wordt de functie speelToon in TuneBeheer aangeroepen.

Hieronder nog de TuneMaker.cpp (wel wat form-opmaak code weggehaald).

C++:

// TuneMaker.cpp : main project file.
#include "stdafx.h"
#include "TuneBeheer.h"

#using <System.dll>
#using <System.Drawing.dll>
#using <System.Windows.Forms.dll>

using namespace Microsoft::DirectX;
using namespace Microsoft::DirectX::DirectSound;
using namespace System; 
using namespace System::ComponentModel;
using namespace System::Drawing;
using namespace System::Windows::Forms;

public ref class TuneMaker:public Form{
    private:
<KNIP>
        
        //Overige objecten
        Device ^devSound; //Sound device, legt de link tussen de applicatie en de geluidskaart
        SecondaryBuffer ^buffer; //Buffer bevat het af te spelen te bestand.

        //TuneBeheer object
        TuneBeheer ^objTuneBeheer;

<KNIP>
    public:
        TuneMaker(){
            //Declaraties en initialisatie
            devSound = gcnew Device();
            //Zorgt ervoor dat de kaart samenwerkt met andere programma's
            devSound->SetCooperativeLevel(this, CooperativeLevel::Normal); 
            //Buffer declaren, anders kan het volume niet meteen ingesteld worden
            buffer= gcnew SecondaryBuffer("test.wav",devSound);

            //Object TuneBeheer aanmaken                    
            objTuneBeheer = gcnew TuneBeheer(devSound);


            //Bouw het scherm op
            Setup_Form();
        };
};


int main(void)
{   
    Console::WriteLine("TuneMaker Form");
    
    //Creeër een formulier
    Application::Run(gcnew TuneMaker());
    return 0;
}

[.oisyn]

Nee, je slaat het niet goed op:

BlackBurn schreef op vrijdag 02 maart 2007 @ 11:59:

In de constructor van de TuneBeheer klasse, staat dit.

C++:

TuneBeheer::TuneBeheer(Device ^sound){
    // ....

    //TuneGenerator aanmaken
    TuneGenerator ^objTuneGenerator = gcnew TuneGenerator(waveFormat); //Maak een object TuneGenerator aan.
};

Je definieert hier een lokale variabele objTuneGenerator. Je slaat 'm niet op in this->objTuneGenerator.

[BlackBurn]

.oisyn schreef op vrijdag 02 maart 2007 @ 14:25:
Nee, je slaat het niet goed op:


[...]

Je definieert hier een lokale variabele objTuneGenerator. Je slaat 'm niet op in this->objTuneGenerator.

Dat was inderdaad de oplossing

Dat ik daar zolang naar moest zoeken Harstikke bedankt!