c++ OOP probleem

[McFreak]

[huiswerk]
http://home.hetnet.nl/~freakangel/c++.rar

Ik ben voor het eerst bezig met c++ proggie, maar ik kom er gewoon echt niet uit, UTFS gebruikt.

Als ik in de mainfile de comment weghaal van de MPVMobile dan zegt ie dat ik een class redefinieer ??
ik snap het niet helemaal, kan iemand mij een duw in de goede richting geven aub ?

Ik heb hiero het boek van bjarne stroustrup, the c++ language in het engels en hiermee kom ik er echt niet uit ..

[Orphix]

Je hebt een grotere kans dat mensen ernaar gaan kijken als je de .cpp en .h even online zet ipv een .rar bestand/

[McFreak]

code:

// ----------------mainfile.cpp-------------
#include <stdlib.h>
#include <string>
#include "Stationwagonmobile.cpp"
//#include "MPVMobile.cpp"     <-- als ik hier de comment weghaal, doet ie het niet meer , wat te doen ??
//#include "Hatchbackmobile.cpp"
using namespace std;

int main()
{
 Stationwagonmobile Stationwagonmobile;
 //MPVMobile MPVMobile       <-- idem


 
 cout <<"\n\n\n";
 system("pause");
}

// ----------------------Stationwagonmobile.cpp --------------
#include "auto.cpp"
class Stationwagonmobile : public Auto
{
public:
Stationwagonmobile();
~Stationwagonmobile();  
}; 

Stationwagonmobile::Stationwagonmobile()
{
cout << "\nEr is een Stationwagonmobile ..";
}

Stationwagonmobile::~Stationwagonmobile()
{
cout << "\nEr is geen Stationwagonmobile meer ..";
}

// -----------------------auto.cpp ---------------------
#include <string>
#include "voertuig.cpp"
#include "statusymbool.cpp"
#include "transportsoort.cpp"
#include <iostream>



class Auto : public Voertuig, public Statusymbool, public Transportsoort
{
  public:
  Auto();
  Auto (int lengte, int breedte, int hoogte, int deuren, string merk, int motor);
  ~Auto();
    int GetAantalDeuren ();
    int GetLengte ();
    int GetBreedte ();
    int GetHoogte ();
    string GetMerk ();
    int GetMotor ();

  
  protected:  
    void SetAantalDeuren (int aantaldeuren);
    void SetLengte (int lengte);
    void SetBreedte (int breedte);
    void SetHoogte (int hoogte);
    void SetMerk (string merk);
    void SetMotor (int motor);  

  private:  
    int aantaldeuren;
    int lengte;
    int breedte;
    int hoogte;
    string merk;
    int motor;

};


Auto::Auto()
{
cout<<"\nEr is een Auto ..";
}

Auto::~Auto(){}

int Auto::GetAantalDeuren ()
{
return aantaldeuren;
}

void Auto::SetAantalDeuren (int d)
{
aantaldeuren=d;
cout << "\nAantal deuren:\t"<<aantaldeuren;
}

int Auto::GetLengte ()
{
return lengte;
}

void Auto::SetLengte (int l)
{
lengte=l;
cout << "\nLengte:\t"<<lengte;
}

int Auto::GetBreedte ()
{
return breedte;
}

void Auto::SetBreedte (int b)
{
breedte=b;
cout << "\nBreedte:\t"<<breedte;
}

int Auto::GetHoogte ()
{
return hoogte;
}

void Auto::SetHoogte (int h)
{
hoogte=h;
cout << "\nHoogte:\t"<<hoogte;
}

string Auto::GetMerk ()
{
return merk;
}

void Auto::SetMerk (string m)
{
merk=m;
cout << "\nMerknaam:\t"<<merk;
}

int Auto::GetMotor ()
{
return motor;
}

void Auto::SetMotor (int m)
{
motor=m;
cout << "\nMotor:\t"<<motor;
}

// ---------------------voertuig.cpp -----------------
#include <string>
class Voertuig
{
public:
Voertuig() ;
~Voertuig();
void rijden();
void stoppen();
};

