Sichtbarkeit von Objekten in switch-Anweisung

[Eisflamme]

Wozu das alles?

[Nasghat]

Für den Moment möchte ich einfach nur, dass wenn der Benutzer in einem Menü "1" eingibt, ein neues Objekt erstellt wird und wenn er eine "2" eingibt, der Status der Objekte ausgegeben wird.

MkG Nasghat

[Gast]

Nasghat schrieb:

Allerdings sind Objekte, die auf dem Heap erstellt worden sind doch gar nicht an Blöcke gebunden, sondern existieren solange, bis sie mit delete gelöscht werden.

Wenn du etwas auf dem Heap anlegst, dann verliert die Variable, mit der du einen Wert (im Heap) ansprichst, ihre Gültigkeit, jedoch das, das hinter der Variable steht, also der Wert im Speicher, bleibt weiterhin geschützt, d.h. der Speicher wird nicht freigegeben.

lg

[SideWinder]

Nasghat schrieb:

Für den Moment möchte ich einfach nur, dass wenn der Benutzer in einem Menü "1" eingibt, ein neues Objekt erstellt wird und wenn er eine "2" eingibt, der Status der Objekte ausgegeben wird.

Die Pointer müssen dazu aber schon vorher existieren:

Einheit *zug1 = null;
...

switch(...) {
    case 1: zug1 = new Einheit(); ... /* was wenn zug1 schon irgendwo hingezegit hat -> falls != 0 sollte hier noch ein delete passieren (*) */ break;
    case 2: zug1->doSomething(); /* achtung besser auf null prüfen */ break;
}

Oder sollen bei () zusätzliche Objekte angelegt werden? Dann brauchst du einen std::vector<Einheit>.

MfG SideWinder

[Gast]

Nasghat schrieb:

Für den Moment möchte ich einfach nur, dass wenn der Benutzer in einem Menü "1" eingibt, ein neues Objekt erstellt wird.

Da dabei ja erst zur Laufzeit entschieden wird, wie viele Objekte letztendlich erstellt werden... dann nimm doch einfach ein std::vector und lege die Objekte darin ab.

lg

[Nasghat]

Ich habe es mit einem Vector probiert, so klappt es auch erstmal...jetzt muss ich nur noch den Fall abfangen, dass, falls noch kein Objekt vom Typ Einheit besteht, der vector nicht auf etwas zeigt, das es nicht gibt.
Hier mal kurz der Code mit dem Vector:
Auf jeden Fall schon mal besten Dank für die Ideen.

//main.cpp
#include <vector>
#include <iostream>
#include "interface.h"
#include "einheiten.h"

using namespace std;

int main(void)
{
	InterfaceMenue *ifm = new InterfaceMenue;
	vector<Einheit> vEinh;
	//vector<Einheit>::iterator vEinh_Iter;

	while (true)
	{
	switch (ifm->start()) // Objektmethode gibt einen Int-Wert zurück
	{
	case 1:
		cout << "Eingabe war 1\n";
		if (ifm->erstellen() == 1) // Nach Benutzereingabe werden die Objekte erstellt
		{
			Einheit *Zug1 = new Einheit;
			vEinh.push_back(*Zug1);
		} else
			ifm->start();
		break;
	case 2:
		{
		cout << "Eingabe war 2\n"; 
		ifm->status(vEinh[0]); 		// Hier soll der Status des Objekts ausgegeben werden	
		break;
		}
	case 3:
		{
		cout << "Eingabe war 3\n";
		break;
		}
	case 4:
		{
		cout << "Eingabe war 4\n";
		break;
		}
	case 5:
		{
		cout << "Eingabe war 5\n";
		exit(1);
		}
	}
	}
	cin.get();
	return 0;
}

[Memorywatcher]

Vermeide new so oft es geht, dabei einstehendir wie hier gerne Memory Leaks. Falls du doch etwas auf dem Heap erstellen willst, solltest du lieber den std::unique_ptr nehmen.

InterfaceMenue *ifm = new InterfaceMenue;
// besser
InterfaceMenue ifm;

// Achtung, Memory Leak!
Einheit *Zug1 = new Einheit;
vEinh.push_back(*Zug1);
// besser
Einheit Zug1;
vEinh.push_back(Zug1);

[SeppJ]

Ich sehe immer noch nicht die Notwendigkeit, überhaupt new zu benutzen. Warum nicht die konkreten Objekte im Vector ablegen?

[314159265358979]

Smartpointer...

[SeppJ]

