Discussion :

[moineauacoustic]

Bonjour,

J'ai encore un souci pour faire des constructeurs multiple, où un constructeur avec moins de paramètre appel un autre avec plus de paramètres ...

Voici ce que donne mon code (vous voyez l'erreur ?!) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
DigitalPin::DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW) :
_device(device), _pin(pin), _mode(mode), _level(defaultlevel)
{
 
  PinMode changedMode = _device.digitalPinMode(_pin, _mode);
  if (changedMode != MODE_ERROR) {
 
    write(_level);
  } 
  else {
 
    _mode = MODE_UNDEFINED;
    _level = LEVEL_UNDEFINED;
  }
}
 
DigitalPin::DigitalPin(Device & device, PinNumber pin, PinMode mode) : DigitalPin(device, pin, mode, LEVEL_LOW)
{
  //DigitalPin(device, pin, mode, LEVEL_LOW);
}
 
DigitalPin::DigitalPin(Device & device, PinNumber pin) : DigitalPin(device, pin, MODE_OUTPUT)
{
  //DigitalPin(device, pin, MODE_OUTPUT);
}

et comme erreur j'ai :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
DigitalPin.cpp: In constructor 'DigitalPin::DigitalPin(Device&, PinNumber, PinMode)':
DigitalPin.cpp:20: error: type 'DigitalPin' is not a direct base of 'DigitalPin'
DigitalPin.cpp:20: error: uninitialized reference member 'DigitalPin::_device'
DigitalPin.cpp: In constructor 'DigitalPin::DigitalPin(Device&, PinNumber)':
DigitalPin.cpp:25: error: type 'DigitalPin' is not a direct base of 'DigitalPin'
DigitalPin.cpp:25: error: uninitialized reference member 'DigitalPin::_device'

[Médinoc]

Il me semble qu'en C++11 et supérieur, c'est possible; mais pas en C++03 ou inférieur.

[moineauacoustic]

Arf

Du coup il y quoi comme alternative "propre" pour faire ça sans dupliquer du code ?
Genre une méthode init(...) avec paramètres par défaut ? (d'ailleurs ça fonctionne les paramètres pas défaut avec du C++03)

[Médinoc]

Tu peux mettre des paramètres par défaut au constructeur aussi.

[moineauacoustic]

J'viens de tester, alors j'ai mis comme ça :

dans le .h (si je met pas aussi les valeur par défaut dans le .h j'ai encore plus d'erreur que ci dessous)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW);

dans le .cpp

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
DigitalPin::DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW) :
_device(device), _pin(pin), _mode(mode), _level(defaultlevel)
{
 
  PinMode changedMode = _device.digitalPinMode(_pin, _mode);
  if (changedMode != MODE_ERROR) {
 
    write(_level);
  } 
  else {
 
    _mode = MODE_UNDEFINED;
    _level = LEVEL_UNDEFINED;
  }
}

... mais j'ai toujours des erreurs :/

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
Arduino : 1.5.7 (Mac OS X), Carte : "Arduino Uno"
 
DigitalPin.cpp:4:117: error: default argument given for parameter 3 of 'DigitalPin::DigitalPin(Device&, PinNumber, PinMode, PinLevel)' [-fpermissive]
 DigitalPin::DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW) :
                                                                                                                     ^
In file included from DigitalPin.cpp:2:0:
DigitalPin.h:15:5: error: after previous specification in 'DigitalPin::DigitalPin(Device&, PinNumber, PinMode, PinLevel)' [-fpermissive]
     DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW);
     ^
DigitalPin.cpp:4:117: error: default argument given for parameter 4 of 'DigitalPin::DigitalPin(Device&, PinNumber, PinMode, PinLevel)' [-fpermissive]
 DigitalPin::DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW) :
                                                                                                                     ^
In file included from DigitalPin.cpp:2:0:
DigitalPin.h:15:5: error: after previous specification in 'DigitalPin::DigitalPin(Device&, PinNumber, PinMode, PinLevel)' [-fpermissive]
     DigitalPin(Device & device, PinNumber pin, PinMode mode = MODE_OUTPUT, PinLevel defaultlevel = LEVEL_LOW);
     ^

[JolyLoic]

Met la valeur par défaut de l'argument dans le .h, et pas dans le .cpp, et ça devrait aller mieux.

[moineauacoustic]

Met la valeur par défaut de l'argument dans le .h, et pas dans le .cpp, et ça devrait aller mieux.

Oh si c'est beau !!! ça marche !
Merci, j'étais en train de chercher en changeant un peu tout mais pas cette solution (c'était trop simple pour que je trouve)

... le C++ ne veux décidément pas de moi !