✉️ Saņem šito visu e-pastā. Tā vietā, lai palaistu garām kaut ko no tā, ko es rakstu savā blogā, tagad vari pierakstīties un saņemt e-pastā visu, ko es te rakstu. Tas nav bieži.

The lifely WTF

2011. gada 5. aprīlī, 33 komentāri

Laiku pa laikam manās rokās nonāk kāds PHP projektiņš, kuram vajag pieķibināt "a" vai salabot "b". Pieredze rāda, ka vislielākās šausmas un sliktus sapņus rada kā reizi legacy projekti, kurus kāds par mazām naudām noalgots students programmētājs - mīlētājs ir veidojis pa naktīm. Skatījos pirmās koda lapas un parāva nostaļģija - kaut ko līdzīgu es rakstīju tālajā 2000/2001. gadā. Bet tā bija tikai mirkļa vājība, jo patiesībā tik traki man nav bijis.

Sāksim ar labo. Cilvēks caurmērā ir bijis pieklājīgs. Konsekventi visos SQL vaicājumos Select tiek rakstīts ar lielo burtu. Tas pats ir arī ar parasti par HTML vienpatni un atstumto uzskatīto <Br> Tas arī ir ar pirmo lielo.

Kodā, kā jau nojautāt, ir diezgan daudz SQL vaicājumu. Lielākā daļa no tiem ir vienā rindiņā. Pat tad, ja tā garums ir 400 simboli. Tukšņus un tabulācijas taustiņus, pie tam, viņš ir rūpīgi taupījis. Tas ir secināms ne tikai no tā, ka pirms un aiz operatoriem, piešķiršanas (=), aiz komatiem, un citur nepastāv nekādas tukšuma pazīmes. Kodā nav novērojamas arī atkāpju paliekas. Pat sestā līmeņa foreach cikls sākas ar kreiso malu.

Turpinājumā pats sākums. Man bija nepieciešams noskaidrot, kā saite no sērijas “index.php?&s=1&s2=2&s3=3&s4=4” tiek apstrādāta, lai gala rezultātā man parādītu vajadzīgo sadaļu. Sākumā tiek savāktas vērtības (ar kļūdu apstrādes atslēgšanu!) @$s = $_GET['s']. Pēc tam pēc dažādām ifošanām (if (!$s1) $s1 = 15; if (!$s2) $s2 = $s1;) un maldīšanās pa sešām dažādām inklūdēm, tiek noskaidrots, ka galu galā svarīgākais notiek pa vidu, kur $q1 = $s2; $q2 = $s1;, kas pēc tam tiek include("mod/$s1.php").

Taču, vissmalkākais OMGWTF gājiens mani gaidīja iekš “mysql.php”. Te es iemetīšu daļu no koda, jo tas nav izstāstāms. Formatējums ir saglabāts.

ob_start();
$db =&DB::connect($DSN,
array('debug'=> 2,'portability' => DB_PORTABILITY_ALL));
if (PEAR::isError($sysdb)) {    $error=$db ->getMessage(); } else {
$db->setFetchMode(DB_FETCHMODE_ASSOC);
$db->query("Set names 'utf8';");}
echo "<script>alert('@$error');";
if (!@$error) {ob_end_clean; } else {
ob_end_flush();}

Un, saprotiet, viss kods šādā stilā. Projekts nav maziņš (bet nav arī nekāds lielais). Projektam ir pat dokumentācija, kas patiesībā ir lietotāja rokasgrāmata, un kuras autors nav koda autors. .

Vēl jāpiebilst, ka visa tā padarīšana ir kodēta ar Zend Guard. Velti tērēta nauda. To kodu tāpat nav iespējams saprast.

Tīri tehniski tas viss projekts darbojas nu jau sešus gadus. To diendienā aktīvi lieto normāla čupiņa ar cilvēkiem. Tas liecina, ka viss nema nav tik ļauni, kā varētu šķist no koda apskates. Diemžēl, neskatoties uz to, ka piekļuve ir tikai no uzņēmuma iekšpuses, aplikācijas autorizācija darbojas ar veco labo ' or 1 = 1 or '...

Es pats esmu rakstījis šausmu lietas savās dienās. Un, domāju (ceru), ka šo projektu rakstījušais (vai rakstījušie) pašlaik to dara kvalitatīvi, pārdomāti un saprotami. Uzrakstīju šo ierakstu nevis ar mērķi paņirgāties par kāda spējām vai to trūkumu, bet gan tāpēc, ka tik daudz WTFs/min man nebija bijuši nekad. Un galu galā, mēs visi esam kādreiz sākuši. Un mums patīk laiku pa laikam par to pareflektēt.

Tu atbildi augstāk redzamajam komentāram. Atcelt

Kā vēlies sevi dēvēt? Šo ieraksti. Epasts. Nav obligāts. Citiem netiks rādīts. Komentārs. Saki visu, kas uz sirds, bet esi klājīgs. Formatēšanai vari izmantot markdown. Saņemt turpmākos šī raksta komentārus norādītajā epastā. Atcerēties vārdu un epastu no tava patreizējā pārlūka.

Endijs Lisovskis

2011. gada 5. aprīlī, plkst. 01:42

Paldies par ierakstu! Ja esam pret sevi un citiem godīgi, tad stipri lielākā daļa savus pirmos kodus (un pat ne pirmos) esam sākuši rakstīt WTF manierē. Tajos kodos ir salikts miksējums no "nedaudz iemācījos" + "nedaudz copy & paste no citu skriptiem bez sajēgas kā tas īsti strādā" + "daudz murgu". Līdz netiekam līdz brīdim, kad esam redzējuši daudzus citus (turklāt smukus kodus), īsti uz skaistu risinājumu cerēt nevajag. Piemēram, es pat jau esot ar it kā zināmu pieredzi rakstīju "brīnumus", kad tiku pie tādas sarežģītības uzdevumiem, kuri iepriekš nav veikti. T.i. - programmētājs var pāris gadus var darīt vienveidīgu darbu, bet tad tikt pie tāda koncepta, kurš viņa prātam ir jauns. Un tad pat programmētājs it kā ar pieredzi sataisa tādus brīnumus kā iesācējs. Protams, ar laiku jau tas uzlabojas un var tikt galā ar teju jebkādas sarežģītības uzdevumu. Taču paradokss - pēc pāris gadiem atskatoties atkal liksies, ka varēja taču labāk. :)

btw - runājot par formatējumu - normālas IDEs māk taču forši kodu pašas saformatēt. Bet miskējumi ar mainīgajiem ir jautri. Tādos brīžos nāk talkā refactoring + debugger. Laiks gan tiek zaudēts pamatīgs, bet citreiz šādos gadījumos citādāk nemaz nevar. Kas attiecas uz to MySQL gabalu - esmu pat gatavs saderēt uz vienu Tērvetes alu, ka to autors bija paņēmis no kāda gatava skripta (vai pamācības) un tad licis visur kur vajag un nevajag.

Endijs Lisovskis

Endijs Lisovskis

BigUgga

faq

jo

laacz Autors

Ingus

Papuass

laacz Autors

vbz

laacz Autors

Juris

vbz

Fricis

laacz Autors

faq

Interesents

laacz Autors

kt315b

e-remit

Dindālis

djhurio

j

j

xxx2

ragulops

laacz Autors

BlackHalt

HIGH-Zen

GG

Mr.Key

Zorge

Zorge