Meine erste frage wäre, wieso kommt er nie in den loop rein(linie 49.)? Ich verstehe einfach nicht was ich falsch mache. Meine zweite frage wäre, ich brauche zugriff auf die Player daten über jeden member. Das soll heißen, dass ich manchmal die Player Daten über die PlayerId/PlayerName usw.. kriegen muss, meine frage ist, gibt es eine leichtere methode als für jeden member eine fkt. zu schreiben? ( FindPlayerById, FindPlayerByName, FindPlayerByXCoord, usw.. ). Ja ich weiß, dass man es schön in einer Klasse haben könnte, aber der code dient nur als bsp.

#include <list>
#include <string>
#include <iostream>

struct PlayerData
{
	int PlayerId;
	std::string PlayerName;
	float PlayerXCoord;
	float PlayerYCoord;

	PlayerData()
		: PlayerId(0)
		, PlayerName("")
		, PlayerXCoord(0.00)
		, PlayerYCoord(0.00)
	{

	}

};

typedef std::list< PlayerData > PlayerList;
typedef PlayerList::iterator PlayerListItr;
typedef PlayerList::const_iterator PlayerListConstItr;

void pauseApp()
{
    std::cin.clear();
    std::cin.ignore(std::cin.rdbuf()->in_avail());
    std::cin.get();
}

bool FindPlayerById( const int PlayerId, PlayerData & tempObject )
{
	PlayerList players;
	PlayerListConstItr Itr;
	PlayerListConstItr ItrEnd;

	Itr = players.begin();
	ItrEnd = players.end();

	for(; Itr != ItrEnd; ++Itr)
	{
		std::cout << "found player:" << Itr->PlayerName; // kommt nie in den loop, wieso?
		if( Itr->PlayerId == PlayerId )
		{
			tempObject = *Itr;
			return true;
		}

	}

	return false;

}

bool FindPlayerByName( const std::string PlayerName, PlayerData & tempObject )
{
	PlayerList players;
	PlayerListConstItr Itr;
	PlayerListConstItr ItrEnd;

	Itr = players.begin();
	ItrEnd = players.end();

	for(; Itr != ItrEnd; ++Itr)
	{
		if( Itr->PlayerName == PlayerName )
		{
			tempObject = *Itr;
			return true;
		}

	}

	return false;
}

void FillList()
{
	PlayerData NewPlayer;
	PlayerList Players;

    // fill the list
	NewPlayer.PlayerId     = 123;
	NewPlayer.PlayerName   = "Test";
	NewPlayer.PlayerXCoord = 0.00;
	NewPlayer.PlayerYCoord = 0.00;

	Players.push_back(NewPlayer);
}

int main()
{

    FillList();
	PlayerData NewPlayer;
	// find the player by id
	bool isPlayerFoundById = FindPlayerById( 123, NewPlayer);

	// print the data
    if ( isPlayerFoundById ) 
	{
	    std::cout << NewPlayer.PlayerId
		          << "\n"
			      << NewPlayer.PlayerName
			      << "\n"
			      << NewPlayer.PlayerXCoord
			      << "\n"
			      << NewPlayer.PlayerYCoord;
	}

	else 
	{
		std::cout << "PlayerId: 123 does not exist";
	}

    // find the player by name
	bool isPlayerFoundByName = FindPlayerByName( "test", NewPlayer);

	// print the data
	if( isPlayerFoundByName ) 
	{ 
		std::cout << NewPlayer.PlayerId
		          << "\n"
			      << NewPlayer.PlayerName
			      << "\n"
			      << NewPlayer.PlayerXCoord
			      << "\n"
			      << NewPlayer.PlayerYCoord;

	}
	else 
	{
		// some msg..
	}

	pauseApp();

}

Gruß

In FindPlayerById() legst du eine neue (und somit leere) Liste an:

PlayerList players;

Also kein Wunder, eine leere Liste kannst du nicht durchiterieren.
Lerne erstmal mehr über die STL, dann können wir weitersehen.

#include <string>
struct PlayerData 
{ 
    int PlayerId; 
    std::string PlayerName; 
    float PlayerXCoord; 
    float PlayerYCoord; 

    PlayerData() 
    :    PlayerId(0), PlayerXCoord(0.f), PlayerYCoord(0.f)
    {}
};

hätte ich zwar im ctor mit standard-parametern gemacht (und den dann noch explicit gemacht), aber ansonsten würde das so zumindest sehr viel besser sein als zu vor ^^
zur erklärung 0.00 ist nen double - du hast aber floats - solltest du also 2 warnings bekommen... 0.f == 0.0f -> 0 als float
der default-ctor von std::string macht genau das, was du willst: es wird ein leerer string erstellt...

zu deiner eigentlichen Frage:

bool FindPlayerById(int PlayerId, PlayerData &tempObject)
{ 
    std::list <PlayerData> players; //ist also leer...
	for(std::list<PlayerData>::const_iterator i(players.begin()), e(players.end()); i != e; ++i) //wir gehen hier also eine leere liste durch...
    { 
        std::cout << "found player:" << Itr->PlayerName; //wenns nichts zum durchgehen gibt, wie oft werden wir dann hier sein...? 
        if( Itr->PlayerId == PlayerId ) 
        { 
            tempObject = *Itr; 
            return true; 
        } 
    } 
    return false; 
}

nen (sehr?) "kluger" compiler macht so was draus:

bool FindPlayerById(int PlayerId, PlayerData &)
{ 
    return false; 
}

dein quellcode ist aber auch an und für sich... naja - nicht gerade sehr gut zu lesen, finde ich ^^
ab und an sind leerzeilen ja gut und bringen übersicht - aber deshalb nur jede zweite zeile zu beschreiben sollte nicht sinn der sache sein...