Voertuig::Voertuig()
{
cout << "\nEr is een Voertuig ..";
}

Voertuig::~Voertuig()
{
}

void Voertuig::rijden ()
{
cout << "\nvoertuig rijdt ";
}

void Voertuig::stoppen ()
{
cout << "\nvoertuig is gestopt";
}

Veel te veel tekst vandaar die rar..
anyway, als ik de comments in de mainfile voor de include MPVMobile weghaal begint DEV-C++ 4.01 te zeiken over een redefinition van voertuig..

[marcusk]

je moet je klassen eerst in een .h bestand declareren.

code:

class MPVMobile : public Auto
{
public:
     MPVMobile();
     ~MPVMobile();
     int GetPersons();    

private:
      int persons;
      void SetPersons(int);    
};

dat gedeelte moet je dus in een .h file zetten, en dan moet er nog wat code omheen:

code:

#ifndef _MPVMobile_h_
#define _MPVMobile_h_

(hier je zooi)

#endif

die .h include je vervolgens, ipv de .cpp

bovenaan mpvmobile.cpp moet je em ook includen.

zelfde met de andere cpp files.

[McFreak]

ok ga ik proberen...tnx

[McFreak]

code:

//-------------------------------------mainfile.cpp
#include <stdlib.h>
#include <string>
#include "Stationwagonmobile.h"
#include "MPVMobile.h"  
using namespace std;

int main()
{
 Stationwagonmobile stationwagonmobile;
 MPVMobile mpvmobile;
 
 cout <<"\n\n\n";
 system("pause");

}



//---------------------------------MPVmobile.h
#ifndef _MPVMobile_h_
#define _MPVMobile_h_

#include "auto.h"
#include <string>

class MPVMobile : public Auto
{
public:
     MPVMobile();
     ~MPVMobile();
     int GetPersons();  

private:
    int persons;
    void SetPersons(int);    
};

#endif
}



//------------------------------MPVmobile.cpp
#include "auto.cpp"
#include <string>

class MPVMobile : public Auto
{
public:
     MPVMobile();
     ~MPVMobile();
     int GetPersons();  

private:
    int persons;
    void SetPersons(int);    
};

MPVMobile::MPVMobile()
{
cout << "\nEr is een MPVMobile ..";
SetAantalDeuren(6);
SetLengte(10);
SetBreedte(3);
SetHoogte(2);
SetMerk("Grover");
SetMotor(4000);
SetPersons(10);
}


MPVMobile::~MPVMobile()
{
cout << "\nEr was een MPVMobile .. ";
}

int MPVMobile::GetPersons()
{
return persons;
}

void MPVMobile::SetPersons(int p)
{
persons=p;
cout << "\nAantal personen:\t"<<persons;
}




//----------------------------------------------auto.h
#include <string>
#include "voertuig.cpp"
#include "statusymbool.cpp"
#include "transportsoort.cpp"
#include <iostream>



class Auto : public Voertuig, public Statusymbool, public Transportsoort
{
  public:
  Auto();
  Auto (int lengte, int breedte, int hoogte, int deuren, string merk, int motor);
  ~Auto();
    int GetAantalDeuren ();
    int GetLengte ();
    int GetBreedte ();
    int GetHoogte ();
    string GetMerk ();
    int GetMotor ();

  
  protected:  
    void SetAantalDeuren (int aantaldeuren);
    void SetLengte (int lengte);
    void SetBreedte (int breedte);
    void SetHoogte (int hoogte);
    void SetMerk (string merk);
    void SetMotor (int motor);

  

  private:
    
    int aantaldeuren;
    int lengte;
    int breedte;
    int hoogte;
    string merk;
    int motor;

};

//-------------------------------------auto.cpp
#include "auto.h"


Auto::Auto()
{
cout<<"\nEr is een Auto ..";
}

Auto::~Auto(){}

int Auto::GetAantalDeuren ()
{
return aantaldeuren;
}

void Auto::SetAantalDeuren (int d)
{
aantaldeuren=d;
cout << "\nAantal deuren:\t"<<aantaldeuren;
}

int Auto::GetLengte ()
{
return lengte;
}

