¿Declaras que las funciones específicas de tu módulo son estáticas?

Question

Creo que es una buena práctica declararlos como estáticos, ya que los hace invisibles fuera del módulo.

¿Qué piensas de esto?

Answer 1

Si es realmente una función que es interna solo para ese archivo .c, entonces sí. Debería ayudar a evitar la contaminación del espacio de nombres global. Además, creo que el compilador puede hacer algunas optimizaciones con convenciones de llamadas si la función es estática ya que no sabe que ningún otro archivo fuente necesita saber cómo llamarlo. Esto solo se aplica realmente a c porque, como otros han notado, C++ tiene espacios de nombres para abordar este problema.

Answer 2

Si con 'módulo' solo quiere decir un archivo CPP, podría simplemente colocar la declaración y la definición en el archivo CPP.

Answer 3

Para C++, una mejor que la estática es ponerlo en un espacio de nombres sin nombre (anónimo). Esta es la forma preferida de evitar la contaminación del espacio de nombres global.

namespace { 
void myLocalFunction() { 
// stuff 
} 
} 

Answer 4

creo que C y C++ tienen diferentes limitaciones relativas static: en C que no tiene espacios de nombres y archivos .c son sus módulos, por lo que es muy, muy importante poner todas las funciones no públicas como estática para prevenir errores!

Answer 5

En C++, se debe utilizar un espacio de nombres en el anonimato, así:

// foo.cpp 
namespace 
{ 
    class Core { ... }; 
    void InternalFandango(Core *); 
} 

void SomeGloballyVisibleFunction() 
{ 
    InternalFandango(&core); 
} 

Ventaja: esto es aplicable a struct declaraciones/clase, también.
En C, simplemente marque las funciones "estático". No hay nada en contra de usar "estático" en C++, también, pero he aprendido a preferir el espacio de nombres, ya que es un concepto único que funciona para todas las declaraciones.

Answer 6

Sobre la única propiedad potencialmente útil que puedo pensar para este uso de "estática" en C++, que los espacios de nombres anónimos no proporcionan, es que hay una advertencia en GCC que puede activar para funciones estáticas no utilizadas (un formulario de código muerto). No lo obtiene para las funciones no utilizadas en espacios de nombres anónimos, por lo que, en el caso improbable de que desee que el compilador le diga cuándo deja de usar la función, hágalo de esa manera.

Answer 7

Se formuló una pregunta muy similar antes del here. Hay un ejemplo de caso límite donde declarar la función resultados estáticos en un comportamiento bastante diferente al de una función de espacio de nombres sin nombre. Ver mi answer.

Answer 8

Hubo muchos detalles sobre la implementación y no demasiado sobre el concepto.

Limitar el alcance de la variable/función, etc. es una buena práctica. Este es un concepto básico de diseño orientado a objetos; desea mantenerlo privado como privado. De esta manera, su interfaz es más limpia y el mantenimiento del código es más fácil. Y un día no encontrará que cambiar algo que consideraba una compilación privada se rompió porque a alguien en otra parte del proyecto le gustaba su función y decidió usarla.

Answer 9

En C, hago todo - funciones y variables - estático en el alcance del archivo hasta que pueda demostrar que son necesarios fuera del archivo. Voy a hacer las cosas estáticas dentro de una función si solo esa función las usará y no son demasiado grandes. Básicamente, si la declaración es más grande que el resto de la función, puedo colocar la declaración fuera de la función. Y, por supuesto, hay un encabezado para los servicios públicos proporcionados por un archivo fuente.

Answer 10

En el código C, establezca sus funciones estáticas de forma predeterminada.Solo realice funciones no estáticas y declaraciones .h para las funciones que necesitarán otros módulos.

En el código C++, coloque las funciones que son locales al archivo en un espacio de nombre anónimo y para que sean estáticas. En el compilador de GNU al menos, esto dará como resultado el mejor y el más pequeño código, porque no se escribirá ninguna función si todos los usos están en línea. Si pretende que esté en línea, entonces, por supuesto, marcarlo en línea es incluso mejor que la estática.

No sé por qué g ++ escribe los cuerpos de funciones no nombradas que están en espacios de nombres anónimos en el resultado, pero lo hace. Las funciones con visibilidad oculta parecen aparecer también; marcados como símbolos ocultos, pero aún produciendo bloques de código no utilizados en el archivo objeto. GCC probablemente no entiende que el código no es necesario en esos casos. O me estoy perdiendo algo, siempre es posible.

Answer 11

Si está utilizando GCC, entonces debería echar un vistazo a la bandera de visibilidad (vea http://gcc.gnu.org/wiki/Visibility para una discusión completa).

Ocultará los símbolos completamente en lugar de marcarlos inalcanzables. Eso reduce la tabla de símbolos y ayuda a disminuir los tiempos de enlace.

No solo eso, abre la puerta a más líneas internas, si eso es lo que buscas.

Answer 12

En C++, te declaran la función privada así:

class MyClass 
{       
public:     
void publiclyAccessibleFunction();    
private:     
    void onlyAccesibleFromWithinTheClass(); 
int some_member_parameter;   
}; 

Nota la función onlyAccesibleFromWithinTheClass().

Answer 13

De acuerdo. Como consecuencia, los prototipos para las funciones estáticas deben ir en la parte superior del archivo .c, no en el archivo .h.