[so] [Tema 2] Nelamurire depunctari

[Mihai Volmer]

2015-04-25 0:36 GMT+03:00 Emma Mirică <emma.mirica at gmail.com>:
>
> Buna Mihai,
>
> Iata raspunsurile mele inline.
>
> 2015-04-24 20:04 GMT+03:00 Mihai Volmer via so <so at cursuri.cs.pub.ro>:
>>
>> Salut,
>>
>> Am observat ca au fost puse rezultatele la tema 2 si am cateva intrebari in legatura cu urmatoarele depunctari pentru tema de pe Linux:
>>
>> -0.1 tab-uri amestecate cu spații pentru indentare; folosiți fie numai spații, fie numai tab-uri
>> Eu am scris tema in vim si am tab-ul setat sa imi introduca 4 spatii in loc de tab, deci nu aveam cum sa amestec tab-uri cu spatii. Singura sursa de tab-uri ar putea fi scheletul de cod (pe care nu l-am modificat).
>
>
> In tot codul apar si taburi si spatii la indentare. Problema este ca ai luat scheletul de cod care era scris cu taburi (fara expand si are dimensiunea de 8 spatii), iar ce ai adaugat tu este scris cu spatii, iar tabul este setat la 4 spatii. Chiar daca primiti schelet de cod trebuie sa mentineti coding style-ul sau sa aranjati tot codul dupa coding style cu care va simtiti mai confortabil.
> Depunctare aceasta se mentine.
>
>>
>>
>> -0.1 indentare inconsecventă (uneori există, alteori nu, un TAB, mai mult TAB-uri)
>> Aici nu am ce explicatie sa dau (in afara de optiunea auto-indent din vim pe care am folosit-o excesiv). As aprecia daca cineva ar putea sa reverifice aceasta depunctare.
>
>
> Din nou apare o indentare inconsecventa. Se vede in parse_command si nu numai. Probabil din aceeasi cauza, ca nu ai indentat tot codul, inclusiv cel primit. Din nou depunctarea se mentine. Ca sa vedeti mai usor toate aceste probleme folositi in vim "set list listchars=". De asemenea, sugerez folosirea comenzii indent.
>
>>
>>
>> -0.2 cod comentat
>> Am avut o linie de cod comentata in toata tema.
>
>
> Niciodata nu se accepta patch-uri cu cod comentat. Ai cod comentat in utils-lin.c:315 si utils-lin.c:355. Depunctarea ramane.
>
>>
>>
>> -0.1 folosire alocare dinamică pentru o dimensiune statică (cunoscută la compilare) (alocare vector pentru pipe, pentru pids)
>> Am alocat astfel:
>> int (*pipes)[2] = malloc(2 * sizeof(int) * length);
>> pid_t *pids = malloc((length + 1) * sizeof(pid_t));
>> Variabila length este calculata chiar inainte de aceste alocari in functie de numarul de pipe-uri din comanda, deci nu este cunoscuta la compilare.
>
>
> Aici este greseala mea. Voi elimina aceasta depunctare.
>
>>
>>
>> De asemenea, as avea aceleasi intrebari in legatura cu depunctarile pentru tab-uri / indentare inconsecventa / trailing whitespaces si pentru tema de pe Windows.
>
>
> Acelasi lucru si pe Windows: ai taburi amestecate cu spatii, ai trailing whitespaces, ai cod comentat (pe care nu l-am depunctat), indentarea este inconsecventa. Depunctarile raman.
>
>
> Multumesc,
> Emma
>

Buna Emma,

Multumesc pentru raspunsul rapid.

In legatura cu tab-urile: nu am acordat mare atentie tab-urilor din
scheletul de cod deoarece am considerat ca toate aspectele legate de
coding style sunt legate de codul pe care l-am scris eu, nu si de
schelet.
In momentul in care am scris tema nu am observat ca scheletul de cod
are tab-uri in loc de spatii (o sa fiu mai atent data viitoare).
In legatura cu indentarea: aceasta problema apare strict din cauza
amestecarii tab-urilor cu spatii. Chiar si asa, eu, avand tab-ul setat
la 4 spatii, nu am observat probleme de indentare. Consider ca nu este
justificat sa fiu depunctat deoarece nu am acelasi tabwidth cu cel al
persoanei care corecteaza tema.
In legatura cu liniile de cod comentate: este vreo diferenta de
depunctare intre blocuri intregi de cod comentat lasate din lene si 2
linii de cod comentat lasate din neatentie? Inainte sa trimit tema am
inspectat codul si am sters toate liniile comentate, dar pe aceastea
nu le-am observat. Nu consider ca este chiar atat de grav incat sa fiu
depunctat dublu.

Multumesc,
Mihai Volmer