void Auto::SetLengte (int l)
{
lengte=l;
cout << "\nLengte:\t"<<lengte;
}

int Auto::GetBreedte ()
{
return breedte;
}

void Auto::SetBreedte (int b)
{
breedte=b;
cout << "\nBreedte:\t"<<breedte;
}

int Auto::GetHoogte ()
{
return hoogte;
}

void Auto::SetHoogte (int h)
{
hoogte=h;
cout << "\nHoogte:\t"<<hoogte;
}

string Auto::GetMerk ()
{
return merk;
}

void Auto::SetMerk (string m)
{
merk=m;
cout << "\nMerknaam:\t"<<merk;
}

int Auto::GetMotor ()
{
return motor;
}

void Auto::SetMotor (int m)
{
motor=m;
cout << "\nMotor:\t"<<motor;
}

Ik heb alles nu verandert maar hij zegt mooi precies hetzelfde..
Fucking redefinition, moet ik misschien met pointers enzo werken.. dat hoeft toch eigenlijk niet perse ?

[Pooh]

Gooi alle code die werkt er eens uit...

code:

Stationwagonmobile stationwagonmobile;
MPVMobile mpvmobile;

Dit is niet zo slim. Normaalgesproken doe je zoiets:

StationWagonMobile* aCar;
MPVMobile* aNotherCar;

Waarna je ze aanmaakt met:

aCar = new StationWagonMobile;

etc...

Dezelfde naam gebruiken voor de class en een instance is niet handig, zelfs niet als ze qua case verschillen.

[Sjaaky]

code:

class MPVMobile : public Auto
{
public:
     MPVMobile();
     ~MPVMobile();
     int GetPersons();    

private:
      int persons;
      void SetPersons(int);    
};

Dit gedeelte moet weg uit MPVMobile.cpp
en daar moet je een include voor in de plaats zetten.
Bij auto doe je het op het eerste gezicht wel goed.

edit:
en bij de #include moet "auto.cpp", "auto.h" worden.
Hij mag denk ik ook helemaal weggelaten worden, omdat hij ook in MPVMobile.h geinclude wordt.

[Orphix]

Op maandag 14 januari 2002 17:40 schreef Poohbear het volgende:
Dit is niet zo slim. Normaalgesproken doe je zoiets:

Is niks mis mee hoor. Hoef je niet met pointers te werken, kan je niet vergeten het geheugen op te ruimen. Dynamisch alloceren is handig bij grote of van tevoren onbekende aantallen objecten.

[.oisyn]

Op maandag 14 januari 2002 17:49 schreef Orphix het volgende:

[..]

Is niks mis mee hoor. Hoef je niet met pointers te werken, kan je niet vergeten het geheugen op te ruimen. Dynamisch alloceren is handig bij grote of van tevoren onbekende aantallen objecten.

ik denk dat ie doelde op het feit dat ze dezelfde naam hadden (alleen andere case), en niet dat ie geen pointers gebruikt

(PS: ik werkt zelf zo min mogelijk met pointers, en ik vind het jammer dat je in java geen objecten op de stack kunt aanmaken en ja ik weet dat dat nogal moeilijk gaat met de manier waarop java in elkaar zit )

[MSalters]

Op maandag 14 januari 2002 17:40 schreef Poohbear het volgende:
Gooi alle code die werkt er eens uit...

code:

Stationwagonmobile stationwagonmobile;
MPVMobile mpvmobile;

Dit is niet zo slim. Normaalgesproken doe je zoiets:

StationWagonMobile* aCar;
MPVMobile* aNotherCar;

Waarna je ze aanmaakt met:

aCar = new StationWagonMobile;

"niet zo slim". Waarom denk je dat?

Ik weet de volgende redenen waarom je het NIET moet doen:
* Memory leaks mogelijk
* new kan exception geven - en wat moet je dan?
* inefficient - stack geheugen is sneller (cached)

Maar ik zou iets kunnen missen ?

edit:

Misschien dubbelop, maar ik miste in 't andere commentaar de redenen waarom de stack beter is

[Verwijderd]

Niet beter, maar ook niet slechter maar in sommige situaties handiger:

