basura en mi programa

[Vaagish]

@ivancea96: Me ganaste de mano jajaja estaba buscando la info para que la vea..

http://c.conclase.net/curso/?cap=021

Me parece que lo mas Pro que podes hacer en ese caso, es sobrecargar la funcion (como ya te dijo ivancea96)

[leosansan]

gracias por sus respuestas, en realidad no son tantos scanf, solo que queria ahorrarme líneas de código para que cada vez que tuviera que ingresar un dato no repitiera fgets +sscanf. Le hice unos cambios al codigo y ya funciona para todos los casos excepto para el de ARITMETICA_BINARIA, que me pide que le ingrese un operando (que es de tipo char) y en la funcion de ingresarDato, me dice que el sscanf me va a guardar  el dato en una variable de tipo int, como puedo corregir esto?? que el sscanf me pueda guardar datos tanto de tipo int como de tipo char??

Una alternativa es, análogamente a lo que haces en el Menú, donde las opciones se corresponden con un valor numérico, hacer lo mismo con las operaciones y así sigues usando la variable en formato int:

Código (cpp) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#define ARITMETICA_SIMPLE 1
#define ARITMETICA_BINARIA 2
#define FACTORIAL 3
#define TABLA_MULTIPLICAR 4
#define SALIR 5

void ingresarDato(char *ch , int *dato);

int main()
{
    //Variables de uso interno del programa
    int continuar = 1,i,factorial = 1;
    //Variables para entrada de datos
    int opcion,numero1,numero2,numero_factorial,multiplicando,multiplicador;
    char operador;
    do
    {
        //Menu principal
        printf("CALCULADORA v1.0\n\n");
        printf("Elige una de las siguientes opciones:\n");
        printf("1. Aritmetica simple con suma de enteros a+b\n");
        printf("2. Aritmetica extendida binaria\n");
        printf("3. Factorial de n (n!)\n");
        printf("4. Detallar una tabla de multiplicar\n");
        printf("5. Salir\n");
        ingresarDato("Opcion a elegir? ",&opcion);
        switch(opcion)
        {
            //Aritmetica simple con suma de enteros a+b
            case ARITMETICA_SIMPLE:
               ingresarDato("Dame el valor de a = ",&numero1);
               ingresarDato("Dame el valor de b = ",&numero2);
               printf("%d + %d = %d",numero1,numero2,numero1+numero2);
               break;
            //Aritmetica extendida binaria (+,-,*,/,%modulo)
            case ARITMETICA_BINARIA:
               ingresarDato("Escribe el operador (1=+, 2=-, 3=*, 4=/, 5=%): ",&operador);
               ingresarDato("\nDame el valor de a = ",&numero1);
               ingresarDato("Dame el valor de b = ",&numero2);
               switch(operador)
               {
                   case 1:
                      printf("%d + %d = %d\n",numero1,numero2,numero1+numero2);
                      break;
                   case 2:
                      printf("%d - %d = %d\n",numero1,numero2,numero1-numero2);
                      break;
                   case 3:
                      printf("%d * %d = %d\n",numero1,numero2,numero1*numero2);
                      break;
                   case 4:
                      printf("%d / %d = %d\n",numero1,numero2,numero1/numero2);
                      break;
                   case 5:
                      printf("%d %% %d = %d\n",numero1,numero2,numero1%numero2);
                      break;
                   default:
                      printf("No hay un resultado calculable\n");
               }
               break;
            //Factorial de n (n!)
            case FACTORIAL:
               ingresarDato("Dame el valor de n = ",&numero_factorial);
               if("Dame el valor de n = ",numero_factorial < 0)
                  printf("El factorial de %d no esta definido\n",numero_factorial);
               else if(numero_factorial == 0)
                  printf("El factorial de 0 es 1\n");
               else
               {
                   for(i = numero_factorial;i > 0;i--)
                      factorial*=i;
                   printf("El factorial de %d es %d\n",numero_factorial,factorial);
               }
               break;
            //Detallar una tabla de multiplicar
            case TABLA_MULTIPLICAR:
               ingresarDato("Dame el multiplicando i = ",&multiplicando);
               ingresarDato("Dame hasta cual multiplicador n= ",&multiplicador);
               printf("Tabla de multiplicar del %d\n",multiplicador);
               i = 1;
               while(i <= multiplicador)
               {
                   printf("%d\tX\t%d\t=\t%d\n",multiplicando,i,multiplicando*i);
                   i++;
               }
               break;
            //Salida del programa
            case SALIR:
               continuar = 0;
               break;
            default:
               printf("Opcion no valida!\n");
        }
        if(continuar)
        system("pause");
    }while(continuar);
    return 0;
}

