🚀 go-pugleaf

On Wed, 05 Dec 2001 21:16:53 +0100, -P!- wrote:

> hallo,
> kan iemand mij vertellen wat er mis is met deze code, ben niet echt
> ervaren met C/C++:

Ik doe mijn best. Kijk onderaan

> #include <string>
> 
> using namespace std;
> 
> void main() {
> 	double dat[29];
> 	double res;
> 	int loop;
> 	res = 0;
> 
> 	dat[0] = 56453578;
[knip>
> 	dat[28] = 54356;
> 	dat[29] = 789552;
> 
> 	for(loop = 0; loop < 30; loop++) {
>         res += (dat[loop] ? (dat[loop] - 50) : 69) ? dat[loop] : 69;
>         while (res > 300000) {
>             res -= dat[loop] ? dat[loop] : 169;
> 	    }
> 	}
> 	cout >> res;
> }
> 
> als ik het probeer te compileren geeft hij de error "fatal error C1189:
> #error :  "eh.h is only for C++!""
> Ik gebruik de C compiler van Visual Studio.

Kan het zijn dat de file waar dit alles insteekt de extensie .c heeft ipv
.cpp? Dan probeert Visual Studio die namelijk te compileren in C-modus
ipv C++, en dat is niet goed want je gebruikt dingen uit de C++ library.

Dat zou het compilatie-probleem al moeten oplossen, maar ik als ik je
code zo eens bekijk heb ik nog wat andere opmerkingen:

- je declareert dat als een array met 29 (double dat[29]), maar je steekt
er 30 in (van 0 tot en met 29). Als je er 30 wil insteken, moet je zo
declareren: double dat[30].

- Je kan dat beter zo declareren en ineens initialiseren:

double dat[] =
{
  56453578, 6543209, // enz. enz.
};

Als je het zo doet, moet je zelfs geen grootte opgeven: dat wordt
automatisch geregeld. Als het niet de bedoeling is de inhoud te
veranderen, zet je er best nog 'const' voor.

- het is een slechte gewoonte 'magic numbers' in je code te gebruiken,
zoals hier bijv. voor de grootte van de array. Beter is werken met
constanten, bijv.
const datsize = 30;
double dat[30];
...
for (int i = 0; i < datsize; i++)
  ...

Of, als je dat declareert en initialiseert zoals hierboven, kan je sizeof
gebruiken:

for (int i = 0; i < sizeof(dat)/sizeof(dat[0]); i++)
  ...

Niet wakker liggen van de performantie daarvan, dat wordt al tijdens
compile-time allemaal berekend en is dus exact even snel.

Ook je andere magic numbers kan je beter vervangen door constanten. Dat
heeft verschillende voordelen: als je later andere waarden zou willen
gebruiken, moet je het slechts op een plaats veranderen. En, zeker zo
belangrijk, het is veel leesbaarder: als je goeie namen kiest voor je
constanten, weet iedereen die je code leest direct wat ze willen zeggen.

- Overdreven gebruik van ? : maakt de code ook nogal onleesbaar. Overweeg
gewoon met if ( ) te werken.

- cout >> res is fout, het is ofwel cout << res ofwel cin >> res. Ik
vermoed dat je het eerste bedoelt.