Differences
This shows you the differences between two versions of the page.
|
so:teme:general [2019/02/24 20:59] mbarbulescu [Recomandări] |
so:teme:general [2020/03/09 09:41] (current) razvan.deaconescu [Listă depunctări] |
||
|---|---|---|---|
| Line 9: | Line 9: | ||
| * Dacă nu sunteți siguri pe modul vostru de indentare, folosiți [[http://www.gnu.org/software/indent/ | indent]] sau [[http://astyle.sourceforge.net/ | astyle]]. | * Dacă nu sunteți siguri pe modul vostru de indentare, folosiți [[http://www.gnu.org/software/indent/ | indent]] sau [[http://astyle.sourceforge.net/ | astyle]]. | ||
| * Pentru cei care folosesc vim, puteți porni de la acest {{ :so:teme:vimrc.zip|vimrc}} | * Pentru cei care folosesc vim, puteți porni de la acest {{ :so:teme:vimrc.zip|vimrc}} | ||
| - | * Creați un README relevant care să nu reproducă enunțul sau comentariile din cod și care să ajute la înțelegerea soluției alese. | + | * Creați un README relevant care să nu reproducă enunțul sau comentariile din cod și care să ajute la înțelegerea soluției alese. Găsiți mai jos în secțiunea README - how to recomandările echipei de SO pentru realizarea README-ului, puteți folosi acestea ca un template când îl scrieți. |
| * Folosiți lista de discuții dacă aveți întrebări referitoare la coding style sau best practices. | * Folosiți lista de discuții dacă aveți întrebări referitoare la coding style sau best practices. | ||
| * Pentru abordări de evitat (bad practices) vă recomandăm lectura articolelor despre [[http://en.wikipedia.org/wiki/Anti-pattern#Software_engineering | Anti-patterns]] și [[https://en.wikipedia.org/wiki/Code_smell | Code smells]]. | * Pentru abordări de evitat (bad practices) vă recomandăm lectura articolelor despre [[http://en.wikipedia.org/wiki/Anti-pattern#Software_engineering | Anti-patterns]] și [[https://en.wikipedia.org/wiki/Code_smell | Code smells]]. | ||
| Line 19: | Line 19: | ||
| ===== Listă depunctări ===== | ===== Listă depunctări ===== | ||
| - | *Temele care nu se compilează sau nu rulează, indiferent de motive, vor primi punctaj 0; folosiți sistemul [[https://elf.cs.pub.ro/vmchecker/ui/ | vmchecker]] pentru a verifica rularea/compilarea corespunzătoare a temelor | ||
| + | * Temele care nu se compilează sau nu rulează, indiferent de motive, vor primi punctaj 0; folosiți sistemul [[https://elf.cs.pub.ro/vmchecker/ui/ | vmchecker]] pentru a verifica rularea/compilarea corespunzătoare a temelor. | ||
| + | * Pe sistemele de operare general purpose de tipul Linux, *OS, Windows, FreeBSD, cu biblioteci standard C complexe, resursele proprii / nepartajate procesului (memorie alocată, fișiere deschise, thread-uri create) sunt eliberate și închise la încheierea procesului (exit). Pe aceste sisteme nu e nevoie să eliberați explicit înainte de exit; pot există medii specifice în care acest lucru nu se întâmplă. Cea mai bună practică (best practice) este să eliberați explicit resursele ca să acoperiți toate situațiile. | ||
| ==== Funcționalitate ==== | ==== Funcționalitate ==== | ||
| *-0.2: accesarea unor zone de memorie nevalide (care nu rezultă în mesaj de eroare de tipul "Segmentation fault"): buffer overflow, lucru neadecvat cu funcții pe șiruri; pentru depistarea accesărilor invalide puteți folosi [[http://ocw.cs.pub.ro/courses/so/laboratoare/laborator-05#valgrind | valgrind]]; | *-0.2: accesarea unor zone de memorie nevalide (care nu rezultă în mesaj de eroare de tipul "Segmentation fault"): buffer overflow, lucru neadecvat cu funcții pe șiruri; pentru depistarea accesărilor invalide puteți folosi [[http://ocw.cs.pub.ro/courses/so/laboratoare/laborator-05#valgrind | valgrind]]; | ||
| Line 71: | Line 72: | ||
| === Documentație === | === Documentație === | ||
| - | *-0.1: surse nesemnate; | ||
| *-0.1: comentarii inadecvate: comentarii nerelevante sau absente; | *-0.1: comentarii inadecvate: comentarii nerelevante sau absente; | ||
| *-0.1: comentarii cu C99 (''%%//%%'' in loc de ''/%%*%% %%*%%/'') | *-0.1: comentarii cu C99 (''%%//%%'' in loc de ''/%%*%% %%*%%/'') | ||
| *-0.2: README necorespunzător: nu se descrie soluția, se indică doar comentarii din sursă, copy paste la enunț sau la comentarii. | *-0.2: README necorespunzător: nu se descrie soluția, se indică doar comentarii din sursă, copy paste la enunț sau la comentarii. | ||
| + | |||
| + | ===== README - how to ===== | ||
| + | |||
| + | === Organizare === | ||
| + | |||
| + | Explicație pentru structura creată (sau soluția de ansamblu aleasă): | ||
| + | |||
| + | **Obligatoriu:** | ||
| + | |||
| + | * De făcut referință la abordarea generală menționată în paragraful de mai sus. Aici se pot băga bucăți de cod/funcții - etc. | ||
| + | * Consideri că tema este utilă? | ||
| + | * Consideri implementarea naivă, eficientă, se putea mai bine? | ||
| + | |||
| + | **Opțional:** | ||
| + | * De menționat cazuri speciale (corner cases), nespecificate în enunț și cum au fost tratate (în special dacă comentariile din cod nu le acoperă) | ||
| + | |||
| + | |||
| + | === Implementare === | ||
| + | |||
| + | * De specificat dacă întregul enunț al temei e implementat sau ce bucăți | ||
| + | * Dacă există funcționalități extra, pe lângă cele din enunț - dați o descriere succintă (maximum 3-4 rânduri/funcționalitate) și motivarea lor (maximum 1-2 fraze) | ||
| + | * De specificat funcționalitățile lipsă din enunț, dar necesare realizării complete a temei (dacă există) și de menționat dacă testele reflectă sau nu acest lucru | ||
| + | * Dificultăți întâmpinate | ||
| + | * Lucruri interesante descoperite pe parcurs | ||
| + | |||
| + | === Cum se compilează și cum se rulează? === | ||
| + | |||
| + | * Explicație, ce biblioteci linkează, cum se face build | ||
| + | * Cum se rulează executabilul, se rulează cu argumente (sau nu) | ||
| + | |||
| + | === Bibliografie === | ||
| + | |||
| + | * Resurse utilizate - toate resursele publice de pe internet/cărți/code snippets, chiar dacă sunt laboratoare de SO | ||
| + | |||
| + | === Git === | ||
| + | |||
| + | Link către repo-ul de git folosit pentru localizarea surselor | ||
| + | |||
| + | === Ce să NU conțină un readme === | ||
| + | |||
| + | * Detalii de implementare despre fiecare funcție/fișier în parte | ||
| + | * Fraze lungi care să ocolească subiectul în cauză | ||
| + | * Răspunsuri și idei neargumentate | ||
| + | * Comentarii copy/paste din cod și TODO-uri | ||
| Line 84: | Line 128: | ||
| <note important> | <note important> | ||
| - | Se preferă abordarea din primul bullet. Dacă un apel low-level de sistem pică este indicată de cele mai multe ori o problemă mai adâncă. | + | Se preferă abordarea din primul bullet (DIE/assert). Dacă un apel low-level de sistem pică este indicată de cele mai multe ori o problemă mai adâncă la nivelul sistemului și preferăm un die general. |
| </note> | </note> | ||
| Line 99: | Line 143: | ||
| char cmd[MAX_CMD_BUF_SIZE]; | char cmd[MAX_CMD_BUF_SIZE]; | ||
| </code> | </code> | ||
| - | + | ||
| * Neînchiderea/necurățarea resurselor folosite: | * Neînchiderea/necurățarea resurselor folosite: | ||
| * Memorie alocată dinamic (cu ''malloc/calloc'') neeliberată folosind ''free'' - se poate verifica utilizând ''valgrind'' | * Memorie alocată dinamic (cu ''malloc/calloc'') neeliberată folosind ''free'' - se poate verifica utilizând ''valgrind'' | ||
| Line 111: | Line 156: | ||
| * implementările funcțiilor de mai sus se fac în sursa ''hashtable.c'' | * implementările funcțiilor de mai sus se fac în sursa ''hashtable.c'' | ||
| * lucrul cu apeluri de sistem (e.g. wrappere) sau macro-uri de assert se pot pune într-un ''util.h'' și ''util.c'' | * lucrul cu apeluri de sistem (e.g. wrappere) sau macro-uri de assert se pot pune într-un ''util.h'' și ''util.c'' | ||
| + | * **NOTĂ:** Funcțiile și variabilele globale interne unui modul (e.g. funcții folosite **doar** în ''list.c'') se declară folosind calificatorul ''static''. | ||
| <note important> | <note important> | ||
