C++

Passage d'arguments par défaut dans constructeur

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Passage d'arguments par défaut dans constructeur

KevinTheProud

Insubmersible

Salut,
J'ai un petit problème(pour note je travaille sur le même projet que DeltaDen):
J'ai une classe Gate et une classe dérivée NOT:public Gate
j'ai un constructeur dans Gate qui a 4 arguments. Pour éviter à avoir à taper 4 constructeurs, j'ai mis
Gate::Gate(char name[],int delai=0, int nb=1, bool inv=false ) dans Gate.cpp

maintenant dans la classe NOT j'ai mis:
NOT::NOT(char name[],int delai=0,int nb=1,bool inv=false) : Gate (name,delai,nb,inv)

dans mon main ensuite je crée l'objet g:
Gate *g=new NOT("NONAME",0,1,false);
là ça compile et ça marche sans problème mais si je mets:
Gate *g=new NOT("NONAME" ); il ya erreur de compilation!
alors où me suis-je trompé?

Publicité

skeye

KevinTheProud a écrit :

Je suis pas tout à fait sûr de ce que tu veux faire, mais si j'ai bien compris tu veux que les valeurs de delai, nb et inv soient fixes, pour pouvoir n'appeler le constructeur qu'avec un seul argument...?
Mais dans ce cas quel intérêt d'avoir ces valeurs en paramètre?
Un constructeur ne prenant que name suffirait, non?
En tout cas l'erreur de compilation me parait normale, puisque ton constructeur à 1 seul paramètre n'existe pas...

PS: la question de ton collègue de projet m'a fait faire quelques révisions déjà, on va bien voir si cette fois-ci j'ai tout compris...

theshockwave

I work at a firm named Koslow

Je n'ai jamais été fan des arguments par défaut ...

Si tu es sur que tu vas soit utiliser tous tes args soit n'en utiliser qu'un, autant faire uniquement 2 constructeurs bien distincts ...

et pourquoi pas faire : NOT::NOT(char name[]):NOT(name, 0, 1, false);

Edit ... Faute de frappes ... :whistle:

Message édité par theshockwave le 18-04-2003 à 13:04:39

---------------
last.fm

theshockwave

I work at a firm named Koslow

skeye a écrit :

En l'occurrence, donner des valeurs par défaut a l'effet de créer les constructeurs avec moins de paramètres (en retirant tout ou partie des arguments auxquels on a affecté des valeurs par défaut)

donc si, le constructeur avec 1 seul arg devrait exister ...

---------------
last.fm

skeye

theShOcKwAvE a écrit :

bon, je retourne lire quelques cours moi...:lol:

theshockwave

I work at a firm named Koslow

En fait, si tu pouvais donner le message d'erreur que tu obtiens, ca pourrait peut-être nous aider ...

---------------
last.fm

theshockwave

I work at a firm named Koslow

skeye a écrit :

bon, je retourne lire quelques cours moi...:lol:

:lol:

je ferais bien de faire pareil ! J'ai pas trop souvent mis en pratique cette fonctionnalité et je ne suis pas trop à l'aise avec ...

---------------
last.fm

KevinTheProud

Insubmersible

skeye a écrit :

Je veux avoir des arguments "optionnels", un objet NOT doit au minimum avoir un nom mais j'aimerai aussi pouvoir spécifier-si besoin est-le nombre d'entrées de l'objet(ici int nb). Or avec 4 arguments ça me fait 4 construteurs, je trouve ça assez lourd, j'avais imaginé que les arguments par défaut étaient la meilleure solution...

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

En fait, si tu pouvais donner le message d'erreur que tu obtiens, ca pourrait peut-être nous aider ...

j'obtiens qu'il ne connait pas de constructeur avec seulement un argument:
no matching function for call to 'NOT::NOT(const char[7])'
candidates are : NOT::NOT(const NOT&)
NOT::NOT(char*,int,int,bool)

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

Effectivement, c'est vrai que cette solution me parait tout à fait adaptée pour ton cas ...

Mais ... Je me demande si les constructeurs générés de cette manière sont bien corrects ... Si ca se trouve, il n'arrive pas à faire le lien entre les valeurs par défaut et les variables que tu veux passer au constructeur de la classe mère ...

Avis de spécialiste demandé ...

---------------
last.fm

Publicité

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

:heink:

Ben ... Apparamment, il ne t'a pas généré les autres constructeurs ... C'est la seule chose que je vois ... MAis bon ... Définir tes 4 constructeurs à la main, c'est pas encore la mort ... D'autant plus que^c'est juste pour appeler le constructeur de la classe mère ... A la limite le corps du constructeur, tu le mets dans une fonction privée que tu appelles depuis chaque constructeur ...

