C++, removing #include or #include in class header

后端 未结 12 1374
感情败类
感情败类 2021-01-03 06:37

I want to remove, if possible, the includes of both and from my class header file. Both string and vector are return types of functions declare

相关标签:
12条回答
  • 2021-01-03 06:59

    I assume your objective here is to speed up compile times? Otherwise I'm not sure why you would want to remove them.

    Another approach (not pretty but practical) is to use macro guards around the include itself.

    e.g.

    #ifndef STDLIB_STRING
    #include <string>
    #define STDLIB_STRING
    #endif
    

    Although this looks messy, on large codebases it does indeed increase the compile times. What we did is create a Visual Studio macro that will automatically generate the guards. We bind the macro to a key for easy coding. Then it just becomes a company coding standard (or habit).

    We also do it for our own includes as well.

    e.g.

    #ifndef UTILITY_DATE_TIME_H
    #include "Utility/DateTime.h"
    #endif
    

    Since we have Visual Studio helpers to auto-generate the guards when we create our own header files, we don't need the #define. The macro knows it's a internal include because we always use the

    #include ""

    format for our own includes and

    #include <>

    for external includes.

    I know it doesn't look pretty but it did speed up our compile times on a largish codebase by over 1/2 hour (from memory).

    0 讨论(0)
  • 2021-01-03 07:06

    With the exception of adding overloads to std::swap (the only exception I can think of right now), you are generally not allowed to add anything to the std namespace. Even if it were allowed, the actual declaration for std::vector is a lot more complicated than the code in the OP. See Nikolai N Fetissov's answer for an example.

    All that aside, you have the additional problem of what your class users are going to do with functions that return a std::vector or std::string. The C++ Standard section 3.10 says that functions returning such objects are returning rvalues, and rvalues must be of a complete type. In English, if your users want to do anything with those functions, they'll have to #include <vector> or <string> anyway. I think it would be easier to #include the headers for them in your .h file and be done with it.

    0 讨论(0)
  • 2021-01-03 07:07

    WARNING

    Expect that doing this will cause uproar.

    The language allows you to derive your own classes:

    // MyKludges.h
    #include <vector>
    #include <string>
    class KludgeIntVector : public std::vector<int> { 
      // ... 
    };
    class KludgeDoubleVector : public std::vector<double> {
      // ...
    };
    
    class KludgeString : public std::string {
      // ...
    };
    

    Change your functions to return KludgeString and KludgeIntVector. Since these are no longer templates, you can forward declare them in your header files, and include MyKludges.h in your implementation files.

    Strictly speaking, derived classes do not inherit base class constructors, destructors, assignment operators, and friends. You will need to provide (trivial) implementations of any that you're using.


    // LotsOfFunctions.h
    // Look, no includes!  All forward declared!
    class KludgeString;
    // 10,000 functions that use neither strings nor vectors
    // ...
    void someFunction(KludgeString &);
    // ... 
    // Another 10,000 functions that use neither strings nor vectors
    
    // someFunction.cpp
    // Implement someFunction in its own compilation unit
    // <string> and <vector> arrive on the next line
    #include "MyKludges.h"
    #include "LotsOfFunctions.h"
    void someFunction(KludgeString &k) { k.clear(); }
    
    0 讨论(0)
  • 2021-01-03 07:10

    This won't help for vector or string, but it might be worth mentioning that there is a forward reference header for iostream, called iosfwd.

    0 讨论(0)
  • 2021-01-03 07:11

    With a very few exceptions, you are not allowed to add things to the std:; namespace. For classes like vector and string, you therefore have no option but to #include the relevant Standard header files.

    Also, notice that string is not a class, but a typedef for basic_string<char>.

    0 讨论(0)
  • 2021-01-03 07:19

    There is no simple obvious way to do it (as others have explained it very well).

    However these headers should be seen as being part of the language (really!), so you can let them in your own headers without any problem, nobody will ever complain.

    If your concern is compilation speed, I encourage you to use pre-compiled header instead and put these std headers in it (among other things). It will significantly increase your compilation speed.

    Sorry the for the "real winner is the one who avoid the fight" kind of answer.

    0 讨论(0)
提交回复
热议问题