¿Mi función está bien escrita?
Sí y No
Está bien escrita porque el compilador la acepta sin mostrar ningún error ni advertencia... es decir, lexicográficamente es correcta.
Peeeero a nivel funcional no es una función con la que a mi me gustaría tener que lidiar. El motivo es que usas variables globales sin necesidad. Las variables globales son una herramienta que es preferible evitar siempre que sea posible:
Las variables globales pueden ser tanto consultadas como modificadas en cualquier parte del programa. Puede ser dificil recordar en qué sitios se usa y entonces es cuando el programa de repente se vuelve errático y no encontramos el motivo.
Las variables globales pueden solaparse con variables locales con el mismo nombre
int a = 0;
void func1()
{
a++;
}
int func2()
{
int a = 10;
a++;
}
int main()
{
func1();
printf("%d", a);
func2();
printf("%d", a);
}
Los depuradores no van a mostrar las variables globales... tendremos que cargarlas a mano para conocer su valor.
En entornos multihilo, las variables globales pueden ser un infierno
Así pues, para que tu función estuviese bien escrita debería adoptar hábitos saludables, entre los que se encuentra evitar el uso de variables globales en la medida de lo posible.
Si la declaración de tu función fuese así:
void Pleno(double * dinero_apostado, double * dinero_jugador, double * dinero_mesa)
Ya no parecería una función tan inofensiva. Dado que recibe tres valores vía puntero (y dado que dichos punteros no son constantes), es bastante probable que la función modifique esas variables en algún momento. A la hora de revisar el código o seguirle la pista a un error, este tipo de prácticas te pueden ahorrar muchos dolores de cabeza.
void
es lo correcto.¡
y¿
.