[C++] Methoden geven foute resultaten

[Darius]

Greetings,

Heb volgende programma geschreven voor opdracht op school maar ben volgens mij iets stoms vergeten.

deBestelling.geefDoorWit() en deBestelling.geefDoorBruin() geven niet de juiste waardes weer.

Kan wel goed met C om maar mijn C++ is nog niet zo goed. Anyway, het programma :

===============

#include "Bestelling.h"
#include "Bon.h"
#include <iostream>
using namespace std;

void main() {
Bestelling bestelling1(5,1.00,4.50,5,0.90,4.00);
Bestelling bestelling2(10,1.05,7.50,4,1.10,3.40);
Bon bon1(bestelling1), bon2(bestelling2);

bestelling1.getAantal();
bon1.printBon();
cout << endl << "Druk op enter om te stoppen";
while (cin.get() != 'd')
cin.get();
}

===========================

//---------------------------------------------------------------------------

#ifndef BestellingH
#define BestellingH
//---------------------------------------------------------------------------

class Bestelling {

private:
int aantalWit;
int aantalBruin;
int aantalWitKorting;
int aantalBruinKorting;
double prijsBruin;
double prijsWit;
double prijs5Wit;
double prijs5Bruin;
public:
Bestelling(int aantalWitKorting, double prijsWit, double prijs5Wit, int aantalBruinKorting, double prijsBruin, double prijs5Bruin);
getAantal();
double bereken();
int geefDoorBruin();
int geefDoorWit();
};


#endif

=======================

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Bestelling.h"
#include <iostream.h>

//---------------------------------------------------------------------------

#pragma package(smart_init)

Bestelling::Bestelling(int aantalWitKorting, double prijsWit, double prijs5Wit, int aantalBruinKorting, double prijsBruin, double prijs5Bruin)
{
this -> aantalWitKorting = aantalWitKorting;
this -> prijsWit = prijsWit;
this -> prijs5Wit = prijs5Wit;
this -> aantalBruinKorting = aantalBruinKorting;
this -> prijsBruin = prijsBruin;
this -> prijs5Bruin = prijs5Bruin;
}

Bestelling::getAantal()
{
cout << "Hoeveel Witte broodjes wilt U ?";
cin >> aantalWit;
cout << endl << "Hoeveel Bruine broodjes wilt U ?";
cin >> aantalBruin;
}

double Bestelling::bereken()
{ int tempWit, tempBruin, restWit, restBruin;
double totaalWit, totaalBruin;
tempWit = aantalWit%aantalWitKorting;
restWit = aantalWit-tempWit;
totaalWit = tempWit*prijsWit + restWit/5*prijs5Wit;
tempBruin = aantalBruin%aantalBruinKorting;
restBruin = aantalBruin-tempBruin;
totaalBruin = tempBruin*prijsBruin + restBruin/5*prijs5Bruin;
return (totaalWit+totaalBruin);
}

int Bestelling::geefDoorWit()
{
cout << "Beta " << aantalWit;
return aantalWit;
}

int Bestelling::geefDoorBruin()
{
return aantalBruin;
}

========================

#include "Bestelling.h"
//---------------------------------------------------------------------------

#ifndef BonH
#define BonH
//---------------------------------------------------------------------------

class Bon {

private:
Bestelling deBestelling;
public:
Bon(Bestelling deBestelling);
printBon();
};

#endif

================================

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Bon.h"
#include <iostream.h>

//---------------------------------------------------------------------------

#pragma package(smart_init)

Bon::Bon(Bestelling deBestelling):deBestelling(deBestelling)
{}

Bon::printBon()
{
cout << "Aantal Wit: " << deBestelling.geefDoorWit() << endl;
cout << "Aantal Bruin: " << deBestelling.geefDoorBruin() << endl;
cout << "Totaal : " << deBestelling.bereken() << endl;
}

==============================

Weet zeker dat het iets dom is. Wie kan mij helpen ? (PLEEEASE)

[.oisyn]

DUIDELIJKE topictitel aub

[ACM]

Ik heb je topictitel maar even verbeterd "Iets stoms" vond ik een beetje stom.

En formuleer je vraag ook even een stuk duidelijker ajb.

