Diskuze: Správný zápis podmínky

V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity

Adam Gajdečka

Člen

Adam Gajdečka:8.5.2018 23:14

Ahoj, prosím o radu. Chci zlepšit můj kód. Mám zeleně podrženou tuto podmínku

if (cam.CommissionFixed == 0 && cam.CommissionPercent != 0)
{
    item.CommissionLabel = cam.CommissionPercent + " % ";
}

Obě proměnné jsou double.

A Warning hlásí:
Severity Code Description Project File Line Suppression State
Warning RECS0018 Fix floating point number comparing. Compare a difference with epsilon.

Díky. Rád bych měl napsánou podmínku korektně.

Odpovědět

8.5.2018 23:14

Ondřej Štorc

Tvůrce

Ondřej Štorc:8.5.2018 23:24

Ahoj, něco o tom se dozvíš třeba zde: http://floating-point-gui.de/…/comparison/ Upřímně se mi to zda trochu zbytečně řešit, jestliže neděláš nějaké matematické aplikace...

Nahoru Odpovědět

8.5.2018 23:24

Život je příliš krátký na to, abychom bezpečně odebírali USB z počítače..

Jan Procházka

Člen

Jan Procházka:9.5.2018 13:40

Z té podmínky nejsem s to vyčíst co a jak má fungovat.

Obecně není moc dobré používat podmínky a==b a a!=b u proměnných typu float a double. Vrací totiž jednu pravdivostní hodnotu pro právě jednu kombinaci a a b. Co se člověku může zdát jako shoda, pro procesor shoda být nemusí. Pro nás je většinou 1e-36 nula; pro počítač ne. V případě, že pracuješ nad celými čísly, takový problém nehrozí, ale i tak je lepší se tomu vyvarovat, když to jde.

if(math.abs(cam.CommissionFixed - 0)) < DostatecneMalyCislo && math.abs(cam.ComissionPercent - 0) > TrebaJinyDostatecneMalyCislo)

Tohle by mělo fungovat pro DostatecneMalyCislo dostatečně malý tak, jak chceš, aby to doopravdy fungovalo. První výrok podmínky nebude totiž pravdivý jen pro právě jedno jediný číslo, ale pro všechna čísla rozumně blízko pro to, aby je mohl člověk prohlásit za rovná.

Nejjednodušší příklad co mě napadá je:

float pseudoCounter=0;
while(pseudoCounter!=1){pseudoCounter+=0.1;} // ukončí se, nebo neukončí?
pseudoCounter=0;
while(pseudoCounter<1){pseudoCounter+=0.1;} // ukončí se, i pro pseudoCounter s hodnotou 1.00000000002

Nahoru Odpovědět

9.5.2018 13:40

termostat

Člen

termostat:9.5.2018 14:20

if (cam.CommissionFixed == 0d && cam.CommissionPercent != 0d)
{
    item.CommissionLabel = cam.CommissionPercent.ToString() + " % ";
}

Nahoru Odpovědět

9.5.2018 14:20

Marian Benčat

Člen

Marian Benčat:9.5.2018 15:15

To co ti hlásí jako warning, asi opravdu řešit nemusíš,.. porovnávání pomocí epsilon se dělává opravdu u SW, kde prostě i malá nepřesnost může způsobit problémy (typicky třeba raytracing, kde to může znamenat, že něco je vidět, nebo není)

Nahoru Odpovědět

9.5.2018 15:15

Totalitní admini..

Luboš Běhounek Satik

Tvůrce

Luboš Běhounek Satik:9.5.2018 16:32

To právě ne, epsilon potřebuješ vždycky při porovnávání obsahu float/double proměnné s nějakou konkrétní hodnotou, protože desetinný čísla mají jen omezenou přesnost a dochází k nepřesnostech při jejich ukládání i při výpočtech.

Ta podmínka nahoře mu nikdy nemusí projít, protože to číslo místo přesný nuly může bejt třeba 0.0000000298...

Adam Gajdečka Hlavní chyba v tvém kódu je, že na peněžní hodnoty nepoužíváš Decimal, který je k tomu vhodnější, protože čísla vnitřně ukládá v desítkové soustavě a má větší rozsah - např. pokud tam máš floaty, tak už při čístce cca 100 000 se ti budou ztrácet halíře, při 10M+ i koruny.

Akceptované řešení
+20 Zkušeností
+2,50 Kč

Nahoru Odpovědět

9.5.2018 16:32

https://www.facebook.com/peasantsandcastles/

Adam Gajdečka

Člen

Adam Gajdečka:9.5.2018 18:39

díky. Neviděl jsem dosud rozdíl mezi double a decimal.

Nahoru Odpovědět

9.5.2018 18:39

Marian Benčat

Člen

Marian Benčat:9.5.2018 18:42

Mas pravdu.:-)

Nahoru Odpovědět

9.5.2018 18:42

Totalitní admini..

Adam Gajdečka

Člen

Adam Gajdečka:10.5.2018 18:25

ušetřili jste mi dost problémů do budoucna. Díky

Nahoru Odpovědět

10.5.2018 18:25

Naučíme tě pracovat na home-office.

Zjistit více...

Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 9 zpráv z 9.