[RESUELTO, AHORA VARIANTES]Hola, tengo un ligero problema

Iniciado por alex030293, 27 Abril 2011, 16:40 PM

0 Miembros y 1 Visitante están viendo este tema.

alex030293

Pues eso, estos días he decidido comenzar a aprender C, un poco por mi cuenta, todo un reto.

Después de leer algunos artículos, hice el típico "Hello world", un programa para sumar, multiplicar etc etc, un comparador de números, y demás chorradas.

Intentando ir un poco por mi cuenta, me dio por hacer un pequeño programilla que resuelva ecuaciones de 2º grado, pero el fallo es que los valores que devuelve están redondeados a enteros, este es mi código:


#include <stdio.h>
#include <conio.h>
#include <math.h>
int main(void)
{

   /*Definir variables*/
   int a;
   int b;
   int c;
   int value1;
   int value2;

   /*Introducir coeficientes*/
   printf("Introduce el coeficiente del la x^2:");
   scanf("%d", &a);
   printf("Introduce el coeficiente del la x:");
   scanf("%d", &b);
   printf("Introduce el termino independiente:");
   scanf("%d", &c);

   /*Para raíces positivas*/
   if((b*b-4*a*c) > 0){                      
   value1 = ((-b)+sqrt(b*b-4*a*c))/(2*a);
   value2 = ((-b)-sqrt(b*b-4*a*c))/(2*a);
   printf("La x vale: %d y %d", value1, value2);
   }

   /*Para raíces 0 o negativas*/
   if((b*b-4*a*c) <= 0){
   printf("No existe solucion");
   }
   
   getch();
}


Creo que el fallo es algún %d, pero no estoy seguro, también creo que debería meter algun float por ahí, pero tras intentarlo no me ha salido gran cosa.

Me encomiendo a vosotros, gracias de antemano.
www.flickr.com/photos/alex030293

Mi equipo: Intel i7 955 @4.0GHz + Nvidia GTX 460 + Nvidia 9800GT para Physx + 12Gb RAM DDR3 1600 Mghz + HDD Seagate Barracuda + Themaltake ProWater 850i.

Bledniss

Tampoco se mucho, tambien estoy empezando en C pero n error que he visto es que utilizas un signo que no debes  printf("%d", &a)
Porque ese singo es el que lee la informacion de dicha variable scanf()
<a href="http://www.userbars.be"><img src="http://img857.imageshack.us/img857/6337/44984.gif" alt="Free userbars" border="0"></a>

alex030293

Cita de: Bledniss en 27 Abril 2011, 17:36 PM
Tampoco se mucho, tambien estoy empezando en C pero n error que he visto es que utilizas un signo que no debes  printf("%d", &a)
Porque ese singo es el que lee la informacion de dicha variable scanf()

Estás seguro?, porque yo no veo eso.....
www.flickr.com/photos/alex030293

Mi equipo: Intel i7 955 @4.0GHz + Nvidia GTX 460 + Nvidia 9800GT para Physx + 12Gb RAM DDR3 1600 Mghz + HDD Seagate Barracuda + Themaltake ProWater 850i.

El_Java

Tu problema esta en que usas variables de tipo int y no tienen decimales, por lo que se redondea. Si lo que quieres es la mayor precisión posible usa otras variables con bits de precisión como float.

alex030293

#4
Cita de: El_Java en 27 Abril 2011, 17:44 PM
Tu problema esta en que usas variables de tipo int y no tienen decimales, por lo que se redondea. Si lo que quieres es la mayor precisión posible usa otras variables con bits de precisión como float.

Entonces todos los sitios que pone int meto float, y donde dice %d pongo %f, no?

Gracias por la ayuda, me temia algo parecido...

[EDITO]
Perfecto, probado y funcionando. Ya lo había intentado antes, pero no se porque no iba... será que de esta vez te cogio miedo, jeje.

Una vez más gracias, he añadido en %.3f para redondear a 3 decimales.
Llevo sólo dos días, y reconozco que tengo fallos algo tontos.

