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

06:25:28VINO SĂ STĂM SUB POM, SAU - FORMULA DE POLITEȚE —» Leo Butnaru
13:13:345G în avioane, Wi-Fi pe șosele – decizia Comisiei Europene deschide noi oportunități de inovare —» Elena Robu
12:12:29ACUM ORI NICIODATĂ. Cartoful fierbinte pe nume Unirea —» Nicolae Federiuc
15:52:04Jesus’ first miracle —» Erik and Elena Brewer's Weblog
11:25:30AZI ȘI DE-A PURURI! —» Leo Butnaru
07:06:26Фотографы —» Бессарабские хроники
06:44:20RELAȚIA CU UN BĂRBAT NARCIS —» Lifestyle
15:11:57God’s Promises for Difficult Days —» Erik and Elena Brewer's Weblog
14:56:21Idealul unității naționale - 1 decembrie 1918: Expoziție virtuală tematică —» BiblioCity
12:14:58Rezultatele Evaluării externe periodice 2021-2022 —» Blogul elevilor din satul Chetrosu
06:23:51POEME, CĂRȚI PRIN ANI... —» Leo Butnaru
15:24:40Prima întâlnire cu un scriitor —» BiblioCity
11:21:17MIȘCĂRILE SUBIECTULUI —» Leo Butnaru
10:00:48Pâinea noastră cea de toate zilele —» Moldova Creștină
10:00:21Să îmi iubesc dușmanul | Editorial Radu Blendarencu —» Moldova Creștină
08:38:02Scriitorul Volodymyr Vakulenko a fost ucis. Trupul său a fost găsit la Izium —» nimic nu e asa cum pare a fi...
08:00:152 NUVELETE. MISTERIOASELE TRANSFORMĂRI DIN... EL, ÎN... EA... —» Leo Butnaru
06:01:15DUPĂ AUSCHWITZ POEZIA A FOST ȘI ESTE OBLIGATORIE —» Leo Butnaru
19:06:54Omonimele – cum le deosebim sensul? —» Moldova Creștină
18:47:23RomStoria: Noi nu suntem țigani! —» Curaj.TV | Media alternativă
17:10:59Au lăsat mașina în intersecție ca să-și ia de mîncare! —» Curaj.TV | Media alternativă
12:50:17Nu-și mai roade unghiile! Cum am rezolvat problema —» Sunt MAMĂ!
11:10:30Jurnaliștii comunitari au participat la Gala Jurnalistului Comunitar —» BPR Ungheni's Blog
08:01:35Сказочная палитра Бессарабии на полотнах Михаила Пейкова —» BiblioCity
06:43:44Să păzim poruncile lui Dumnezeu şi, cu dânsele, sufletele noastre! —» Portalul Tineretului Ortodox din Moldova