Funciones de clases privadas vs Funciones en el espacio de nombre sin nombre

Question

Me he encontrado que tiendo a no tener funciones de clase privadas. Si es posible, todos los candidatos a la clase privada funcionan en lugar de poner en el espacio de nombres sin nombre y pasar toda la información necesaria como parámetros de función. No tengo una buena explicación de por qué estoy haciendo eso, pero al menos me parece más natural. Como consecuencia, necesito exponer menos detalles internos en el archivo de encabezado.

¿Cuál es su opinión? ¿Es una práctica correcta?

Answer 1

En los proyectos semi grandes en los que suelo trabajar (más de 2 millones de líneas de código), prohibiría las funciones de clase privadas si pudiera. La razón es que una función de clase privada es privada, pero aún así es visible en el archivo de encabezado. Esto significa que si cambio la firma (o el comentario) de todos modos, a veces me recompensan con una recompilación completa que cuesta varios minutos (u horas dependiendo del proyecto).

Solo di no a eso y oculta lo que es privado en el archivo cpp.

Si comenzara de cero en un gran proyecto de C++ haría cumplir PIMPL Idiom: http://c2.com/cgi/wiki?PimplIdiom para mover aún más detalles privados en el archivo cpp.

Answer 2

Creo que esta es una buena práctica. A menudo tiene el beneficio de ocultar estructuras auxiliares y tipos de datos también, lo que reduce la frecuencia y el tamaño de las reconstrucciones. También hace que las funciones sean más fáciles de dividir en otro módulo si resulta que son útiles en otros lugares.

Answer 3

Básicamente se trata de si la función en cuestión realmente tiene sentido como parte de la clase. Si su única intención es mantener los detalles de la clase fuera del encabezado, consideraría usar el idioma pimpl en su lugar.

Answer 4

He hecho esto en el pasado, y siempre ha terminado mal. No puede pasar objetos de clase a las funciones, ya que necesitan acceder a los miembros privados, presumiblemente por referencia (o termina con listas de parámetros intrincados) por lo que no puede llamar a los métodos de clase pública. Y no puede llamar a funciones virtuales, por el mismo motivo. Creo firmemente (basado en la experiencia) que esta es una mala idea.

En pocas palabras: Esto suena como el tipo de idea que podría funcionar cuando el "módulo" de implementación tiene algún acceso especial a la clase, pero este no es el caso en C++.