Jak zjistit, ze skript byl dobre napsany?

Jak poznate skript od profesionala a od "studenta", pokud vynechame OOP a framework systemy? Citelnost? Komentare? Zbytecnosti? Bezpecnost? Rychlost?

Bez potřeby hlubší znalosti algoritmů a jazyka: První pohled - názvy proměnných, které vyjadřují smysl v terminologii řešeného problému. Konzistentní a čisté formátování kódu poukazuje na logickou konzistenci a čistotu. Třetí pohled u webových aplikací a stránek - zpracování a očištění uživatelského vstupu.

Dáš to sem na zhodnocení a uvidíš.Jinak k dobrému kódu patří určitě čitelnost (názvy funkcí, proměnných) a bezpečnost. OOP a frameworky je vhodné na větší aplikace, takže jak kdy.Rychlost je diskutabilní, záleží na situaci.

ukazka skriptu teda:news.phpinclude("config.php");include("sidebar.php");if(isset($_GET)){$page=ochrana_sql($_GET);$add_title=" - $page";}else{$page=1;$add_title='';}$limit=5;$start = ($page - 1) * $limit;$news=show_news($start,$limit,1,'novinky');$pager=pager("select id from articles where active=1 and article_category=13","/novinky/",$page,$limit,5);for($i=0;$i<count($news);$i++){ $title= $news; $datum=$news.".".changeto_cs_month($news)." ".$news; $nr_comments=$news ; $author=$news; $contentik=$news; $id=$news; $seo_title=$news; //$tags=make_tags($news); $tags=$news; $thumb=$news; $viewed=$news; if($thumb !=''){ $pix=" <a class='tooltip' href='/novinky/$id-$seo_title' title='$title'><img src='$thumb' class='my_newsthumb attachment-feed-vertical wp-post-image' alt='$title'/></a> "; }else{ $pix=""; } $content.="<div class='feed-vertical'> <div class='inner'> <div class='article-image'> $pix </div> <div class='article'> <div class='comments'> <div class='number'> <a href='/novinky/$id-$seo_title#comments'>$nr_comments</a> </div> <div class='label'> Komentáře </div> </div> <!--<a class='category adelle tooltip' href='sss' title='sss'><b>sss</b></a>--> <h2 class='adelle'><a class='post-title' href='/novinky/$id-$seo_title'>$title</a></h2> <div class='desc'> Přidáno $datum uživatelem <a href='#'>$author</a> </div> <div class='excerpt-wrapper'> <div class='excerpt'> <p>$contentik </p> </div> </div> <a class='read-more' href='/novinky/$id-$seo_title'><strong>Celý článek »</strong></a> </div> </div> </div> <br class='clearer'/> ";}$smarty->assign('page_title',"xxx$add_title");$smarty->assign('pager',$pager);$smarty->assign('news_list',$content);$smarty->assign('title',"xxx | $base_title");$smarty->assign('description',"xx.");$smarty->display("news.tpl");sidebar.php<?php//include_once("includy/dbz.php");//include_once("includy/main_funkce.php");$vtip=mysql_fetch_assoc(mysql_query("select content from special_uses where cat_id=4 order by rand() limit 1"));//****recent news$recent_n=show_news(0,7,1,'novinky');for ($i=0;$i<5;$i++){ $recent_news.=" <li> <a href='/novinky/".$recent_n."-".$recent_n."' title='".$recent_n."'>".$recent_n."</a> </li> ";}//****end news//****recent postavy$recent_p=show_news(0,7,1,'postavy');for ($i=0;$i<5;$i++){ $recent_postavy.=" <li> <a href='/postavy/".$recent_p."-".$recent_p."' title='".$recent_p."'>".$recent_p."</a> </li> ";}//****end psotavy//****recent postavy$recent_a=show_news(0,7,1,'anime-a-filmy');for ($i=0;$i<5;$i++){ $recent_anime.=" <li> <a href='/postavy/".$recent_a."-".$recent_a."' title='".$recent_a."'>".$recent_a."</a> </li> ";}//****end psotavy//****new comments//recent comments$sql_comments=mysql_query("select c.article_id,k.seo_category,c.comment,a.seo_title from comments c left join articles a on a.id=c.article_id left join categories k on k.id=a.article_category where c.active='1' order by c.id desc limit 8");while($zaz_comments=mysql_fetch_assoc($sql_comments)){ $zaz_comments=str_replace("</span>","</strong>",$zaz_comments); $zaz_comments=str_replace("<span style="font-weight:bold;">","<strong>",$zaz_comments); $zaz_comments=strip_tags($zaz_comments); $recent_comments.=" <li><a href='/$zaz_comments/$zaz_comments-$zaz_comments#comments'>"$zaz_comments"</a></li> ";}//konec recent comments//**** $smarty->assign('recent_news',$recent_news);$smarty->assign('recent_postavy',$recent_postavy);$smarty->assign('recent_comments',$recent_comments);$smarty->assign('recent_anime',$recent_anime);$smarty->assign('vtip',$vtip);?>config.php<?phpsession_start();include_once("./includy/dbz.php");include_once("./includy/main_funkce.php");require('smarty/Smarty.class.php');//pollif(isset($_GET)){ vote(ochrana_sql($_GET),ochrana_sql($_GET)); }//$cache_dir="./cache";//$cachetime = 60 * 60 * 24; // 1 daydefine('_SITE','');$base_title='xxx.cz';$setup=mysql_fetch_assoc(mysql_query("select * from settings"));//print_r($_GET);$smarty = new Smarty;$smarty->template_dir = "./templates/sdsds/";$smarty->compile_dir = "./templates_c"; $smarty->assign('poll',show_poll());$smarty->assign('keywords',"");?>

To vypadá na nějakou šablonu. SQL, PHP a HTML by od sebe měly být oddělené, tenhle mišmaš se hrozně špatně spravuje. Jinak není moc co hodnotit.

Kód se mi moc nelíbí, míchá dohromady HTML a PHP, ale dokázal bych odhadnout, co by měl kód dělat.

Tohle profesional rozhodne nepsal. Kod je z pohledu programatora dost spatny, kombinuje ceske a anglicke nazvy metod, nelogicky zpusob zapisu, duplikuje promenne atp. Proceduralni zpusob resit nebudu, i kdyz osobne mu nehovim kvuli citelnosti. Plus spousta dalsich chyb, ktere funkcnost aplikace nijak neohrozi, ale ja bych ho nezamestnal ani jako juniora. Tipuju to na 17-18leteho studenta spise prumerneho programatora.---------- Příspěvek doplněn 21.02.2012 v 17:54 ----------Edit: mimochodem proc sakra micha PHP a HTML kdyz pak pouziva Smarty?

Jako lajk to nemáš moc šanci poznat. Jsou jisté věci, které většinou dobří programátoři dodržují, ale nemusí to být pravidlem.1. Bezpečnost a rychlost nemáš šanci poznat vůbec, můžeš jen vycházet z faktu, že lepší programátor bude asi psát i bezpečnější a rychlejší kód. Na druhou stranu rychlost není v tvém případě asi moc podstatná. Pokud budeš dělat web kam bude chodit hodně lidí, tak budeš mít budget na zaplacení dobrého programátora a pokud tam přijde pár lidí, tak se nějaký ten kód navíc ztratí.Jinak na co se zaměřit:- jak již někdo zmínil oddělení HTML a PHP (každé ve vlastním souboru)- objektovost- smysluplné názvy proměnných a funkcí, vše se dnes standardně píše v angličtině.- komentáře kódu- adresář nazvaný "includy" ve mě také nevyvolává pocit profesionálnosti:)- v news ten for cyklus s tím začátečním inicializováním asi 20 proměnných je také zajímavý, ale to jako lajk také nepoznášJinak můžeš také zkusit porovnávání, stáhni si nějaký open-source PHP script (třeba Nette framework www.nette.org) otevři si nějaké PHP a porovnej vizuálně, jak to celé vypadá.Nicméně pokud někomu zaplatíš za podobný skript třeba 100-200Kč, tak nemůžeš čekat, že kód bude kvalitní. Buď musíš vsadit na osvědčené značky, které bývají dražší nebo se budeš potácet s podobnými problémy. Teď mě napadá, že by byla dobrá činnost "najímatel programátoru" což by byl člověk, který se vyzná v oboru a ty si ho najmeš, aby ti vybral kvalitního programátora:) Taková obdoba personální agentury, ale nevím jestli dnešní personálky jsou schopné zajít až tak daleko, že zjistí kvalitu programátora jinak než podle CV.