Hola, lo primero saludarles por primera vez ya que me acabo de registrar. Lo segundo decidles que procurare no cagarla con las normas del foro.

Bueno pues al lio: estoy haciendo una practica en la que me obligan a hacer un menu mediante un switch e invocar distintas funciones que he tenido que crear yo mismo. Tras largo tiempo tiempo pensando he decidio preguntar por aqui cua es mi error (errores mas bien) en este codigo:

#include<stdio.h>
int factorial(int n);

int main (void){

void menu();

}



void menu(){

int opcion;

printf( "Marque la opción que prefiera\n");
printf( "0 Salir del programa\n");
printf( "1 Calcular el factorial de una número\n");
printf( "2 Comprobar si un número es primo\n");
printf( "3 Calcular el mínimo común múltiplo\n");
printf( "4 Calcular el máximo común divisor\n");
printf( "5 Determinar si un año es bisiesto\n");
printf( "6 Mostrar los años bisiestos\n");
printf( "7 Mostrar número de dias de un mes y año\n");
printf( "8 Calcular números combinatorios\n");
printf( "9 Operar números racionales\n");
printf( "10 Jugar al doble o nada\n");
***(solo les pido ayuda la opcion 1 porque las demas son de igual manera)
scanf("%d",&opcion);



switch (opcion){

   

   case 1:
       
     int numero, resultado;
       printf("Introduzca el número");
        scanf("%d", &n);
        resultado = factorial(n);
      printf(“El factorial de %d es %d”, numero, resultado);
       
       return 0;

      break;



     default:
       printf("Válido\n");

}




int factorial(int n){
if(n > 1)
return n * factorial(n-1);
return 1;

}


No hace falta decir que esta mal, pero no consigo ver y resolver el problema, se que no gacen tarea de los demas pero si me guian sobre esta las otras nueve invocaciones seran mas sencillas dado que son del mismo tipo y dificultad.

Por cierto pra matar dos pajaros de un tiro: ¿como puedo invocar una funcion dentro de otra funcion para despues invocarla en main?

MUchas gracias y espero haberme explicado medianamente bien, saludos y gracias de ante mano.

Hola! primero los errores:

- En el swtich, usas la variable n para el scanf pero esa variable nunca la declaras

- El return 0 en menu esta demas, primero porque menu no retorna nada, es void, y segundo porque no hace falta que obligues a que la funcion termine desde ese punto(o cualquier otro)


Para una mejor estructuracion y generalizacion de funciones, te doy algunos consejos: si la funcion se llama menu, lo que deberia hacer esta funcion es solo mostrar las opciones(los printf) y nada mas. Luego seria conveniente poner el switch dentro del main y finalmente en cada case haces las llamadas a las distintas funciones; ejemplo:

int main() {
  menu();
  /* ...... */
  switch(opcion) {
     case 1:
          funcion1(); /* llamo a funcion */
     /* ........*/
    case 2:
         funcion2();
   /* ......... */
  }
 
return 0;
}

Saludos

Advertencia - mientras estabas escribiendo, una nueva respuesta fue publicada. Probablemente desees revisar tu mensaje.

trabquilo todo es cuestion de tiempo y practica.

creo q eso depende de cada uno, si declaras las variables en el main esta bien y si lo haces en las funciones tambien. Solo que si haces todo en la funcion entonces la funcion no recibiria ningun parametro y no retornaria nada(lo cual no me gusta mucho xD)
Nota: segun lo que dijo avesudra no se pueden declarar variables en el case, pero por lo que tengo entendido,segun el C99, se puede hacer en cualquier parte. Igual solo es cuestion de probar y listo

Solo ordene un poco lo que tenias:

#include <stdio.h>
 
void menu(void);
int factorial(int n);
int primo(int n); /* nose como hiciste esta funcion, pero podrias hacer que si el numero ingresado es primo entonces retorne 1, de lo contrario retorne 0 */
 
int main() {
 
  int numero,opcion, resultado;
  menu();
 
  scanf("%d",&opcion);
 
  switch (opcion)
   {
   case 1:
 
       printf("Introduzca el número");
       scanf("%d", &numero);
       resultado = factorial(numero);
       printf("El factorial de %d es %d", numero, resultado);
       break;
   case 2:
       printf("Introduzca el número");
       scanf("%d", &numero);
       resultado=primo(numero);
 
      if(resultado==1)
         printf("EL numero es primo\n");
      else
         printf("EL numero no es primo \n");
      break;
   } /* fin del switch */
 
return 0;
} /* fin del main */
 
/* .................. Aca van todas las funciones */  

Saludos

PD: el error nose que es

La función primo no estaba bien implementada y la que tenías de factorial inicialmente era mejor.
Te he incluido la librería "#include<locale.h>" con la instrucción en el main de "setlocale(LC_ALL, "Spanish");", lo que permite que se vean los acentos y no símbolos extraños.:

#include <stdio.h>
#include<locale.h>
void menu(void);
int factorial(int n);
int primo(int n);
int main() {
    int numero,opcion, resultado;
    setlocale(LC_ALL, "Spanish");
    while (1){
        menu();
        scanf("%d",&opcion);
        switch (opcion){
           case 0: return 2;
           case 1:
               printf("Introduzca el número: ");
               scanf("%d", &numero);
               resultado = factorial(numero);
               printf("El factorial de %d es %d\n\n\n", numero, resultado);
               break;
           case 2:
                printf("Introduzca el número: ");
                scanf("%d", &numero);
                resultado=primo(numero);
                if(resultado==1)
                    printf("EL número %d es primo\n\n\n",numero);
                else
                    printf("EL número %d no es primo \n\n\n",numero);
                break;
           }
    }
return 0;
}
/******************* MENU **********************/
void menu(){
 
    printf( "Marque la opción que prefiera\n");
    printf( "0 Salir del programa\n");
    printf( "1 Calcular el factorial de una número\n");
    printf( "2 Comprobar si un número es primo\n");
    printf( "3 Calcular el mínimo común múltiplo\n");
    printf( "4 Calcular el máximo común divisor\n");
    printf( "5 Determinar si un año es bisiesto\n");
    printf( "6 Mostrar los años bisiestos\n");
    printf( "7 Mostrar número de dias de un mes y año\n");
    printf( "8 Calcular números combinatorios\n");
    printf( "9 Operar números racionales\n");
    printf( "10 Jugar al doble o nada\n");
 
}
/****************** FACTORIAL ******************/
int factorial(int n)
{
    if(n > 1)
        return n * factorial(n-1);
    else
        return 1;
}
/***************** PRIMO ***********************/
int primo(int n){
    int x=2;
    while(x<n){
        if (n%x==0)
            return 0;
          x=x++;
    }
    return 1;
}
/*********************************************/
 

Saluditos!.