245494 cazuri COVID-19 în Moldova
9856 – cazuri active
230090 – cazuri vindecate
5548 – cazuri fatale
Actualizarea datelor: 18 aprilie 2021 ora 07:20


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

21:06:21Primarul din Căzănești te bate dacă filmezi. Polițiste în acțiune! —» Curaj.TV | Media alternativă
20:11:27Poliţist blajin fără mască, călare pe motor, defila prin zona Obor —» Curaj.TV | Media alternativă
21:15:08What’s New in the Block Editor: New Page Layout Picker, Better Block Transformation Options, and More —» costica
19:59:00Despre limitele/pericolele libertății de exprimare #palavre —» Curaj.TV | Media alternativă
08:22:14Ce semnifică Termenul Creativitate ? —» Biblioteca de Arte 'Tudor Arghezi'
20:22:00Inspecție civică la Ciulnița (2 p) —» Curaj.TV | Media alternativă
19:53:37Introducing Milestone Notifications —» costica
19:01:05”Cei care așteaptă rușii...”(parodie)* —» Andrei LANGA. Blogul personal
15:24:37Conducerea Ministerului Sănătății a fost demisă. E o reacție rapidă a mafiei? —» Curaj.TV | Media alternativă
14:55:00Poliția nu poartă centura decît cu mască —» Curaj.TV | Media alternativă
14:44:00Despre fanteziile sexuale —» Curaj.TV | Media alternativă
14:27:00Paznicul fără mască se caută la prohab cînd îi arăt svastica —» Curaj.TV | Media alternativă
13:40:26Телятина (говядина) с овощами —» XStrafer's WebPlace : unconformable point of view.
13:36:21Ucraina în limba română: De ce Rusia insistă să sfideze Ucraina? —» Curaj.TV | Media alternativă
05:38:47Reguli pentru elevi ( participrea la activitățile on-line) —» Liceul Teoretic “Mihail Sadoveanu”, Călăraşi
04:09:51Astăzi, consiliera prezidențială nemerenco, îl va face pe „otmazcikul” său Oleg Lozan, șef pe sănătate —» un alt blog
16:59:56Inspecție civică la Bilciurești! Geană și activistul au pus poliția la treabă —» Curaj.TV | Media alternativă
15:06:39Certificatul digital de vaccinare ar putea intra în vigoare la sfârșitul lunii iunie. În ce scopuri va fi folosit —» Elena Robu
09:21:11Bilanț AFP: Peste un milion de oameni au murit în Europa din cauza COVID-19 —» Elena Robu
07:03:12Vin pentru înălțime —» Fine Wine
06:05:20La mulți ani – sora noastră Claudia! —» Blog de biblioteconomie şi ştiinţa informării
20:30:11Specialiștii psihologi în continuă creștere profesională —» Asociaţia Obştească "Demos"
17:49:42©️ Dacă iubirea… —» Licurici de suflet
15:36:47Norocosul Gagarin —» Gheorghe Erizanu
14:27:4450 documente falsificate, identificate în zilele de weekend de către Poliția de Frontieră —» Elena Robu