🚀 go-pugleaf

Jo Vermeulen wrote:
> Ben een abstracte klasse aan het compileren met Visual C++ maar krijg een
> heel rare error. De fout zit in de constructor volgens VC++.
> Dit is de error:
> 
> interface.h(26) : error C2234: '<Unknown>' : arrays of references are illegal
> 
> Onder g++ compileert het wel.
> Iemand een idee wat ik fout doe?

De foutmelding lijkt me duidelijk: je maakt een array van references aan, en dat
mag niet volgens de C++ standaard. g++ heeft een niet-standaard uitbreiding die
het blijkbaar wel toestaat.

De oorzaak van de fout is mij niet geheel duidelijk, maar zal wel in de definitie
van Dictionary of Dna zitten. Ik ben dus benieuwd naar de definitie van die typen.

Overigens is het slordig om m_pDna, m_pC, m_pJ en m_pV niet te initialiseren
in de constructor.

Ook het gebruik van "using namespace std;" in een header is niet zo handig.
Daardoor, en door het gebruik van de rare naam "stdin" voor een boolean, moest
ik toch even kijken wat er nou met stdin gebeurde en twee keer kijken of er
nou std::stdin of een lokale stdin werd gebruikt. Het laten staan van die "std::"
overal is wel zo duidelijk.

Verder lijkt het erop dat een File en een Dictionary steeds met elkaar verband
houden. Waarom heb je geen aparte klasse voor die combinatie?

Maar laat ik maar ophouden met commentaar op het ontwerp. Ik kan daar nog
wel even mee doorgaan, maar daar zit je waarschijnlijk niet op te wachten.
De code zou in ieder geval, zelfs na bovenstaande correcties, op mijn werk
niet door de code review heenkomen.

Sjoerd Schreuder