Construction Kolektiv autorů

Transkript

1 Construction Kolektiv autorů

2 Připomenutí Analýza Návrh Implementace Co Jak

3 Opakování

4 Příklad Vyhodnocení výrazu o 3 + ((5 + 9) * 2)

5 Příklad Vyhodnocení výrazu boolean isoperator = (operator!= null) && (leftchild!= null) && (rightchild!= null); int leftvalue = leftchild.evaluate(); int rightvalue = rightchild.evaluate(); switch (operation) { case PLUS: return leftvalue + rightvalue; case MINUS: return leftvalue - rightvalue; case TIMES: return leftvalue * rightvalue; case DIVIDE: return leftvalue / rightvalue; default: throw new IllegalStateException( "Unknown operator: " + operator); }

6 Příklad Vyhodnocení výrazu

7 Příklad Vyhodnocení výrazu

8 Příklad Vyhodnocení výrazu public class ValueNode implements Node { public int evaluate() { return value; } } public class OperationNode implements Node { public int evaluate() { return operation.evaluate( leftchild.evaluate(), rightchild.evaluate()); } } public enum DefaultOperations implements Operation { PLUS { public int evaluate(int a, int b) {return a + b;} }, MINUS { public int evaluate(int a, int b) {return a - b;} }, TIMES { public int evaluate(int a, int b) {return a * b;} }, DIVIDE { public int evaluate(int a, int b) {return a / b;} } }

9 Příklad - Cache public class CacheMap<K, V> extends LinkedHashMap<K, V> { private ReentrantReadWriteLock lock = new ReentrantReadWriteLock(); public void lockread() { lock.readlock().lock(); } public void unlockread() { lock.readlock().unlock(); } } public void lockwrite() { lock.writelock().lock(); } public void unlockwrite() { lock.writelock().unlock(); } try { cache.lockwrite(); cache.put("a", "B"); } finally { cache.unlockwrite(); } cache.putall(someothercache); cache.remove("a"); try { cache.lockread(); cache.put("a", "B"); } finally { cache.unlockread(); }

10 Příklad - Cache public class CacheMap<K, V> { private final CustomLinkedHashMap<K, V> map; private final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(); public V get(final K key) { return withreadlock(() -> map.get(key)); } public void put(final K key, final V value) { withwritelock(() -> map.put(key, value)); } public void remove(final K key) { withwritelock( ); } } public int size() { return withreadlock(() -> map.size()); }

11 Pár úvah o kvalitním kódu

12 Na cestě ke kvalitnímu kódu o Každý to chce psát jednoduše

13 Na cestě ke kvalitnímu kódu o o Každý to chce psát jednoduše Každý to chce psát čistě

14 Na cestě ke kvalitnímu kódu o o o Každý to chce psát jednoduše Každý to chce psát čistě Každý to chce psát jinak

15 Na cestě ke znovupoužitelnosti o Developers love to reuse code!

16 Příklad Join řetězců o Jak implementovat spojení kolekce řetězců o [ A, B, C ] -> A, B, C

17 Příklad Join řetězců o Možná implementace public static String join(list<string> strings) { StringBuilder result = new StringBuilder(); for (String p : strings) { result.append(p).append(", "); } } return result.substring( 0, result.length() - ", ".length());

18 Příklad Join řetězců o Hotová řešení v Java světě: Apache commons-lang StringUtils.join Google Guava Joiner.on Java String.join

19 Co je to kvalitní kód?

20 Kvalitní kód je Bezchybný Rychlý / Efektivní Krátký? Rafinovaný? Srozumitelný Udržovatelný S unit testy

21

22 Proč psát srozumitelný kód Kód píšete jednou, číst ho (vy nebo někdo jiný) budete pravděpodobně vícekrát Pokud váš kód někdo nepochopí, snadno v jeho použití nebo úpravě udělá chybu Nesrozumitelný kód nikdo nebude chtít používat Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. Code for readability.

