Suggesties over code

[Verwijderd]

WIl iemand opmerkingen maken over deze code die ik vandaag geschreven heb ( goede dingen en slechte dingen )

hier is de code:
ftp://213.51.164.191:2121/RdObject.h

[Verwijderd]

niemand???
of gebruiken jullie het alleen voor je eigen bestwil (=jatten)

[Verwijderd]

wat voor code is het in gods naam?? beeetje vaag

[marcusk]

of gebruiken jullie het alleen voor je eigen bestwil (=jatten)

ja, echt wel! bedankt man!

code:

// methods
int refcount;

dit is geen methode maar een member var.
het is logischer om dit private te maken trouwens.

[curry684]

Goh die had je beter hiervandaan kunnen rippen

[curry684]

Okee ff doorgekeken, en complimenten met de hopeloze spaghetti... kun je dit echt fatsoenlijk lezen?

Als in de volgende regel vind ik echt geniaal, maar totaal onleesbaar en ondebugbaar:

code:

SmartPointer& operator= (const SmartPointer& p) { return *this = p.t_pointer; }

Leuk dat het werkt maar dit slaat echt als een tang op een varken. Roep dan gewoon als volgt aan:

code:

return operator=(p.t_pointer);

Of nog beter (VEEL beter), maak een protected Assign functie die overal wordt aangeroepen en automagisch je references beheert...

Ennuh ik zou de opzet van die smart pointer zo veranderen dat ie hetzelfde inheritance path volgt als de referenced classes, zodat je feitelijk nog iets aan inheritance hebt (bij jou zal RMyBaseClass = CSomeSuperclass niet werken).

Lees voor de lol dat topic van mij eens door, misschien schrijf ik die tutorial ooit nog wel eens af...

[Verwijderd]

ook wel fijn als die ftp het doet, kan ik ook ff een veeg erover halen

[Verwijderd]

He nog bedankt he
Ik zal maar weer eens wat gaan sleutelen aan mijn code

Okee ff doorgekeken, en complimenten met de hopeloze spaghetti... kun je dit echt fatsoenlijk lezen?

Ja dit kan ik echt goed lezen hoor. Wat vind jij er onduidelijk aan dan???

Ennuh ik zou de opzet van die smart pointer zo veranderen dat ie hetzelfde inheritance path volgt als de referenced classes, zodat je feitelijk nog iets aan inheritance hebt (bij jou zal RMyBaseClass = CSomeSuperclass niet werken).

Kan je dit aub wat nader uitleggen waarom dat niet werkt

[curry684]

Op donderdag 27 september 2001 22:03 schreef Liquidus het volgende:
Ja dit kan ik echt goed lezen hoor. Wat vind jij er onduidelijk aan dan???

Je hebt de server offline, maar ik doelde vooral op de enorme indents, weinig uitlining, geen logische volgorde in opzet (macro's midden tussendoor), en nog wat dingen. Ik kijk het vanavond nog wel een keer structureel door als je dat wil (mail maar).

Kan je dit aub wat nader uitleggen waarom dat niet werkt

Ik zei het fout: die werkt wel. Daarentegen laat je de compiler nodeloos lange omwegen maken om van RSuperClass naar RBaseClass te komen, in bijvoorbeeld:

code:

RBaseClass    Base  = new CBaseClass;
RSuperClass   Super = new CSuperClass;

Base = Super;

Er is geen direct verband tussen classes Base en Super, alleen tussen de pointers waar ze naar wijzen. Ik vraag me af of de compiler dit in iedere situatie kan resolven, en zo ja hoeveel langzamer dit is dan een inherited smart pointer (hij moet dan een omweg maken via implicit conversions). En een langzame smart pointer is wel het laatste dat je wilt....

[curry684]

Op donderdag 27 september 2001 21:16 schreef Liquidus het volgende:
niemand???
of gebruiken jullie het alleen voor je eigen bestwil (=jatten)

Even hierop terugkomen:
- Omhoogschoppen binnen anderhalf uur wordt niet op prijs gesteld.
- Impliceren dat we je code zouden jatten nog minder (moet je eerst met betere code komen sowieso).

[Verwijderd]

Op vrijdag 28 september 2001 09:26 schreef curry684 het volgende:

[..]

Even hierop terugkomen:
- Omhoogschoppen binnen anderhalf uur wordt niet op prijs gesteld.
- Impliceren dat we je code zouden jatten nog minder (moet je eerst met betere code komen sowieso).

Excuse moi voor dit