Jak overit kvalitu PHP kodu od dodavatele?

Napsal ItSnowsInHellAgain;1476814

Olegu, máš to prostě těžké.

1) Full-time programátor se ti nevyplatí (podle těch počtů webů by měl)

2) Máš hodně kontaktů na programátory, avšak většina nemá čas a čekat nechceš.

3) Chceš perfektní kód, ale nemůžeš najít programátora.

Po úvaze mám 3 řešení, jiné neexistuji.

1) Outsourcing do IT firmy, s kterou podepíšeš NDA a nějaký ceník.

2) Naučit se PHP sám - např. https://www.phptherightway.com/

3) Změnit obor podnikání

Prece nebudes mi spravovat weby na cms, ktere jsem vyjmenoval. Nebo ano ale bude to na bad practice & not dry? To jsem zkousel outsourcovat. Predpodledni web stal pre 100k psany v nette, kdyz se na to podival muj znamy nette proc tak se vyvratil, ale cas na opravu nema, protoze ho ted bavi delat Android hry.Nemam cas se sam ucit PHP a to, ze umim Python se v tom vsechno neda delat a uz programovat opravdu nechci :) Weby neni muj hlavni obor, ja je nedelam. Mame online projekty, ktere generuji urcitou cast zisku. To ale neznamena, ze chci shit zdrojak a neoptimalizovanou aplikaci. No jic, zacnu shanet auditory. Ale prece jen bych chtel vedet jak reklamovat shit kod. Chci udelat jednu vec a nerad bych vyhodil 50k oknem---------- Příspěvek doplněn 28.07.2018 v 18:55 ----------

Napsal sentosa;1476813

Poptavat dodrzovani zakladnich programatorskych principu, jak sem psal. Pokud se dodrzovat nebudou, zakazku nelze povazovat za dokoncenou

Okey takze pozadovat SOLID princip a TDD?

Napsal Oleg;1476815

Prece nebudes mi spravovat weby na cms, ktere jsem vyjmenoval. Nebo ano ale bude to na bad practice & not dry? To jsem zkousel outsourcovat. Predpodledni web stal pre 100k psany v nette, kdyz se na to podival muj znamy nette proc tak se vyvratil, ale cas na opravu nema, protoze ho ted bavi delat Android hry.

Nemam cas se sam ucit PHP a to, ze umim Python se v tom vsechno neda delat a uz programovat opravdu nechci :)

Weby neni muj hlavni obor, ja je nedelam. Mame online projekty, ktere generuji urcitou cast zisku. To ale neznamena, ze chci shit zdrojak a neoptimalizovanou aplikaci.

No jic, zacnu shanet auditory.

Ale prece jen bych chtel vedet jak reklamovat shit kod. Chci udelat jednu vec a nerad bych vyhodil 50k oknem

---------- Příspěvek doplněn 28.07.2018 v 18:55 ----------

Okey takze pozadovat SOLID princip a TDD?

Tvůj známý byl další programátor. Je to smutné, ale auditoři tě budou stát peníze a bude ti to k ničemu - uvidíš pouze audit navíc. Nedovedu si představit, že někdo podepíše smlouvu, která by určovala platbu za práci dle auditu - dál jsem příklad právníky. Pokud ti to generuje jen část zisku, nechceš full-time programátora, outsourcing, programovat v PHP či nechceš opustit IT, tak shit kód ti bude muset stačit.---------- Příspěvek doplněn 28.07.2018 v 19:07 ----------Jeden můj klient utahoval rozpočet tak dlouho, až se mu rozpadl tým, který se mu staral o ziskový web. Nahrazoval jednotlivé pozice juniory. Web časem přestal být ziskový, proto se klient rozhodl povolit rozpočet a složil původní tým. Pointa: Pokud chceš být v plusu, musíš do toho investovat.

Napsal Oleg;1476815

Okey takze pozadovat SOLID princip a TDD?

SOLID se da 'snadno' auditovat, testy jsou hlavne k tomu, aby se clovek nebal zmeny - tim ze neco upravi muze dalsich X veci rozbit. Testy to odhali, zaroven podle napsanych testu lze vice porozumet chovani systemu. TDD je metodika psani testu - testy se pisi pred implementaci funkcionality = 100% kodu otestovano. Psat testy az nakonec je neskutecne neefektivni. Dodrzovani SOLID principu ty zmeny jeste vice usnadni.pokud programator bude dodrzovat tyto 2 veci, dodane kody budou o nekolik radu kvalitnejsi

...

Napsal cmelo;1476807

Obcas robim code review cudzich projektov (php, nette, symfony, laravel) a vacsinou je to katastrofa.. ale vzdy sa s tym da nieco robit, len je to otazka casu a penazi.

