C/C++ Fast way to test for existence of a file in C++?

[ahrkron]

Here's the situation:

I'm running over a few thousand files, some of which may have already been processed. In order to decide if a file has to be processed, I check if the corresponding output file exists.

The problem:

I tried using FILE *f = fopen(<filename>,"r"), and test for f!=0; however, it takes a long time to just open the files.

Any suggestions?

Edit: FILE instead of TFILE.[/color]

 

[dduardo]

How about this:

#include <sys/stat.h>

int fexist( char *filename ) {
  struct stat buffer ;
  if ( stat( filename, &buffer ) ) return 1 ;
  return 0 ;
}

This is under Unix, so stat.h may not be under sys/ for windows.

 

[ramollari]

If you're programming for Windows check if the following works:

System::IO::File::Exists(file);

which returns true if the file exists, false otherwise.

 

[dduardo]

Here is the msdn docmentation for stat.h:

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vclib/html/_crt__fstat.2c_._fstati64.asp

It looks microsoft is up to their dirty tricks again. Instead of calling their function stat(...), they conveniently change it to _fstat(...). If anyone wants to port their code to unix they would need to change their function calls. Nice job. Even borland uses stat(...).

 

[ahrkron]

Thanks!

That is fast enough.
Just it seems to have the logic reversed: stats(...) returns -1 for non-existing files and 0 for existing files.

(or it may be my expectation of 1 to mean "true" and 0 to be "does not exist").

 

[ahrkron]

OK. I was just being dumb. I had a piece of code before the actual test of file existence that was using some serious IO. Now both versions (dduardo's and my original FILE use) work ok.

Thanks anyway!

 

[master_coda]

Here is the msdn docmentation for stat.h:

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vclib/html/_crt__fstat.2c_._fstati64.asp

It looks microsoft is up to their dirty tricks again. Instead of calling their function stat(...), they conveniently change it to _fstat(...). If anyone wants to port their code to unix they would need to change their function calls. Nice job. Even borland uses stat(...).

But you code you provided will work with Microsoft's compiler.

 

[verwall]

You also have the posix

#include <unistd.h>
int access(const char *path, int amode);

Not sure it works on *dows, though

 

[jtbell]

Um... did you notice that the posts you responded to are six and a half years old?

 

[Timo]

Um... did you notice that the posts you responded to are six and a half years old?

That only makes it only more important to check if the file still exists.