Ich habe die Klassen Team und mit der Klasse TeamDB möchte ich eine unbekannte Anzahl Teams verwalten.

class Team
{
Team(char* TeamName);
//....
}

class TeamDB
{
vector<Team*> Teams;
void LadeDatei(char* DateiPfad);
void ZeigeDBInhalt();
//.....
}

void LadeDatei(char* DateiPfad)
{
//....

while(!feof(NameFile))
{
//Lese Zeile mit Teamname aus
Teams.push_back(new Team(Zeile);
}

//....
}

void main()
{
TeamDB* Test = new TeamDB();
LadeDatei();
ZeigeDBInhalt();
}

Bei der Ausgabe wird aber bei allen Teams der Name des zuletzt eingelesenen Teams ausgegeben. Wenn ich jetzt aber folgenden Code anwende...

void LadeDatei(char* DateiPfad)
{
//....

//Lese Zeile mit Teamname aus
Teams.push_back(new Team(Zeile);
//Lese nächste Zeile aus
Teams.push_back(new Team(Zeile);
//usw.

//....
}

...werden die Teamnamen richtig gespeichert!

Wo liegt das Problem? Es spielt doch keine Rolle ob das push_back in einer while-Schleife ausgeführt oder nicht. Ich habe zuerst gedacht es läge an den Pointern und alle Pointer zeigen auf die selbe Adresse (darum die selben Werte), aber wieso funktioniert dann das zweite Beispiel? Ist ja eigentlich dasselbe!

Verwende std::string, dann hast du das Problem nicht.

Mal abgesehen davon, dass bei den push_backs eine schliessende Klammer fehlt sehe ich hier erstmal nichts. Es fehlen aber noch weitere Informationen. Zeig mal den Konstruktor von Team und ZeigeDBInhalt().
Warum nimmst du hier überhaupt Zeiger?

...Bei der Ausgabe wird aber bei allen Teams der Name des zuletzt eingelesenen Teams ausgegeben. Wenn ich jetzt aber folgenden Code anwende...werden die Teamnamen richtig gespeichert!

1. Wo ist die Ausgebenfunktion, ich tippe eher da auf einen Fehler
2. Warum versuchen sich eigentlich alle an C-Strings statt an std::string der deutlich weniger Anfängerfehler mit sich bringt (Ich hoffe du hast auch einen Destruktor um den Speicher wieder freizugeben wenn du schon mit Pointern arbeitest).

Rookie1 schrieb:

Wo liegt das Problem? Es spielt doch keine Rolle ob das push_back in einer while-Schleife ausgeführt oder nicht. Ich habe zuerst gedacht es läge an den Pointern und alle Pointer zeigen auf die selbe Adresse (darum die selben Werte), aber wieso funktioniert dann das zweite Beispiel? Ist ja eigentlich dasselbe!

3. Es liegt wohl ein Fehler in der Benutzung von std::vector in den Bereichen vor die du nicht gepostet hast (Wäre auch gut wenn du an sich keine Fehler wie "Teams.push_back(new Team(Zeile);" postest, wo z.B. die schließende Klammer fehlt).

cu André

while(!feof(NameFile))
{
// hier musst du für die Zeile neuen speicher allociren
Zeile = new char[...];
//Lese Zeile mit Teamname aus
Teams.push_back(new Team(Zeile);
}

Team(char* strName) {this->Name = strName;}

void TeamDB::ZeigeInhaltDB()
{
cout << "Show All Teams" <<endl;
for (i=0;i < Teams.size();i++)
{
cout << i << " "<< Teams[i]->getName() << endl;
}
}

Team::getName() gibt nur mit einem return den Teamnamen zurück.

Ich würde fast wetten das du dem Konstruktor von Team nur einen Pointer übergibst, den du auch so in Team speicherst. Somit verweisen nachher alle Pointer auf die gleiche Adresse. Vermutlich auch noch eine temporäre Variable in der Methode zum Auslesen.

Dass das Problem bei den Pointern liegt habe ich auch vermutet, aber wieso läft denn das 2.Beispiel???

Team(char* strName)
{
this->Name = new char[20];
this->Name = strName;
}

Also etwa so?

Team(char* strName)
{
this->Name = new char[20];
this->Name = strName;
}

Da hast du 1. das gleiche Problem immernoch und 2. ein Speicherleck.

Team() : Name(0){ }
Team(char* strName){
  this->Name = new char[strlen(strName)+1];
  strncpy(this->Name, strName, strlen(strName)+1);
}
~Team(){
  if (Name) delete[] Name;
}

So in eta.

Aber wenigstens bin ich jetzt mal aus meiner Gedanklichen Sackgasse raus und weiss wo mein Problem liegt. Nochmals vielen Dank für die Hilfe.

Team() : Name(0){ }
Team(char* strName){
  this->Name = new char[strlen(strName)+1];
  strncpy(this->Name, strName, strlen(strName)+1);
}
~Team(){
  if (Name) delete[] Name;
}

So in eta.

Wenn Du Anfänger bist und C++ (nicht C) lernen willst, dann eher so:

Team() { }
Team(string const& strName)
   : Name(strName) { }

private:
string Name;

Team(const char* name)
    : m_name(new char[std::strlen(name) + 1])
{    
    std::strcpy(m_name, name); 
}

~Team()
{
    delete [] m_name;
}

und das ganze kurz erklären:
Team-Konstruktor bekommt einen const char-Zeiger übergeben, da du ja nichts an dem String verändern willst sondern ihn kopieren willst(damit du sicher gehen kannst das er auch noch existiert!).

Nun wird in der Initialisierungsliste m_name ein Speicherbereich aufm Heap zugewiesen von char * (länge des Strings + 1(für Nullterminierungszeichen!)).

Damit kannst du sichergehen das dein String name auch in m_name reinpasst und kannst ihn einfach kopieren (std::strcpy).

Da du Speicher aufm Heap angefordert hast und somit dieser nicht automatisch freigeben wird, musst du diesen selbst freigeben. Da ein Array delete [] anstatt delete.

Soweit sollte das als Erklärung reichen!

Noch eine Frage. Warum packst du in deinen vector denn Zeiger von Team? (wenn du das machst musst du dich wieder um's aufräumen kümmern!)

2. Warum versuchen sich eigentlich alle an C-Strings statt an std::string der deutlich weniger Anfängerfehler mit sich bringt

Weil "alle" Opfer von solch schwachsinnigen "C/C++"-Büchern sind deren Autoren meinen C++ sei eine Fortsetzung von C und nicht mitgekriegt haben dass es eine Standardbibliothek mit std::string etc. gibt. Diese Autoren haben einfach nicht mitbekommen dass sich C++ in den letzten 15-20 Jahren so weit von C entfernt hat dass const char* - Gefrickel, printf() &Co in den meisten Fällen nurnoch Altlasten sind. Wenigstens hat er in seiner Ausgabefunktion cout benutzt

...Autoren meinen C++ sei eine Fortsetzung von C und nicht mitgekriegt haben dass es eine Standardbibliothek mit std::string etc....

... und oftmals der Ansicht sind, dass Anfänger dieselbe Sch*** fressen müssten, die ihnen damals auch schon nicht geschmeckt hat (weil "man nur so wirklich versteht, was da passiert" = "den Orden 'Programmier' verliehen bekommen darf").

Gruß,

Simon2.