[so] Depunctare Tema 1

[Adrian Stanciu]

2015-04-01 0:51 GMT+03:00 Codrin-Victor Poienaru via so <so at cursuri.cs.pub.ro>:
> Salut,
>
> In urma corectarii temei, am primit urmatoarele observatii:
>     - leak-uri de memorie
>     - nu sunt verificate toate valorile de retur ale functiilor
> si as dori o reevaluare.

Salut Codrin,

Am reevaluat tema ta.

>
> Pentru leak-uri de memorie, am rulat tema mea manual, pentru fiecare din
> cele 28 de teste publice folosite pentru testare, in valgrind, si se pare ca
> pentru fiecare test aceste leak-uri nu exista defapt. A fost folosit cumva
> alt utilitar pentru detectarea leak-urilor ?

Am folosit tot valgrind, dar am rulat un alt test. Scenariul în care
aveai probleme era următorul:
* adaug un cuvânt
* șterg cuvântul
* îl adaug iar

> Referitor la valorile de retur ale functiilor, primesc o referinta in
> hashtable.c de exemplu, in care mi se spune ca eu nu verific rezultatul
> alocarii hashtable-ului. Verificarea daca s-a reusit sau nu alocarea se
> realizeaza in main. Acolo aloc un hashtable, si daca nu am reusit, afisez un
> mesaj corespunzator si inchei programul (goto exit). Am considerat
> nepotrivit sa afisez mesajul de eroare in aceeasi functie care aloca
> hashtable-ul din considerente de modularizare. In opinia mea, printf-urile
> ar trebui sa se realizeze numai in functia principala (acesta este si modul
> in care am gandit tema, defapt).
>

Am observat acest lucru dar ți-a scăpat un caz: la operația de resize
nu verifici dacă alocarea a reușit. Din acest motiv nu am aplicat
inițial depunctarea completă de 0.2 puncte, ci una mai "blândă" de 0.1
puncte.

Am pus explicații extra pentru cele două depunctări pe vmchecker [1],
sper că e mai clar acum. Te rog să le urmărești acolo. Pentru mine
aceste depunctări sunt corecte astfel încât au rămas în vigoare.

[1] https://vmchecker.cs.pub.ro


Adrian