Ich schreibe gerade ein Adressbuch.

Ich habe in meinen Konstruktor eingebaut dass er beim initialisieren die Daten
aus der txt Datei ausliest und sie in ein vector speichert.

Address::Address()
{
	char buffer[bufferSize];
	std::ifstream iaddresstxt;
	iaddresstxt.open("address.txt");

	if(!iaddresstxt)
	{
		std::ofstream oaddresstxt;
		oaddresstxt.open("address.txt");
		oaddresstxt.close();
	}

	else
	{
		do
		{
			iaddresstxt.getline(buffer, bufferSize);	firstName.push_back(buffer);	buffer[0] = '\0';
			iaddresstxt.getline(buffer, bufferSize);	lastName.push_back(buffer);		buffer[0] = '\0';
			iaddresstxt.getline(buffer, bufferSize);	dateOfBirth.push_back(buffer);	buffer[0] = '\0';
			iaddresstxt.getline(buffer, bufferSize);	phoneNumber.push_back(buffer);	buffer[0] = '\0';
			iaddresstxt.getline(buffer, bufferSize);	address.push_back(buffer);		buffer[0] = '\0';
			iaddresstxt.getline(buffer, bufferSize);	location.push_back(buffer);		buffer[0] = '\0';
		}while(!iaddresstxt.eof());
	}
	iaddresstxt.close();
}

wen ich dass Programm dann neu starte hat er meine Daten und ein zweiten mit leeren strings

was mach ich falsch ?

std::ifstream iaddresstxt;
iaddresstxt.open("address.txt");

wird zu

std::ifstream iaddresstxt("address.txt");

Dito für das ofstream. Und .close() musst du nicht aufrufen, da die filestreams beim Scopeende automatisch geschlossen werden.

MfG, EOutOfResources

Edit: Um Savefunktion erweitert.

Quellcode:

#include <iostream>
#include <fstream>
#include <string>
#include <vector>

class Person
{
private:
	std::string name;
	std::string vorname;
	std::string geburtstag;
	std::string telefonnummer;
	std::string adresse;
	std::string ort;

public:
	friend std::ostream& operator << (std::ostream& out, const Person& pers);
	friend std::istream& operator >> (std::istream& in, Person& pers);
};

std::ostream& operator << (std::ostream& out, const Person& pers)
{
	out << pers.name << "\n"
		<< pers.vorname << "\n"
		<< pers.geburtstag << "\n"
		<< pers.telefonnummer << "\n"
		<< pers.adresse << "\n"
		<< pers.ort << "\n";
	return out;
}

std::istream& operator >> (std::istream& in, Person &pers)
{
	in  >> pers.name
		>> pers.vorname
		>> pers.geburtstag
		>> pers.telefonnummer
		>> pers.adresse
		>> pers.ort;

	return in;
}

class Adressen
{
private:
	std::vector<Person> vecAdressen;

public:
	bool Load(std::string datei);
	bool Save(std::string datei);
	friend std::ostream& operator << (std::ostream& out, const Adressen& adr);
};

bool Adressen::Load(std::string datei)
{
	std::ifstream in(datei);
	if(!in) return false;

	while (!in.eof())
	{
		Person pers;
		in >> pers;

		vecAdressen.push_back(pers);
	}

	return true;
}

bool Adressen::Save(std::string datei)
{
	std::ofstream out(datei);
	if(!out) return false;

	for(std::vector<Person>::const_iterator i = vecAdressen.begin(); i != vecAdressen.end(); ++i)
		out << *i;

	return true;
}

std::ostream& operator << (std::ostream& out, const Adressen& adr)
{
	for(std::vector<Person>::const_iterator i = adr.vecAdressen.begin(); i != adr.vecAdressen.end(); ++i)
		out << *i << "\n";
	return out;
}

int main()
{
	Adressen adr;
	if(adr.Load("test.txt"))
		std::cout << adr;

	adr.Save("test_save.txt");
}

Testdatei:

Mustermann
Max
12.07.2010
0123/456789
Hauptstr.777
Berlin
Musterfrau
Maria
20.12.2008
0110/456789
Hauptstr.2
Muenchen