---------------
last.fm

skeye

theShOcKwAvE a écrit :

:lol:

je ferais bien de faire pareil ! J'ai pas trop souvent mis en pratique cette fonctionnalité et je ne suis pas trop à l'aise avec ...

je vois rien chez christian casteyde...donc si tu as un meilleur lien hésite pas!

skeye

KevinTheProud a écrit :

Et pour le cas ou tu as aussi besoin de nb tu l'appelles comment?
Gate *g=new NOT("NONAME",nb);
me parait bancal, car aucune manière pour lui de savoir quelle argument représente nb...
p-e
Gate *g=new NOT("NONAME",,nb,);
auquel cas ca donnerait
Gate *g=new NOT("NONAME",,,);
:??:

gloop

skeye a écrit :

non, quand tu veux spécifier un paramètre optionnel, tous les précédents doivent aussi être donnés

> KevinTheProud: avec un const char *, ce serait déjà mieux.

polo021

j'ai peut etre pas tout compris mais pourquoi ne pas faire :
- soit un constructeur par defaut (donc vide) + un constructeur avec 1 param + 1 constructeur avec 4 params
- soit un constructeur par defaut + un constructeur avec des parametres qui ont des valeurs par defaut mon_constructeur(param1 = 0, param2 = "truc", param 3 = 2.3, param 4 = "mais bien sur" )
{
val1 = param1;
val2 = param2;
val3 = param3;
val4 = param4;
}
il me semble, je n'en suis plus du tout sur, que si tu fait
new (4)
ton objet sera initialise avec val1 = 4, val 2 = "truc", val3 = 2.3 et val 4 = "mais bien sur"

Est ce que ca irait pour resoudre ton probleme?

EDIT : en relisant le debut du topic je viens de m'apercevoir que je ma reponse a un peu rien a voir
EDIT 2 : enfin si mais bon je n'ai fait que de repeter ce que les autres ont dit :ange:

Message édité par polo021 le 18-04-2003 à 13:47:15

KevinTheProud

Insubmersible

gloop a écrit :

non, quand tu veux spécifier un paramètre optionnel, tous les précédents doivent aussi être donnés

> KevinTheProud: avec un const char *, ce serait déjà mieux.

ça change rien

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

ça change rien

tu peux même mettre des const partout dans ton entête de constructeur ... Ca ne changera rien au schmilblick ... Simplement, ce sera plus explicite quant à tes intentions ...

Mais en règle générale, on ne modifie pas les valeurs passées dans un constructeur ...

---------------
last.fm

theshockwave

I work at a firm named Koslow

Code :

class Stash {
Stash(int size, int initQuantity = 0);
}
//...
Stash A(100), B(100, 0);

will produce exactly the same results. The identical constructor is called in both cases, but for A, the second argument is automatically substituted by the compiler when it sees the first argument is an int and that there is no second argument. The compiler has seen the default argument, so it knows it can still make the function call if it substitutes this second argument, which is what you?ve told it to do by making it a default.

Bruce Eckel, Thinking in C++ 2nd Edition Volume 1 ...

Ben ... D'après lui, ca devrait marcher ... (puisque ce serait même le même constructeur qui serait appelé ...)
Je ne sais pas si ce bouquin est une bonne référence, mais bon ... C'est le seul que j'aie sous la main à l'instant ...

Message édité par theshockwave le 18-04-2003 à 14:13:44

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

Code :

Stash(int size, int initQuantity = 0);
//...
Stash A(100), B(100, 0);

will produce exactly the same results. The identical constructor is called in both cases, but for A, the second argument is automatically substituted by the compiler when it sees the first argument is an int and that there is no second argument. The compiler has seen the default argument, so it knows it can still make the function call if it substitutes this second argument, which is what you?ve told it to do by making it a default.

Bruce Eckel, Thinking in C++ 2nd Edition Volume 1 ...

Ben ... D'après lui, ca devrait marcher ... (puisque ce serait même le même constructeur qui serait appelé ...)
Je ne sais pas si ce bouquin est une bonne référence, mais bon ... C'est le seul que j'aie sous la main à l'instant ...

Bah en fait moi aussi j'en ai un sous la main,Claude Delannay, 5ème édition, ils mettent en plus un exmple qui marche chez moi

skeye

Le problème viendrait de l'héritage?

