[so] [Tema2] Contestatie

[Mihai Barbulescu]

Salut Dan,

Pe Linux la cd in caz de eroare nu afisezi nimic. M-am uitat din nou
pe Linux si as putea sa iti acord cele -0.2 puncte pierdute. Desi cd e
foarte greu de evaluat in cazul temei tale. Ar fi trebuit poate, din
pacate, -0.1 pentru ne-existenta unui mesaj de eroare/imposibilitatea
verificarii de functionalitate a redirect-ului cd-ului. Desi asta e
doar functionalitate de Bash, nu-s sigur in celelalte shell-uri. Pt ca
e prea tipic am decis sa nu iti scad nimic

Pt variabilele hardcoded cam avea dreptate pe windows. Solutia cu
macro era cea mai buna, cea propusa de tine, intrucat depunctarea nu
se regasea pe Linux am scos-o si de pe Windows

Ai 10 pe ambele teme si karma de la mine


2017-05-01 19:07 GMT+03:00 Dan Ungureanu via so <so at cursuri.cs.pub.ro>:
> Salut,
>
> Doresc recorectarea temei 2 intrucat depunctarile mentionate pe
> VmChecker nu se aplica in cadrul temei mele:
>
>> -0.1: mesajele de eroare nu sunt foarte relevante/ajutatoare la localizarea problemei
>
> Majoritatea acestor mesaje de eroare vin sunt emise de macro-ul DIE().
> Acesta afiseaza fisierul si linia care sunt arhisuficiente pentru
> localizarea problemei. Cu toate acestea, am incercat sa adaug
> comentarii descriptive pentru fiecare aparuta (i.e. "Invalid command."
> in cazul in care era specificata o comanda invalida, "Failed waiting
> for child.", "Could not reallocate memory.", etc.)
>
>> -0.1: nu se face redirectare pentru comanda internă cd
>
> Se face. Unul din teste chiar verifica asta si mai mult, se vede si in
> cod (functia "execute_simple", sectiunea descrisa in comentarii ca
> "Built-ins", mai exact liniile 191 si 193 ale fisierului cmd.c in
> cazul variantei pe Linux, asemanator si pe Windows).
>
>> -0.1: folosire de valori hard-coded, în loc să se definească macrodefiniții (e.g. nr de threaduri)
>
> Singura valoare hard-coded din intreaga tema este "2" care reprezinta
> numarul de capaturi ale unui pipe si numarul de thread-uri necesare
> pentru a executa un pipe. Nu cred ca ar fi ajutat daca as fi folosit
> doua variabile pentru capetele pipe-ului sau daca as fi avut ceva de
> genul int pipefds[MAX_PIPE_ENDS], unde MAX_PIPE_ENDS ar fi 2.
>
> O zi buna,
> Dan Ungureanu
> _______________________________________________
> http://ocw.cs.pub.ro/courses/so/info/lista-discutii



-- 
Cu stimă,
Mihai Bărbulescu