Moin... mal wieder ein kleines c++ Anfängerproblem.

Ich habe mir eine Klasse StackAsFixedVector geschrieben.

Also einen Stack der auf einem nicht dynamischen Array basiert.

Hier die Implementierung der Klasse StackAsFixedVector.

Der ganze Spass steht in der Datei "J_StackAsFixedVector":

//   int read_top          Oberstes Element lesen

//   void clear_top        Oberstes Element entfernen

//   void write_top(wert)  Ein Element obendrauflegen

//   void clear            Stack leeren

//   bool empty            Ist der Stack leer?



class StackAsFixedVector

{

private:

   int data[5];

   int stack_size;

public:

   StackAsFixedVector() //Konstruktor

   {

   int stack_size = -1;

   }

   int read_top()//Oberstes Element lesen

   {

   return data[stack_size];

   }

   void clear_top()//Oberstes Element entfernen

   {

   stack_size = stack_size-1;

   }

   void write_top(int wert)//Ein Element obendrauflegen

   {

   stack_size = stack_size + 1;

   data[stack_size] = wert;

   }

   void clear()//Stack leeren      

   {

   stack_size = -1;

   }

   bool empty()//Ist der Stack leer?      

   {

   return stack_size == -1;

   }   

};

und nun hier die Implementierung der main.cpp:

#include <iostream>

#include "J_StackAsFixedVector.h"

using namespace std;


int main()

{

StackAsFixedVector stack; //Stack erzeugen


cout<<"Zahl eingeben: "; int zahl; cin>>zahl;

stack.write_top(zahl);



cin.get();

cin.get();

}

Das Programm wird zwar kompiliert, bricht aber mit der folgenden Fehlermeldung ab nachdem die Zahl eingegeben wurde und stack.write_top(zahl) aufgerufen wird:

"Unbenannt3.exe hat ein Problem festgestellt und muss beendet werden."

Kann mich irgendjemand auf den Fehler im Code aufmerksam machen den ich gemacht haben muss?

mfg

Jonsc1

   StackAsFixedVector() //Konstruktor

   {

   int stack_size = -1;

   }[/code]
Hier definierst du eine lokale Variable. Die Membervariable stack_size änderst du damit nicht.

Logisch... soll ja auch ein Konstruktor sein...

Also einfach aus "int stack_size = -1" ein "stack_size = -1" gemacht und schon läufts... und wegen so was hab ich mir ne Stunde lang den Kopf zerbrochen...

Danke dir!

Mit einer Intialisierungsliste wäre das nicht passiert:

   StackAsFixedVector() : stack_size(-1)

   {

   }[/code]

Mit einer Intialisierungsliste wäre das nicht passiert:

   StackAsFixedVector() : stack_size(-1)

   {

   }

So weit bin ich noch nicht

Noch nicht gelernt...