Constructeur avec paramètre objet par réference, et valeur par défaut

Version imprimable

11/08/2014, 20h48
moineauacoustic

Constructeur avec paramètre objet par réference, et valeur par défaut
Juste une petite question/avis ...

Pour mettre une valeur pas défaut dans un constructeur dans lequel je passe des objets j'ai fais ça, est ce que c'est bien ?
Code:

1 2 MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Counter & counterLeft = *(Counter*)0, Counter & counterRight = *(Counter*)0);
... ça me donne l'impression d'être du "bricolage", donc bon, je préfère être sûr... est ce que c'est bon ? ou pas ?

Bon bah j'ai ma réponse, ça fonctionne pas ma solution ...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Arduino : 1.5.7 (Mac OS X), Carte : "Arduino Uno"
 
kiserary010002.ino:51:105: error: no matching function for call to 'MotorManagementSystem::MotorManagementSystem(Motor&, Motor&, Power, Counter (&)(), Counter (&)())'
kiserary010002.ino:51:105: note: candidates are:
In file included from kiserary010002.ino:12:0:
MotorManagementSystem.h:13:5: note: MotorManagementSystem::MotorManagementSystem(Motor&, Motor&, Power, Counter&, Counter&)
     MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Power power = OFF, Counter & counterLeft = *(Counter*)0, Counter & counterRight = *(Counter*)0);
     ^
MotorManagementSystem.h:13:5: note:   no known conversion for argument 4 from 'Counter()' to 'Counter&'
MotorManagementSystem.h:9:7: note: MotorManagementSystem::MotorManagementSystem(const MotorManagementSystem&)
 class MotorManagementSystem
       ^
MotorManagementSystem.h:9:7: note:   candidate expects 1 argument, 5 provided

Comment faire !? :?

11/08/2014, 21h23
foetus

Pour moi c'est une erreur de logique :aie:
Lorsqu'on préfère une référence (X&) à un pointeur (X*) c'est pour garantir de toujours avoir une valeur concrète.

Une référence nulle :weird: Et une question: que se passe-t-il si tu utilises ta référence nulle?

Bin, c'est un peu spécial de C++ de l'Arduino j'ai l'impression.

Si je met ça :

Code:

1
2
 
MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Power power = OFF, Counter & counterLeft = NULL, Counter & counterRight = null);

... ca marche pas non plus ...

Code:

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
27
28
29
30
31
32
 
Arduino : 1.5.7 (Mac OS X), Carte : "Arduino Uno"
 
In file included from /Applications/Arduino.app/Contents/Java/hardware/tools/avr/avr/include/stdio.h:49:0,
                 from /Applications/Arduino.app/Contents/Java/hardware/arduino/avr/cores/arduino/Print.h:24,
                 from /Applications/Arduino.app/Contents/Java/hardware/arduino/avr/cores/arduino/Stream.h:26,
                 from /Applications/Arduino.app/Contents/Java/hardware/arduino/avr/cores/arduino/HardwareSerial.h:29,
                 from /Applications/Arduino.app/Contents/Java/hardware/arduino/avr/cores/arduino/Arduino.h:221,
                 from Common.h:4,
                 from kiserary010002.ino:2:
MotorManagementSystem.h:13:109: error: could not convert '0' from 'int' to 'Counter&'
     MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Power power = OFF, Counter & counterLeft = NULL, Counter & counterRight = null);
                                                                                                             ^
In file included from kiserary010002.ino:12:0:
MotorManagementSystem.h:13:140: error: 'null' was not declared in this scope
     MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Power power = OFF, Counter & counterLeft = NULL, Counter & counterRight = null);
                                                                                                                                            ^
kiserary010002.ino:51:105: error: no matching function for call to 'MotorManagementSystem::MotorManagementSystem(Motor&, Motor&, Power, Counter (&)(), Counter (&)())'
kiserary010002.ino:51:105: note: candidates are:
In file included from kiserary010002.ino:12:0:
MotorManagementSystem.h:13:5: note: MotorManagementSystem::MotorManagementSystem(Motor&, Motor&, Power, Counter&, Counter&)
     MotorManagementSystem(Motor & motorLeft, Motor & motorRight, Power power = OFF, Counter & counterLeft = NULL, Counter & counterRight = null);
     ^
MotorManagementSystem.h:13:5: note:   no known conversion for argument 4 from 'Counter()' to 'Counter&'
MotorManagementSystem.h:9:7: note: MotorManagementSystem::MotorManagementSystem(const MotorManagementSystem&)
 class MotorManagementSystem
       ^
