Cómo volver a incluir encabezado en aplicación

Question

supongamos que tengo una cabecera foo.h así:

#ifndef FOO_H 
#define FOO_H 
#include <string> 
#include "non_standard_class.h" 

std::string foo(MyClass a); 
... 
#endif 

y la implementación foo.cpp habrá

#include <vector> 

#include "foo.h" 

std::string foo(MyClass a) 
{ 
    std::vector<int> x; 
    MyClass b; 
    ... 
} 

es una buena pratice volver a incluir <string> y non_standard_class.h en foo.cpp ? El punto es: si leo foo.cpp, ¿cómo puedo entender de dónde viene MyClass? Necesito mirar foo.h pero será más difícil.

Answer 1

La práctica I sigue:

• foo.h debe incluir directamente todos los encabezados de sus necesidades de código fuente (es decir #include <string> si hay std::string en foo.h)
• foo.cpp debe incluir directamente foo. h más todos los demás encabezados que necesita su propio código fuente, excepto los que ya incluye directamente foo.h

Otra formulación de esta práctica: en foo. *, incluya solo los encabezados que necesite el código fuente. Si solo se necesita un encabezado por foo.cpp (pero no por foo.h), inclúyalo desde foo.cpp, de lo contrario, inclúyalo desde foo.h.

Por lo tanto, en su caso, no me #include <string> en foo.cpp, porque ya está incluido por foo.h.

Supongamos que string hace un #include <vector>, y foo.h contiene std::vector. En este caso, tendría #include <vector> en foo.h, porque su código fuente lo necesita – y no importa que string ya lo incluya.

Answer 2

Yo diría que sí, porque de lo contrario su foo.cpp se basa en un detalle de foo.h que puede cambiar. Obviamente, esto es más un problema con otros archivos .h que no están bajo tu control, pero aún así lo hago por consistencia.

Answer 3

Personalmente, sigo la práctica de inclusión mínima (cuestionable): solo incluyo lo que se necesita para que la compilación funcione.

Las personas que dicen que para obtener un mejor compilador/máquina me hacen reír, de alguna manera, el proyecto en el que estoy trabajando diariamente se compila en aproximadamente 8 minutos (desde cero) en un quad-core (fue de aproximadamente 1 hora antes Eliminé las dependencias) y me parece demasiado malditamente. Ciertamente no quiero ningún cambio para hacer que todo vuelva a compilarse, y por lo tanto estoy preocupado de incluir.

Durante el desarrollo, siempre siento que el tiempo de compilación es tiempo perdido. Un poco podría permitir un descanso seguro, pero cuando quieres probar algo ahora, es frustrante esperar a que la maldita cosa esté finalmente lista (especialmente si solo suena en tu mano cuando realizas las pruebas y te das cuenta de que tendrás que modificar de nuevo y vuelva a compilar nuevamente).

Incluir las cosas dos veces parece ser una buena manera de olvidarse de eliminar el include de la fuente cuando ya no es necesario por el encabezado (hurrah for forward declaration), no veo el punto.