.' =
TECHNICKA UNIVERZITA V LIBERCI ' .4' : : :r' i .i;,.''.i.i. !' ,lri'|.,r r,-=
oPoN ENTN I posu oer zdvEneeru E mrall H xneN I pRAcE
Autor zdv6re6n6 prdce: Vladimir Anto5
N{zev pr6ce: NAVRH lr,tiRy pRo Wpodef ToxtctTy KoDU A
teti lvpterrlteNtncr
Oponentpr6ce Ing. pavel Khail
PracoviSt6 opon€nta pREGIS a.s.
A. Kvalita abstraktu, klieov6 slova odpovidajindptni
prece Viborne
(1)B. Rozsah a zpracov6ni
reSer5e
Vyborne minus (1-)C. Re5eni prace po teoreticke str6nce VYborne (1)
Vyborne minus (1-)
Viborn6
('l ) V:iborne minus (1-) D. Vhodnost, pfimdienost pouzite metodikyE. Uroveri zpracoveni Wsledk0 a diskuse F. Vlastni
piinos
k iesen6 problematiceG. Formulace z6v6ru
prdce
V)rborne minus (1-)H. Splneni zadani (cilfi)
prace . .
Splnenol.
Skladba, spravnost a oplnost citaci liter6rnich0daj0
Vyborne (1)J. Typografickd a jazykov6
iroveri
(vd. pravopisu)Viborne
minus(1,
K. Formdlni nalezitosti
prace
Vyborne (1)(struktura textu, iazenf kapitol, piehlednost ilustraci)
Komentaie ei piipominky:
Celkove vyborna prace - zajimav6 a uzitedn6 tema.
ReSerZe - nCkter6 metriky by si zaslouZily lepSi popis a piiklady (napi. Metoda funkcnich
bodt),
v popisu Halsteadovy metriky velikosti programu.jsou zie.jm6 chybn6 popsany n1 a n2 (stejn6 vysvefleni). Celkov6 byreser:e
mohla byt podrobn6jSi.Nektere piiklady nevhodne voleny (napi. Calculator), n6kde naopak chybi (Fp).
Vyborn6 zpracovany popis nestroJi pomahajicich s psanim dist6ho kOdu.
Osobne polemizuji s metrikou poetu koment6i0,
z
me zkusenosti se na komentAie lze divat jako na metadata poskytujici dalsi informace o k6du, ktere nejsou na prvni pohled patrne ausnadiujijeho
pochopeni. Naopak mi chybi napf. velmi d&lezite metrika hlidajici duplicitu k6du, ktere je jednou z nejeastejSich prohiesk0 programatortr.Skoda
nekterich
pravopisnich chyb v textu prace (meli x mely).Vlastni aplikace je pekne - chybi pouze dotehnout do konce (9rafy, referendni metriky - viz nize,
alespoi strueni
uzivatelsky navod), z pohledu vlastniho k6du - kalibraeni data bych spise ukladal do XML, ne plain textu, nekter6 konstrukce v k6du lze vylepsit-
napi. vicenasobne pouziti stejnych retezcowch konstant aDod....pokraduje na strane
2
TECHNfCKA UNtVERztTA V L|BERC| ;
r,riu.
Srudentskd 14ozt2 461 17 Lrberec 1tet : +42O 485 353 429 theno.pnjment@tul
cz
www tutcz . DIC CZ 467 47 ABsi::ffi
rI
TECHNrcKA UNVERZFAV L|EERC|\A_/ f.kultl
{r1e(hat.oniky, rnformatrky a frtfztoboroviahstudti I
Celkov6 zhodnoceni:
Dobte zvolen6 t6ma, z me vlastni zkuSenosti (>20 let programovani) Je taio vpravdd osv6tova 6innost velmi podceiovena a je proto nutne na ni dasteji
upozoriovat
aktivitamiiako je tato - nejEpe tedypiimo
pii vzd6l6vani studentt. Vyu2it aplikaci pllmo ve Skolach je dobrym nepadem. Bylo by vSak nutne aplikaci dotehnout do finalniho stavu, toto bychbraljako vezi
0.9.Libi se mi vyuziti API kompil6toru Roslyn jako
chytri
aWhodni
krok pfi imp6mentaci aplikace. Zavedeni referendni metriky pro porovnAni vlastnlho k6du jeskv6l!
n6pad, st6lo by v5ak za to, vytvolit jakousi repository referenenich metrik, zekterich
by bylo mozne vybirat. Architektura aplikace je dobie popsana.Osobn6 mi chybi d|]lerit6 metrika duplicitniho k6du (podrobnAji
Wie).
Prace m6 drobn6 nedostatky (viz
vise)
amotnosti
ke zlepseni, celkou! plinos je vsak patrny!orazky k obhajob6:
1. Jak byste navrhoval
rozllleni
aplikace o dalsi mekiky (napi. duplicitni k6d), 5lo byvyutlt
API Roslynu?2.
Ziakych
zdrojit jste cerpal pii zapojeni meiriky pro Poeet koment6l0?3. Vidite n6jaky prostor pro zlepseni zdrojoveho k6du CleanCodeAnalyzeru?
Celkovd klasifiksce:
Prace
spliuje
pozadavky na ud6leni akademickeho titulu, a proto.ii doporueuJi k obhajobeNavrhuji tuto prAci klasifikovat stupndm Vfborn6 (1)
V
Janovd nad Nisoudne
5.6.2016Podpisem soudasnE potvzuji, Ze neisem v 26dn6m osobnim vztahu k autorovi pr6ce
TEcHt{lcKA uNlvERzTA v
uEERcl
FMIMS studenlskd 1402/2 461 17 Liberec 1tel.: +420 485 353 429 t jneno.ptijnen@tut.Q I www tul cz \ DIC: CZ 467 47 885j