MotorManagementSystem.h:9:7: note:   candidate expects 1 argument, 5 provided
 
  Ce rapport contiendrait plus d'informations si l'option
  "Montrer les informations de sortie pendant la compilation"
  était activée dans Fichier > Préférences.

11/08/2014, 21h46
foetus

Citation:

Envoyé par moineauacoustic

Bin, c'est un peu spécial de C++ de l'Arduino j'ai l'impression.

Et en quoi? :weird:
Les erreurs sont très claires et en plus tu sembles convertir un pointeur de fonction en référence

Citation:

Envoyé par moineauacoustic

error: could not convert '0' from 'int' to 'Counter&'

Citation:

Envoyé par moineauacoustic

error: 'null' was not declared in this scope

Citation:

Envoyé par moineauacoustic

note: no known conversion for argument 4 from 'Counter()' to 'Counter&'

Pour le pointeur de fonction pourquoi ne pas passer en paramètre une énumération enum { FUNCTION_1 = 0, FUNCTION_2, FUNCTION_3 } et ensuite faire un switch pour appeler la bonne fonction :?
11/08/2014, 21h52
moineauacoustic

C'est un objet "Counter" que je veux passer en paramètre a mon constructeur, mais je veux qu'il soit optionnel et donc mettre un paramètre par défaut à "rien".

Je n'ai pas de fonction ... ou alors j'ai pas compris un truc ?
11/08/2014, 22h01
foetus

Citation:

Envoyé par moineauacoustic

C'est un objet "Counter" que je veux passer en paramètre a mon constructeur, mais je veux qu'il soit optionnel et donc mettre un paramètre par défaut à "rien".

Et comment veut-savoir s'il est passé en paramètre ou pas? :mrgreen:

Citation:

Envoyé par moineauacoustic

Je n'ai pas de fonction ... ou alors j'ai pas compris un truc ?

Et la surcharge cela te parle? :mrgreen:

Citation:

MotorManagementSystem(/* all parameters */);

MotorManagementSystem(/* all parameters */, Counter& c);

Astuce: Dans la liste d'initialisation de ton constructeur qui prend en paramètre ton objet, tu peux appeler ton constructeur qui ne le prend pas :ccool:
11/08/2014, 22h02
Médinoc

Références et paramètres par défaut ne font pas bon ménage: Une référence doit toujours être valide, et l'objet référencé doit toujours avoir une durée de vie supérieure à la référence.
Au pire, tu peux passer un objet compteur global ou static par défaut.
12/08/2014, 11h38
therwald

Logiquement, objet référencé optionnel => pointeur (et test !=nullPtr avant utilisation), non?
Comme le faisait remarquer foetus, une rérérence est censés TOUJOURS exister, non? prendre 0 pour initialiser une référence, c'est passer la corde à l'arbre et s'en faire une cravate, non?
12/08/2014, 19h24
moineauacoustic

Au final j'ai pas trop trouvé de solution technique pour ce que je voulais faire, mais j'ai fait autrement :) ...

Mais sinon, mon idée c'était d'avoir une classe "Controlleur de moteurs" qui prend en paramètre les moteurs, et aussi les compte-tour des roue, et rendre optionnel les comptes tour pour si on moteur (roue) en possède un ou pas. Certaines roue en on un, d'autre pas, et prévoir si j'en rajoute un, etc. ...
12/08/2014, 19h38
foetus

Citation:

Envoyé par moineauacoustic

Au final j'ai pas trop trouvé de solution technique pour ce que je voulais faire, mais j'ai fait autrement :) ...

Mais sinon, mon idée c'était d'avoir une classe "Controlleur de moteurs" qui prend en paramètre les moteurs, et aussi les compte-tour des roue, et rendre optionnel les comptes tour pour si on moteur (roue) en possède un ou pas. Certaines roue en on un, d'autre pas, et prévoir si j'en rajoute un, etc. ...

Et via l’encapsulation, mais il faut voir si cela ne fait pas trop de classes :mrgreen:

Pourquoi ne pas mettre le compte-tour (en utilisant soit la composition soit l'agrégation) dans le moteur et demander au moteur s'il a un compte-tour? :mrgreen:
13/08/2014, 00h00
therwald

moteur avec pointeur sur compte-tour. pointeur nullPtr si pas de compte-tour.
le moteur expose bool hasCompteTour() const qui retourne true si pointeur renseigné, false si nullPtr.
A toi de voir le comportement de double Moteur::nbTours() qui correspond le mieux à ton modèle dans le cas pas de compte-tour (retour NaN?)