Thread: Including default headerfiles in custom headerfiles

Ok, So I've just got the hang of working with header files, and source files together. I am making a text game, to better learn how header files work with the source files. I made a header file called race.h for the different races. And in this headerfile is the class race. In the class I have a bunch of int variables for max health,stamina, etc. I also want to have a string or array declared in this class. But is it "ok" to include <string> in my custom header file? I am doing this because it will declare a name variable. Ex. string NAME;. So is it ok to do this?

What is a "default" header file?

I also want to have a string or array declared in this class. But is it "ok" to include <string> in my custom header file?

If you declare a string type in the file, you have to include <string>. The string type is defined in the string header file, so the only way the compiler knows what "string" means is if you include the <string> header file.

>> So is it ok to do this?

Yes. However, the convention is to not use the using directive in header files, so it would be std::string NAME; (or even better std::string name; since upper case names are usually constants by convention).

Ok, that's cool. What I mean by default header file, are header files such as <iostream> , <cstdlib>, <string>. I didn't have any other word to describe them. Just wanted to make sure I can use those header files in MY headerfile. Thanks.

> Just wanted to make sure I can use those header files in MY headerfile. Thanks.
What you need to avoid doing though is using the namespace, like this

Code:

#include <string>
using namespace std;
class foo {
  string mString;
};

This is very bad, everything which includes the header file will get the whole of the standard namespace whether it likes it or not.


It's much better to fully specify what you want, like this

Code:

#include <string>
class foo {
  std::string mString;
};

Each file which includes this header can then import as much or as little of the standard namespace as it wants to.

Yeah, someone mentioned that before. I wasn't going to use it anyway, I would just use std:: if I needed something that used the namespace.