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ă. 


Sursa
2013-10-26 22:06:42



Comenteaza





Ultimele 25 posturi adăugate

11:14:40Living Room Mats —» Andrei Fornea
10:49:36Magia ştiinţei —» Biblioteca Hristo Botev
09:20:46Prefab Garages Nh —» Andrei Fornea
06:41:42Strada copilăriei mele a primit numele bunicii – Taisa Primac —» Andrei Albu - omul alb cu gînduri negre
05:03:06Valorile familiei abordate cu elevii Centrului de Excelență în Construcții —» Tineretul Ortodox
01:17:15Clipe vazute la Forumul MASS-MEDIA 2019 si relatate numai la Curaj.TV —» Curaj.TV | Media alternativă
22:23:29Amman in imagini —» Viata pe Tinder – Viata amoroasa pe Tinder
21:41:38Pe cât de important poate fi adevărul? —» Victor Timotin | creează, inovează, dezvoltă și fii exemplu
20:03:40Je ne veux pas gâcher ma vie en t'aimant mal. —» turn up the silence
19:25:00Balul bobocilor ( cl. V-VI ) —» Liceul Teoretic “Mihail Sadoveanu”, Călăraşi
19:24:00Cea mai mare victorie a FMF: Lecții de fotbal în școli —» Sandu GRECU
19:23:08Ucraina în limba română 28: Bugetul 2020 – priorități sociale, de apărare și diplomație —» Curaj.TV | Media alternativă
18:15:44Lipsa creșterii spirituale te siacalifica de la slujirea in Evanghelie —» Moldova Creștină
16:59:01Imagini apocaliptice: 90% din Veneţia, înghiţită de apă de până la doi metri —» Elena Robu
16:35:20Am susținut un briefing în cadrul căruia am comunicat decizia oficială —» Igor Dodon
16:12:13eBook Cartier. Top vânzări octombrie 2019 —» Gheorghe Erizanu
16:09:30Ion Chicu, premierul desemnat de Igor Dodon —» Elena Robu
15:42:57©️ Învață să fii fericit aici și acum… —» Licurici de suflet
15:09:47Vor discotecă în sat, dar distrug scaunele —» Curaj.TV | Media alternativă
15:01:26Essencial Home Office Interior Design Tips —» ajna-blogging-press
14:59:29Amman —» Viata pe Tinder – Viata amoroasa pe Tinder
14:06:50Am avut discuții separate cu toate fracțiunile din Parlamentul Republicii Moldova —» Igor Dodon
12:04:34Învățarea alfabetului la preșcolari: căutăm literele cu pensula —» Planeta Mami
11:58:19Light Up Air Jordans —» Andrei Fornea
11:40:08Majoritatea critică socialiștii, dar tot aceștia câștigă alegerile. Cum e posibil? —» Elena Robu