Autor
|
Tema: Ayuda con estos códigos (Actualizado 22/06/2012) (Leído 10,004 veces)
|
LoLo2207
Desconectado
Mensajes: 9
|
Hola, estoy intentando hacer un ejercicio sobre aritmética racional (sumar, restar, multiplicar y dividir fracciones). Al mostrar los resultados hay que simplificarlos, pero según veo en el depurador, no se pasan correctamente los valores a simplificar a la función. Qué debo cambiar? Seguramente sea muy obvio, pero soy malísimo con punteros y estructuras xD Gracias. /* Fuente: RACIONAL.C Programa: ARITMETICA BASICA CON NUMEROS RACIONALES Descripción: Este programa permite operar con números racionales. Un número racional se caracteriza por estar expresado con un denominador y un denominador. Las operaciones que contempla son: - suma - resta - producto - división Además utiliza una función para proporcionar siempre la expresión irreducible del número racional */ #include <stdio.h> #include <math.h> #include <ctype.h> /* ---------------------------------------------------------- */ /* Definiciones globales */ /* ---------------------------------------------------------- */ typedef struct { int num; /* Numerador */ int denom; /* Denominador */ } Racional; /* ---------------------------------------------------------- */ /* Prototipos */ /* ---------------------------------------------------------- */ void Modulo_Principal (void); void Presentacion (void); void LeerRacional (Racional *); void MostrarRacional (Racional); void SumaRacional (Racional, Racional, Racional *); void RestaRacional (Racional, Racional, Racional *); void MultiplicaRacional (Racional, Racional, Racional *); void DivideRacional (Racional, Racional, Racional *); void SimplificaRacional (Racional *); int mcd (int, int); /* ---------------------------------------------------------- */ int main (void) { /* ---------------------------------------------------------- */ char respuesta; Presentacion(); do Modulo_Principal(); do { printf("¿Otra ejecución (S/N)?: "); fflush(stdin ); /* Limpia el buffer de teclado */ } while (respuesta != 'S' && respuesta != 'N'); } while (respuesta != 'N'); return 0; } /* Fin del main */ /* ---------------------------------------------------------- */ void Modulo_Principal (void) { /* ---------------------------------------------------------- */ Racional rac1, rac2, rsum, rres, rprod, rdiv; printf("Introduzca el primer número racional:\n"); LeerRacional(&rac1); printf("Introduzca el segundo número racional:\n"); LeerRacional(&rac2); SumaRacional (rac1, rac2, &rsum); MostrarRacional(rsum); RestaRacional (rac1, rac2, &rres); MostrarRacional(rres); MultiplicaRacional (rac1, rac2, &rprod); MostrarRacional(rprod); DivideRacional (rac1, rac2, &rdiv); MostrarRacional(rdiv); } /* ---------------------------------------------------------- */ void Presentacion (void) { /* ---------------------------------------------------------- */ puts("ARITMETICA BASICA CON NUMEROS RACIONALES"); puts("========================================"); puts("Realiza la suma de números racionales"); puts("Siempre presenta la expresión irreducible del número racional"); } /* ---------------------------------------------------------- */ void LeerRacional (Racional *r) { /* ---------------------------------------------------------- */ scanf("%d/%d", &(r ->num ), &(r ->denom )); } /* ---------------------------------------------------------- */ void SumaRacional (Racional a, Racional b, Racional *r) { /* ---------------------------------------------------------- */ if(a.denom == b.denom){ (r->num)=(a.num + b.num); (r->denom)=(a.denom); }else{ (r->denom)=(a.denom * b.denom); (r->num)=(((r->denom/a.denom)*a.num) + ((r->denom/b.denom)*b.num)); } SimplificaRacional(&r); } /* ---------------------------------------------------------- */ void RestaRacional (Racional a, Racional b, Racional *r) { /* ---------------------------------------------------------- */ if(a.denom == b.denom){ (r->denom)=(a.denom); (r->num) = (a.num - b.num); }else{ (r->denom)=(a.denom * b.denom); (r->num)=(((r->denom/a.denom)*a.num) - ((r->denom/b.denom)*b.num)); } SimplificaRacional(&r); } /* ---------------------------------------------------------- */ void MultiplicaRacional (Racional a, Racional b, Racional *r) { /* ---------------------------------------------------------- */ (r->num)=(a.num*b.num); (r->denom)=(a.denom*b.denom); SimplificaRacional(&r); } /* ---------------------------------------------------------- */ void DivideRacional (Racional a, Racional b, Racional *r) { /* ---------------------------------------------------------- */ (r->num)=(a.num*b.denom); (r->denom=a.denom*b.num); SimplificaRacional(&r); } /* ---------------------------------------------------------- */ void MostrarRacional (Racional r) { /* ---------------------------------------------------------- */ if (r.denom != 1) } /* ---------------------------------------------------------- */ void SimplificaRacional (Racional *r) { /* ---------------------------------------------------------- */ // Calcula el mcd del numerador y del denominador // Divide numerador y denominador por su mcd // Si la fraccion debe llevar segno negativo (num*dem <0) // entonces el signo se coloca al numerador int min; min=mcd(r->num, r->denom); r->num = (r->num)/min; r->denom =(r->denom)/min; if((r->num * r->denom)<0){ r->num = - r->num; } } /* ---------------------------------------------------------- */ int mcd (int n, int d) { /* ---------------------------------------------------------- */ /* Calcula el máximo común dividor de n y d */ /* Precisa que d sea distinto de cero */ /* Precisa que n > d */ int c; if(n!=0 && d!=0 && n>d){ do{ c=n%d; if(c!=0){ n=d; d=c; } }while(c!=0); }else{ return; } return d; }
EDIT: Para evitar abrir más hilos, pongo en este todas mis dudas personales. Nuevo programa añadido el 22/06/2012.
|
|
« Última modificación: 22 Junio 2012, 02:43 am por LoLo2207 »
|
En línea
|
|
|
|
durasno
Desconectado
Mensajes: 373
|
Hola LoLo2207! el problema es cuando llamas a la funcion SimplificaRacional. Vos le pasas la direccion del puntero r, (&r), lo que estas haciendo aca es pasarle a la funcion un puntero a puntero. Solamente deberias pasarle el puntero, osea SimplificaRacional(r);
Saludos
PD: el programa tambien funcionaria con puntero a puntero solo que tendrias que saber como se utiliza
|
|
|
En línea
|
Ahorrate una pregunta, lee el man
|
|
|
LoLo2207
Desconectado
Mensajes: 9
|
Gracias, ya funciona perfectamente A ver si he entendido bien, al llamar a la función, como en el prototipo ya sabe que recibirá un puntero, sólo tengo que poner el nombre del puntero, no la dirección, porque si no sería la direccion de un puntero, que a su vez sería la dirección de una direccíon, no? Y lo de usar dobles punteros pues como que no, que además los prototipos me los da el profesor y nosotros sólo tenermos que rellenar el "esqueleto".
|
|
|
En línea
|
|
|
|
durasno
Desconectado
Mensajes: 373
|
A ver si he entendido bien, al llamar a la función, como en el prototipo ya sabe que recibirá un puntero, sólo tengo que poner el nombre del puntero, no la dirección, porque si no sería la direccion de un puntero, que a su vez sería la dirección de una direccíon, no? SI, es como vos decis Saludos
|
|
|
En línea
|
Ahorrate una pregunta, lee el man
|
|
|
LoLo2207
Desconectado
Mensajes: 9
|
Nuevo programa: se supone que tiene que leer una ficha de un alumno creada mediante estructuras, y luego imprimirla. No me acaba de salir Qué lío de punteros, estructuras, typedef y la madre que lo trajo... #include <stdio.h> #include <stdlib.h> #define MAXN 10 /* Número máximo de notas del alumno */ typedef struct{ int dia; int mes; int anno; }tipo_fecha; typedef struct{ char nombre[51]; int curso; tipo_fecha fechaNacimiento; char direccion[61]; float notas[MAXN]; }tipo_alumno; /* PROTOTIPOS */ void leerAlumno(tipo_alumno *); void imprimiralumno(tipo_alumno *); /* FUNCIONES */ void leerAlumno(tipo_alumno * pa){ int i; printf("Introduzca el nombre y apellidos del alumno (max. 50 caracteres): "); printf("Fecha de Nacimiento (dd/mm/aaaa): "); scanf("%2d/%2d/%4d", &pa ->fechaNacimiento. dia, &pa ->fechaNacimiento. mes, &pa ->fechaNacimiento. anno); for(i=0; i<MAXN; i++){ scanf("%f", &pa ->notas [i ]); } } void imprimirAlumno(tipo_alumno * pa){ int i; printf("Nombre: %s", pa ->nombre ); printf("Curso: %d", pa ->curso ); printf("Fecha de Nacimiento: %2d/%2d/%4d\n", pa ->fechaNacimiento. dia, pa ->fechaNacimiento. mes, pa ->fechaNacimiento. anno), printf("Direccion: %s\n", pa ->direccion ); for(i=0; i<MAXN; i++){ printf("Nota[%d]: %2.1f\n", i +1, pa ->notas [i ]); } } int main (void){ tipo_alumno alumno; tipo_alumno * ptalumno; ptalumno=&alumno; leerAlumno(ptalumno); imprimirAlumno(ptalumno); return 0; }
Por cierto, a ver si me aclaro con las estructuras. En mi libro vienen varias formas de declarar tipos y crear variables mediante el typedef typedef struct tipo_alumno{ tipo1 miembro1; tipo2 miembro2; ... tipoN miembroN; } var1, var2,...;
El tipo_alumno es opcional si no quieres ponerle una etiqueta al tipo no? Pero si no se la pones no puedes crear por ejemplo una var3 después, o sí porque está puesto el typedef? Qué forma me recomendáis para definir un tipo de dato fuera del main?
|
|
« Última modificación: 13 Abril 2012, 16:54 pm por LoLo2207 »
|
En línea
|
|
|
|
durasno
Desconectado
Mensajes: 373
|
Hola! el programa te anda?? porque a mi me funciona bien... typedef struct tipo_alumno{ //En este caso tipo_alumno es opcional tipo1 miembro1; tipo2 miembro2; ... tipoN miembroN; } var1, var2,...;
Lo que estas haciendo aca es declarar tipos de datos no variables; var1 y var2 no son variables, esto es por el typedef. Para crear una variable vas a tener que declarar, ej: var1 MI_ESTRUCTURA; // MI_ESTRUCTURA es la variable
No es lo mismo hacer lo anterior q hacer: struct tipo_alumno{ // En este caso tipo_alumno tambien es opcional tipo1 miembro1; tipo2 miembro2; ... tipoN miembroN; } var1, var2,...;
Aca si var1 y var2 son variables declaradas de forma globalOtra forma es no declarando variable globales: struct tipo_alumno{ // [b]En este caso tipo_alumno tiene que ir si o si[/b] tipo1 miembro1; tipo2 miembro2; ... tipoN miembroN; };
Para declarar una variable en este caso seria: main() { struct tipo_alumno MI_ESTRUCTURA; // esta variable es local al main ...... }
Ahora cual es la forma mas conveniente depende de cada uno, en mi caso no trabajo con variables globales y tampoco uso el typedef. Es por eso q prefiero el tercer ejemplo, me parece mas facil de entender Espero te sirva, Saludos
|
|
|
En línea
|
Ahorrate una pregunta, lee el man
|
|
|
LoLo2207
Desconectado
Mensajes: 9
|
Gracias por responder. He estado haciendo mientras tanto otros ejercicios, y para todos ellos he utilizado lo siguiente (siempre tras los #include y los #define, y fuera del main o de cualquier otra función): typedef struct tipo_jugador{ tipo1 miembro1; tipo2 miembro2; ... tipoN miembroN; }tipo_jugador;
Y luego en el main: tipo_jugador equipo[MAX_JUG];
Y me funciona bien. Lo que quiero saber es si es muy redundante o no, y qué diferencia hay enter los dos tipo_jugador, el de antes y después de las llaves. De todas formas, me he dado cuenta de que una función puede devolver una estructura por valor, ya que creía que como esa un tipo de dato compuesto pues se tenía que hacer sí o sí por referencia. Probaré a empezar de 0 con el programa anterior, ya que aunque me compilaba, no se guardaban bien los valores en las variables (y obviamente se imprimían cosas aleatorias).
|
|
|
En línea
|
|
|
|
durasno
Desconectado
Mensajes: 373
|
Practicamente no hay ninguna diferencia, el tipo_jugador antes de la llave es el nombre de la estructura y el tipo_jugador despues de la llave es el nuevo tipo de dato definido por vos, en este caso lleva el mismo nombre
Saludos
|
|
|
En línea
|
Ahorrate una pregunta, lee el man
|
|
|
LoLo2207
Desconectado
Mensajes: 9
|
Otra dudilla. Esta vez estoy con listas simplemente enlazadas. Tengo que hacer un programa que cree una lista enlazada, que en su interior tenga un num. aleatorio >0 y que la muestre, junto con el nº de nodos. (hasta aquí todo bien) El problema es que también tengo que pedir un nº por teclado y eliminar todos los nodos que contengan un nº igual o mayor al introducido, y me falla esta última función. Os pongo sólo la función para ahorrar espacio. Si fuera necesario, luego pondría el programa completo. void EliminarNodos(NODO *lst, int maximo){ NODO *indice=NULL, *anterior=NULL; if(lst!=NULL){ indice=lst; while((indice->sig)!=NULL){ if((indice->num)>=maximo){ if(anterior==NULL){ lst=(indice->sig); indice=lst; }else{ (anterior->sig)=(indice->sig); indice=anterior->sig; } } } } }
A ver si véis en qué me falla. Porque también tengo que hacer otro prog. muy parecido al anterior, pero en vez de eliminar nodos con el nº mayor, tengo que ordenar la lista y quitar los repetidos. Pero eso ya para luego, que si me sale este a lo mejor entendiendo lo que tenía mal puede que me salga el otro. Saludos y gracias.
|
|
|
En línea
|
|
|
|
durasno
Desconectado
Mensajes: 373
|
Hola! lo que subiste no funciona xq anterior siempre es igual a NULL, nunca entra al else; pero no esta del todo mal... De todas formas el error principal esta en que no tenes claro el concepto de pasaje por valor y por pseudoreferencia. A la funcion vos le pasas el puntero lst, es decir en la llamada haces EliminarNodos(lst, maximo); // pasas por valor a lst
El problema es que si vos tratas asi a lst cualquier modificacion que hagas en la funcion(es decir haces q apunte a otro lado) cuando retornes al main no va a tener efecto... Para resolver esto tenes q pasar a lst por pseudoreferencia// en la llamada EliminarNodos(&lst, maximo); // paso la direccion de lst
y en la funcion: void EliminarNodos(NODO **lst, int maximo) // se trata como un puntero a puntero { }
Entonces, el codigo seria mas o menos asi: void EliminarNodos(NODO **lst, int maximo){ NODO *indice=NULL, *anterior=NULL; int flag; if(*lst!=NULL){ indice=*lst; while((indice)!=NULL){ flag=0; if((indice->num)>=maximo){ if(indice==*lst){ // es decir es el inicio de la lista *lst=indice->sig; // ahora el inicio de la lista es el siguiente indice=*lst; }else{ // no es el inicio anterior->sig=indice->sig; indice=anterior->sig; } flag=1; } if(!flag) { /* nose borran nodo */ anterior=indice; indice=anterior->sig; /* cambia al sig nodo */ } } /* fin del while */ } }
Fijate q solo agregue un par de cosas a lo q tenias(ya q no estaba tan mal). Te falto considerar que pasa cuando indice->num>=maximo es falso, el bucle se vuelve infinito ya que no cambias de nodo, te falto agregar indice=anterior->sig; pero para q esto funcione bien agregue un flag. Cuando flag=0 es porque no se cumplio que num>=maximo(nose borran nodos) entonces se ejecuta if(!flag) { anterior=indice; indice=anterior->sig; /* cambia al sig nodo */ }
de lo contrario flag=1 nose ejecuta lo anterior mencionada... Si funciona lo que puse, analiza porque lo hice asi teniendo en cuenta todas las consideraciones. Saludos
|
|
« Última modificación: 22 Junio 2012, 08:27 am por durasno »
|
En línea
|
Ahorrate una pregunta, lee el man
|
|
|
|
Mensajes similares |
|
Asunto |
Iniciado por |
Respuestas |
Vistas |
Último mensaje |
|
|
[Aporte] Factorización Relativamente Rápida - Actualizado 18/08/2012 -
.NET (C#, VB.NET, ASP)
|
Keyen Night
|
0
|
5,262
|
15 Marzo 2011, 04:38 am
por Keyen Night
|
|
|
Como se desencriptan códigos como estos: ''sºÁÓrX''
Programación General
|
Ngeooz
|
0
|
2,026
|
29 Diciembre 2013, 23:58 pm
por Ngeooz
|
|
|
¿Cómo mejorar estos códigos?
PHP
|
Richter{};
|
1
|
1,898
|
16 Noviembre 2014, 00:09 am
por dimitrix
|
|
|
Alguien me explicaría que son estos codigos
Programación C/C++
|
naidmen
|
2
|
2,351
|
13 Febrero 2017, 21:17 pm
por dato000
|
|
|
(AYUDA POR FAVOR) determinar salida en estos códigos
Programación C/C++
|
victorportes
|
1
|
2,129
|
10 Octubre 2018, 01:25 am
por Beginner Web
|
|