[whoami]

Wat geven ze dan weer en wat moeten ze teruggeven?

[.oisyn]

het zit m denk ik hier

code:

Bon::Bon (Bestelling deBestelling) : deBestelling(deBestelling)
{}

hier gebruik je een ambigue naam, want deBestelling is zowel een classmember als een functie parameter

dus even de naampjes veranderen, bijvoorbeeld

code:

Bon::Bon (Bestelling bestelling) : deBestestelling (bestelling)
{
}

een kleine toelichting: wat dus bij je eigen code gebeurt is je initialiseert de functieparameter met zichzelf, waardoor de classmember deBestelling ongeinitializeert blijft (of tenminste, de default constructor wordt aangeroepen), waardoor de waarden die je later gebruikt niet kloppen

[Darius]

Op het laatst drukt ie het volgende af :

Aantal Witte Broodjes : 0
Aantal Bruine Broodjes : 0
Totaal : 0

Ik heb het verander maar ik krijg nog steeds dezelfde waarde. Als ik de functies geefDoorWit/Bruin onder main{} worden wel de goeie waardes doorgegeven. Ik gebruik Borland Cbuilder5. Weet iemand wat het nog meer zou kunnen zijn ?

BTW Sorry voor de titel, zal het nooit meer doen
BTW++ Bedankt voor de replies. Ik weet dat dit allemaal saai en flauw is.

[.oisyn]

effe de code gekoppiepeest naar mijn visual C++, en ik heb de fout ontdekt

Je maakt eerst een bon van de bestelling
daarna vraag je de aantallen. Daar wordt dus alleen bestelling1 gewijzigd, maar bon1 niet, want die heeft een kopie van de bestelling toen je m maakte

Wat jij waarschijnlijk wilt is dat die Bon eigenlijk linkt naar de juiste bestelling ipv dat ie zijn eigen kopie heeft. Dan moet je het op gaan lossen met pointers:

code:

class Bon
{   
private:
    Bestelling * deBestelling;
public:
    Bon (Bestelling * deBestelling);
    printBon();
};

Bon::Bon (Bestelling * pBestelling) : deBestelling (pBestelling)
{
}

Bon::printBon()
{
    cout << "Aantal Wit: " << deBestelling->geefDoorWit() << endl;
    cout << "Aantal Bruin: " << deBestelling->geefDoorBruin() << endl;
    cout << "Totaal : " << deBestelling->bereken() << endl;
}

en natuurlijk niet dat regeltje in je main veranderen in

code:

    Bon bon1 (&bestelling1), bon2 (&bestelling2);

Ik hoop dat dit duidelijk is (je kent C, dus ik neem aan dat je ook weet hoe pointers werken)

[Darius]

Hij doet het. Thanks Wou het zoveel mogelijk Object georienteerd doen, maar pointers roelen toch.

[.oisyn]

pointers doen niets af aan OO hoor

[Korben]

Op dinsdag 27 november 2001 01:43 schreef OiSyN het volgende:
pointers doen niets af aan OO hoor

Yup.

* Korben wijst naar sig...

[MSalters]

Op maandag 26 november 2001 22:37 schreef OiSyN het volgende:
het zit m denk ik hier

code:

Bon::Bon (Bestelling deBestelling) : deBestelling(deBestelling)
{}

hier gebruik je een ambigue naam, want deBestelling is zowel een classmember als een functie parameter

Geen enkele "deBestelling" is ambigue. De eerste is de
parameternaam, daar zijn de members niet in scope. De tweede, achter de :, slaat altijd op een member. De derde is het meest ingewikkeld, maar dit staat duidelijk in de standaard. Method parameters verbergen members, dus deBestelling slaat in het laatste geval op de parameter, zoals bedoeld.

Dit is een standaard truc, als je geen zin hebt om parameternamen te verzinnen.

[.oisyn]

hmmm dat 2e wat je zei wist ik niet, ik gebruik altijd gewoon een andere naam om onduidelijkheden te voorkomen

(dat de parameter declaratie zelf niet ambigue is is natuurlijk logisch)

[Darius]

Dus met andere woorden (zoals ik het begrepen heb) is dat hetzelfde als

this deBestelling -> deBestelling