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. Lo principal es recordar que puntero a int es un tipo en si mismo:
int a = 10; // tipo de a -> int
int *b = &a; // tipo de b -> puntero a int
char *str = "hola, mundo !!"; // tipo de str -> puntero a caracter
int c = a; // mismo tipo
int d = b; // NO. TIPOS DISTINTOS.
int *z = b; // mismo tipo
char ch1 = str; // NO. TIPOS DISTINTOS.
char ch2 = *str; // mismo tipo
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;
}