NOVINKA - Online rekvalifikační kurz Java programátor. Oblíbená a studenty ověřená rekvalifikace - nyní i online.
NOVINKA – Víkendový online kurz Software tester, který tě posune dál. Zjisti, jak na to!

Diskuze: Programovací konvence a doporučení C#

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

Aktivity
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Luboš Běhounek Satik:23.9.2013 22:13

Až se budete nudit, tak doporučuji projít http://msdn.microsoft.com/…f926074.aspx .

Zjistíte spoustu zajímavých věcí ke zpřehlednění vašeho kódu a často i k ušetření si nějaké práce při jeho psaní :)

Odpovědět
23.9.2013 22:13
https://www.facebook.com/peasantsandcastles/
Avatar
Jan Vargovský
Tvůrce
Avatar
Jan Vargovský:23.9.2013 22:30

Používám vše, takže easy :P

 
Nahoru Odpovědět
23.9.2013 22:30
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Luboš Běhounek Satik:23.9.2013 22:31

Tak to je jen dobře :)

Nahoru Odpovědět
23.9.2013 22:31
https://www.facebook.com/peasantsandcastles/
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Luboš Běhounek Satik
Kit:23.9.2013 22:33

Některá z těch pravidel se dají použít i v jiných jazycích, používají poměrně často var, některá běžná doporučení však chybí.

Na můj vkus mají moc roztahané try ... catch ... finally, které tím ztrácí přehlednost. Používám kompaktní formu i pro podmínky a cykly.

Nahoru Odpovědět
23.9.2013 22:33
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Kit
Jan Vargovský:23.9.2013 22:35

Co je na tom roztahaného? u try-catch / try-finally bloku tam ty blokové závorky musí být. Jedině, jak bys to mohl stáhnout je, že bys u té podmínky kde ničíš ten objekt, je nedáš ... Ale to už je na člověku, jestli je tam dává, i když je jeden příkaz.

Editováno 23.9.2013 22:36
 
Nahoru Odpovědět
23.9.2013 22:35
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Kit
Luboš Běhounek Satik:23.9.2013 22:39

Var pouzivaji v pripadech, kdy tim netrpi prehlednost (dokonce prave casto tu prehlednost zvysi, protoze se kod zkrati).

Me roztahane try-catch-finaly vyhovuje, jsou to preci jen vyjimky, takze by se mely stavat vyjimecne a jsem rad, kdyz je v kodu na prvni pohled vidim.

Mohlo by tam toho samozrejme byt vic, ale pro zacatek to alespon trochu pomoci muze.

Nahoru Odpovědět
23.9.2013 22:39
https://www.facebook.com/peasantsandcastles/
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Luboš Běhounek Satik:23.9.2013 22:44

Nemyslel tim Kit neco jako

try {
  riskyFunction();
} catch (Exception ex) {
  log(ex);
}

? :)

Nahoru Odpovědět
23.9.2013 22:44
https://www.facebook.com/peasantsandcastles/
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Luboš Běhounek Satik
Jan Vargovský:23.9.2013 22:49

Je to možné. Ale u žádného C# kódu jsem neviděl, že by dával ty blokové závorky už na konec řádku, je to spíše jeho zvyk z Javy. (Mačkám stejně úpravu kódu celkem často a to mi pokaždé upraví do té klasické varianty) Mimochodem, přijde mi to méně přehlednější, než ta klasická "roztahaná" varianta :)

 
Nahoru Odpovědět
23.9.2013 22:49
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Kit:23.9.2013 22:50

Pro mne je obvyklý zápis try ... catch v kontextu odvozeném z toho webu např:

static string GetValueFromArray(string[] array, int index) {
    Console.WriteLine("Začínám...");
    try {
        return array[index];
    } catch (System.IndexOutOfRangeException ex) {
        Console.WriteLine("Index is out of range: {0}", index);
        throw;
    } finally {
        if (index == 0) {
            Console.WriteLine("Warning: Index byl nulový");
        }
    }
}

Pro mne je ten zápis přehledný, proto ho používám. Na první pohled vidím, co k čemu patří, pro jistotu si to vždy nechám přerovnat editorem. Tím velice snadno najdu chybějící nebo přebývající závorky - odsazení se udělá jinak, než bych očekával.

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

