NOVINKA - Online rekvalifikační kurz Python programátor. Oblíbená a studenty ověřená rekvalifikace - nyní i online.
Hledáme nové posily do ITnetwork týmu. Podívej se na volné pozice a přidej se do nejagilnější firmy na trhu - Více informací.

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.

Jak se ti líbí článek?
Před uložením hodnocení, popiš prosím autorovi, co je špatněZnaků 0 z 50-500
Jak se ti kurz líbí?
Tvé hodnocení kurzuZnaků 0 z 50-500
Aktivity
Avatar
Adam Gajdečka
Člen
Avatar
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
Avatar
Ondřej Štorc
Tvůrce
Avatar
Odpovídá na Adam Gajdečka
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
+2
8.5.2018 23:24
Život je příliš krátký na to, abychom bezpečně odebírali USB z počítače..
Avatar
Jan Procházka
Člen
Avatar
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
Avatar
termostat
Člen
Avatar
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
Avatar
Marian Benčat
Člen
Avatar
Odpovídá na Adam Gajdečka
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
-2
9.5.2018 15:15
Totalitní admini..
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Marian Benčat
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č
Řešení problému
Nahoru Odpovědět
+4
9.5.2018 16:32
https://www.facebook.com/peasantsandcastles/
Avatar
Adam Gajdečka
Člen
Avatar
Odpovídá na Luboš Běhounek Satik
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
Avatar
Marian Benčat
Člen
Avatar
Odpovídá na Luboš Běhounek Satik
Marian Benčat:9.5.2018 18:42

Mas pravdu.:-)

Nahoru Odpovědět
+2
9.5.2018 18:42
Totalitní admini..
Avatar
Adam Gajdečka
Člen
Avatar
Odpovídá na Luboš Běhounek Satik
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
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.