PS: je lance des idées, mais si vous voulez que j'arrete dites-le...en tout cas moi j'aime bien votre projet, ca m'apprend des choses...

theshockwave

I work at a firm named Koslow

C'est ce que je supposais depuis le début ...

Je n'arrive pas à trouver d'exemple qui mette en avant les paramètres par défaut d'un constructeur ainsi que l'utilisation d'un constructeur hérité ...

---------------
last.fm

KevinTheProud

Insubmersible

Voici l'exemple de mon bouquin:
#include <iostream>
#include <stdlib.h>

using namespace std;
//Classe point
class point
{
int x,y;
public:
point(int abs=0,int ord=0) //constructeur de point
{cout<<"++ constr. point : "<<abs<<" "<<ord<<"\n";
x=abs; y =ord;
}
~point()
{cout<<"-- destr. point : "<<x<<" "<<y<<"\n";
}
};
class pointcol : public point
{
short couleur;
public:
pointcol(int,int,short);
~pointcol()
{cout<<"-- dest. pointcol - couleur: "<<couleur<<"\n";
}
};
pointcol::pointcol(int abs=0,int ord=0,short cl=1): point (abs,ord)
{cout<<"++constr. pointcol : "<<abs<<" "<<ord<<" "<<cl<<"\n";
couleur=cl;
}
int main()
{pointcol a (10,15,3);
pointcol b (2,3);
pointcol c (12);
pointcol * adr;
adr= new pointcol (12,25);
delete adr;

system("PAUSE" );
return 0;
}

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

Voici l'exemple de mon bouquin:

Code :

#include <iostream>
#include <stdlib.h>
using namespace std;
//Classe point
class point
{
int x,y;
public:
point(int abs=0,int ord=0) //constructeur de point
{
cout<<"++ constr. point : "<<abs<<" "<<ord<<"\n";
x=abs; y =ord;
}
~point()
{
cout<<"-- destr. point : "<<x<<" "<<y<<"\n";
}
};
class pointcol : public point
{
short couleur;
public:
pointcol(int,int,short);
~pointcol()
{
cout<<"-- dest. pointcol - couleur: "<<couleur<<"\n";
}
};
pointcol::pointcol(int abs=0,int ord=0,short cl=1): point (abs,ord)
{
cout<<"++constr. pointcol : "<<abs<<" "<<ord<<" "<<cl<<"\n";
couleur=cl;
}
int main()
{
pointcol a (10,15,3);
pointcol b (2,3);
pointcol c (12);
pointcol * adr;
adr= new pointcol (12,25);
delete adr;
system("PAUSE" );
return 0;
}

Je me permet de faire une petite mise en forme ....

Message édité par theshockwave le 18-04-2003 à 14:34:17

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

Je me permet de faire une petite mise en forme ....

j'ai mis le code plus par intérêt de copier/coller

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

j'ai mis le code plus par intérêt de copier/coller

