Ayuda! Soy nuevo en programacion "C" Necesicito que me ayuden con FUNCIONES

[smdiegoarmando]

La verdad hice el esfuerzo de terminar el programa, pase toda la noche intentando que corra y no lo logre lo ideas seria que cada opcion llevara funciones pero se me hizo dificil Uso Dev-C ... AYUDAAAA

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include<conio.h>
int cubo ( int n ) {
     return n*n*n;
}
int multiplo ( int x, int y) {
   return x%y;
}
int main(){
int i, temp,n1,n2;
int _cont;
int opc,numero,resultado;
srand(time(NULL));
for(i=0 ; i<5 ; i++){

printf(" \n Menu  ");
printf(" \n 1...Generar y mostrar  5 números aleatorios del 1 al 10  ");
printf(" \n 2...Cubo de un número entero  ");
printf(" \n 3...Determinar si un numero es múltiplo de otro  ");
printf(" \n 4...Mostrar los números pares entre 1 y 40  ");
printf(" \n 5...Salir  ");
printf(" \n ESCOJA UNA OPCION  ");
scanf("%d",opc);
switch (opc)

{
case 1: puts("Generar y mostrar  5 números aleatorios del 1 al 10");
       printf("Numero aleatorio %d es : %d\n",i+1, rand()%10 );
       getch();
       main();
       break;
case 2: puts("Cubo de un número entero");
       printf(" \n Ingrese un numero");
       scanf("%d",&numero);
       resultado = cubo(numero);
       printf("El cuadrado del numero es %d", resultado);
       getch();
       main();
       break;
case 3: puts("Determinar si un numero es múltiplo de otro");
       printf(" \n Ingrese un numero");
       scanf("%d",&n1);
       printf(" \n Ingrese otro numero");
       scanf("%d",&n2);
       resultado=multiplo(n1,n2);
       if (n1%n2==0)
       {
       printf(" \n Los numeros son multiplos entre si"); }
       else
       {
       printf(" \n Los numeros no son multiplos entre si"); }
       getch();
       main();
       break;  
case 4: puts("Mostrar los números pares entre 1  y 40");
        while(_cont<=40){
        if(_cont%2==0){
        printf("%d, ",_cont);
        }
        _cont++;
    }
        getch();
        main();
        break;
case 5:puts("salir");
      printf("Fin del programa");
      getch();
      break;
      default: printf(" opcion invalalida");
      }
      getch();        
                       }            
system("pause");
}

[amchacon]

Código (cpp) [Seleccionar] Expandir

case 1: puts("Generar y mostrar  5 números aleatorios del 1 al 10");
       printf("Numero aleatorio %d es : %d\n",i+1, rand()%10 );
       getch();
       main();
       break;
WTF? ¿Porque haces una llamada al main?.

Eso es super inestable, cada vez que llames a una función se iran creando llamadas proguesivamente gastando memoria y tiempo de ejecución.... Cuando te quedes sin pila el programa te crasheara estrepidosamente. Si lo que quieres esque el programa se repita continuamente ponlo dentro de un while:

Código (cpp) [Seleccionar] Expandir

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include<conio.h>

int cubo ( int n )
{
   return n*n*n;
}
int multiplo ( int x, int y)
{
   return x%y;
}