void ingresarDato(char *ch , int *dato)
{
    printf("%s ",ch);
    fflush (stdout);
    scanf ("%d",dato);
}


Saluditos! ....

[dato000]

Bueno, creo ya tienen cubierto todas las opciones, no?? pero yo me pregunto porque no separas las opciones de cada menu en otras funciones, se veria bueno, mejor no??

Código (c) [Seleccionar] Expandir


void ingresarDato(char *ch , int *dato)
{
    printf("%s ",ch);
    fflush (stdout);
    scanf ("%d",dato);
}


Me parece una buena opción usar parametros por referencia, así puede sobrecargarse sin mayor inconveniente cada variable, yo iba a decir que solo se usara para el valor númerico, pero puede que funcione bien para el caracter.

Lo que me pregunto es por el fflush(stdout), no seria necesario realizar uno para el scanf?? es decir, fflush(stdin) para limpiar el buffer de entrada del teclado??

[rir3760]

ya solo le faltaria otro argumento para indicarle el tipo de dato, pero como se le puede pasar por argumento un tipo de dato y que este sea int?

No pasas el tipo, pasas un valor entero que indique el tipo. Por ejemplo:

Código (c) [Seleccionar] Expandir

enum tipo {INT, DOUBLE, CHAR};

/* ... */

void leer_valor(char const *msj, int tipo, void *p)
{
   int ch;
   
   printf("%s", msj);
   fflush(stdout);
   
   switch (tipo){
   case INT:
      scanf("%d", (int *) p);
      break;
   case DOUBLE:
      scanf("%lf", (double *) p);
      break;
   case CHAR:
      scanf("%c", (char *) p);
      break;
   }
   
   while ((ch = getchar()) != EOF && ch != '\n')
      ;
}

Para utilizar la función basta con indicar que leer y pasar la dirección del objeto:

Código (c) [Seleccionar] Expandir

int main(void)
{
   int num;
   char ch;
   
   leer_valor("Introduce un entero: ", INT, &num);
   printf("El numero es %d\n", num);
   
   leer_valor("Introduce un caracter: ", CHAR, &ch);
   printf("El caracter es %c\n", ch);
   
   return EXIT_SUCCESS;
}
Falta agregar la validación en el caso de que la función scanf falle.

Lo que me pregunto es por el fflush(stdout), no seria necesario realizar uno para el scanf?? es decir, fflush(stdin) para limpiar el buffer de entrada del teclado?

fflush(stdout) es necesario para asegurarnos de que el texto se envié a la salida estándar, fflush(stdin) no se recomienda (se explica en el tema |Lo que no hay que hacer en C/C++. Nivel basico|), en su lugar se puede eliminar el resto de la linea con un bucle.

Un saludo

[dato000]

ah vale no me acordaba de eso, gracias

[m@o_614]

Ya pude hacer que el código me hiciera lo que le pido, sólo que en la función de ingresar dato en vez de ponerle un switch lo hice con if's, no se cuál de los dos será mejor o más eficiente,pero funciona

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#define TAM 5
#define ARITMETICA_SIMPLE 1
#define ARITMETICA_BINARIA 2
#define FACTORIAL 3
#define TABLA_MULTIPLICAR 4
#define SALIR 5

enum tipo{INT,CHAR};
void ingresarDato(void *dato,int tipo);

