[vmchecker-dev] Roadmap scripted_checker

[Lucian Adrian Grijincu]

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