Chci geek triko! Chci geek triko!
Extra 10 % bodů navíc a tričko zdarma při zadání kódu "TRIKO10"

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

C# .NET .NET (C# a Visual Basic) Správný zápis podmínky American English version English version

Aktivity (3)
Avatar
Adam Gajdečka:8. května 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. května 23:14
Avatar
Odpovídá na Adam Gajdečka
Ondřej Štorc:8. května 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. května 23:24
Život je příliš krátký na to, abychom bezpečně odebírali USB z počítače..
Avatar
Jan Procházka:9. května 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. května 13:40
Avatar
termostat
Člen
Avatar
termostat:9. května 14:20
if (cam.CommissionFixed == 0d && cam.CommissionPercent != 0d)
{
    item.CommissionLabel = cam.CommissionPercent.ToString() + " % ";
}
 
Nahoru Odpovědět 9. května 14:20
Avatar
Marian Benčat
Redaktor
Avatar
Odpovídá na Adam Gajdečka
Marian Benčat:9. května 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. května 15:15
Totalitní admini..
Avatar
Luboš Satik Běhounek
Autoredaktor
Avatar
Odpovídá na Marian Benčat
Luboš Satik Běhounek:9. května 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í
+1 bodů
Řešení problému
Nahoru Odpovědět  +4 9. května 16:32
https://www.facebook.com/peasantsandcastles/
Avatar
Odpovídá na Luboš Satik Běhounek
Adam Gajdečka:9. května 18:39

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

 
Nahoru Odpovědět 9. května 18:39
Avatar
Marian Benčat
Redaktor
Avatar
Nahoru Odpovědět  +2 9. května 18:42
Totalitní admini..
Avatar
Odpovídá na Luboš Satik Běhounek
Adam Gajdečka:10. května 18:25

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

 
Nahoru Odpovědět 10. května 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.