und jetzt, wo du weist, was dort falsch ist, guck dir mal das hier an:

void FillList()
{
    PlayerData NewPlayer;
    PlayerList Players;

    // fill the list
    NewPlayer.PlayerId     = 123;
    NewPlayer.PlayerName   = "Test";
    NewPlayer.PlayerXCoord = 0.00;
    NewPlayer.PlayerYCoord = 0.00;

    Players.push_back(NewPlayer);
}

für so etwas gibts übrigens construktoren:

PlayerData NewPlayer;
    NewPlayer.PlayerId     = 123;
    NewPlayer.PlayerName   = "Test";
    NewPlayer.PlayerXCoord = 0.00;
    NewPlayer.PlayerYCoord = 0.00;

also versuch es mal noch mal - tipp: den container kannst du auch in der main anlegen und per (const) reference an die funktionen übergeben...

mal gucken - ich bin ja ma gespannt auf deine Lösung ^^

bb

Zu deiner ersten Frage:

In FindPlayerById() legst du eine neue (und somit leere) Liste an:

PlayerList players;

Also kein Wunder, eine leere Liste kannst du nicht durchiterieren.
Lerne erstmal mehr über die STL, dann können wir weitersehen.

Hast recht.. Keine ahnung wie ich so ein fehler machen konnte..

ty

@unskilled

auch danke, habs atm so gelöst..

class.hpp

#include <list>
#include <string>
#include <iostream>

struct PlayerData
{
	int PlayerId;
	std::string PlayerName;
	float PlayerXCoord;
	float PlayerYCoord;

	explicit PlayerData(int playerId, std::string playerName, float playerXcoord, float playerYcoord )
		: PlayerId(playerId)
		, PlayerName(playerName)
		, PlayerXCoord(playerXcoord)
		, PlayerYCoord(playerYcoord)
	{}

	explicit PlayerData()
        : PlayerId(0)
		, PlayerXCoord(0.f)
		, PlayerYCoord(0.f)
	{}

};

class MyPlayerClass
{
      public:
	  bool FindPlayerById(const int PlayerId, PlayerData & tempObject);
	  void FillList();

      private:
      typedef std::list< PlayerData > PlayerList;
      typedef PlayerList::iterator PlayerListItr;
      typedef PlayerList::const_iterator PlayerListConstItr;
      PlayerList Players_;

};

class.cpp

#include "class.hpp"

bool MyPlayerClass::FindPlayerById( const int PlayerId, PlayerData & tempObject )
{
    PlayerListConstItr Itr;
	PlayerListConstItr ItrEnd;

    Itr = Players_.begin();
    ItrEnd = Players_.end();

	for(; Itr != ItrEnd; ++Itr)
	{

		if( Itr->PlayerId == PlayerId )
		{
			tempObject = *Itr;
			return true;
		}

	}

	return false;

}

void MyPlayerClass::FillList()
{
	PlayerData NewPlayer(123, "Test123", 1.0, 1.0);
	Players_.push_back(NewPlayer);

}

main.cpp

#include "class.hpp"

void pauseApp()
{
    std::cin.clear();
    std::cin.ignore(std::cin.rdbuf()->in_avail());
    std::cin.get();
}

int main()
{
    MyPlayerClass NewClass;
    NewClass.FillList();

	PlayerData tempPlayer;
	NewClass.FindPlayerById(123, tempPlayer);
	std::cout << tempPlayer.PlayerId;

	pauseApp();

}
}

Ich habe "explicit" benutzt, aber wo ist der unterschied? Ich merke keinen.. Und ich bräuchte hilfe bei meiner 2ten frage, ist so etwas überhaupt möglich zu basteln? ^^

btw:

PlayerId(0), PlayerXCoord(0.f), PlayerYCoord(0.f)

find ich hässlicher als

: PlayerId(0)
        , PlayerXCoord(0.f)
        , PlayerYCoord(0.f)

Aber das kommt ja auf die Person an, und ich mag es halt so. ^^ Trotzdem danke für deine hilfe.

Grüße

jz macht das explicit auch Sinn...

bb

zu explicit - obwohl man da auch bei google ne ganze menge finden sollte...:

struct foo
{
  foo(int i=0, int j=0) {}
};

struct bar
{
  explicit bar(int i=0, int j=0) {}
};

foo GetFoo()
{
 return 1;
}
foo GetFoo2()
{
 return foo(1);
}

bar GetBar()
{
 return 1;
}
bar GetBar2()
{
 return bar(1);
}

Kann mir jemand meine 2te frage beantworten?

Gruß

naja:

Das soll heißen, dass ich manchmal die Player Daten über die PlayerId/PlayerName usw.. kriegen muss, meine frage ist, gibt es eine leichtere methode als für jeden member eine fkt. zu schreiben? ( FindPlayerById, FindPlayerByName, FindPlayerByXCoord, usw.. ). Ja ich weiß, dass man es schön in einer Klasse haben könnte, aber der code dient nur als bsp.

Du kannst std::find_if verwenden und dann musst du nur noch die compare-funktoren schreiben...
Irgendwer (wahrscheinlich Dravere oder vll auch Nexus) hatte mal so etwas komplett mit templates gezeigt - da war es zwar zum Sortieren aber das sollte hier auch klappen...

Jz hab ichs doch noch aus den Bookmarks rausgekramt:
http://www.c-plusplus.net/forum/viewtopic-var-t-is-236521
es war Nexus ^^

Wenn du dann noch fragen dazu hast, dann zeig aber auch, dass du dich mit dem Code auseinander gesetzt hast - dann wird dir das auch jmd erklären, falls du iwas nich verstehen solltest...

bb