[C++] error met cout-> bug?

namnatulco · **05-08-2005, 10:46**

ik heb een probleem:
ik heb een programma geschreven (de hele versie staat hier)
en nu blijkt: als ik em run, loopt ie hier fout:

Code:

cout << "you see a goblin! it is attacking you!";

(dit stuk code staat bijna aan het eind)

het programma geeft gewoonweg niets weer.
ik heb ongeveer hetzelfde probleem net opgelost, maar toen stond dit in een while-lus, waar ik volgens mij een eeuwige lus heb gemaakt, en ik heb de while-lus weggehaald, en toen werkte het wel, maar dit keer heb ik geen lus gebruikt.
help!

Engadin · **05-08-2005, 11:16**

Waarom hij het niet doet heb ik nog niet gekeken, maar wat mij wel meteen opvalt is dat je op regel 71 kijkt of end=0, wat volgensmij end==0 moet zijn.

Was dit programma ooit een basic progje ofzo, vanwege al die goto's?

namnatulco · **05-08-2005, 11:36**

Citaat:

Engadin schreef op 05-08-2005 @ 12:16 :
Waarom hij het niet doet heb ik nog niet gekeken, maar wat mij wel meteen opvalt is dat je op regel 71 kijkt of end=0, wat volgensmij end==0 moet zijn.

Was dit programma ooit een basic progje ofzo, vanwege al die goto's?

die goto's zijn omdat het met while-lussen niet werkte, en ik zo snel ff niets anders kon bedenken. wat je zegt over regel 71 kan wel kloppen, maar ik ben nu ff druk met backups maken

dragonstorm · **05-08-2005, 12:08**

een paar dingen:

1): Je licentie
Ik ben bang dat je huidige licentie zo waterdicht is als een zwembroek. Het beste kun je een licentie van opensource.org gebruiken. Het ziet eruit alsof je een BSD/MIT licentie zou willen.

2): Ik ga geen programma's met goto's debuggen
3): doe aan indentatie, serieus

dragonstorm · **05-08-2005, 12:22**

oh ja
heeeel, heeel vaak kun je een do-while loop gebruiken ipv een goto

edit:

en gebruik in vredesnaam structs

**05-08-2005, 13:20**

^^ with dragonstorm, en nu we toch bezig zijn, bij je switchcases heb je hardcoded case 1: case2: etc... staan. Dit is niet echt handig als je over een tijdje weer je code bekijkt. Gebruik constanten hiervoor.

GVR · **05-08-2005, 14:45**

Citaat:

namnatulco schreef op 05-08-2005 @ 11:46 :

Code:

cout << "you see a goblin! it is attacking you!\n";

linebuffered

vbmot · **05-08-2005, 14:49**

Het meeste is al gezegd, maar ik heb ook nog een toevoeging. Als je hiermee verder gaat, maak dan eerst een degelijk OO ontwerp. Zomaar beginnen met coden is leuk als je gaat scripten, maar (geavanceerde) RPG's kun je haast niet procedureel schrijven.

GVR · **05-08-2005, 15:35**

Citaat:

namnatulco schreef op 05-08-2005 @ 12:36 :
die goto's zijn omdat het met while-lussen niet werkte, en ik zo snel ff niets anders kon bedenken. wat je zegt over regel 71 kan wel kloppen, maar ik ben nu ff druk met backups maken

Je while loops werken niet omdat:

= is toewijzen (altijd waar)
== is vergelijken

en je zet ; achter je while terwijl dat denk niet je bedoeling is...

Verder, haal die comments uit je functie aanroepen. Als ik wil weten wat
die functie doet ga ik er wel heen.

En lees eens boek of ga andermans sources eens bekijken, daar kan je een hoop van leren.

namnatulco · **05-08-2005, 19:50**

ok bedankt voor de tips iedereen, ik ga er druk mee bezig.

@vbmot ik had gewoon zin om zoiets te maken, ik heb er zeker wel veel van geleerd, wat toch eigelijk het belangrijkste doel is.

edit:
om terug te komen op die licenties, ik wil graag dat iedereen mijn source mag lezen/aanpassen, zolang ze mijn source bijvoegen als ze het verkopen/publiceren. hoe heet deze licentie? ik ben namelijk niet zo thuis in al dat licentie-gedoe.

dragonstorm · **05-08-2005, 21:15**

de GNU GPL.

05-08-2005, 11:16
Engadin	Waarom hij het niet doet heb ik nog niet gekeken, maar wat mij wel meteen opvalt is dat je op regel 71 kijkt of end=0, wat volgensmij end==0 moet zijn. Was dit programma ooit een basic progje ofzo, vanwege al die goto's? __________________ Jongeren - Natuur: http://www.njn.nl      Kom mee op zomerkamp: http://www.zomerkampen.njn.nl

05-08-2005, 12:08
dragonstorm	een paar dingen: 1): Je licentie Ik ben bang dat je huidige licentie zo waterdicht is als een zwembroek. Het beste kun je een licentie van opensource.org gebruiken. Het ziet eruit alsof je een BSD/MIT licentie zou willen. 2): Ik ga geen programma's met goto's debuggen 3): doe aan indentatie, serieus

05-08-2005, 12:22
dragonstorm	oh ja heeeel, heeel vaak kun je een do-while loop gebruiken ipv een goto edit: en gebruik in vredesnaam structs Laatst gewijzigd op 05-08-2005 om 12:24.

05-08-2005, 13:20
Verwijderd	^^ with dragonstorm, en nu we toch bezig zijn, bij je switchcases heb je hardcoded case 1: case2: etc... staan. Dit is niet echt handig als je over een tijdje weer je code bekijkt. Gebruik constanten hiervoor.

05-08-2005, 14:49
vbmot	Het meeste is al gezegd, maar ik heb ook nog een toevoeging. Als je hiermee verder gaat, maak dan eerst een degelijk OO ontwerp. Zomaar beginnen met coden is leuk als je gaat scripten, maar (geavanceerde) RPG's kun je haast niet procedureel schrijven. __________________ With everyone unique, you can't be perfect because I already am.

Advertentie