Názory k článku Ministerstvo financí dalo zdrojáky Supervizora na GitHub

Tak pánové, vidím tu 8 expertů, rád bych tam viděl alespoň 8 forků.

Ať je kvalita kódu front-endu či back-endu jakákoliv, jejich přístupu si cením. A neremcám.

Jojo, pokud to je napsane v PHP nebo to nejede na cool frameworku (nejlepe grudl approved), nema to sto dvacet milionu testu a samozrejme neobsahuje zadne chyby tak je to k nicemu. Co na tom, ze to jen prechroustava verejny data a i pres ty vsechny nedokonalosti to funguje, podle "expertu" je to proste nahovno. Ceska klasika, tady se prace jinych neceni.

Trefovat se do kvality kódu je hrozně jednoduchý. Myslím, že mnohem důležitější je to, že se jedná o první vlaštovku ve zveřejňování zdrojáků státními institucemi a tu je potřeba podpořit.

Takže kluci, dobrá práce!

Mně zas přijde správné, že co vzniká za veřejné peníze, má být defaultně veřejné. Což zatím moc v ČR není zvykem. Takže krůček správným směrem.

Děkuji, pane Babiši, ujistil jsem se, že kontrolní hlášení, EET a další data jsou zcela v bezpečí a v těch nejlepších rukou.

Chápu a dobrý nápad to určitě. Pointa je jiná - to vám nevadí, že to je prezentováno jako produkt MF vytvořený cituji "za peníze daňových poplatníků"?

Rozumějte, já ten princip považuju za správný, cokoliv vytvořeného skutečně za peníze daňových poplatníků by opravdu mělo být zdarma k dispozici veřejnosti včetně zdrojových kódů. Ovšem pod tím si nepředstavuju to, že nějaký nadšenec po večerech cosi doma zadarmo tvoří (zcela mimo náplň své práce) a pak si nějaký politik přivlastní a udělá si na tom PR a začne halasně prezentovat, jak je úřad moderní a otevřený.

nerozumim tomu proc tady probiha tolik kritickych komentaru na Supervizora. Kod je otevren, hostovan na Githubu a v issue trackeru neni zalogovana ani jedna chyba. Podnety na zlepseni a chyby patri tam.

A ten cloud za 13 miliard jiz dava zcela smysl.

To spíš naopak testovací u někoho na lokále, ne?

To neznáš státní zakázky :-P

Díky za info.

Pokud to dělate ve volném čase a ještě neprogramujete, tak to beru a proti kódu nic nemám.

Spíše celý článek vyzněl, že programatoři z MFČR udělali apku v rámci pracovní naplně (hlavní by bylo, že jsou programatoři ; dotace času pro dobrovolnou apku se dáí pochopit). To by pak byl velký průser.

Akorát nechápu, kde se stála "chyba v překladu"

Víte oni existují dva typy programátorů - ti co chtějí mít dokonalý kód a ti kteří něco dotáhli dokonce.

Na githubu vidím 0 pull requestu a 0 issues, tak to spíš vypadá, že tu všichni chtějí jen plácat prázdnou slámu než opravdu hledat chyby na něčem co očividně funguje (a jde použít).

Namachrovaný dem.nty jako jsi ty mám nejradši. Ještě nenapsali žádnej kód, kterej by k něčemu byl, umí programovat jen v "týmu" a musí je vést za ručičku zkušenej senior. Pravděpodobně máš za sebou pár Hello worldů někde na škole co?

A tohle asi byla původní produkční konfigurace:

//$pdo = new PDO("mysql:dbname=zpevniky;host=127.0.0.1","root","",$driver_config);

:-DDD

Kód je zajímavý, ale trochu (moc) nepřehledný. Vysvětlím proč

Celá webappka je ukázkou neudržovatelného, "špagetového" kódu. Nikde náznak jakékoliv architektury, prakticky neexistuje nějaká objektová architektura, není oddělena prezentační vrstva a datový model, ... prrostě neodpovídá úrovni požadované současným businessem.

Doporučuji celou aplikaci (pokud ji chcete dál rozšiřovat), přes nějaký PHP framework např. nette.org a trošku zapracovat na designu celé aplikace.

Šel bych do Laravel nebo do něčeho vlastního.

https://github.com/SmallhillCZ/Supervizor/blob/master/www/app/startup.php:

/*
if($_SERVER["RE­MOTE_ADDR"] !== "90.180.72.190"){
header("Content-type:text/pla­in;charset=utf-8");
die("Na webu něco důležitého měním, opět spuštěn bude co nevidět.");
}
*/

