Nesprávný výsledek kvadratické rovnice – C / C++ – Fórum – Programujte.com

 Dobrý den,mohli by jste mi prosím poradit kde je chyba v níže uvedeném kódu? Pracuje správně,ale výpočet je nesprávný děkuji.

#include <stdio.h>
#include <MATH.H>
#include <conio.h> 



int main(void)
{
float a, b, c, x1, x2, d, i, r, n;


	printf("Zadej 3 cleny kvadraticke rovnice a,b,c\n");
	while(scanf("%lf %lf %lf",&a, &b, &c)==0)
		{
	printf("Toto neni cislo...");
	fflush(stdin);
		}
	d=(b*b-(4*a*c));
	int A = (1*(d>0)) + (2*(d==0)) + (3*(d<0));
	switch(A)
	{
	case 1 : 
	x1=(-b+sqrt(d))/2*a;
	x2=(-b-sqrt(d))/2*a;
	printf("Koren x1 je: %f. Koren x2 je: %f",x1,x2);
	break;
	case 2 :
	x1=(-b)/(2*a);
	x2=x1;
	printf("Koren x1 je: %f. Koren x2 je: %f",x1,x2);
	break;
	case 3 :
	d=fabs (d);
	x1=(-b+sqrt(d))/2*a;
	x2=(-b-sqrt(d))/2*a;
	r=sqrt(d);
	n=-(sqrt(d));
	printf("Koren x1 je: %lf+%lf*i. Koren x2 je: %lf+%lf*i",x1,x2,r,n);
	break;
	}
	return(0);
	
	
}

x1=(-b+sqrt(d))/2*a je zle na konci ma byt .../(2*a)

a to v kazdom pripade mozno tam je viac chyb ale toto mi udrelo do oci :)

#2 jano
ty jo díky moc kamaráde :) taková hrubka

float a, b, c, x1, x2, d, i, r, n;
další chyba má tam být double a ne float :)

#4 Jakub
Vyloženě špatně to není. Float je také číslo s plovoucí desetinnou čárkou. Jen má menší přesnost.

Aha, taky dobré vědět nevím, kde může být daná chyba, mám pocit ze je možná i někde problém ve switchi,protože jestli se mi to nezdá, tak ten program vynechává některou z case.

co ma proboha delat tenhle radek?

int A = (1*(d>0)) + (2*(d==0)) + (3*(d<0));

to sem teda nevidel 

#7 bob
Na tohle jsem taky koukal 

On využívá toho, že podmínka nebo resp. true/false je 1 a 0, čili kdyz d>0 a vrátí true, tak A = 1, protože ostatní budou 0 díky podmínce

Ale osobně jsem takovýhle řešení ještě neviděl, ale dal bych +1 za kreativitu =D

A co kdyz vyjde treba d = 0,3 ?

To pak jako bude 1 * 0,3 = 0.3 => Prevedeno se ztratou desetinne casti na int A = 0 ???

Zadny switch pro 0 tam nevidim.

Pro D > 0 bude A = 1, pro D == 0 bude A = 2 a pro D < 0 bude A = 3.

#9 bob
ale on nenásobí nic s 0.3 :)

Pokud má:

int A = (1*(d>0)) + (2*(d==0)) + (3*(d<0));

a pak.. d = 0.3; tak...

0.3>0 = true = 1 =1*1 = 1 => A = 1 :) ostatní co se provede je bezcenný, protože podmínky selžou a 2*0 nebo 3*0 = 0 a 1+0+0 = nečekaně 1 :)

Uz je mi to jasny, musel sem to dat do VS, protoze tenhle zpusob podminky/nepodminky jsem teda nikdy nevidel. 

#12 bob
jj, ale správně by to ale měl napsat jako: 

int A = d > 0 ? 1 : !d ? 2 : d < 0 ? 3 : 0;

Jen tak mimochodem ... výpočet hodnoty A je samoúčelné, zbytečné, zhoršuje čitelnost kódu a taky prodlužuje jeho vykonání, takže oba zápisy jsou stejně dobře. Proč nejít tou nejjednodušší cestou (a vyhnout se switch):

if (d > 0) {
	x1=(-b+sqrt(d))/2*a;
	x2=(-b-sqrt(d))/2*a;
	printf("Koren x1 je: %f. Koren x2 je: %f",x1,x2);
}
else if (d == 0) {
	x1=(-b)/(2*a);
	x2=x1;
	printf("Koren x1 je: %f. Koren x2 je: %f",x1,x2);
}
else {
	d=fabs (d);
	x1=(-b+sqrt(d))/2*a;
	x2=(-b-sqrt(d))/2*a;
	r=sqrt(d);
	n=-(sqrt(d));
	printf("Koren x1 je: %lf+%lf*i. Koren x2 je: %lf+%lf*i",x1,x2,r,n);
}

Kdyby na tu proměnnou A měl navázané třeba texty v poli apod., tak bych to bral, ale jinak ....

Ta silenost fflush(stdin) by tam taky nemela byt. Ze to funguje je tim, ze MS nedodrzuje standardy u C kompilatoru (viz letity problem s nemoznosti deklarace promennych kdekoliv uvnitr kodu).

Jinak GCC by vyhodilo chybu. Je to bezny jev, zacatecnici si tim ulehcuji praci.

http://faq.cprogramming.com/…smartfaq.cgi?…