23 Kdy nepsat kvalitní kód?

24 Jak psát srozumitelný kód

25 Co bude uživatel vaší metody číst 1. Jméno metody 2. Deklarované typy parametrů a návratové hodnoty 3. Jména deklarovaných parametrů 4. Javadoc 5. Samotný kód metody Čím dříve se dozví vše, co potřeboval, tím dříve může přestat číst (a ušetřit čas)

26 Příklad: StringUtils (Commons Lang) /** join()

27 Příklad: StringUtils (Commons Lang) /** public static String join(object[], String, int, int)

28 Příklad: StringUtils (Commons Lang) /** public static String join(object[] array, String separator, int startindex, int endindex)

29 Příklad: StringUtils (Commons Lang) /** Joins the elements of the provided array into a single String containing the provided list of elements. No delimiter is added before or after the list. A null separator is the same as an empty String (""). Null objects or empty strings within the array are represented by empty strings. StringUtils.join(null, *) = null StringUtils.join([], *) = "" StringUtils.join([null], *) = "" StringUtils.join(["a", "b", "c"], "--") = "a--b--c" StringUtils.join(["a", "b", "c"], null) = "abc" StringUtils.join(["a", "b", "c"], "") = "abc" StringUtils.join([null, "", "a"], ',') = ",,a" Parameters: array the array of values to join together, may be null separator the separator character to use, null treated as "" startindex the first index to start joining from. It is an error to pass in a start index past the end of the array endindex the index to stop joining from (exclusive). It is an error to pass in an end index past the end of the array Returns: the joined String, null if null array input */ public static String join(object[] array, String separator, int startindex, int endindex)

30 Příklad: StringUtils (Commons Lang) /** Joins the elements of the provided array into a single String containing the provided list of elements. No delimiter is added before or after the list. A null separator is the same as an empty String (""). Null objects or empty strings within the array are represented by empty strings. StringUtils.join(null, *) = null StringUtils.join([], *) = "" StringUtils.join([null], *) = "" StringUtils.join(["a", "b", "c"], "--") = "a--b--c" StringUtils.join(["a", "b", "c"], null) = "abc" StringUtils.join(["a", "b", "c"], "") = "abc" StringUtils.join([null, "", "a"], ',') = ",,a" Parameters: array the array of values to join together, may be null separator the separator character to use, null treated as "" startindex the first index to start joining from. It is an error to pass in a start index past the end of the array endindex the index to stop joining from (exclusive). It is an error to pass in an end index past the end of the array Returns: the joined String, null if null array input */ public static String join(object[] array, String separator, int startindex, int endindex)

31 Příklad: StringUtils (Commons Lang) public static String join(object[] array, char separator, int startindex, int endindex) { if (array == null) { return null; } int bufsize = (endindex - startindex); if (bufsize <= 0) { return EMPTY; } bufsize *= ((array[startindex] == null? 16 : array[startindex].tostring().length()) + 1); StrBuilder buf = new StrBuilder(bufSize); } for (int i = startindex; i < endindex; i++) { if (i > startindex) { buf.append(separator); } if (array[i]!= null) { buf.append(array[i]); } } return buf.tostring();