Ako casto sa ten programator ktory robi web, vrati ku kodu ktory robil dajme tomu pred pol rokom a potrebuje donho nieco dorobit. Tak isto ten zakaznik. Ak potrebuje zakaznik nieco urobit raz za 3 roky, tak to je fakt jednoduksie to napisat odznova. Za poslednych 5 rokov, som videl len jeden pekne navrhnuty projekt. No len na tom sedeli 2 architekti niekolko mesiacov, kym dali finalny navrh a ten system bezi niekolko rokov a priebezne na nom maka tak 5 ludi. Takze je na nom stravenych tak 20-30 cloveko rokov. Tam sa uz velmi tvrdo hladi na to aky je kod. Teraz sa mi napr. dostal do ruky jeden projekt. Desktopova aplikacia. Vytvoreny pred 2 rokmi. Potrebuje tam urobit len par drobnych zasahov. Nic velke, len nejakych 15 formularov. No trieda obsluhujuca hlavny formular ma 23 000 riadkov kodu a velkost 1.7 MB. Co mu mam povedat, ze to potrebuje cele prepisat, lebo cely kod je nahovno ? 2 roky do toho zakaznik nepotreboval sahnut a ked mu tych par uprav spravym, tak zase bude mat na 2 roky pokoj. Ano, ta oprava co budem robit koli blbemu kodu bude mozno trvat 2 dni. Pokial by to bolo pekne urobene tak by mi to trvalo 1.5 dna. Takze jemu sa to neoplati to prerabat. Predpokladam, ze ta aplikacia bola lacna. Daval si ju robit v Indii.

Napsal qwertr;1476836

Ako casto sa ten programator ktory robi web, vrati ku kodu ktory robil dajme tomu pred pol rokom a potrebuje donho nieco dorobit. Tak isto ten zakaznik. Ak potrebuje zakaznik nieco urobit raz za 3 roky, tak to je fakt jednoduksie to napisat odznova.

Za poslednych 5 rokov, som videl len jeden pekne navrhnuty projekt. No len na tom sedeli 2 architekti niekolko mesiacov, kym dali finalny navrh a ten system bezi niekolko rokov a priebezne na nom maka tak 5 ludi. Takze je na nom stravenych tak 20-30 cloveko rokov. Tam sa uz velmi tvrdo hladi na to aky je kod.

Teraz sa mi napr. dostal do ruky jeden projekt. Desktopova aplikacia. Vytvoreny pred 2 rokmi. Potrebuje tam urobit len par drobnych zasahov. Nic velke, len nejakych 15 formularov. No trieda obsluhujuca hlavny formular ma 23 000 riadkov kodu a velkost 1.7 MB. Co mu mam povedat, ze to potrebuje cele prepisat, lebo cely kod je nahovno ? 2 roky do toho zakaznik nepotreboval sahnut a ked mu tych par uprav spravym, tak zase bude mat na 2 roky pokoj. Ano, ta oprava co budem robit koli blbemu kodu bude mozno trvat 2 dni. Pokial by to bolo pekne urobene tak by mi to trvalo 1.5 dna. Takze jemu sa to neoplati to prerabat. Predpokladam, ze ta aplikacia bola lacna. Daval si ju robit v Indii.

Nemůžeš dát část kódu na pastebin? Mě by hrozně zajímalo, jak to dokázal.

Napsal sentosa;1476810

Dnesni programatori jsou lina verbes, co si o sobe mysli kdovi co, ale zvladnou vypotit tak maximalne if/else. Koukej hodne na vek, protoze kdo programuje 5 let, prakticky se do toho teprve dostava. Nejhorsi jsou studenti a mladi obecne.

Dalsi vec je, ze 90% freelanceru produkuje mizerny kod, protoze se nijak nerozvijeji - nikdo je nekontroluje, s nikym kod nediskutuji, zakaznikovi staci ze to funguje, takze se zameruji na to, aby stim meli co nejmene prace, schrabli love a rozloucili se.

Zkuseny kvalitni programator typicky neshani ksefty na webtrhu, ale ma staly job, dela co ho bavi a ma slusne penize.

Kdo dela z domova tak je vetsinou flakac a nijak se nazajima o nejaky rozvoj, chce mit jen snadno vydelane penize - proto nad kazdou praci breci ze je to zprasene - nejradeji by kodil na zelene louce podle sebe se svymi "vychytavkami", at pak nadava nekdo jiny.

Jak uz tady padlo - idealne mit nekoho dobryho na fulltime, ale konkurence je velika, musel bys dobre platit jinak pujde jinam.