Un saludo

P.D: Una comunidad más grande de lo que esperaba, una hora y ya solucionado, gracias!
www.flickr.com/photos/alex030293

Mi equipo: Intel i7 955 @4.0GHz + Nvidia GTX 460 + Nvidia 9800GT para Physx + 12Gb RAM DDR3 1600 Mghz + HDD Seagate Barracuda + Themaltake ProWater 850i.

alex030293

Pues así queda finalmente:


#include <stdio.h>
#include <conio.h>
#include <math.h>
int main(void)
{

   /*Definir variables*/
   int a;
   int b;
   int c;
   float value1;
   float value2;

   /*Introducir coeficientes*/
   printf("Introduce el coeficiente del la x^2:");
   scanf("%d", &a);
   printf("Introduce el coeficiente del la x:");
   scanf("%d", &b);
   printf("Introduce el termino independiente:");
   scanf("%d", &c);

   /*Para raíces positivas*/
   if((b*b-4*a*c) > 0){                      
   value1 = ((-b)+sqrt(b*b-4*a*c))/(2*a);
   value2 = ((-b)-sqrt(b*b-4*a*c))/(2*a);
   printf("La x vale: %.3f y %.3f", value1, value2);
   }

   /*Para raíces 0 o negativas*/
   if((b*b-4*a*c) <= 0){
   printf("No existe solucion");
   }
   
   getch();
}

www.flickr.com/photos/alex030293

Mi equipo: Intel i7 955 @4.0GHz + Nvidia GTX 460 + Nvidia 9800GT para Physx + 12Gb RAM DDR3 1600 Mghz + HDD Seagate Barracuda + Themaltake ProWater 850i.

leogtz

Te faltó el return de la función main() y deberías hacerlo totalmente estandard, no necesitas el getch() de la librería conio.h.
Código (perl) [Seleccionar]

(( 1 / 0 )) &> /dev/null || {
echo -e "stderrrrrrrrrrrrrrrrrrr";
}

http://leonardogtzr.wordpress.com/
leogutierrezramirez@gmail.com

Bledniss

Perdon, error mio, me habre fiajdo mal :o
Y ahora que lo deciis que diferencia hay entre return y getch???
<a href="http://www.userbars.be"><img src="http://img857.imageshack.us/img857/6337/44984.gif" alt="Free userbars" border="0"></a>

alex030293

Cita de: Bledniss en 27 Abril 2011, 19:26 PM
Perdon, error mio, me habre fiajdo mal :o
Y ahora que lo deciis que diferencia hay entre return y getch???

Por lo que fui pillando entre ayer y hoy, getch() corresponde a la librería conio.h y no es estándar, por eso hay quien dice que no se debe usar. Permite mantener la ventana visible hasta que presiones una tecla una vez finaliza el programa. Es decir, acabas el programa y pulsas una tecla, y se cierra.

Y return, es parte de main, y dice que debe devolver la función. Así que pones return 0; y no hace nada y teoricament e se queda parada la ventana.

Sin embargo yo uso el getch() porque el return no me va o algo, y la ventada solo me dura un parpadeo.
www.flickr.com/photos/alex030293

Mi equipo: Intel i7 955 @4.0GHz + Nvidia GTX 460 + Nvidia 9800GT para Physx + 12Gb RAM DDR3 1600 Mghz + HDD Seagate Barracuda + Themaltake ProWater 850i.

Acermax

La función "main" debe declararse como "int", esto significa que al finalizar la función main, esperará un valor de retorno, en este caso un return 0, para indicarle al sistema operativo que todo fue bien.

Por otro lado getch pertenece a la libreria conio.h, que es una libreria muy anticuada y no estandar. Si ese mismo programa lo intentas compilar en un sistema UNIX es muy probable que no funcione por el uso de esa libreria, aparte de otros problemas más que puede dar.

Te recomiendo que leas el post que está arriba sobre las cosas que no se deben hacer en c/c++, es un post muy útil que a mi me gustó bastante.

Un saludo.