Jo, vždyť říkám, takhle se píšou běžne závorky v Jave, tak jsi nato zvyklý. My .NETáři, jsme zvyklí na poněkud jiné věci. A editovat si VS kvůli try-catch bloku, který stejně používám jen na pár důležitých místech ... V C# to beru jako konvenci, že mají být složené závorky na samostatném řádku (když neberu příklad nastavování properties přímo u volání konstruktoru atd...) je to pro mě čitelnější a řekl bych že i pro ostatní :)

 
Nahoru Odpovědět
23.9.2013 22:58
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Kit:23.9.2013 22:59

Je to jedna z konvencí otevírat závorky už na konci řádku s podmínkou.

Možná je to u mne i tím, že fonty v editoru mám nastavené na 14px a neviděl bych začátek či konec třídy, kterou právě edituji. Tím, že mám větší písmo, mi nevadí, že je text vertikálně zhuštěný.

Nahoru Odpovědět
23.9.2013 22:59
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 Jan Vargovský
Kit:23.9.2013 23:01

try ... catch de facto stačí jen jedno v celé aplikaci, ale většinou chceš, aby se aplikace dokázala z drobných chyb rozumně zotavit.

Nahoru Odpovědět
23.9.2013 23:01
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Kit
Jan Vargovský:23.9.2013 23:05

Hlavně je to dost výkonově náročné, takže to používám velmi zřídka a jen tam, kde moc chybu nejde jinak ošetřit.

 
Nahoru Odpovědět
23.9.2013 23:05
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Kit:23.9.2013 23:07

To je jen mýtus. Když výjimky použiješ správně, tak naopak šetří výkonem. Hlavně programátora.

Jak to může být výkonově náročné, když k výjimce dojde jen výjimečně?

Editováno 23.9.2013 23:09
Nahoru Odpovědět
23.9.2013 23:07
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Kit
Jan Vargovský:23.9.2013 23:10

To není mýtus. Ušetří ti to práci, to ano, ale aplikaci ti to opravdu zpomalí.

Já spíše narážím na úplné začátečníky, co si myslí, že narvat to do každé metody je nejlepší řešení a pak se diví, že je to jaksik pomalé.

 
Nahoru Odpovědět
23.9.2013 23:10
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Kit:23.9.2013 23:13

Pokud někdo dělá i flow control pomocí výjimek, tak je to fakt špatně. Výjimka, ke které nedojde, však program vůbec nezpomalí. Naopak se zbavíš zbytečných podmínek, program se vyčistí a zrychlí.

Podmínkou je, že k těm výjimkám nedochází, alespoň ne moc často.

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

Jednoduše, ten kód se musí chovat jinak, protože musí zabránit té aplikaci, aby nespadla, když k té chybě dojde. Kdyby to bylo jak ty říkáš, bylo by to implementované explicitně už všude né ? :)

 
Nahoru Odpovědět
23.9.2013 23:16
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Kit
Luboš Běhounek Satik:23.9.2013 23:22

V C# když nějaký krátký kód hodíš do try-catch, a k chybě nedojde, tak je režie v řádech jednotek až několika málo desítek procent, pokud k výjimce dochází častěji, tak je režie v řádech desetisíců procent i víc.

Nahoru Odpovědět
23.9.2013 23:22
https://www.facebook.com/peasantsandcastles/
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Kit
Jan Vargovský:23.9.2013 23:25

Schválně jsem ti nato udělal úplně triviální test. http://www.itnetwork.cz/dev-lighter/211

Na linuxu to asi nespustíš, ale abys viděl výsledky tak tu máš i obrázek:

http://2i.cz/…6b4192.f.png

Je to na ticky procesoru. Abys viděl rozdíly v ms. Ofc záleží na tom, jestli je kritická pouze jedna operace nebo všechny ty části, ale obecně se vyhýbám try-catch bloku co nejvíce, když jsem začínal tak sem si to vychvaloval a teď když vidím, jak prasácky někdo ošetřuje vyjímky a myslí si, že je to správně, tak je mi akorát špatně :)

 
Nahoru Odpovědět
23.9.2013 23:25
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Luboš Běhounek Satik:23.9.2013 23:29

obrazek nejede :)

Nahoru Odpovědět
23.9.2013 23:29
https://www.facebook.com/peasantsandcastles/
Avatar
Jan Vargovský
Tvůrce
Avatar
Odpovídá na Luboš Běhounek Satik
Jan Vargovský:23.9.2013 23:30

