#pragma once
#include <string>
#include <iostream>
#include <fstream>
#include <list>
#include <iterator>
#include <algorithm>
#include <iomanip>
#include <vector>

#include "item.h"
using namespace std;

class Item;
class Player
{
protected:
	std::list<Item*> inventory;
public:

	Player(const string& vName, const int& vMaxiWeight);
	virtual ~Player(void);

	string	getName()			const;
	int		getCurrentWeight()	const;
	int		getMaximumWeight()	const;

	void	setName();
	void	setCurrentWeight(int& plCurrentWeight);
	void	setMaximumWeight(int& plMaximumWeight);

	void	showInventory() const;
	void	addItem(Item& items);

	string makeCompareable(const string& str) const;
	bool  operator<(const Player& value) const; 
            bool  operator<(const Player& value) const; 

	 ostream& write(std::ostream& os, std::vector<Player>& allPlayers)
	 {
		 std::vector<Player>::const_iterator iter; 

	 for(iter=allPlayers.begin(); iter!=allPlayers.end(); ++iter) 
	 { 
		 cout<<"test1";
		 os	<<iter->getName()<<"\n"
			 <<iter->getMaximumWeight()<<"\n"
			 <<iter->getCurrentWeight()<<"\n";
	 }

	 std::list<Item*>::const_iterator iterInventory;
	 //inventory existiert nicht?
	 for(iterInventory=inventory.begin(); iterInventory!=inventory.end(); ++iter) 
	 {
		 cout<<"test2";
		 os<<(*iterInventory)->write(os)<<"\n"; //wieso schreibt der hier nicht rein?
	 }

	 return os;
	 }//end write()

private:
	string name;
	int currentWeight;
	int maximumWeight;

};

//hier kann ich zugreifen 
inline ostream& operator<<( ostream& out,  Player const &p ) 
{ 
	out <<"\n**********************************************"
	    <<" \nName:\t\t\t"<<p.getName()
		<<" \nMaximales Gewicht:\t"<<p.getMaximumWeight()
		<<" \naktuelles Gewicht:\t"<<p.getCurrentWeight()<<endl;	
	p.showInventory();
	out<<"**********************************************"<<endl;
	return out; 
}

folgende main

#include "stdafx.h"
#include "Item.h"
#include "Player.h"
#include "Food.h"
#include "Weapon.h"
#include <iterator>
#include <iostream>
#include <fstream>
#include <string>
#include <vector>
#include <algorithm>
#include <stdexcept>
#include <conio.h>// für _getch()

using namespace std;

void menu();