Myslím že to je najlepší príspevok, až na koncovku, ktorú autor nezvládol. Lepšiu radu nedostanete. Zabudnite na auditi, a audity auditov a pod.

máte moc binární vidění, kód není jen špatný nebo dobrý, ale plynule přechází od jednoho stavu k druhému podle úhlu (a cíle) pohledu.Velké či komunitní projekty mají tzv. code style guide, který poměrně přesně popisuje co se smí a co, dokonce je možné pak takové kontroly dělat automatizovaně. Pěkný příkladem jsou ty od googlu http://google.github.io/styleguide/. Na PHP je poměrně dost rozšířený (ale také méně konkrétní) PSR, jak už zmiňoval node, https://www.php-fig.org/psr/psr-2/. Velké týmu se jinak řídit nedají, samozřejmě se musí platit codereview.Tyhle pravidla musí být ale dána dopředu, programátor musí dopředu vědět čím se má řídit a jak to má tvořit, pokud mu to dopředu neřekneš, těžko se s ním pak můžeš hádat, že tady nemá DRY, tady má funkce side effect, tady má nevhodné pojmenování, tady to naprasil přes konstanty a tady mu chybí testy.Potřebuji-li od týmu výsledek v určité kvalitě, poskytnu jim mentora (seniorního člověka), který je pohlídá, vysvětlí, ukáže, zaručí určitou technickou úroveň. Platí tady ale jasné pravidlo, musím to zaplatit já, veškeré opravy, úpravy a účení jdou na moje bendra a do mého rozpočtu. Spousta mentorů, které mám k dispozici jsou freelanceři.Drtivá většina programátorů nadává na ty předchozí, že to je celé špatně, bohužel často mi právě vychází, že tihle lidé ani sami nejsou schopní to udělat lépe. Nehraje tady roli jenom vlastní znalost a povědomí o tom, jak to udělat lépe, ale čas a peníze, jakýkoliv refactoring bez pokrytí testy je dost riziková a drahá věc, tak se raději volí horší varianta, ale bez kaskádového efektu. Ti lidé si tím jenom zvyšují vlastní kreditabilitu, protože přece když poukáží na chyby někoho jiného, jsou lepší. Správná reakce na dost obtížný (=prasácký) kód je, "hele bude mi to trvat o 1/3 víc čas, v kódu se dostatečně rychle nevyznám", pokud ti tohle někdo řekne, ceň si ho!sentosa: nesouhlasím :), spousta open source vývojářů jsou freelanceři, kteří pracují v homeoffice, openBSD takhle žije 20 let, linux Kernel opět rozvíjí převážně lidé ze svých domovů a drivery poskytují firmy z kanceláří. Chápu, ale že to bereš z pohledu webového vývoje malých projektů v ČR, kde by to při troše "štěstí" tak mohlo být

Napsal TomášX;1476893

máte moc binární vidění, kód není jen špatný nebo dobrý, ale plynule přechází od jednoho stavu k druhému podle úhlu (a cíle) pohledu.

Velké či komunitní projekty mají tzv. code style guide, který poměrně přesně popisuje co se smí a co, dokonce je možné pak takové kontroly dělat automatizovaně. Pěkný příkladem jsou ty od googlu http://google.github.io/styleguide/. Na PHP je poměrně dost rozšířený (ale také méně konkrétní) PSR, jak už zmiňoval node, https://www.php-fig.org/psr/psr-2/. Velké týmu se jinak řídit nedají, samozřejmě se musí platit codereview.

Tyhle pravidla musí být ale dána dopředu, programátor musí dopředu vědět čím se má řídit a jak to má tvořit, pokud mu to dopředu neřekneš, těžko se s ním pak můžeš hádat, že tady nemá DRY, tady má funkce side effect, tady má nevhodné pojmenování, tady to naprasil přes konstanty a tady mu chybí testy.

Potřebuji-li od týmu výsledek v určité kvalitě, poskytnu jim mentora (seniorního člověka), který je pohlídá, vysvětlí, ukáže, zaručí určitou technickou úroveň. Platí tady ale jasné pravidlo, musím to zaplatit já, veškeré opravy, úpravy a účení jdou na moje bendra a do mého rozpočtu. Spousta mentorů, které mám k dispozici jsou freelanceři.

Drtivá většina programátorů nadává na ty předchozí, že to je celé špatně, bohužel často mi právě vychází, že tihle lidé ani sami nejsou schopní to udělat lépe. Nehraje tady roli jenom vlastní znalost a povědomí o tom, jak to udělat lépe, ale čas a peníze, jakýkoliv refactoring bez pokrytí testy je dost riziková a drahá věc, tak se raději volí horší varianta, ale bez kaskádového efektu. Ti lidé si tím jenom zvyšují vlastní kreditabilitu, protože přece když poukáží na chyby někoho jiného, jsou lepší. Správná reakce na dost obtížný (=prasácký) kód je, "hele bude mi to trvat o 1/3 víc čas, v kódu se dostatečně rychle nevyznám", pokud ti tohle někdo řekne, ceň si ho!