Zvláštní, mi jde :D
http://www.itnetwork.cz/…trycatch.png

 
Nahoru Odpovědět
23.9.2013 23:30
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Jan Vargovský
Kit:23.9.2013 23:45

Spustil jsem to i na Linuxu, ale trochu jsem to upravil. Počet cyklů jsem zestonásobil, abych měl přesnější výsledek a u B jsem přidal test validity (aby se srovnávalo srovnatelné). Místo převodu string->int jsem použil dělení, při kterém teoreticky může dojít k dělení nulou, ale dělitel jsem zvolil tak, aby nebyl nulový. Výsledky byly prakticky totožné.

http://www.itnetwork.cz/dev-lighter/212

Editováno 23.9.2013 23:45
Nahoru Odpovědět
23.9.2013 23:45
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Kit
Luboš Běhounek Satik:24.9.2013 1:17

Tvůj kód:
http://screenshot.cz/MVVYT/exc.png

Dokud k chybě nedochází vůbec nebo zcela výjimečně, tak má try-catch vliv malý (jednotky až desítky procent, podle velikosti a typu kódu), ale pokud by k němu mělo docházet častěji, třeba i jen u jednoho průchodu z 1000, tak už je to znát hodně a kód s výjimkami je několikrát pomalejší.

edit: ještě kód testování s výjimkami:
http://www.itnetwork.cz/dev-lighter/213

Editováno 24.9.2013 1:19
Nahoru Odpovědět
24.9.2013 1:17
https://www.facebook.com/peasantsandcastles/
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Luboš Běhounek Satik
Kit:24.9.2013 6:53

Jenže k výjimkám dochází méně často, než jeden průchod z 1000, takže měřit dobu jejich provádění je pro praktické účely zcela bezvýznamné.

Nahoru Odpovědět
24.9.2013 6:53
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Luboš Běhounek Satik
Tvůrce
Avatar
Odpovídá na Kit
Luboš Běhounek Satik:24.9.2013 9:08

To záleží na použití, jsou i lidé, co je dají do opakovaně volaného kódu i do místa, kde k nim dochází častěji :)

Nahoru Odpovědět
24.9.2013 9:08
https://www.facebook.com/peasantsandcastles/
Avatar
Kit
Tvůrce
Avatar
Odpovídá na Luboš Běhounek Satik
Kit:24.9.2013 9:54

To je pak použití výjimek k řízení toku programu (např. k vyskakování z cyklu), což je nežádoucí. Je nutné se toho vyvarovat, aby k vyvolávání výjimek docházelo skutečně výjimečně.

Tuhle jsem někde na fóru viděl dotaz, že nějaký nešťastník nemohl najít chybu ve svém programu, nefungoval jak měl. Měl tam 3 bloky try..catch a ve všech třech měl větev "catch" prázdnou. Takovým obvykle není pomoci...

Nahoru Odpovědět
24.9.2013 9:54
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 Jan Vargovský
Kit:24.9.2013 9:59

Obvykle se obálka try..catch nachází mimo cyklus a tím, že se ušetří jeden validační test uvnitř cyklu, se ušetří i nějaký čas. Vždycky to záleží na tom, jakým způsobem ty výjimky použiješ. Nelze paušálně tvrdit, že jsou rychlejší nebo pomalejší.

Nahoru Odpovědět
24.9.2013 9:59
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
vitamin
Člen
Avatar
vitamin:24.9.2013 10:28

V desnej dobe byvaju vynimky dizajnovane tak ze spracovanie chyb je oddelene od hlavneho toku programu (tzv. Zero cost exception). Takze ak nenastane vynimka tak program moze byt teoreticky aj rychlejsi ako testovanie cez if else switch... Ak nastane vynimka, tak sa presunie tok programu a zacne sa postupne spracovavat vynimka. Samotne spracovanie vynimky je ovela pomalsie, preto sa vynimky maju pouzivat na "vynimocne" udalosti ako napr fatalne chyby. Kedysi boli vynimky implementovane cez setjmp/longjmp a preto maju relativne zlu povest. Dnes sa vecsinou pouzivaju vynimky podla DWARF standartu a vo windowse SEH (ms musi mat vzdy nieco co je nekompatibilne so ostatnymi...)

 
Nahoru Odpovědět
24.9.2013 10:28
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 28 zpráv z 28.