Ačkoliv v PHP nevyvíjím, kód je pro mě těžko čitelný. Většinou jsem schopen pochopit dobře napsaný kód i v jiných jazycích, než v těch, které jsem byl schopen poznat.

Tento mi moc čitelný nepřijde.

Já bych ve vašem prvním bodě veřejnost zaměnil za "jiné vývojáře". Sám mám co dělat, když otevřu svůj několik měsíců starší kód, abych zjistil co jsem tam vlastně dělal. Když otevřu několik let starý kód, tak si mnohdy říkám co za prase to psalo...

Jenže tím, že je kód OpenSource neznamená že jej každý den pročítají stovky vývojářů jen aby v tam našli nějakou chybu (pokud se nejedná o Amazon, Google, Microsoft apod. a vývojáře přímo k hledání chyb vyzývají). Za svou kariéru jsem nepotkal člověka, který by procházel open source projekty ostatních a hledal tam chyby.

A na hledání už existujících řešení tu máme Stackoverflow.

Ehm... rozumím tomu správně, že vás pan Bureš nutí zdarma ve volném čase programovat zaměstnanecká díla, aby si pak s nimi budoval PR?

Jedno načtení CSV za miliony? To je snad přemrštěný odhad.

Tohle je mega hardcore https://github.com/SmallhillCZ/Supervizor/blob/master/www/app/api/dodavatel.php

"Jirka" je zjevně Jirka Skuhrovec. http://blog.aktualne.cz/blogy/jiri-skuhrovec.php

V tom případě OK, je fajn, že se tomu věnujete.

Bohužel jak se píše výše, tak je to spíše na komplet přepis a absence jakýchkoli rozumných komentářů (typu "tohle má dělat tohle" ne "regional settings" před třemi řádky nastavení locales) taky orientaci moc neusnadňuje.

podle linkedin.com/in/mar­tinkopecek máte velké zkušenosti, takže nechápu.

Třeba mě osobně nejvíc děsí je nedoržování jednotných standardů kódu, například jen udržování jednotné indentace, což Vám dneska dokáže zajistit jakýkoliv editor kódu, včetně těch, co jsou zdarma.

To potlačování pomoci @ je taky bingo.

ccs v iframu nedává smysl a ani by nefungovalo. SPA je jen buzzworld a zbytečnost.

Ne nemohl, kód je přišernej a neodpovídá úrovni, která se klade na PHP programátory, seniorní úroveň tomu je vzdálená na míle.

Je to ale hezké gesto a pokud se nebudou bránit PR, rád něco přišpějii, od toho je komunita.

ten senior architekt byl sarkasmus, takhle nejak si predstavuju cloveka co jeste dneska dela PHP.

css ve vlastnim souboru a vlozene jak do indexu tak do iframe pagi by fungovalo.

Co je na tom závadného?

Ten namachrovaný *** se kterým mluvíte, má vystudovaného magistra v oboru programování (a ne na ekonomce), má za sebou 15+ let zkušeností s mnoha technologiemi, samostatně i v týmech, několik open source projektů a vychoval už několik juniorů.

Zřejmě vůbec nechápete, k čemu existuje rozdělění senior a junior. Senior nikoho nemá vést za ručičku, ale má hlavně dohlédnout na to, aby junior získal zkušenosti a naučil se správné programátorské návyky.

Ano, takový kód, který zveřejnili, jsem psal taky. Ale když mi bylo 20 let a dělal jsem samostatně. První kontakt se zkušenými mě to velmi rychle odnaučil.

Nebudu je poplácávat po rameni za to, že zveřejnili pár mizerných řádek kódu, opravdu ne. Hlavně když vím, že všechna ministerstva si platí vývoj aplikací za miliardu, kde přistup ke zdrojovému kódu nemají ani v ceně.

Při pohledu na ty zdrojáky se mi chce zvracet.

Zajimalo by mě, je tohle běžná úroveň PHP programátorů? Nebo je autor extrém?

Můžete si o kritice myslet, co chcete, ale faktem je, že daný kód jen ukazuje velmi nekvalitní práci (holt ministerstvo nezaplatí kvalitní lidi?).

Open source má dva účely:
1. Umožnit veřejnosti najít chyby
2. Umožnit někomu dalšímu použít to samé

V tom stavu, v jakém to je, je 2 absolutně mimo. A vy se rozčilujete, že v tom kritici hledají chyby?

