[vmchecker-dev] Roadmap scripted_checker

[Szekeres Adriana]

Ceau,

Multumesc pentru review. :)
 
OK. O sa-l aranjez acuma.

Adriana

--- On Wed, 7/16/08, Lucian Adrian Grijincu <lucian.grijincu at gmail.com> wrote:

> From: Lucian Adrian Grijincu <lucian.grijincu at gmail.com>
> Subject: Re: [vmchecker-dev] Roadmap scripted_checker
> To: vmchecker-dev at lists.rosedu.org
> Date: Wednesday, July 16, 2008, 3:07 AM
> Salut Adriana,
> 2008/7/13 Szekeres Adriana <aaa_sz at yahoo.com>:
> > Astept code-review  :)
> 
> Am trei comentarii mari și late:
> 1) big-ass long function: sparge-o în mai multe
> funcțioare.
> 2) valori hardcodate ("./file.zip" & co.)
> 3) folosești ceva de genu:
> 
> 	s->home[s->len]='\0';
> 	strcat(s->home,tests);
> 	do stuff(			s->home)
> 	........
> 	s->home[s->len]='\0';
> 	strcat(s->home,compiler);
> 	do other stuff(			s->home)
> refolosirea lui s->home nu arată prea frumos și nici
> nu prea are nici
> o legătură cu numele "home".
> Premature optimisation is teh root of all that is evil
> & kinky.
> 
> Dacă nu vrei să treci pe Python, cred că ai putea și ar
> fi mai bine să
> treci pe C++ și să folosești clasa standard de stringuri
> și excepții.
> 
> Tu de ex. ai limite la dimensiunile numelor fișierelor (de
> ex. #define
> MAX 20). Not nice. În loc să faci implementările astea
> manuale ai
> putea să faci
>    "str += tests;" sau
>    "str += compiler;"
> 
> 
> -- 
> Lucian
> _______________________________________________
> vmchecker-dev mailing list
> vmchecker-dev at lists.rosedu.org
> http://lists.rosedu.org/cgi-bin/mailman/listinfo/vmchecker-dev