Thread: Including string with header?

Sorry to be a bother or anything with my assult of questions, but currently I am working on a class to handle files. The problem I currently have (and I'm sure it is something simple I am missing here) is that my header file needs the string class to be defined, and even though I included it, it still says it is not defined.

Visual Studio gives me this error:

Code:

Error C2061: syntax error : identifier 'string'

And here is the header:

Code:

#include <string>
#pragma once

class File
{
public:
	File();
	~File();
	bool Exists();							//Returns true if exists

	string Read();							//Read file, returns contents of file if successful.
	bool Write(string &sContents);			//Write to end of file, returns true if successful.

	bool Overwrite(string &sContents);		//Completely erases contents of file with given contents, returns true if successful.
	
	bool Move(string &sDir);				//Moves the file to another directory, and changes the path stored in the object. Returns true if successful.
	bool Copy(string &sDir,string &sName);	//Copies the file to another directory. Doesn't change the path stored in the class. Returns true if successful.

	bool Delete();							//Deletes the given file.
private:
	string filename;
};

As always any help you can offer is to great use to me.

Originally Posted by theJ89

Sorry to be a bother or anything with my assult of questions, but currently I am working on a class to handle files. The problem I currently have (and I'm sure it is something simple I am missing here) is that my header file needs the string class to be defined, and even though I included it, it still says it is not defined.

...
As always any help you can offer is to great use to me.

Remove #pragma once, as it's non-portable, and in place of it put: using namespace std;

- xeddiex

std::string

The string class is part of the std namespace.

xeddiex, your solution is quick and simple, but not really recommended.

Hah, I forgot to use the namespace. Thank you.

Also I might mention that the file should only be included once. Is there any replacement for #pragma once?

Originally Posted by jafet

std::string

The string class is part of the std namespace.

xeddiex, your solution is quick and simple, but not really recommended.

Very true, My bad.

- xeddiex

Originally Posted by theJ89

Hah, I forgot to use the namespace. Thank you.

Also I might mention that the file should only be included once. Is there any replacement for #pragma once?

The following prevents any file that's including this file from being included again.

#ifndef HEADER_FILE
#define HEADER_FILE
class File {...
.. other declarations
#endif

- xeddiex

#ifndef HEADER_FILE
#define HEADER_FILE

class File {...
.. other declarations

#endif

"HEADER_FILE" needs to be a unique name, so consider using the file name:

#ifndef FILE_H
#define FILE_H