int _tmain(int argc, _TCHAR* argv[])
{
	std::vector<Player> allPlayers;
	std::ostream_iterator<Player>output(cout, " ");
	Player *pPlayer=NULL;

	//*********************************
			string plname;
			int plweight;
			cout<<"\nNeuer Spielername eingeben: ";
			cin>>plname;
			cout<<"Maximum-Gewicht eingeben:";
			cin>>plweight;;
			pPlayer= new Player(plname,plweight);
			allPlayers.push_back(*pPlayer);
			std::sort(allPlayers.begin(),allPlayers.end());

			string foodname;int feedingValue; int foodweight;
						cout<<"\nWelches Nahrungmittel wollen sie aufnehmen?:";
						cin>>foodname;
						cout<<"\nGeben Sie den Nährwert an:";
						cin>>feedingValue;
						cout<<"\nGeben Sie nun das Gewicht ein:";
						cin>>foodweight;

						Item *pItem= new Food(foodname,feedingValue,foodweight);
						std::copy(allPlayers.begin(), allPlayers.end(),output);
						cout<<" Zu welchem Spieler soll der Gegenstand hinzugefuegt werden?:";
						string pname;
						cin>>pname;
						bool exist=false;
						for (std::vector< Player >::iterator location=allPlayers.begin(); location!=allPlayers.end(); location++)
						{
							if (location->getName()==pname)
							{
								location->addItem(*pItem);
								exist=true;
							}
						}

				cout<<" Alle Spieler anzeigen:";
				std::copy(allPlayers.begin(), allPlayers.end(),output);

				ofstream outfile("test.txt", ios::out);
				pPlayer->write(outfile, allPlayers);
delete []pPlayer;
delete []pItem;

ausgabe der txt-datei ist nur der eintrag von Objekt zu sehen
aber nicht der inhalt der liste: inventory. warum?

virtual ostream &write(ostream&)=0;

in der klasse Food ist dann write definiert mit

virtual ostream &write(ostream&os)
	{
		os<<getFoodName()<<"\n"
			<<getFeedingValue()<<"\n"
			<<getFoodweight()<<"\n";
		return os;

	}

leider bekomme ich nur in die Txtdatei geschrieben die Player-daten aber keine Inventory-daten von Food-klasse oder Weaponklasse. weiterhin weiss ich auch das in der inventory nichts drin steht , jetzt die frage wie bekomme ich da was rein bzw abgespeichert in txt. dachte das wurde in main gemacht mit

pPlayer->write(outfile, allPlayers);.

hoffe das ihr mir helfen könnt.

Zweitens solltest du mal dein Design überdenken. Player::write() in der Form, wie du sie hier eingesetzt hast, ist unsinnig. Entweder du änderst sie so, daß sie nur die Daten von this ausgibt oder du machst eine statische Methode (oder gar eine globale Funktion) draus.

Und drittens ist os<<(*iterInventory)->write(os) sinnlos - write() schreibt schon selbst und gibt den fertig geschriebenen Stream zurück, den mußt du nicht nochmal auf sich selbst ausgeben.

PS: Und viertens hast du ein gewaltiges Speicherleck in deinem Hauptprogramm - du legst kiloweise Player-Objekte per new an und lässt sie dann auf dem Heap verrotten.

Erstmal versucht du, das Inventar von "this" zu schreiben, und nicht das eines der vector-Elemente.

Zweitens solltest du mal dein Design überdenken. Player::write() in der Form, wie du sie hier eingesetzt hast, ist unsinnig. Entweder du änderst sie so, daß sie nur die Daten von this ausgibt oder du machst eine statische Methode (oder gar eine globale Funktion) draus.

Und drittens ist os<<(*iterInventory)->write(os) sinnlos - write() schreibt schon selbst und gibt den fertig geschriebenen Stream zurück, den mußt du nicht nochmal auf sich selbst ausgeben.

PS: Und viertens hast du ein gewaltiges Speicherleck in deinem Hauptprogramm - du legst kiloweise Player-Objekte per new an und lässt sie dann auf dem Heap verrotten.

ok ich hab die mit delete gelöscht hab nur net das ganze main geschrieben..
das os<<(*iterInventory)->write(os) ruft die virtual funktion von Item auf die dann in der unterklasse definiert ist mit :

virtual ostream &write(ostream&os) 
    { 
        os<<getFoodName()<<"\n" 
            <<getFeedingValue()<<"\n" 
            <<getFoodweight()<<"\n"; 
        return os; 

    }

aber das problem liegt das in inventory nichts drin steht bin immoment bissl confused

Zweitens: Deine virtuelle Funktion kümmert sich doch schon selber um die Ausgaben. Der Rückgabewert ist der beschriebene Ausgabestrom - und den auf sich selbst zu schreiben macht wenig Sinn.

Und drittens: An der Aktion sind zwei(+x) Player-Objekte beteiligt - das Objekt hinter pPlayer und das/die Elemente in deinem Vector. In den Vector-Elementen hast du das Inventar umsortiert, aber in der write()-Methode gibst du das Inventar von pPlayer aus (und der wurde beim push_back() kopiert und hat von den Aktionen im vector nichts mitbekommen).
(wie ich schon sagte, du solltest den Sinn der Player::write()-Methode nochmal hinterfragen)

if(!allPlayers.empty())
					{
						string dateiname;
						cout<<"\nBitte Dateiname eingeben *.txt: ";
						cin>>dateiname;
						ofstream outfile(dateiname.c_str(), ios::out);
						if(!outfile)
						{
							cerr<<"File could not be opened"<<endl;
							exit(1);
						}
						std::vector<Player>::const_iterator iter; 
						for(iter=allPlayers.begin(); iter!=allPlayers.end(); ++iter) 
						{ 
							outfile<<iter->getName()<<"\n"
								<<iter->getMaximumWeight()<<"\n"
								<<iter->getCurrentWeight()<<"\n";

                   //outfile<< inventory....... nur wie?????		
		                            }//endfor

						//pPlayer->write(outfile, allPlayers);						
						outfile.close();

					}else {cout<<" zuerst Datensatz einlesen!\n";}

so damit habe ich die Player attribute in der Txtdatei nun fehlt nur
das was in der Liste drin steht bei inventory. das heisst ich muss irgendwie auf die attribute der abgeleiteten klassen zugreifen können bzw eben auf das was in inventory abgespeichert ist.

Schreib die Methode write() doch so um, daß sie EINEN Player (*this) ausgeben kann und ruf ihn dann in der Schleife auf (oder du wendest einfach den operator<<() aus deinem ersten Beitrag auf alle Player des vector's an).

ich hab jetzt in player nee funktion geschrieben

ostream& Player::getInventory(ostream& os)const
{
  cout<<"Inventar:"<<endl;
  std::list<Item*>::const_iterator iter;
  for(iter=inventory.begin(); iter!=inventory.end(); ++iter) 
  { 
	 return (*iter)->write(os);
  } 
	//return (*iter)->write(os); //hmm muss net sein
}

und rufe jetzt mit :

ofstream outfile(dateiname.c_str(), ios::out);
						if(!outfile)
						{
							cerr<<"File could not be opened"<<endl;
							exit(1);
						}
						std::vector<Player>::const_iterator iter; 
						for(iter=allPlayers.begin(); iter!=allPlayers.end(); ++iter) 
						{ 
							outfile<<iter->getName()<<"\n"
								<<iter->getMaximumWeight()<<"\n"
								<<iter->getCurrentWeight()<<"\n";
							outfile<<iter->getInventory(outfile);
						}

						outfile.close();

so jetzt schreibt der auch das inventar
allerdings bringt der noch nee merkwürdige adresse hinterher das muss ich noch wegbekommen.
Wie meinst du das mit dem

(oder du wendest einfach den operator<<() aus deinem ersten Beitrag auf alle Player des vector's an).

??
wie soll das aussehen? weil das hab ich mir auch schon überlegt hmmm?

ostream& Player::getInventory(ostream& os)const
{
  cout<<"Inventar:"<<endl;
  std::list<Item*>::const_iterator iter;
  for(iter=inventory.begin(); iter!=inventory.end(); ++iter) 
  { 
     (*iter)->write(os);
     os<<endl;
  }
  return os;
}

(über den Namen der Methode könnte man ja streiten ;))

Und zweitens hast du schon den Punkt entdeckt, den ich oben angesprochen habe:

CStoll schrieb:

Zweitens: Deine virtuelle Funktion kümmert sich doch schon selber um die Ausgaben. Der Rückgabewert ist der beschriebene Ausgabestrom - und den auf sich selbst zu schreiben macht wenig Sinn.

Die Anweisung outfile<< iter->getInventory(outfile); führt erst die obige Methode getInventory() aus (die schreibt das Inventar in deine Datei) und schreibt anschließend den Rückgabewert (der identisch ist mit outfile) hinterher - ein ofstream kann nicht ge"schrieben" werden, aber er hat einen Umwandlungsoperator nach void* (hauptsächlich für Konstrukte ala "while(datei)...") und wie er einen void* schreiben soll, weiß der Stream - als "nee merkwürdige Adresse". Lass einfach den Teil "outfile<<" an dieser Anweisung weg, um das zu umgehen.

(oder du wendest einfach den operator<<() aus deinem ersten Beitrag auf alle Player des vector's an).

??
wie soll das aussehen?

Dazu mußt du den operator von oben noch so erweitern, daß er auch das Inventar ausgibt - und schon kann der Schleifenrumpf zusammengestaucht werden auf ein einfaches outfile<<*iter; .

so jetzt zum speicherleck problem, ich hoffe es ist jetzt nicht so dreist
die komplette main zu posten. aber ich weiss nicht wo ich die delete´s reinsetzten soll, muss!

// KLSS04.cpp : Definiert den Einstiegspunkt für die Konsolenanwendung.
//

#include "stdafx.h"
#include "Item.h"
#include "Player.h"
#include "Food.h"
#include "Weapon.h"
#include <iterator>
#include <iostream>
#include <fstream>
#include <string>
#include <vector>
#include <algorithm>
#include <stdexcept>
#include <conio.h>// für _getch()

using namespace std;

void menu();

int _tmain(int argc, _TCHAR* argv[])
{
	std::vector<Player> allPlayers;
	std::ostream_iterator<Player>output(cout, " ");

	Player *pPlayer=NULL;

	menu();
	bool end=false;
	do
	{

		char choice;
		choice = _getch();

		switch(choice)
		{
		case '1': 
			{	string plname;
			int plweight;
			cout<<"\nNeuer Spielername eingeben: ";
			cin>>plname;
			cout<<"Maximum-Gewicht eingeben:";
			cin>>plweight;;
			pPlayer= new Player(plname,plweight);
			allPlayers.push_back(*pPlayer);
			std::sort(allPlayers.begin(),allPlayers.end());
			cin.clear();

			}break;
		case '2':	
			{
				cout<<" Alle Spieler anzeigen:";
				std::copy(allPlayers.begin(), allPlayers.end(),output);

			}break;

		case '3':
			{
				if(!allPlayers.empty())
				{
					cout<<" Gegenstand aufnehmen:\n";

					cout<<"Was wollen Sie aufnehmen Food oder Weapon?:";
					cout<<"\n0 Food:\n"
						<<"1 Weapon:\n"
						<<"ihre Wahl?:";

					char choice;
					choice = _getch();

					switch(choice)
					{
					case '0':{
						string foodname;int feedingValue; int foodweight;
						cout<<"\nWelches Nahrungmittel wollen sie aufnehmen?:";
						cin>>foodname;
						cout<<"\nGeben Sie den Naehrwert an:";
						cin>>feedingValue;
						cout<<"\nGeben Sie nun das Gewicht ein:";
						cin>>foodweight;

						Item *pItem= new Food(foodname,feedingValue,foodweight);
						std::copy(allPlayers.begin(), allPlayers.end(),output);
						cout<<" Zu welchem Spieler soll der Gegenstand hinzugefuegt werden?:";
						string pname;
						cin>>pname;
						bool exist=false;
						for (std::vector< Player >::iterator location=allPlayers.begin(); location!=allPlayers.end(); location++)
						{
							if (location->getName()==pname)
							{
								location->addItem(*pItem);
								exist=true;
							}
						}
						if (exist==false){cout<<"Name nicht vorhanden!\n!";}
							 }
						break;
					case'1':{
						string weaponname;int attackValue; int weaponweight;
						cout<<"\nWelcher Waffe wollen sie aufnehmen?:";
						cin>>weaponname;
						cout<<"\nGeben Sie den Vernichtungsgrad an:";
						cin>>attackValue;
						cout<<"\nGeben Sie nun das Gewicht ein:";
						cin>>weaponweight;

						Item *pItem= new Weapon(weaponname,attackValue,weaponweight);
						std::copy(allPlayers.begin(), allPlayers.end(),output);
						cout<<" Zu welchem Spieler soll der Gegenstand hinzugefuegt werden?:";
						string pname;
						cin>>pname;
						bool exist=false;
						for (std::vector< Player >::iterator location=allPlayers.begin(); location!=allPlayers.end(); location++)
						{
							if (location->getName()==pname)
							{
								location->addItem(*pItem);
								exist=true;
							}
						}
						if (exist==false){cout<<"Name nicht vorhanden!\n!";}

							}break;

					default:{
						cerr<<"Incorrect choice"<<endl;
							}break;
				}//end switch	
				}else{cout<<" Kein Spieler vorhanden, erst hinzufuegen!\n";}		

			}break;

		case'4': { 
					cout<<"\nSpiel speichern";
					if(!allPlayers.empty())
					{
						string dateiname;
						cout<<"\nBitte Dateiname eingeben *.txt: ";
						cin>>dateiname;
						ofstream outfile(dateiname.c_str(), ios::out);
						if(!outfile)
						{
							cerr<<"File could not be opened"<<endl;
							exit(1);
						}
						std::vector<Player>::const_iterator iter; 
						for(iter=allPlayers.begin(); iter!=allPlayers.end(); ++iter) 
						{ 

							//outfile<<*iter; ausgabe über <<operator
							outfile<<iter->getName()<<"\n"
								<<iter->getMaximumWeight()<<"\n"
								<<iter->getCurrentWeight()<<"\n";
							outfile<<iter->getInventory(outfile);
						}
						outfile.close();

					}else {cout<<" zuerst Datensatz einlesen!\n";}
				 }break;
		case'5':{
					cout<<"\nSpiel laden\n";
				}break;

		case '6':{
					cout<<"\nProgramm wird beendet\n";
					end=true;
				 } break;

			default:
				cerr<<"Incorrect choice"<<endl;
				break;
		}//end switch	
		if(!end)
		{	
			menu();
		}

	}while(!end==true);

	delete []pPlayer;

	return 0;
}

void menu()
{
	cout<<"Enter your choice:\n"
		<<"1 Neuer Spieler\n"
		<<"2 Alle Spieler anzeigen\n"
		<<"3 Gegenstand aufnehmen\n"
		<<"4 Spiel Speichern\n"
		<<"5 Spiel Laden\n"
		<<"6 Spiel beenden\n bitte waehlen:";
}

wo , an welchen stellen , sollte ich delete verwenden? hmmm

case '1': 
{
    string plname;
    int plweight;
    cout<<"\nNeuer Spielername eingeben: ";
    cin>>plname;
    cout<<"Maximum-Gewicht eingeben:";
    cin>>plweight;;
    Player pl(plname,plweight);
    allPlayers.push_back(pl);
    std::sort(allPlayers.begin(),allPlayers.end());
    break;
}

(PS: bist du zufällig Java-Programmierer?)

sten<<"student is"

grade am erlernen von beiden. in 10 tagen ist meine prüfung in c++ und habe noch einige schwächen. grade bei dynamischer speicherverwaltung und vererbung.
leider muss ich mit dynamischer erzeugung dieses programm bewältigen .frag mich auch was das soll, aber iss wohl so.. deshalb muss ich new und delete verwenden. fühl mich schon wie ein spaghetti-code-programmierer und bin in manchen dingen einer . Jedoch bin ich für alle hilfe die du, ihr mir bietet sehr dankbar und hab durch das forum schon viel gelernt , desshalb vielen dank!