Damit sollte es gehen (<string>, <fstream>)!

greetz, Swordfish

file_exist("C:/Programme/Adobe/Adobe Reader 6/Reader.exe");

was geht daran nicht?

file_exists("Reader.exe");

aufruft und dabei die Festplatte nach einer solchen Datei, in welchem Verzeichnis auch immer, sucht.

Ich hatte es so verstanden, dass man zB

file_exists("Reader.exe");

aufruft und dabei die Festplatte nach einer solchen Datei, in welchem Verzeichnis auch immer, sucht.

file_exists("Reader.exe");
Nimmt die Funktion das als std:: string const oder als const char* ?

?

file_exist("C:/Programme/Adobe/Adobe Reader 6/Reader.exe");

was geht daran nicht?

erstens kein '/' sondern '\'

zweitens damit du das Zeichen '\' bekommst musst du '\' schreiben

"C:\\Programme\\Adobe\\Adobe Reader 6\\Reader.exe"

drittens:

const std::string file_name("C:\\Programme\\Adobe\\Adobe Reader 6\\Reader.exe");
file_exist(file_name);

wäre es nich auch so besser

bool exist(std::string name){
     std::ifstream file_stream(name.c_str()); 
     bool open = file_stream;
     file_stream.close();
     return open;
}

erstens […]

zweitens […]

drittens: […]

Klugscheißen will gelernt sein. Try again.

kiba91 schrieb:

wäre es nich auch so besser

bool exist(std::string name){
     std::ifstream file_stream(name.c_str()); 
     bool open = file_stream;
     file_stream.close();
     return open;
}

Hm, was sollte daran besser sein?

Und wofür willst du extra ein konstantes String-Objekt erzeugen? Wo ist da der Sinn? Es ist doch gerade das schöne an const-Ref das man das nicht machen muss ...

@kiba91: Nun, close brauchst du nicht extra aufrufen, das wird vom Destruktor von std::ifstream automatisch erledigt. Der Rest erledigt sich dann, oder?

Boost.FileSystem ist eine gute und auch portable alternative hierfür.

BR
Vinzenz