[so] [Tema 2] Nelamurire depunctari

[Emma Mirică]

2015-04-25 10:48 GMT+03:00 Mihai Volmer <mihaivolmer at gmail.com>:

> 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.
>

Buna Mihai,

Nu este problema ca nu ai acelasi tabwidth cu cel care corecteaza tema, ci
nu aveai acelasi tabwidth cu cel din codul existent.
Datorita acestor probleme, am adaugat un exemplu de vimrc la [0]. Puteti
lua acel vimrc si sa-l puneti in ~/.vimrc. Oricum, ma bucur sa aflu ca
folositi vim.


> 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.
>
>
Ok. Voi aplica o depunctare de -0.1 pentru cod comentat.

Emma

[0] http://ocw.cs.pub.ro/courses/so/teme/general?&#recomandari
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://cursuri.cs.pub.ro/pipermail/so/attachments/20150425/e0522315/attachment.html>