ayuda con programa

[Mephisto]

Primero: todas las funciones definelas como void Funcion(Parametros); no requieres de regresar valore ya que estas imprimiendo tu resultado en la funcion.

Segundo: en el switch, para cada case igualas una variable int con un char
opcion = '1'; la verdad no le veo el caso a esa igualdad.

Tercero: Para cada caso del switch mandas llamar a la fucion menu(), en vez de eso, tienes dos opciones, colocar un do-while para verificar sea pro ejemplo 0 y es cuando acaba o defines int menu (void); y cuando sea el valor 0 le mandas un return para finalizar la ejecución de la funcion...