32 Příklad: StringUtils (Commons Lang) private static final String[] ARRAY_LIST = {"foo", "bar", "baz"}; private static final String[] NULL_ARRAY_LIST = {null}; private static final String[] EMPTY_ARRAY_LIST = {}; private static final String[] MIXED_ARRAY_LIST = {null, "", "foo"}; private static final Object[] MIXED_TYPE_LIST = {"foo", Long.valueOf(2L)}; private static final String SEPARATOR = ","; private static final String TEXT_LIST_NOSEP = "foobarbaz"; private static final String TEXT_LIST = public void testjoin_arraystring() { assertnull(stringutils.join((object[]) null, null)); assertequals(text_list_nosep, StringUtils.join(ARRAY_LIST, null)); assertequals(text_list_nosep, StringUtils.join(ARRAY_LIST, "")); assertequals("", StringUtils.join(NULL_ARRAY_LIST, null)); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, null)); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, "")); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, SEPARATOR)); assertequals(text_list, StringUtils.join(ARRAY_LIST, SEPARATOR)); assertequals(",,foo", StringUtils.join(MIXED_ARRAY_LIST, SEPARATOR)); assertequals("foo,2", StringUtils.join(MIXED_TYPE_LIST, SEPARATOR)); } assertequals("/", StringUtils.join(MIXED_ARRAY_LIST, "/", 0, MIXED_ARRAY_LIST.length - 1)); assertequals("", StringUtils.join(MIXED_ARRAY_LIST, "", 0, MIXED_ARRAY_LIST.length - 1)); assertequals("foo", StringUtils.join(MIXED_TYPE_LIST, "/", 0, 1)); assertequals("foo/2", StringUtils.join(MIXED_TYPE_LIST, "/", 0, 2)); assertequals("2", StringUtils.join(MIXED_TYPE_LIST, "/", 1, 2)); assertequals("", StringUtils.join(MIXED_TYPE_LIST, "/", 2, 1));

33 Název metody To první a často jediné, co z vaší metody bude většina lidí číst věnujte mu pozornost Měl by být stručný a přitom dostatečně popisovat, co metoda dělá Pokud se vám nedaří vhodný název vymyslet, může to být špatným návrhem metody Metoda dělá více různých věcí (měla by dělat vždy jen jeden svůj úkol) Nemáte vlastně jasno, co má metoda dělat Pokud jste při vymýšlení jména přišli na špatný návrh metody, ušetřili jste si dost času, který byste jinak strávili později přepisováním kódu Kvíz: co dělá tato metoda? dontstoredatatocache()

34 Název metody issavebasicdataandtemplateattributevaluesdeactivatedocumentreturnpossible()

35 Jedno slovo o One Word per Concept fetch retrieve get read find

36 Jména a typy parametrů Jméno by mělo popisovat, co daný objekt reprezentuje nejen sám o sobě, ale i v kontextu použití parametru: void merge(graph graph1, Graph graph2) vs. void merge(graph sourcegraph, Graph targetgraph) U typovaných jazyků využívejte datové typy Toto by měla být naprostá samozřejmost Speciálně v Javě se ale často setkávám zejména s ignorováním enums (místo enumu se často používají booleany nebo soustavy konstant) Zdá se, že velmi populární je místo strongly typed používat stringly typed přístup: Text: "abc" Číslo: "42" Objekt obsahující jednu textovou a dvě číselné položky: "abc_42_666" Přístup k první číselné položce objektu: myobjectstring.split("_")[1]

37 Co nepovažuji za užití datových typů Map<Vertex, Set<Boolean>> List<List<List<String>>> Map<List<Object>, HashSet<Pair<Integer, Integer>>>

38 Parametry Metoda s příliš velkým počtem parametrů je nepřehledná protected PdfPTable getprijemceplnenitable(float documentwidth, String nazevleasingovky, String cisloleasingovesmlouvy, String smluvniservis, String prijemcecislouctu, String prijemcejmeno, String prijemceulicecp, String prijemceobecpsc, boolean pojistenijindeexistuje, String pojistenijindenazev, Boolean dphanonenull, String zpusobuhradykey, String datum, boolean isdph, String stextpodpodpisem) Parametry typu boolean nejsou příliš vhodné Nutno v názvu parametru jasně deklarovat, co znamená true a co false Jméno parametru ale nemusí být vždy dostupné (např. v Javě, mám-li knihovnu bez javadoc a zkompilovanou bez jmen parametrů, vidím jen typ bez jména) Při volání metody není na první pohled význam boolean parametru obvykle zřejmý newtable(tablename, parentdatabase, true) newtable(tablename, parentdatabase, TableType.GLOBAL_TEMPORARY)

