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

[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:

Código [Seleccionar] Expandir

#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.

[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()

[alex030293]

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.....

[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]

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!

[alex030293]

Pues así queda finalmente:

Código [Seleccionar] Expandir

#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();
}

[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.

[Bledniss]

Perdon, error mio, me habre fiajdo mal
Y ahora que lo deciis que diferencia hay entre return y getch???

[alex030293]

Perdon, error mio, me habre fiajdo mal
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.

[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.