int main()
{
   int i, temp,n1,n2;
   int _cont;
   int opc,numero,resultado;
   srand(time(NULL));

   while (true)
   {

       printf(" \n Menu  ");
       printf(" \n 1...Generar y mostrar  5 números aleatorios del 1 al 10  ");
       printf(" \n 2...Cubo de un número entero  ");
       printf(" \n 3...Determinar si un numero es múltiplo de otro  ");
       printf(" \n 4...Mostrar los números pares entre 1 y 40  ");
       printf(" \n 5...Salir  ");
       printf(" \n ESCOJA UNA OPCION  ");
       scanf("%d",opc);

       switch (opc)
       {

       case 1:
           puts("Generar y mostrar  5 números aleatorios del 1 al 10");
           printf("Numero aleatorio %d es : %d\n",i+1, rand()%10 );
           getch();

           break;
       case 2:
           puts("Cubo de un número entero");
           printf(" \n Ingrese un numero");
           scanf("%d",&numero);
           resultado = cubo(numero);
           printf("El cuadrado del numero es %d", resultado);
           getch();

           break;
       case 3:
           puts("Determinar si un numero es múltiplo de otro");
           printf(" \n Ingrese un numero");
           scanf("%d",&n1);
           printf(" \n Ingrese otro numero");
           scanf("%d",&n2);
           resultado=multiplo(n1,n2);
           if (n1%n2==0)
           {
               printf(" \n Los numeros son multiplos entre si");
           }
           else
           {
               printf(" \n Los numeros no son multiplos entre si");
           }
           getch();

           break;
       case 4:
           puts("Mostrar los números pares entre 1  y 40");
           while(_cont<=40)
           {
               if(_cont%2==0)
               {
                   printf("%d, ",_cont);
               }
               _cont++;
           }
           getch();

           break;
       case 5:
           puts("salir");
           printf("Fin del programa");
           getch();
           break;
       default:
           printf(" opcion invalalida");
       }
       getch();
       system("CLS"); // Comando para borrar la pantalla
   }
   system("pause");
}

También te he quitado el for del principio porque no servía para nada...

[avesudra]

A parte de lo dicho por amchacon  te lo he puesto en funciones, pero trabaja un poco más porque es que el código es sucio.Y no te digo nada de la portabilidad...Además te faltaba un ampersand al introducir la opción en ese scanf. En fín:

Código (c) [Seleccionar] Expandir

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include<conio.h>//Esto sería bueno quitarlo pero bueno
//PROTOTIPOS DE FUNCIONES
int multiplo ( int x, int y);
void aleatorio();
void cubo();
void esMultiplo();
void pares();
void salir();

int main()
{
   int opc;
   do
   {
       printf(" \n Menu  ");
       printf(" \n 1...Generar y mostrar  5 números aleatorios del 1 al 10  ");
       printf(" \n 2...Cubo de un número entero  ");
       printf(" \n 3...Determinar si un numero es múltiplo de otro  ");
       printf(" \n 4...Mostrar los números pares entre 1 y 40  ");
       printf(" \n 5...Salir  ");
       printf(" \n ESCOJA UNA OPCION  ");
       scanf("%d",&opc);
       switch (opc)
       {
       case 1:
           aleatorio();
           break;
       case 2:
           cubo();
           break;
       case 3:
           esMultiplo();
           break;
       case 4:
           pares();
           break;
       case 5:

           break;
       default:
           printf("Opcion invalida");
       }
       if(opc!=5)
       {
           printf("Presione una tecla para volver al menu principal . . . ");
           getch();
           system("cls");
       }
   }
   while(opc!=5);
   puts("Salir");
   printf("Fin del programa");
   getch();
   return 0;
}
//DEFINICIÓN DE FUNCIONES
int multiplo(int x,int y)
{
   return x%y;
}
void aleatorio()
{
   puts("Generar y mostrar  5 números aleatorios del 1 al 10");
   srand(time(0));
   int i;
   for(i=0; i!=5; ++i)
   {
       printf("Numero aleatorio %d es : %d\n",i+1, rand()%10 );
   }
}
void cubo()
{
   int numero=0;
   int resultado=0;
   puts("Cubo de un número entero");
   printf(" \n Ingrese un numero");
   scanf("%d",&numero);
   resultado = numero*numero*numero;
   printf("El cuadrado del numero es %d", resultado);
}
void esMultiplo()
{
   int n1,n2,resultado;
   puts("Determinar si un numero es múltiplo de otro");
   printf(" \n Ingrese un numero");
   scanf("%d",&n1);
   printf(" \n Ingrese otro numero");
   scanf("%d",&n2);
   resultado=multiplo(n1,n2);
   if (n1%n2==0)
   {
       printf(" \n Los numeros son multiplos entre si");
   }
   else
   {
       printf(" \n Los numeros no son multiplos entre si");
   }
}
void pares()
{
   int _cont;
   puts("Mostrar los números pares entre 1  y 40");
   while(_cont<=40)
   {
       if(_cont%2==0)
       {
           printf("%d, ",_cont);
       }
       _cont++;
   }
}

