O mică analiză, a unei prostii mari

Astăzi voi scrie despre o experiență a anui programator. Ideea este că la pe mîinele mele a căzut un proiect, să îi spunem că îi popular căci 1000-2000 de utilizatori zilnici este o cifră bună.
Azi voi vorbi doar despre cod, știți deja că sunt o persoană care îmi place să critic fără ”să mă uit la nasul meu”, astăzi asta o voi face, puțină analiză a unui proiect, web proiect.

Din păcate am fost nevoie să șterg / să cenzurez unele secțiuni de cod din motive de securitate și de păstrare a numelui curat a site-ului.

Deci preîntîmpin că proiectul nu este realizat pe un framework și celor cu sistemul nervos sensibil rog să nu citească articolul în continuare.

1. Totul se începe de la routing, de core nici nu vorbesc că nu a fost găsit, deci routingul, a fost fatal:
 
Și așa ifocheisuri sunt mii, și asta e trist, cred că știi de ce.

2. Urmează lucru cu templates, și aici fatal, există un fișier cu html ce este în calitate de layout și în el este undeva un echo la o variabilă.


Deci iesă că în acestă variabilă trebuie să transmitem tot blocul rezultat de modulul care este inclus cu ifocheisu de mai sus, în rezultat primim așa variante:

Ce evident nu este comod.

3. Programatorul precedent era preocupat de performanță, da nu nafig, de unde performanță, fie, măcar avem funcție de făcut loguri :D

păcat că comentată la moment, ah da, proiectul este in production. Imediat urmează:

Hmmm, bine, doar cu o excepție, ”они повсиуду”:

Și eu nu vreu să comentez sensul la asta.

4. Nu am spus nimic de XXX din denumirea funcției precedente, voi încă nu ați văzut:

Evident creatorul nu este inferent de numărul de X-uri.

5. Dacă tot e vorba de performanță programatorul ne uimește cu niște update-uri la cod cu cifre senzaționale în comentarii
 
Asta schimbă tot, eu mor, cum naiba o interogare să se execute 30s, îmi inchipui ce era acolo înainte de miracolul care se execută 3 secunde, blin, da nui nimic că 3 sec tot îi prea mult?
A, nu, tăt normal:


 6. Am mai promis ifocheisuri, poftim:
 
Important că aici numai variabilele se schimbă, sql-ul nu, paprobui acuma modifică un cîmp din select.

7. Încă o minune:

Domnul a realizat o paginare de minune, o algoritmică cosmică.

Eu numai pot, sunt prea multe exemple, e plictisitor. Unica interesant mi-a părut că în proiect sunt mai multe rînduri de cod comentate decît decomentate, И это хорошо, nu trebuie să se execute așa prostii, pentru binele tuturor.

Hai să facem o concluzie, evident pentru dezvoltarea ulterioară a acestui proiect trebuie de șters tot. În urma acestui articol nu am dorit să jignesc pe nimeni, din acest motiv nu postez cine e programatorul și ce proiect este, e o simplă analiză, poate pe cineva îl va amuza poate pe altcineva îl va învăța cum nu se poate. 

PS. Nici nu am pomenit că în tot proiectul se utilizează mysql_* care nu este recomandata nici sub o formă, dar asta sunt fleacuri pe lîngă restul ce este și ce s-a enumerat. Baftă.