Skip to main content
4 de 5
se añadieron 1218 caracteres en el cuerpo
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Primero, lo que está mal en multi( ), siendo aplicable a resta( ):

int * multi (int a, int b ) {
  int *r;

  r = ( a * b );

  return r;
}

Al hacer r = ( a * b ), estas asignando a un puntero el resultado de una multiplicación. Aunque fuerces al compilador a aceptarlo, el resultado sera, cuando menos, indefinido; estas haciendo que el puntero apunte a una dirección arbitraria, que creo que no es lo que pretendes.

Nota la diferencia entre asignar a un puntero o al dato apuntado; no es lo mismo. En otras partes de tu código, como *punts = a + b lo haces bien, asignas el valor al dato apuntado. El asterisco (*) delante de la variable marca la diferencia. Sin el, asignas al puntero. Con el, asignas al dato apuntado.

Ventajas de hacerlo asi: en tu código, absolutamente ninguna. En otros casos, solo depende de tu imaginación. Piensa, por ejemplo, en la función malloc( ), que asigna bloques de memória. ¿ Como lo harías sin devolver un puntero ? (vale, es una pregunta trampa. Se podría hacer con punteros a punteros ** ). Un uso muy común es en funciones que crean e inicializan struct u otros TAD (Tipo Abstracto de Dato), y devuelven punteros a los datos creados.

Por lo que deduzco del código, el objetivo del ejercicio es que reutilices una misma variable resultado para todas las operaciones, y practiques con las posibles formas de usarla mediante punteros: pasando un puntero como argumento, o bien devolviendo un puntero como resultado.

Mi versión del mismo ejercicio. Como ejercicio :-p , busca mis cambios e intenta comprenderlos según lo dicho aquí; veo en tu código que ya usas todo lo necesario (excepto el do):

#include <stdio.h>

int results; // FUERA de main, para que sea accesible por todas las funciones.

void suma( int, int, int * );
void divi( int, int, int * );
int *multi( int, int ); 
int *resta( int, int );

int main( ) {
  int num1, num2, op;

  printf( "Ingrese el primer numero:  " );
  scanf( "%d", &num1 );

  printf( "Ingrese el segundo numero:  " );
  scanf( "%d", &num2 ); 

  do {
    printf( "Elija una de las siguientes opciones: \n" );
    printf( " 1-Suma \n 2-Division \n 3-Multiplicacion \n 4-Resta \n 0-Salir \n" );
    scanf( "%d", &op );

    switch( op ) {
    case 1:
      suma( num1, num2, &results );
      printf( "El resultado es: %d \n", results );
      break;

    case 2:
      divi( num1, num2, &results );
      printf( "El resultado es: %d \n", results );
      break;

    case 3:
      printf("El resultado es: %d \n", *multi( num1, num2 ) );
      break;

    case 4:
      printf("El resultado es: %d \n", *resta( num1, num2 ) );
      break;
    }
  } while( op );

  system( "PAUSE" );
  return 0;
}

void suma( int a, int b, int *punts ) {  
  *punts = a + b;
}

void divi( int a, int b, int *puntd ) {
  *puntd = a / b;
}

int *multi( int a, int b ) {
  results = a * b;
  return &results;
}

int *resta( int a, int b ) {
  results = a - b;
  return &results;
}

Pd.: lo se, soy muy quisquilloso con el formato del código xD.

EDITO

El trabajo con punteros es igual que el trabajo con datos normales. Solo podemos asignar un valor a una variable del tipo correcto. Si para asignar un entero hacemos

int a = 10;
int b = a;

para hacerlo con punteros, igual:

int *a = &algo.
int *b = a;

Añado un pequeño ejemplo; una función swapInt( ) que intercambia el valor de 2 int mediante punteros; devuelve 0 si los datos son iguales, 1 si son distintos.

#include <stdio.h>

int swapInt( int *, int * );

int main( void ) {
  int dato1 = 10, dato2 = 20;
  int inter;

  printf( "dato1 = %d, dato2 = %d\n", dato1, dato2 );

  inter = swapInt( &dato1, &dato2 );

  printf( "dato1 = %d, dato2 = %d, se intercambiaron: %d\n", dato1, dato2, inter );

  return 0;
}

void swapInt( int *a, int *b ) {
  int tmp;

  // Comparamos los punteros.
  if( a == b )
    return 0;

  // Comparamos los valores de los datos apuntados.
  if( *a == *b )
    return 0;

  tmp = *a; // tmp == 10.
  a = *b;   // a == 20.
  *b = tmp; // b == 10.

  return 1;
}
Trauma
  • 26.9k
  • 5
  • 42
  • 68