Oui, mais perso, je préfère lire le code que de l'essayer sans chercher à voir ce que ca fait avant ... D'où la mise en forme ... (en plus, ca permet à ceux qui n'ont pas un compilo directement sous la main de se faire une idée plus facilement ...)

Bon ... Toujours est-il que je ne vois pas pourquoi cet exemple passe et pas ton projet ... En soi, c'est exactement le comportement que tu veux avoir ...

---------------
last.fm

theshockwave

I work at a firm named Koslow

Tes 2 classes de portes logiques ne doivent sans doute pas être monstrueuses ... Tu pourrais poster la déclaration dans son intégralité ?

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

ça fait une journée que je suis dessus :sweat:

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

Tes 2 classes de portes logiques ne doivent sans doute pas être monstrueuses ... Tu pourrais poster la déclaration dans son intégralité ?

bon ok,attention les yeux(monstrueuse?): Gate.h

Code :

/*------------Définition de la classe Gate--------------------------*/
#ifndef Gate_h
#define Gate_h
#include <string>
using namespace std;
class Gate
{
//Déclaration des membres privés
int * input_values;//Tableau d'entiers contenant les entrées
char * Name;//String contenant le nom de la porte
bool * OpenEntries;//Tableau répertoriant les entrées libres
//Tampon
int point;//Itérateur du Buffer
int delay;//Délai interne de la porte(en cycles)
int *Buffer;//Tableau d'entier contenant le tampon, initialisé à 0 de taille delay+1
Gate * GateInTab;//Tableau contenant des pointeurs vers les portes d'entrées
//Fonctionnalités "répertoire des sorties"
bool hasOutput();//renvoie true si une sortie est une sortie circuit
protected:
/*
Gate();
Gate(char *);
Gate(char *,int);
Gate(char *,int,bool);
const char * noname;*/
Gate(char*,int,int,bool);//constructeur permettant de définir le délai à l'instanciation
int entriesNumber;//Nombre d'entrées
int *truth_table;//Table de vérité-tableau de booléens
//Fonctionnalités diverses
void Inverse();
//Déclaration des membres protégés
void EntryInit();//Initialisation des tableaux d'entrée
int CalculateValue();//Renvoie la réponse de la porte en fonction de la table de vérité
/*ATTENTION: CalculateValue se base sur un formatage précis de la table de vérité!*/
//Déclaration des membres publics
public:
//Fonctions
//Niveau circuit
void connectOut(Gate *,int);//Connecte la sortie de la porte à l'entrée spécifiée
void disconnectOut(Gate *,int);//Déconnecte la sortie de la porte de l'entrée spécifiée
void connectIn(int,Gate *);//Connecte l'entrée de la porte à la sortie spécifiée
void disconnectIn(int);//déconnecte l'entréé de la porte de la sortie attachée
//Niveau Simulation
void setValue(int,int);//attribue la valeur(1er argument) à l'entrée(2eme argument)
int getValue();//Renvoie la valeur courante de la sortie
void reset();//Réinitialise les valeurs contenues relatives à la simulation
//cad:intput_values est initialisé à un tableau de zéros
//Sauvegarde
int * getTruthTable();//Renvoie un pointeur vers la table de vérité
void setDelay(int);//permet de définir le délai
int getDelay();//Renvoie le délai interne de la porte
char * getName();//Renvoie le nom de la porte
int getNbEntries();
string ConnectoString();//Renvoie un string des connexions sorties
/*
*****************************************************
Implémentation de la liste contenant les sorties
*****************************************************
*/
private:
//************Définition de structure noeud de liste
struct node
{node * next; //pointeur sur l'élément suivant
Gate * gate; //pointeur vers une porte de porte
int EntryNumber;//Numero de la porte
};
node* current;//pointeur sur la sortie courante
node * start;//pointeur sur le début de la liste
void Begin();//initialise le pointeur current sur le début de la liste
bool End();//renvoie true si current se trouve en fin de liste
void Next();//déplace current sur le noeud suivant dans la liste
void transmitBit(int);
void toString(); //Renvoie un string représentatif de la liste
};
#endif

maintenant Gates.h:

Code :

#ifndef Gates_h
#define Gates_h
#include "Gate.h"
using namespace std;
class NOT:public Gate
{
public:
NOT();
NOT(char *);
NOT(char *,int);
NOT(char *,int,int);
NOT(char*,int,int,bool);
};
//class AND:public Gate{};
//class OR:public Gate{};
//class XOR:public Gate{};
#endif

theshockwave

I work at a firm named Koslow

KevinTheProud a écrit :

Code :

/*------------Définition de la classe Gate--------------------------*/
#ifndef Gate_h
#define Gate_h
#include <string>
using namespace std;
class Gate
{
int * input_values;//les entrées
char * Name;//nom de la porte
bool * OpenEntries;//entrées libres
//Tampon
int point;//Itérateur du Buffer
int delay;//Délai interne
int *Buffer;//le tampon
Gate * GateInTab;//Tableau de portes d'entrées
//Fonctionnalités "répertoire des sorties"
bool hasOutput();//a une sortie circuit ?
protected:
Gate(char*,int,int,bool);//définir le délai à l'instanciation
int entriesNumber;//Nombre d'entrées
int *truth_table;//Table de vérité-tableau de booléens
//Fonctionnalités diverses
void Inverse();
//Déclaration des membres protégés
void EntryInit();//Initialisation des tableaux d'entrée
int CalculateValue();//Renvoie la réponse de la porte en fonction de la table de vérité
//Déclaration des membres publics
public:
//Fonctions
//Niveau circuit
void connectOut(Gate *,int);
void disconnectOut(Gate *,int);
void connectIn(int,Gate *);
void disconnectIn(int);
//Niveau Simulation
void setValue(int,int);
int getValue();
void reset();
//Sauvegarde
int * getTruthTable();
void setDelay(int);
int getDelay();
char * getName();
int getNbEntries();
string ConnectoString();
/*
*****************************************************
Implémentation de la liste contenant les sorties
*****************************************************
*/
private:
// ************Définition de structure noeud de liste
struct node
{node * next; //pointeur sur l'élément suivant
Gate * gate; //pointeur vers une porte de porte
int EntryNumber;//Numero de la porte
};
node* current;
node * start;
void Begin();
bool End();
void Next();
void transmitBit(int);
void toString();
};
#endif

maintenant Gates.h:

Code :

#ifndef Gates_h
#define Gates_h
#include "Gate.h"
using namespace std;
class NOT:public Gate
{
public:
NOT();
NOT(char *);
NOT(char *,int);
NOT(char *,int,int);
NOT(char*,int,int,bool);
};
#endif

C'est déjà un peu mieux ... Si tu veux être explicite et éviter d'avoir à écrire 3 tonnes de commentaires qui alourdissent la lecture du code, tu devrais directement nommer tes variables dans les déclarations de tes méthodes ... Enfin ... Ce n'est que mon humble avis ....

Message édité par theshockwave le 18-04-2003 à 15:19:53

---------------
last.fm

theshockwave

I work at a firm named Koslow

:heink: ... J'arrive pas à trouver le destructeur ... Ne me dis pas que tu as "oublié" le destructeur ...

Enfin ... Ca ne devrait pas te causer ce genre de pb pour autant ...

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

:heink: ... J'arrive pas à trouver le destructeur ... Ne me dis pas que tu as "oublié" le destructeur ...

Enfin ... Ca ne devrait pas te causer ce genre de pb pour autant ...

Euh... disons que je l'ai "consciemment oublié" , je compte l'implémenter "plus tard"
EDIT: c'est aussi mon premier prog C++, donc le destructeur je n'ai pas encore très bien capté l'utilité...

Message édité par KevinTheProud le 18-04-2003 à 15:59:52

gloop

KevinTheProud a écrit :

bon ok,attention les yeux(monstrueuse?): Gate.h
[cpp]
/*------------Définition de la classe Gate--------------------------*/

int * input_values;//Tableau d'entiers contenant les entrées
char * Name;//String contenant le nom de la porte
bool * OpenEntries;//Tableau répertoriant les entrées libres

avec des vector, string, bitset ou vector<bool>, ce sera mieux et plus simple.

skeye

KevinTheProud a écrit :

Euh... disons que je l'ai "consciemment oublié" , je compte l'implémenter "plus tard"
EDIT: c'est aussi mon premier prog C++, donc le destructeur je n'ai pas encore très bien capté l'utilité...

au hasard, libérer la mémoire? :ange:

theshockwave

I work at a firm named Koslow

Petite question simple : Tu as essayé de spécifier les valeurs par défaut dans le .cpp ET dans le .h ?

Edit : typo

Message édité par theshockwave le 22-04-2003 à 00:34:30

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

Petite question simple : Tu as essayé de spécifier les valeurs par défaut dans le .cpp ET dans le .h ?

Edit : typo

merci pour ta réponse, j'ai essayé de les mettre dans les 2 ça marchait pas, mais en fait il faut juste les mettre dans le .h et par exemple écrire GATE(char*name="NONAME"... et donc dans le .cpp j'ai juste GATE(char*name,...

theshockwave

I work at a firm named Koslow

ok ... Donc ca y est, ca marche ?

---------------
last.fm

KevinTheProud

Insubmersible

theShOcKwAvE a écrit :

ok ... Donc ca y est, ca marche ?

vi ça marche j'avaic pas pensé à mettre ça direct dans le .h parce je trouve bizarre de déclarer la variable dans l'interface,
mais bon si ça marche...

theshockwave

I work at a firm named Koslow

Dans un sens, ca devient un peu plus logique ... Tu déclares plusieurs constructeurs qui sont en fait des redirections vers le seul que tu auras implémenté ... Donc logiquement, si ce ne sont que des déclarations alternatives pour ledit constructeur, il est plus logique de laisser lesdites déclarations dans le header ...

---------------
last.fm

Publicité

FORUM HardWare.fr

Programmation

C++

Passage d'arguments par défaut dans constructeur

Sujets relatifs
[VB] selection d un champ par defaut dans un combo	mysql: komment kon fait pour retourner une valeur par defaut?
VBS regwrite et passage d'une varible	question sur le passage d'un pointeur en paramètre
[Session] Passage de l'id de session de pages en pages	[Access] Insertion d'une valeur par défaut
Des arguments pour expliquer que les normes, c'est bien ?	Passage erroné d'un tableau de chaînes de caractères à une procédure..
Probleme de passage de variable en parametre (type=file)	[mysql] Laisser une valeur null par défaut ou non?
Plus de sujets relatifs à : Passage d'arguments par défaut dans constructeur

Page générée en 0.108 secondes