[MQL4/OOP] Klasse in een klasse initialiseren lukt niet

[KappuhH]

Ik ben na lange tijd weer wat aan het programmeren (in mql4, wat lijkt op C++) en kom niet helemaal uit de volgende case. Ik zal hem iets versimpeld weergeven.

Stel ik heb een fiets en van die fiets wil ik een klasse maken. Dit doe ik met de volgende code:

code:

class Fiets
{
   private:
      int      variableA;
      bool     variableB; 
   public:
      void Fiets(int A, bool B)
      {
         variableA = A;
         variableB = B;
         
      };
      
      void     DoeIets();
};

Die fiets kan iets doen dmv de methode DoeIets.

Nu zitten er wielen aan een fiets en die Dat wiel definieer ik met de wiel klasse:

code:

class Wiel
{
   private:
      int VariableC;
   public:
      void Wiel(int C)
      {
         VariableC = C;
      };
      
      double Draai();
};

Elk wiel heeft dus een methode Draai.

De methoden heb ik uiteraard ook in de code staan:

code:

void Fiets::DoeIets()
{  
    //Doe iets
}

double Wiel::Draai()
{
    return(2.0);
}

Nu wil ik dus dat elke fiets standaard 2 wielen meekrijgt op het moment dat er in mijn programma een Fiets aangemaakt wordt. Ik dacht dit te doen in de klasse Fiets dit op te nemen:

code:

class Fiets
{
   private:
      int variableA;
      bool variableB;
      Wiel wiel1;
      Wiel wiel2;
   public:
      void Fiets(int A, bool B)
      {
         variableA = A;
         variableB = B;
         wiel1 = wiel1(1);
         wiel2 = wiel2(2);
      };
      
      void DoeIets();
};

Zodat ik binnen de methodes van de klasse Fiets altijd een Wiel en zijn functie kan aanroepen met bijvoorbeeld wiel1.Draai(); Het probleem echter is dat ik in de constructor van Fiets geen variabelen aan mag maken van het type Wiel, alleen de standaard int, bool, double, enz. Ik krijg dan de error 'declaration without type'.

Welke fout maak ik hier? Ik wil Wiel wel als klasse hebben omdat ik de optie wil hebben om binnen Fiets nog meer objecten van Wiel aan te maken.

[Merethil]

Normaalgesproken zou ik zeggen dat het moet zijn

code:

wiel1 = new Wiel(1);

Let op het keyword "new".
Dit is echter in de meeste talen zo, weet niet hoe mql4 in elkaar zit verder.

[Cloud]

Ik ken de taal mql4 niet maar heb in een grijs verleden wel wat met C++ gedaan.

Even op de gok en met de aanname dat je alle source van je class hier toont; moet je in je Fiets-class niet aangeven dat je van de Wiel-class gebruik wilt maken? Een #include ofzo? Met andere woorden, is de fout niet zozeer dat je een declaratie doet zonder type maar eigenlijk dat het type wat je gebruikt in de declaratie niet bekend is op die positie?

Of is het juist het ontbreken van een 'new' keyword bij het instantiëren?

[Verwijderd]

Variabele2 moet zijn variabeleB? Wat verwacht je van wiel1 = wiel1(1)?

[ Voor 37% gewijzigd door Verwijderd op 04-09-2018 23:14 ]

[KappuhH]

Verwijderd schreef op dinsdag 4 september 2018 @ 23:11:
Variabele2 moet zijn variabeleB?

Die was ik net aan het corrigeren, foutje uit m'n voorbeeld om het te versimpelen ;-)

Merethil schreef op dinsdag 4 september 2018 @ 23:06:
Normaalgesproken zou ik zeggen dat het moet zijn

code:

wiel1 = new Wiel(1);

Let op het keyword "new".
Dit is echter in de meeste talen zo, weet niet hoe mql4 in elkaar zit verder.

Initiëren met new kom ik verder niet tegen. Hoeft ook niet als ik een klasse Fiets initieer in m'n programma. Het gaat er echt om dat ik hem niet kan aanmaken in het declaratiestuk van Fiets. De error komt dus ook op de regel [code]Wiel wiel1;[code]