39 Primitive Obsession Parametry void bookticket(string name, String firstname, String film, int count, String cinema); bookticket(string arg0, String arg1, String arg2, int arg3, String arg4); void bookticket(name name, FirstName firstname, Film film, Count count, Cinema cinema); bookticket(name arg0, FirstName arg1, Film arg2, Count arg3, Cinema arg4);

40 Primitive Obsession - Parametry o Příklad Vyhledávání produktů podle cenového intervalu? searchbyprice (?);

41 Primitive Obsession - Parametry o Příklad Vyhledávání podle cenového intervalu Vector searchbyprice(double start, double end); List<Product> searchbypricerange(price start, Price end); List<Product> searchbypricerange(pricerange pricerange);

42 Primitive Obsession Třídy public class Customer { } private int id; private String firstname; private String lastname; public class Customer { } private CustomerId id; private FirstName firstname; private LastName lastname;

43 Javadoc Dohodněte se na jazyku a dodržujte ho V angličtině může být problém se vyjádřit (zejména odborné pojmy) Komentáře v češtině vyžadují neustále přepínat klávesnici a jazyk v hlavě Komentáře v cestine nevypadají dobře Dokumentujte chování vůči null (všude tam, kde to není očividné) U atributů objektu / třídy zda může někdy být null a co to znamená U parametrů metod zda může být null a co se v tom případě stane U návratových hodnot zda a v jaké situaci může metoda vrátit null U kolekcí a polí vnímejte rozdíl mezi prázdnou kolekcí a null Většinou dává lepší smysl prázdná kolekce ( seznam nalezených prvků je prázdný ) než null ( seznam nalezených prvků neexistuje ) Nenechte se vést leností nebo dojmem větší efektivity / úspory paměti při použití null (Java: Collections.emptyList())

44 Javadoc /** * Metoda cislo zaznamu milestone v cache nebo -1 firstrow lastrow extrarecords ExpiredDataException */ public CachedData gettimemilestonecacheddata(..)

45 Javadoc: StringUtils (Commons Lang) /** Joins the elements of the provided array into a single String containing the provided list of elements. No delimiter is added before or after the list. A null separator is the same as an empty String (""). Null objects or empty strings within the array are represented by empty strings. StringUtils.join(null, *) = null StringUtils.join([], *) = "" StringUtils.join([null], *) = "" StringUtils.join(["a", "b", "c"], "--") = "a--b--c" StringUtils.join(["a", "b", "c"], null) = "abc" StringUtils.join(["a", "b", "c"], "") = "abc" StringUtils.join([null, "", "a"], ',') = ",,a" Parameters: array the array of values to join together, may be null separator the separator character to use, null treated as "" startindex the first index to start joining from. It is an error to pass in a start index past the end of the array endindex the index to stop joining from (exclusive). It is an error to pass in an end index past the end of the array Returns: the joined String, null if null array input */ public static String join(object[] array, String separator, int startindex, int endindex)

46 Javadoc : StringUtils (Spring) o o Anotace pro popis okrajových podmínek javax.annotation.nullable javax.annotation.nonnull Není nutný popis v Javadocu /** * Check whether the given {@code String} is empty. * <p>this method accepts any Object as an argument, comparing it to * {@code null} and the empty String. As a consequence, this method * will never return {@code true} for a non-null non-string object. * <p>the Object signature is useful for general attribute handling code * that commonly deals with Strings but generally has to iterate over * Objects since attributes may e.g. be primitive value objects as well. str the candidate String */ public static boolean isempty(@nullable Object str) { return (str == null "".equals(str)); }