314159265358979 schrieb:

Smartpointer...

Nochmal: Wieso überhaupt Pointer? Distanziert euch doch mal von dem was der Threadersteller gemacht hat und schaut auf das was er will. Und dann überlegt, wie ihr das machen würdet.

[314159265358979]

Sorry, habe die Seite 2 des Threads komplett übersehen. Kannst meinen Post (und diesen) löschen.

[SeppJ]

314159265358979 schrieb:

Sorry, habe die Seite 2 des Threads komplett übersehen. Kannst meinen Post (und diesen) löschen.

Ach, ist schon ok, zum Thema Pointer passt es und vielleicht will ja noch jemand etwas dazu lernen.

[Rauschig]

#include <vector> 
#include <iostream>
#include <cstdlib> /* --> exit(1) */ 
#include "interface.h" 
#include "einheiten.h"

using namespace std;

typedef void (COMMAND*)(InterfaceMenu*, vector<Einheit>*);

void FirstCmd(InterfaceMenu* ifm, vector<Einheit> *vEinh){
 if(ifm->erstellen() == 1)
  vEinh->push_back(Einheit::Einheit());
}

void SecondCmd(InterfaceMenu* ifm, vector<Einheit> *vEinh){
 ifm->status(&vEinh[0]);
}

void DoNothingCmd(InterfaceMenu* ifm, vector<Einheit> *vEinh){
}

void QuitCmd(InterfaceMenu* ifm, vector<Einheit> *vEinh){
 exit(1);
}

COMMAND Reaktion[] = {DoNothingCmd, FirstCmd, SecondCmd, DoNothingCmd, DoNothingCmd, QuitCmd};

int main(){
 int iStartRet;
 vector<Einheit> vEinh;
 InterfaceMenu ifm;
 do{
  if((iStartRet = ifm.start()) < 6 && iStartRet > 0){
	cout << "Eingabe war " << iStartRet << "\n";
	Reaktion[iStartRet](&ifm, &vEinh);
  }
 }while(true);
 cin.get();
 return 0;
}

Ungetestet, aber könnte (ev. mit ein paar Korrekturen) so klappen.
Immerhin das new und das switch hätte man so wegbekommen,
weil die haben ja die Probleme verursacht? Um das ging's doch?

[Rauschig]

ifm->status(&(*vEinh)[0]); /* Zeile 17 */

[Nasghat]

Das erste new ist in der Tat unnütz...ist mir heute auch in den Sinn gekommen. Danke für den Hinweis.
Das etwas Verwirrende an meinem Code ist: so wie er da steht ist das new überflüssig. Allerdings soll das Programm später zur Laufzeit Objekte erstellen, die von dem Benutzer geändert werden können, ebend durch das Menü (ifm Objekt).
Wenn ich das eichtig gelesen habe, dann ist die einzige Alternative zu Pointer eine Referenz oder?
Dann werde ich als erstes mal die Pointer aus dem Code verbannen. Mit Typedef und den smart_pointern habe ich mich noch nicht beschäftigt. Ich denke von der Benutzung der STL/Boost bin ich auch noch ein Stück entfernt, mal abgesehen von vector vielleicht

mkG Nas

Edit: mein Lösung bis jetzt, ohne new und ohne Pointer
Danke für die Hilfestellungen!

//main.cpp
#include <vector>
#include <iostream>
#include "interface.h"
#include "einheiten.h"

using namespace std;

int main(void)
{
	InterfaceMenue ifm;
	vector<Einheit> vEinh;

	while (true)
	{
	switch (ifm.start()) // Objektmethode gibt einen Int-Wert zurück
	{
	case 1:
		cout << "Eingabe war 1\n";
		if (ifm.erstellen() == 1) // Nach Benutzereingabe werden die Objekte erstellt
		{
			vEinh.push_back(Einheit::Einheit());	// neues Objekt wird in den vector eingefügt
		} else
			ifm.start();
		break;
	case 2:
		{
		cout << "Eingabe war 2\n"; 
		for (size_t i = 0; i < vEinh.size(); i++)
		{
			ifm.status(vEinh[i]); 		// Hier soll der Status des Objekts ausgegeben werden
		}
		break;
		}
	case 3:
		{
		cout << "Eingabe war 3\n";
		break;
		}
	case 4:
		{
		cout << "Eingabe war 4\n";
		break;
		}
	case 5:
		{
		cout << "Eingabe war 5\n";
		exit(1);
		}
	}
	}
	cin.get();
	return 0;
}