Zou dan met een include oid te maken kunnen hebben omdat hij binnen die klasse een Wiel niet kent. Ga ik even uitzoeken.

[Gropah]

Als je je code daadwerkelijk op deze manier hebt, schat ik dat dit ook een probleem is:

code:

wiel1 = wiel1(1);
wiel2 = wiel2(2);

wiel1 en wiel2 zijn geen functie's, methodes of constructors, dus dat gaat niet goed.

Anyways, iemand anders die ongeveer hetzelfde probleem had (volgens mij) heeft het opgelost door een pointer te gebruiken. Niet de netste oplossing, maar als je er daardoor mee verder kan...

edit: overigens is new wel een ding in mql4

[ Voor 15% gewijzigd door Gropah op 04-09-2018 23:34 ]

[Merethil]

KappuhH schreef op dinsdag 4 september 2018 @ 23:14:
[...]
Die was ik net aan het corrigeren, foutje uit m'n voorbeeld om het te versimpelen ;-)
[...]

Initiëren met new kom ik verder niet tegen. Hoeft ook niet als ik een klasse Fiets initieer in m'n programma. Het gaat er echt om dat ik hem niet kan aanmaken in het declaratiestuk van Fiets. De error komt dus ook op de regel [code]Wiel wiel1;[code]

Zou dan met een include oid te maken kunnen hebben omdat hij binnen die klasse een Wiel niet kent. Ga ik even uitzoeken.

Het probleem echter is dat ik in de constructor van Fiets geen variabelen aan mag maken van het type Wiel, alleen de standaard int, bool, double, enz. Ik krijg dan de error 'declaration without type'.

Initiëren met new kom ik verder niet tegen. Hoeft ook niet als ik een klasse Fiets initieer in m'n programma. Het gaat er echt om dat ik hem niet kan aanmaken in het declaratiestuk van Fiets. De error komt dus ook op de regel

code:

Wiel wiel1;

Ah, dat had ik niet door, want de quote uit je TS gaf mij het gevoel dat het juist misging met de constructor.

Hoe dan ook, ik denk dat je inderdaad een include moet doen via https://docs.mql4.com/basis/preprosessor/include

[KappuhH]

Gropah schreef op dinsdag 4 september 2018 @ 23:22:
Als je je code daadwerkelijk op deze manier hebt, schat ik dat dit ook een probleem is:

code:

wiel1 = wiel1(1);
wiel2 = wiel2(2);

wiel1 en wiel2 zijn geen functie's, methodes of constructors, dus dat gaat niet goed.

Anyways, iemand anders die ongeveer hetzelfde probleem had (volgens mij) heeft het opgelost door een pointer te gebruiken. Niet de netste oplossing, maar als je er daardoor mee verder kan...

edit: overigens is new wel een ding in mql4