47 Javadoc: ServerRequest (Spring) o o Optional monáda pro návratové hodnoty Optional.isEmpty() Optional.map(value -> ).orelse( ) Není nutný popis v Javadocu public interface ServerRequest { /** * Get the request URI. */ URI uri(); /** * Get the request attribute value if present. name the attribute name the attribute value */ Optional<Object> attribute(string name); } /** * Get a mutable map of request attributes. the request attributes */ Map<String, Object> attributes();

48 Javadoc: Flux (Reactor) o Když slova nestačí Obrázky, diagramy /** * Transform the items emitted by this {@link Flux} by applying a synchronous function * to each item. * <p> * <img class="marble" src="..." alt=""> * <p> mapper the synchronous transforming {@link Function} <V> the transformed type * a transformed {@link Flux} */ public final <V> Flux<V> map(function<? super T,? extends V> mapper) {

49 Javadoc : Flux (Reactor)

50 Javadoc - asciiart o Když nemáte obrázek, můžete vzít asciiart /** * Reads complete offer. * <pre> * * * Offer * * * * \/ * * Business Product < Offered Product * * * * \/ \/ * * Business Feature < Offered Feature * * </pre> */ Optional<FinancialProduct> getoffer(productid productid, OfferId offerid);

51 Kód Dodržujte konvence daného jazyka (jména, formátování, ) Orientace na čtenáře kód bude čten víckrát, píšete jej jednou Lenost zde není platným argumentem Jména proměnných (a všeho ostatního) volte dostatečně srozumitelná Příklad z jedné revize: v cele třídě mnoho nevhodných názvů proměnných bi, is, bos, asr, s, m, baos, dos a můj favorit, pole s názvem array Kvíz: co ošetřuje následující podmínka? if(lv >= 0 && df >= 0 && this.hasrows() && df < this.getrows().length) { } Všeho s mírou obecně čím větší rozsah platnosti, tím důležitější je jasné jméno Srozumitelnost!= délka např. význam proměnné i je obecně známý Nepoužívejte jednu proměnnou ke dvěma účelům Ternární operátor je efektní, ale leckdy poněkud nečitelný Kvíz: co vrátí následující výraz? (0 < 1)? 2 : 3 + 4

52 Programming by Coincidence

53 Programming by Coincidence if (typzadosti == ZALOZENI_SMLOUVY) { zalozsmlouvu(); } else { zrussmlouvu(); }

54 Programming by Coincidence if (typzadosti == ZALOZENI_SMLOUVY) { zalozsmlouvu(); } else if (typzadosti == ZRUSENI_SMLOUVY) { zrussmlouvu(); }

55 Programming by Coincidence if (typzadosti == ZALOZENI_SMLOUVY) { zalozsmlouvu(); } else if (typzadosti == ZRUSENI_SMLOUVY) { zrussmlouvu(); } else { } throw new IllegalArgumentException( Nepodporovany typ zadosti: + typzadosti);

56 Programming by Coincidence if (typzadosti == ZALOZENI_SMLOUVY) { zalozsmlouvu(); } else if (typzadosti == ZRUSENI_SMLOUVY) { zrussmlouvu(); } // else do nothing

57 Programming by Coincidence 2 int pocetcelychmesicudokoncekvartalu = 0; switch(mesicvroce%3) { case 0: pocetcelychmesicudokoncekvartalu = 0;break; case 1: pocetcelychmesicudokoncekvartalu = 2;break; case 2: pocetcelychmesicudokoncekvartalu = 1;break; }

58 Programming by Coincidence 2 int pocetcelychmesicudokoncekvartalu = 0; switch(mesicvroce%3) { case 0: pocetcelychmesicudokoncekvartalu = 0;break; case 1: pocetcelychmesicudokoncekvartalu = 2;break; case 2: pocetcelychmesicudokoncekvartalu = 1;break; default: throw new IllegalStateException( "Nastala situace, ke ktere nemuze dojit"); }

59 Je tento konstruktor prázdný záměrně? protected MyObject() { };

60 Je tento konstruktor prázdný záměrně? protected MyObject() { }; /* empty */

61 Dřív to fungovalo entity.id = new EntityId(this.allEntities.size());

62 Fail Fast o Kdo si pamatuje?

63 Fail Fast o o Je-li v programu chyba, měl by selhat co nejdříve Aktivně kontrolujte konzistenci a ošetřujte možné chyby o o Dead programs tell no lies Nejhorší je na chybu přijít až po databázovém commitu

64 Fail Fast - Kontrola parametrů public void setformatter(dateformatter dateformatter) { Assert.notNull(dateFormatter, "DateFormatter must not be null"); } this.dateformatter = dateformatter;

65 Fail Fast - Kontrola stavu objektu public Transaction createtransaction(string name, int timeout) throws NotSupportedException, SystemException { Assert.state(transactionManager!= null, "No JTA TransactionManager available"); }...

66 Unit testy

67 Unit testy o o Kdo píšete? Pravidla?

68 Unit testy o o o o o Psaní unit testů je absolutní samozřejmost. Kvalitní kód bez unit testů je nesmysl. Nejdůležitější je Psát hodně Psát průběžně Aktualizovat Pravidla (podle Uncle Boba): One assert per test Readable Fast Independent Repeatable One assert per test dodržuje málokdo.

69 Příklad: StringUtils (Commons Lang) private static final String[] ARRAY_LIST = {"foo", "bar", "baz"}; private static final String[] NULL_ARRAY_LIST = {null}; private static final String[] EMPTY_ARRAY_LIST = {}; private static final String[] MIXED_ARRAY_LIST = {null, "", "foo"}; private static final Object[] MIXED_TYPE_LIST = {"foo", Long.valueOf(2L)}; private static final String SEPARATOR = ","; private static final String TEXT_LIST_NOSEP = "foobarbaz"; private static final String TEXT_LIST = public void testjoin_arraystring() { assertnull(stringutils.join((object[]) null, null)); assertequals(text_list_nosep, StringUtils.join(ARRAY_LIST, null)); assertequals(text_list_nosep, StringUtils.join(ARRAY_LIST, "")); assertequals("", StringUtils.join(NULL_ARRAY_LIST, null)); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, null)); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, "")); assertequals("", StringUtils.join(EMPTY_ARRAY_LIST, SEPARATOR)); assertequals(text_list, StringUtils.join(ARRAY_LIST, SEPARATOR)); assertequals(",,foo", StringUtils.join(MIXED_ARRAY_LIST, SEPARATOR)); assertequals("foo,2", StringUtils.join(MIXED_TYPE_LIST, SEPARATOR)); } assertequals("/", StringUtils.join(MIXED_ARRAY_LIST, "/", 0, MIXED_ARRAY_LIST.length - 1)); assertequals("", StringUtils.join(MIXED_ARRAY_LIST, "", 0, MIXED_ARRAY_LIST.length - 1)); assertequals("foo", StringUtils.join(MIXED_TYPE_LIST, "/", 0, 1)); assertequals("foo/2", StringUtils.join(MIXED_TYPE_LIST, "/", 0, 2)); assertequals("2", StringUtils.join(MIXED_TYPE_LIST, "/", 1, 2)); assertequals("", StringUtils.join(MIXED_TYPE_LIST, "/", 2, 1));