int main()
{
    //Variables de uso interno del programa
    int continuar = 1,i,factorial = 1;
    //Variables para entrada de datos
    int opcion,numero1,numero2,numero_factorial,multiplicando,multiplicador;
    char operador;
    do
    {
        //Menu principal
        printf("CALCULADORA v1.0\n\n");
        printf("Elige una de las siguientes opciones:\n");
        printf("1. Aritmetica simple con suma de enteros a+b\n");
        printf("2. Aritmetica extendida binaria\n");
        printf("3. Factorial de n (n!)\n");
        printf("4. Detallar una tabla de multiplicar\n");
        printf("5. Salir\n");
        printf("Opcion a elegir? ");
        ingresarDato(&opcion,INT);
        switch(opcion)
        {
            //Aritmetica simple con suma de enteros a+b
            case ARITMETICA_SIMPLE:
               printf("Dame el valor de a = ");
               ingresarDato(&numero1,INT);
               printf("Dame el valor de b = ");
               ingresarDato(&numero2,INT);
               printf("%d + %d = %d",numero1,numero2,numero1+numero2);
               break;
            //Aritmetica extendida binaria (+,-,*,/,%modulo)
            case ARITMETICA_BINARIA:
               printf("Escribe el operador (+,-,*,/,%%): ");
               ingresarDato(&operador,CHAR);
               printf("\nDame el valor de a = ");
               ingresarDato(&numero1,INT);
               printf("Dame el valor de b = ");
               ingresarDato(&numero2,INT);
               switch(operador)
               {
                   case '+':
                      printf("%d %c %d = %d\n",numero1,operador,numero2,numero1+numero2);
                      break;
                   case '-':
                      printf("%d %c %d = %d\n",numero1,operador,numero2,numero1-numero2);
                      break;
                   case '*':
                      printf("%d %c %d = %d\n",numero1,operador,numero2,numero1*numero2);
                      break;
                   case '/':
                      printf("%d %c %d = %d\n",numero1,operador,numero2,numero1/numero2);
                      break;
                   case '%':
                      printf("%d %c %d = %d\n",numero1,operador,numero2,numero1%numero2);
                      break;
                   default:
                      printf("No hay un resultado calculable\n");
                      printf("%d %c %d = 0\n",numero1,operador,numero2);
               }
               break;
            //Factorial de n (n!)
            case FACTORIAL:
               printf("Dame el valor de n = ");
               ingresarDato(&numero_factorial,INT);
               if(numero_factorial < 0)
                  printf("El factorial de %d no esta definido\n",numero_factorial);
               else if(numero_factorial == 0)
                  printf("El factorial de 0 es 1\n");
               else
               {
                   for(i = numero_factorial;i > 0;i--)
                      factorial*=i;
                   printf("El factorial de %d es %d\n",numero_factorial,factorial);
               }
               break;
            //Detallar una tabla de multiplicar
            case TABLA_MULTIPLICAR:
               printf("Dame el multiplicando i = ");
               ingresarDato(&multiplicando,INT);
               printf("Dame hasta cual multiplicador n= ");
               ingresarDato(&multiplicador,INT);
               printf("Tabla de multiplicar del %d\n",multiplicador);
               i = 1;
               while(i <= multiplicador)
               {
                   printf("%d\tX\t%d\t=\t%d\n",multiplicando,i,multiplicando*i);
                   i++;
               }
               break;
            //Salida del programa
            case SALIR:
               continuar = 0;
               break;
            default:
               printf("Opcion no valida!\n");
        }
        if(continuar)
           system("pause");
    }while(continuar);
    return 0;
}

void ingresarDato(void *dato,int tipo)
{
    char buffer[TAM];
    fgets(buffer,sizeof(buffer),stdin);
    if(tipo == INT)
       sscanf(buffer,"%d",dato);
    else if(tipo == CHAR)
       sscanf(buffer,"%c",dato);
}


gracias

[Vaagish]

sólo que en la función de ingresar dato en vez de ponerle un switch lo hice con if's, no se cuál de los dos será mejor o más eficiente,pero funciona

Es indiferente,, pero para algo tan corto como 4 sentencias, creo que es mejor usar el if

Saludos!