Lijkt inderdaad dat het met een pointer moet. Kom deze code tegen op een pagina van de mql4 documentatie (https://www.mql5.com/en/docs/basis/types/classes):

code:

class CTetrisField 
  { 
private: 
   int               m_score;                            // Score 
   int               m_ypos;                             // Current position of the figures 
   int               m_field[FIELD_HEIGHT][FIELD_WIDTH]; // Matrix of the well 
   int               m_rows[FIELD_HEIGHT];               // Numbering of the well rows 
   int               m_last_row;                         // Last free row 
   CTetrisShape     *m_shape;                            // Tetris figure 
   bool              m_bover;                            // Game over 
public: 
   void              CTetrisField() { m_shape=NULL; m_bover=false; } 
   void              Init(); 
   void              Deinit(); 
   void              Down(); 
   void              Left(); 
   void              Right(); 
   void              Rotate(); 
   void              Drop(); 
private: 
   void              NewShape(); 
   void              CheckAndDeleteRows(); 
   void              LabelOver(); 
  };

Lijkt er op dat dit precies is wat ik nodig heb. Dacht dit al geprobeerd te hebben maar blijkbaar niet helemaal juist. Zal het vanavond thuis even opnieuw proberen.

[expor]

Ik ben niet thuis in MQL maar als ik zo snel even Google zou de C++ stijl moeten werken:

code:

class Fiets
{
   private:
      int variableA;
      bool variableB;
      Wiel wiel1;
      Wiel wiel2;
   public:
      void Fiets(int A, bool B) :
         variableA(A),
         variableB(B),
         wiel1(1),
         wiel2(2)
      {
      };
      
      void DoeIets();
};

[KappuhH]

expor schreef op woensdag 5 september 2018 @ 11:42:
Ik ben niet thuis in MQL maar als ik zo snel even Google zou de C++ stijl moeten werken:

code:

class Fiets
{
   private:
      int variableA;
      bool variableB;
      Wiel wiel1;
      Wiel wiel2;
   public:
      void Fiets(int A, bool B) :
         variableA(A),
         variableB(B),
         wiel1(1),
         wiel2(2)
      {
      };
      
      void DoeIets();
};

Deze manier geeft op regel 6 een error ivm decleration withouten type.

[Raynman]

Maar misschien komt dat doordat Wiel dan nog niet gedeclareerd is. Zie eerdere opmerking over #include.

[KappuhH]

Raynman schreef op woensdag 5 september 2018 @ 15:33:
Maar misschien komt dat doordat Wiel dan nog niet gedeclareerd is. Zie eerdere opmerking over #include.

Oh wat dom, ik moet de Wiel klasse natuurlijk wel voor de Fiets klasse zetten.

De volgende code lijkt te werken. (geeft in ieder geval geen compilation error, functionaliteit moet ik nog even checken).

code:

class Fiets
{
   private:
      int variableA;
      bool variableB; 
      Wiel *wiel1;
   public:
      void Fiets(int A, bool B) 
      {
         variableA = A;
         variableB = B;
         wiel1 = new Wiel(1);
      };
      void DoeIets();
};

[KappuhH]

Het lijkt allemaal te werken. Dank jullie wel!

[g4wx3]

de implementatie kan volgens mij toch beter, door de wielen mee te geven aan de constructor. Depency injection. Dan kan je ook testen met slechte wielen, of ander soorten wielen, zolang ze maar wiel implementeren.

[dogtagz]

Speaking of constructor, waar is deze?
Het object wordt niet aangemaakt als de klasse wordt aangeroepen. In C++ is dat dus wel nodig. Geen idee hoe dat met mql4 zit?

Een voorbeeld met de fiets klasse met jouw private variabelen en een standaard constructor(C++ stijl).

code:

#include "wiel.hpp"

class Fiets
{
   private:
      int variableA;
      bool variableB; 
      Wiel wiel1;
   public:
      
      Fiets(int A, bool B, Wiel wiel1) :
      variableA  ( A ),
      variableB ( B ),
      Wiel1 ( wiel1),
      { //hier voer ik dus niets uit bij het aanmaken van het object. Wat een gebruikelijke methode is. //}

      void DoeIets();
};

Wat de onderstaande code doet(per notatie even een cijfer toegevoegd).

code:

class Fiets
{
   private:
      int variableA;
      bool variableB; 
      Wiel *wiel1;       #1
   public:
      void Fiets(int A, bool B) 
      {                              #2
         variableA = A;       
         variableB = B;
         wiel1 = new Wiel(1);  #3
      };
      void DoeIets();
};

1. Hierbij gebruik je een pointer naar wiel1. Dit is dus een geheugenadres wat je wilt aanroepen. Heb je uberhaupt wiel wel geinclude dat hij deze kan aanroepen?

2. De krulhaakjes is code wat hij gaat uitvoeren bij de constructor. Is dit wel de manier hoe je een constructor maakt in mql4? Het voorbeeld daarboven staat hoe je een constructor in C++ maakt. Je assigned daar de waardes naar je private variablen die je meegeeft bij het aanroepen van de constructor.

3. Eerst gebruik je een pointer van wiel1, wat dus al aangemaakt moet zijn voordat je de constructor aanroept. Vervolgens moet de klasse fiets, de pointer gebruiken die je niet meegeeft als parameter?


Excuses voor deze late reactie, misschien kan je toch nog wat met deze informatie. Ik denk alleen dat het gebruik van pointers een beetje overbodig is..

[ Voor 6% gewijzigd door dogtagz op 25-09-2018 12:01 ]