IT rekvalifikace s garancí práce. Seniorní programátoři vydělávají až 160 000 Kč/měsíc a rekvalifikace je prvním krokem. Zjisti, jak na to!
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í.
Avatar
matesax
Tvůrce
Avatar
matesax:27.6.2012 15:12

Dobrý den,
konečně jsem zrealizoval svoji myšlenku - funguje dokonale. Jen bych chtěl poprosit o vytknutí nedostatků - či co byste udělali lépe a tak,...

http://www.text-upload.com/…852202514082

 
Odpovědět
27.6.2012 15:12
Avatar
matesax
Tvůrce
Avatar
Odpovídá na matesax
matesax:27.6.2012 15:39

Opravil jsem sekci hledání funkcí - místo contains jsem dal ty příkazy se 4mi znaky do catch bloku, zbytek pod,...

 
Nahoru Odpovědět
27.6.2012 15:39
Avatar
matesax
Tvůrce
Avatar
Odpovídá na matesax
matesax:27.6.2012 16:10

Oprava:

 string[] dohr = new string[2];

try
{
    try
    {
        dohr[0] = this.Vzorec.Substring(pozz - 4, 4);
        dohr[1] = this.Vzorec.Substring(pozz - 3, 4);
    }
    catch
    {
        dohr[0] = "nic";
        dohr[1] = this.Vzorec.Substring(pozz - 3, 4);
    }
}
catch
{
    dohr[0] = "nic";
    dohr[1] = "nic";
}

if (dohr[0] == "sin" || dohr[1] == "sin")
    this.matf = "sin";
else if (dohr[0] == "sinh" || dohr[1] == "sinh")
    this.matf = "sinh";
else if (dohr[0] == "asin" || dohr[1] == "asin")
    this.matf = "asin";
else if (dohr[0] == "cosh" || dohr[1] == "cosh")
    this.matf = "cosh";
else if (dohr[0] == "acos" || dohr[1] == "acos")
    this.matf = "acos";
else if (dohr[0] == "tanh" || dohr[1] == "tanh")
    this.matf = "tanh";
else if (dohr[0] == "atan" || dohr[1] == "atan")
    this.matf = "atan";
else if (dohr[0] == "cos" || dohr[1] == "cos")
    this.matf = "cos";
else if (dohr[0] == "tan" || dohr[1] == "tan")
    this.matf = "tan";
else
    this.matf = "";
 
Nahoru Odpovědět
27.6.2012 16:10
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:27.6.2012 16:31

Tak nevím, mně se to nezdá. Nebylo by lepší použít na parsování stavový automat? Tohle věčné nahrazování má vysokou režii.

Nahoru Odpovědět
27.6.2012 16:31
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:27.6.2012 16:37

No tak z celého toho kódu se porvede ani ne čtvrtina,... Nevím, co máš na mysli - konečný automat?

 
Nahoru Odpovědět
27.6.2012 16:37
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:27.6.2012 16:50

Až z toho budeš kreslit grafy...

Mám na mysli Mooreův nebo Mealyho automat. Je jedno, který použiješ, jsou mezi sebou převoditelné. Mooreův je na pochopení jednodušší a pamatuje si poslední stav.

Prostě bereš ze vstupu znak po znaku. Až z toho detekuješ symbol, dáš ho na výstup jako dvojici (symbol, atribut) nebo ho rovnou zpracuješ.

Nahoru Odpovědět
27.6.2012 16:50
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:27.6.2012 16:58

Kdybys místo špagety "else if" použil slovník, bylo by to kratší a efektivnější.

Také nechápu, proč se před nahrazováním znaků v řetězci ptáš, jestli tam ten znak je. A také nechápu, proč to nemáš v jedné funkci, ale děláš to pro každý znak zvlášť. Že by v C# nebyl ekvivalent funkce tr()?

Nahoru Odpovědět
27.6.2012 16:58
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:27.6.2012 17:22

Ptám se proto, že pokud to v tom stringu není, tak to hodí error - protože to tam musí být - tak se ptám,... :) Nevím, jak bych to měl jinak udělat - ani switch se na to nehodí,...

 
Nahoru Odpovědět
27.6.2012 17:22
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:27.6.2012 17:37

Error nebo výjimku? Výjimka se dá ošetřit a dá se z ní zotavit.

Podívej se, jak se dělá slovník. Ovšem překládat "asin" jako "asin" mi připadá poněkud hloupé. Když už si dáš tu práci a rozpoznáš symbol, měl bys mu přiřadit místo stringu třeba výčet. I když netuším, jak to dál zpracováváš. Snad ne jako string.

Nahoru Odpovědět
27.6.2012 17:37
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:27.6.2012 17:44

Vždyť se můžeš podívat do přiloženého zdrojáku,... :)

 
Nahoru Odpovědět
27.6.2012 17:44
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:27.6.2012 17:48

To mám louskat tu hrůzu, ve které je každý druhý řádek zbytečný? Navíc jsem v C# nikdy nedělal.

Nahoru Odpovědět
27.6.2012 17:48
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:27.6.2012 18:01

Co jsem našel na internetu byly kilometrové vzorce - a neuměli vše - jeden třeba jen základní operace + pow a toť vše - a o desetinných číslech aby si člověk nechal zdát - mě fungují všechny funkce, i destinná čísla,... Celou dobu se snažím srazit kód na minimum - i cykly maximálně omezit,... Ještě s tím zkusím něco udělat,...

 
Nahoru Odpovědět
27.6.2012 18:01
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:27.6.2012 18:01

Nemohu to celé strčit do try bloku - pak by to neproběhlo vším,...

 
Nahoru Odpovědět
27.6.2012 18:01
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na matesax
David Hartinger:27.6.2012 18:08

A kam ukládáš ta čísla, máš nějaký zásobník?

Nahoru Odpovědět
27.6.2012 18:08
New kid back on the block with a R.I.P
Avatar
matesax
Tvůrce
Avatar
Odpovídá na David Hartinger
matesax:27.6.2012 18:13

Která? Totiž mám to rozdělené na tři části. Pokud výraz obsahuje závorky, vyhledám tu nejvnitřnější - to spočítám do dočasného double a nahradím ve vzorci. Takto pokračuji, dokud výraz neobsahuje žádnou závorku. Pak to již spočítám do výsledného double,...

 
Nahoru Odpovědět
27.6.2012 18:13
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na matesax
David Hartinger:27.6.2012 18:17

Takže výpočet převádíš zpět do stringu?

Nahoru Odpovědět
27.6.2012 18:17
New kid back on the block with a R.I.P
Avatar
matesax
Tvůrce
Avatar
 
Nahoru Odpovědět
27.6.2012 18:20
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 17 zpráv z 17.