70 Příklad JpaRepository (Spring Data JPA) public class JpaRepositoryTests { CrudRepository<WithIdClass, WithIdClassPK> // DATAJPA-50 public void executescrudoperationsforentitywithidclass() { PersistableWithIdClass entity = new PersistableWithIdClass(1L, 1L); idclassrepository.save(entity); assertthat(entity.getfirst(), is(notnullvalue())); assertthat(entity.getsecond(), is(notnullvalue())); PersistableWithIdClassPK id = new PersistableWithIdClassPK( entity.getfirst(), entity.getsecond()); } assertthat(idclassrepository.findbyid(id), // DATAJPA-266 public void testexistsfordomainobjectswithcompositekeys() {... } // DATAJPA-527 public void executesexistsforentitywithidclass() {... }

71 Unit testy Názvy metod o o o Špatné testfind testvalidate Dobré shouldfindbyidwhenexists shouldthrowexceptionwhenagelessthan18 should_throwexception_when_agelessthan18 executesexistsforentitywithidclass Doporučení Testujeme vlastnosti třídy, ne jednotlivé metody Pro jednu metodu typicky existuje několik testů

72 Nástroje

73 SonarQube o o o Nástroj pro analýzu kvality kódu Statická analýza Testy a coverage Podporuje Java C# JavaScript Doporučení Pouštět analýzu pravidelně Kontinuálně řešit nálezy Nenechat nahromadit technický dluh

74 SonarQube - Přehled

75 SonarQube Issues

76 SonarQube Code smell

77 Perličky

78 Kdy se zobrazí nabídka úvěrů? /** * Method returns true if credit lines should be shown. * true if credit lines should be shown otherwise false */ public boolean isshowcreditlines() { return // Showing depends of dynamic property and if user is in pilot mode (show only in pilot to // users in pilot mode if constant is true, if false, show to all) ((ebankinguser.getpilot() == null) (!EbankingConstants.CREDIT_LINES_FUNCTIONS_PILOT) (Boolean.TRUE.equals(ebankingUser.getPilot()) && EbankingConstants.CREDIT_LINES_FUNCTIONS_PILOT)) // and (home tab is shown and there are no campaigns) && ((menubean.ishometab() &&!bcs.isshowcampaigns()) // or ((we are on loans tab (TAB_LOANS) on specific menu item (MNU_LST_CRE_MORA_GET) // or on eshop (TAB_ESHOP)) and client has no approved application) (((menubean.iswantedtab(menunames24.tab_loans.name()) // THU: tento kod je zakomentovan, protoze nefuguje nastavovani actualmenuitem spravne /*&& (menubean.getactualmenuitem()!= null) && menubean.getactualmenuitem().name().touppercase().equals( MenuNameS24.MNU_LST_CRE_MORA_GET.name())*/ ) menubean.iswantedtab(menunames24.tab_eshop.name())) && bcs.getapprovedapplicationid() == null)) // and there exist some credit lines &&!isitemlistempty(); }

79 Map není nikdy dost public interface IValidator { public void validate( Object[] objects, Map<Object, Object> globalparams, Map<Object, Object> localparams, Map<Object, Object> rowparams, Locale locale); }

80 DB procedura... ** Nazev ulozene procedury: esipo_s_prehl_rotv ** ** Funcionalita: Procedura slouzi ** ** Vstupni parametry:...

81 Trocha logiky public void setplatnostdoodvolani(boolean platnostdoodvolani) { if (platnostkontaktniadresy == null) { platnostkontaktniadresy = new IntervalOdDo(); } else { platnostkontaktniadresy = new IntervalOdDo( platnostkontaktniadresy.getzacatek(), null); } }

82 Čím se od sebe metody liší? /** * Directory interface poskytovany modulum skrze WebService webovou * aplikaci datovych schranek. */ public interface IDSDirectoryService { public ListResponse<Issuer> getissuer( IssuerSearchCriteria searchcriteria, String personid); } public ListResponse<Issuer> findissuer( IssuerSearchCriteria searchcriteria, String personid);

83 Handle stuff dataprocessor.processdata(elementsonpages);

84 Literatura o The Pragmatic Programmer: From Journeyman to Master Andrew Hunt, David Thomas o Clean Code: A Handbook of Agile Software Craftsmanship Robert C. Martin

85 Dotazy? 85

86 Děkuji za pozornost Profinit, s.r.o. Tychonova 2, Praha 6 Telefon Web LinkedIn linkedin.com/company/profinit Twitter twitter.com/profinit_eu