[smdiegoarmando]

Ciertamente no entiendo porque el programa corre pero al ingresar una de las opciones del 1 al 5 se cierra automaticamente muestra -> NombreDelPrograma.exe dejo de funcionar.

Sera que tego problema con el Dev-C ?

[avesudra]

Pero cual de los códigos, ¿el que tu has puesto, el que ha puesto amchacon o el que he puesto yo? El que tu y amchacon habeis puesto debería dar error en ejecución como dices por esta sentencia, donde falta un ampersand:

Código (c) [Seleccionar] Expandir

scanf("%d",opc);
Debería estar así:

Código (c) [Seleccionar] Expandir

scanf("%d",&opc);

[smdiegoarmando]

Pero cual de los códigos, ¿el que tu has puesto, el que ha puesto amchacon o el que he puesto yo?

Avesudra utilice tu codigo

[avesudra]

Pues es por el Dev-Cpp sin duda. Porque en Codeblocks con la ultima versión de MinGW funciona a la perfección.Puedes descargarlo e instalarlo desde aquí:
http://sourceforge.net/projects/codeblocks/files/Binaries/12.11/Windows/codeblocks-12.11mingw-setup_user.exe/download?_test=updater&utm_expid=65835818-0&utm_referrer=http%3A%2F%2Fwww.codeblocks.org%2Fdownloads%2F26

[dato000]

Pues es por el Dev-Cpp sin duda. Porque en Codeblocks con la ultima versión de MinGW funciona a la perfección.Puedes descargarlo e instalarlo desde aquí:
http://sourceforge.net/projects/codeblocks/files/Binaries/12.11/Windows/codeblocks-12.11mingw-setup_user.exe/download?_test=updater&utm_expid=65835818-0&utm_referrer=http%3A%2F%2Fwww.codeblocks.org%2Fdownloads%2F26

Si señor, poco más puedo agregar a los gurus de aqui, dev-c++ apesta a millones, CodeBlocks con MinGW es mucho pero millones años luz mejor.

[smdiegoarmando]

Pues es por el Dev-Cpp sin duda. Porque en Codeblocks con la ultima versión de MinGW funciona a la perfección.Puedes descargarlo e instalarlo desde aquí:
http://sourceforge.net/projects/codeblocks/files/Binaries/12.11/Windows/codeblocks-12.11mingw-setup_user.exe/download?_test=updater&utm_expid=65835818-0&utm_referrer=http%3A%2F%2Fwww.codeblocks.org%2Fdownloads%2F26

Ya instale el Codeblocks y me va de maravilla el programa es genial y todo gracias a ti aunque por una extraña forma la opcion 4...Mostrar los números pares entre 1 y 40  " no me muestra los numeros :/

[avesudra]

No es extraño es un fallo de programación, ha sido error mío al declarar la variable, perdón jeje  . Para arreglarlo, en la función:

Código (c) [Seleccionar] Expandir

void pares()
{
   int _cont;
   puts("Mostrar los números pares entre 1  y 40");
   while(_cont<=40)
   {
       if(_cont%2==0)
       {
           printf("%d, ",_cont);
       }
       _cont++;
   }
}
Debes inicializar _cont en 0 pues sino se inicializa con un valor aleatorio.Quedando así:

Código (c) [Seleccionar] Expandir

void pares()
{
   int _cont=0;
   puts("Mostrar los números pares entre 1  y 40");
   while(_cont<=40)
   {
       if(_cont%2==0)
       {
           printf("%d, ",_cont);
       }
       _cont++;
   }
}