sentosa: nesouhlasím :), spousta open source vývojářů jsou freelanceři, kteří pracují v homeoffice, openBSD takhle žije 20 let, linux Kernel opět rozvíjí převážně lidé ze svých domovů a drivery poskytují firmy z kanceláří. Chápu, ale že to bereš z pohledu webového vývoje malých projektů v ČR, kde by to při troše "štěstí" tak mohlo být

Vypozoroval jsem, že v ČR dost osob chce platit pouze za čístý vývoj a nikoliv za věci, které popisuješ (senior, učení, úpravy apd.). Dle všeho je to proto, že v tom nejsou schopní / nechtěji vidět práci, ale jen něco luxusního, což je právo pouze těch nahoře.

Napsal ItSnowsInHellAgain;1476897

Vypozoroval jsem, že v ČR dost osob chce platit pouze za čístý vývoj a nikoliv za věci, které popisuješ (senior, učení, úpravy apd.). Dle všeho je to proto, že v tom nejsou schopní / nechtěji vidět práci, ale jen něco luxusního, což je právo pouze těch nahoře.

Nerekl bych, ze je to jen v CR :) Teda pokud nejde o klienty, kteri maji firmy se ziskem $500k+ rocne nebo startupy, ktere chteji vyzdimat rozpocet investoru.

Napsal Oleg;1476902

Nerekl bych, ze je to jen v CR :) Teda pokud nejde o klienty, kteri maji firmy se ziskem $500k+ rocne nebo startupy, ktere chteji vyzdimat rozpocet investoru.

Toto jsou jedny z mnoha rozhodujících maličkosti, zda projekt bude úspěšně dokončen nebo ne.

Napsal TomášX;1476893

sentosa: nesouhlasím :), spousta open source vývojářů jsou freelanceři, kteří pracují v homeoffice, openBSD takhle žije 20 let, linux Kernel opět rozvíjí převážně lidé ze svých domovů a drivery poskytují firmy z kanceláří. Chápu, ale že to bereš z pohledu webového vývoje malých projektů v ČR, kde by to při troše "štěstí" tak mohlo být

No to bude tech 10% schopnych a zodpovednych. Vem si kolik lidi dnes programuje. To co popisujes je ta hrstka "vyvolenych"... No a sel by takovy clovek delat tuctovou zakazku za par supu?Jinak strasne tu resite code standardy, ale to je ta posledni vec, ktera by mela programatora trapit - dnesni IDEcka to zvladnou preformatovat raz dva. Nejvetsi problem je design - kdyz zdanlive jednoducha uprava nuti k velkemu refaktoringu, nebo rozbije X featur. Nechapu, jak dnes nekdo muze vysilovat nad tim, ze je zavorka na stejnem nebo dalsim radku, nebo jestli odsazuje tabama ci mezerama, ale to uz je krapet mimo tema.

jedina spravna cesta pre teba je bud:1 mat zaplatenu firmu ktora sa ti stara o vyvoj a udrzbu2 mat interny tym ktory ti vyvoj zastresi3 mat jedneho cloveka -architekta pre body 2 a 3 nasledne plati: ze vzdy je tam minimalne 1 architekt ktory sa stara o navrh a kontroluje implementaciu a robi code review + zauca novych clenov tymu, dava im guildline ako co implementovat.bohuzial toto stoji peniaze ale ak sa to vsetko dodrzuje tak ako ma, tak v konecnom dosledku sa peniaze setria tym ze sa veci nemusia milion krat prerabat, novy ludia skor pochopia o com projekt je, lahsie sa implementuju nove funkcionality - usetreny cas = mensie naklady = vacsi potencialny zisk.

Nejsem tedy odborník, ale měl jsem za to, že kromě nějakých validatoru hlidající syntax si bez pochopeni kódu ověřím velké nic. Nicméně je fakt, že php používají patlalove a ne programátoři, takže už to by bylo pro mě asi znamení...

Napsal Panxdz;1476945

Nejsem tedy odborník, ale měl jsem za to, že kromě nějakých validatoru hlidající syntax si bez pochopeni kódu ověřím velké nic.

Nicméně je fakt, že php používají patlalove a ne programátoři, takže už to by bylo pro mě asi znamení...

Mlceti zlato, vid?Kdyz ma 150 projektu na PHP, tak koho ma hledat, Javistu?