Osobně bych přispěl max přepsáním celýho kódu. Tady nic lepšího nepomůže.

Zveřejňovat open source s dokumentací a komentáři v Češtině?
Základní nepochopení, k čemu slouží na verzovacím systému popis commitu? "Update readme", "Opravy chyb" atd. je opravdu ideální, aby člověk zpětně něco nalezl.

Proboha, dejte tomu autorovi k ruce nějakého seniora, který se v tom vyzná, a ukáže mu, kde jsou chyby.

Největší peklo v programování jsou samouci, kteří sice umí programovat, ale nikdy nedělali ve větším týmu vedle zkušených lidí.

Funguje jak ma? Nie, nefunguje.

Spoliehat na nejake deterministicke spravanie sa funkcie/metody zavana problemom. Tie warningy tam nie su pre srandu kralikov. Vstupy sa maju osetrit a kontrolovat.

Proč jsem něco takového čekal...

Je to takove cele PHPkove, ale kdyby tam bylo to CSS nekde pohromade a nacitalo se to aspon do iframu v indexu kdyz uz ne javascriptem jako SPA tak me to ani prilis neurazi.

Myslim ze pan by klidne mohl delat senior PHP architekta v nejake webove agosce.

Proč? Funguje to tak jak má, nebo ne?

Přesně tak.

Nette je mrtvé. To jste jim to poradil :)

nemyslím si. obsahuje několik featuers, které by třeba takový zend mohl závidět (robotloader).

Tady nejde o počty features, ale o celkový přístup komunity k Nette, což výrazně ovlivňuje životnost.

A dle mých zkušenosti je komunita akční jen ve stylu "Jak udělam toto?".

To je naprosto běžny styl psání kódu programátory samouky. Není to špatné, ale je tam vždy vidět, že nikdy nedělali ve větším týmu zkušených lidí. To má za následek chybějící standardy psaní kódu, komentáře v Češtině, nic neříkající commit zprávy. Kdyby něco takového udělal v týmu, tak ho to ti ostatní velmi rychle odnaučí.

Spoliehat na nejake deterministicke spravanie sa funkcie/metody zavana problemom.

wat? i dont even

ne, nerozumis

Vzhedem k tabulkovému platu bych řekl, že práce odpovída ceně.

Ale zase je zde otázka, zda takové kódy nepracuji se citlivými daty.

Nedělejte to ani doma na testování. Čert vem roota bez hesla, ale vy přece potřebujete uživatele s minimálními oprávněními k databázi, která tak akorát dostačují pro účely té aplikace. Protože když to někdo vezme a nahodí jinde, tak určitě ne se superuserem s neomezenými právy. No a zjistí, že mu to nefunguje správně.

Dobrá práce, díky za kódy, nápad a že je to na Gitu. Použijeme i na našem projektu ke zveřejnění obdobných dat. Pro nás úspora časová i finanční. Vzhled aplikace bez problémů, dělá to co má a přehledně. Klobouk dolů, že jsou ještě ve státní správě nadšenci. Máte u mne pivo a já u šéfa kladné body, když jsem mu to předvedl. Především myšlenka vynechání VZMR, tlachání s analytikem, projekťákem, ředitelem vývoje, ředitelem divize, ředitelem PR a ředitelem .... a čert ví ještě kým ho potěšila. Pokecal bych klidně s programátorem, jenže ten se k zákazníkovi pro jistotu raději nedostane a ještě by to mohl za 2-3 dny udělat a celou zakázku zlevnit :-)

Tak, když jsi tak ostřílenej profík, předpokládám že jim teď ukážeš jak se to má správně dělat. Když tu z tebe vypadlo tolik kritiky.
Mimochodem jsme tu v česku tak proč by nemohly být komentáře v češtině? Stydíš se za to že jsi čech?

A teď čekají, že jim komunita napíše fízlovací aplikaci sama...

A potom su taki ktori vobec nie su programatori - co sami priznali. Ostatne nic take ako dokonaly kod neexistuje.

A k comu by bolo opravovat to? Aby to potom ministerstvo pouzilo pre svoje PR tak ako k tomu pouzil ten kod co zlatala partia ajtakov po veceroch doma?

Asi som prehliadol oznamenie ministerstva ktore by audit a nasledne prepisanie zaplatilo. Potom si s tym mozu robit co chcu - kludne znova zverejnit na githube alebo pouzit k chvalospevoch o EET.