code:

int main(){
  Object mijnObject;

  mijnObject.setup();
  return mijnObject.run();
}

[Orphix]

vicz, wat probeer je nu te zeggen
Dat objecten op de stack declararen beter is dan op de heap? Of vind je beide methodes 'evengoed'?

(Ik snap niet helemaal de bedoeling van je stukje code)

[McFreak]

gaste ik kom er niet uit..
ik wil ff het onmogelijke vragen..
Wie kan deze zooi, http://home.hetnet.nl/~freakangel/c++.rar, ff door zijn compiler jagen en mij vertellen wat ik fout doe?

Hij blijft zeggen dat er een redefinition van voertuig is zodra ik

code:

#include MPVMobile

en

code:

#include Stationwagonmobile

allebei in mijn mainfile zet..

code:

#include <stdlib.h>
#include <string>
           
#include "Stationwagonmobile.cpp"
//#include "MPVMobile.h" 
using namespace std;

int main()
{
 Stationwagonmobile stationwagonmobile;
 //MPVMobile *mpvmobile;     
 cout <<"\n\n\n";
 system("pause");
}

[Orphix]

RaR, over al je bestanden mag maar op 1 plek de declaratie staan van een class. Deze staat vrijwel altijd in een header bestand.
Check dus goed in alle bestanden of je op meerdere plekken dezelfde class beschrijft.

Dit:

code:

header1.h:
class Auto { ... };
header2.h:
class Auto { ... };
program.cpp:
#include "header1.h"
#include "header2.h"

Auto mijnAuto;

is dus fout.

[McFreak]

hmm..

[Verwijderd]

Op maandag 14 januari 2002 23:23 schreef Orphix het volgende:
vicz, wat probeer je nu te zeggen
Dat objecten op de stack declararen beter is dan op de heap? Of vind je beide methodes 'evengoed'?

(Ik snap niet helemaal de bedoeling van je stukje code)

Beide evengoed... bij de een moet je wat beter opletten...
met de code bedoelde ik dat het soms wat compacter is als je het op de stack gooit.

[Orphix]

Op maandag 14 januari 2002 23:57 schreef vicz het volgende:
Beide evengoed... bij de een moet je wat beter opletten...
met de code bedoelde ik dat het soms wat compacter is als je het op de stack gooit.

MSalters heeft 3 punten genoemd waarom op de stack beter is (en waar ik het mee eens ben), nu noem jij nog een voordeel. Heb je ook argumenten om het tegenovergestelde te bewijzen?

[Sjaaky]

als in een aantal headerbestanden dit zet:

code:

#include <iostream>
#include <string>
using namespace std;

gaat het al een stuk beter.

* Sjaaky is zo stom geweest om alles te downloaden en uit te vogelen, inmiddels werkt het wel.

[Verwijderd]

Op dinsdag 15 januari 2002 00:13 schreef Orphix het volgende:

[..]
MSalters heeft 3 punten genoemd waarom op de stack beter is (en waar ik het mee eens ben), nu noem jij nog een voordeel.

Ik zie alleen een lijst waarom de heap niet beter is...

de stack
- meestal sneller
- Makkelijker voor de programmeur... automatisch cleanup
Natuurlijk kan je nog steeds problemen krijgen met pointers naar die objecten. Het object is al weg, maar je zit zelf er nog mee te spelen.

extreem voorbeeld...

void getAddress(void * add) {
Object mijnObject;
add = &mijnObject;
}

de heap
- flexibeler...
- volgens mij wordt dit erg expensive:

code:

void func(){
  Object mijnObject();
  func();
}

[marcusk]

Op dinsdag 15 januari 2002 01:40 schreef vicz het volgende:
- volgens mij wordt dit erg expensive:

code:

void func(){
  Object mijnObject();
  func();
}

dit wordt nog meer expensive:

code:

void func()
{
    Object * mijnObject = new Object();
    func();
}

[farlane]

- declareer je classes in een headerbestand
- definieer je classes in een cpp bestand
- do not include cpp bestanden in je source
- include cpp bestanden in je makefile/projectfile whatever..