Diskuze: C# aplikace na výpočet provizí
Obávám se, že přesně takovýto kód je neobjektový kód. Nebudu ti to přepisovat, ale dám ti jednoduchý příklad na kterém to třeba pochopíš.
1)
class pocitani : textove_prvky // textove_prvky dědí MainWindows - takže jsou v ní ty protected Listy v MainWindow
{Už todle je hodně špatně.. Ty máš Model, tedy věc čisté logiky.. ten tvuj class Pocitani nic jineho ani neni.. a ty abys dostal hodnoty textboxu, tak si podedis od Main_WINDOW????
Pokud vim, tak ty potrebujes dostat jen ty jejich hodnoty,.. spravny pristup (je jich spoustu) je poslat si do vypocitej ty hodnoty tech textboxu.
- Mas tam asi tak 100 podminek, ktere ti kontroluji urcite meze a na zaklade toho nejak modifikuji odecet, procenta atp,
Todle je naprosto ideální čas na to udělat si nějaký třeba List "mezí a modifikací". Dám ti jednoduchý příklad v pseudokodu.
class Procenta {
int horniMez;
intDolniMez;
int procenta;
getProcenta() {
return procenta;
}
bool vyhovuje(int value) {
return value>=dolniMez && <hornimez;
}
}
List<Procenta> procentaList = new...;
procentaList.Add(new Procenta(10000, 20000, 0.04);
procentaList.Add(new Procenta(20000, 40000, 0.08);a použití místo těch 10 000 podmínek:
procenta = procentaList.First(x=>x.vyhovuje(bodovy_soucet)).getProcenta();Samozřejmě by si mohl ten LINQ nahradit vlastní třídou atp. Toto řešení má spoustu i dalších výhod.. například... pokud se ti změní ty hodnoty a meze pro procenta.. změníš jen to přidávání do listu.. nebo to můžeš číst z configu atp.
Toto je objektový přístup.Cela ta metoda vypocitej, by se dala pomoci ruznych navrhovych vzoru (State, Behavior) a nastroju OOP (polymorfismus) smrsknout tak na 1/10 a hlavne by to bylo zcela pochopitelne co to dela.. Takhle je to jen dlouhy nic nerikajici kod.
Píši ti to protože tě očividně zajímá OOP... Správné OOP se ale nikde úplně neučí a tak ti povím, co ti může dobře napovědět, že nějaký kód nen napsaný jako spávné OOP, ale jen imperativní kod s classama...
- Obsahuje tam prave podobný decision tree pomocí 20ti ifu, co tam ty máš - tomudle se říká branching a v OOP se řeší často pomocí nějakého vnitřního stateu (tam se používá návrhový vzor state, předek- potomek a polymorfismus.)
- Objekty mají public settery.
+20 Zkušeností
+2,50 Kč
Ja jsem z toho kodu silne ponekud nesvuj , mozna kdyby si to popsal matematickou funkcí, poznas ze to jde pomoci matematickyh pravidel pretransformovat na neco rozumnejsiho. Takhle to vypada jako nejaka matematicka posloupnost, z toho co jsem pochopil.
Ten kod co si daval nahore moc nedava smysl sam o sobe..
- Kouknes se jestli nejaky "SEZNAMTRICETDVOJEK" ma jeden prvek a pokud ano, tak to projizdis foreachem? :-/
- Neomezuj se v C# zadnyma programovacíma paradigmatama. C# zvládá velmi dobře libovolnou kombinace OOP, imperativního a funkcionalniho programovani.
Opět příklad:
if (seznamtricetdvojek.Count() == 1)
{
int pomocna_promenna = 0;
foreach (int s in seznamtricetdvojek)
{
pomocna_promenna = pomocna_promenna + s;
}
if (bodovy_soucet - pomocna_promenna >= 300000)
{
procenta = 0.35;
}
else
procenta = 0.32;
}Lze převézt na třeba:
procenta = (bodovy_soucet - seznamtricetdvoje.Sum()) >= 30000 ? 0.35 : 0.32;a stále je to velmi čitelné = stačí pohled na to a hned je každému jasné, co to dělá.. Je mu totiž hned jasné, že v tom není žádná složitá logika.
kdežto pokud tam dáě těch 25řádek kódu, tak bude každý luštit, co to vlastně dělá.
Přesto, že jsem mluvil o tom, že tvůj kód není OOP, tak nesdílím názor Čápky (autora těch článků), že OOP je samospásné a neOOP kód je spaghetti. Využij jedné z největších výhod C# a sice to, že jde o tzv. multi-paradigmatický jazyk.
Zatímco pro řízení programu můžeš dobře použít OOP, na matematické výpočty (jako jsou tvoje procenta) se hodí třeba zase jeho funkcionální číst, případně kombinace funkcionální+OOP.
NO.. a pak je tam ještě další level. A sice reactive extensions.. ale tim
te nebudu zatezovat 
Zobrazeno 7 zpráv z 7.