Débutant dans les constructeurs

**dot-_-net** · 29/03/2008, 18h34

Voila je suis encor débutant en C++ et je suis entrain de comprendre les constructeurs je viens d’écrire un code mais quand je l’exécute ça me donne un bug

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include "stdafx.h"
#include <string.h>
#include <stdio.h>
class client{
public:
	client();
	client(const client & evo);
	client(char * nom,int age,int code);
	client::~client();
protected:
 
char *_nom;
int age;
int code;
 
};
 
client::client(){
this->_nom =0;
}
 
client::client(char *nom,int age,int code){
this->_nom =new char[strlen(nom)+1];
strcpy(nom,this->_nom);
this->age =age;
this->code =code;
}
 
client::~client()
 	{
 		if (_nom)
 			delete [] _nom;
 	}
 
int main(int argc, char* argv[])
{
 
client c();
client a("dotnet",5,32);
client b(a);
client *d;
d=new client(b);
 
 
delete d;
	return 0;
}

Ou se trouve l’erreur et merci d’avance

**Montag** · 29/03/2008, 19h22

Salut,

Déjà remplace :

Code :

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

strcpy(nom,this->_nom)

par :

Code :

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

strcpy(this->_nom,nom)

Ensuite, évite d'utiliser les char * qui sont sources d'erreurs (ton code en est la preuve), préfère leur std::string.
Et pour finir ton code ne compile pas (en particulier, il manque la définition du constructeur de copie).

**corrector** · 29/03/2008, 21h03

Envoyé par dot-_-net

Voila je suis encor débutant en C++ et je suis entrain de comprendre les constructeurs je viens d’écrire un code mais quand je l’exécute ça me donne un bug

Code :

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

#include "stdafx.h"

C'est quoi stdafx.h?

Envoyé par dot-_-net

Code :

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

client c();

Ceci ne déclare pas un objet initialisé par le constructeur par défaut, mais une fonction ne prenant aucun argument renvoyant un client.

Envoyé par dot-_-net

Ou se trouve l’erreur et merci d’avance

Peut-être dans le code que tu n'as pas posté?

**dot-_-net** · 29/03/2008, 21h14

Envoyé par corrector

C'est quoi stdafx.h?

Ceci ne déclare pas un objet initialisé par le constructeur par défaut, mais une fonction ne prenant aucun argument renvoyant un client.

Peut-être dans le code que tu n'as pas posté?

J’ai déclaré stdafx car je travaille sous visual c++ v6

**corrector** · 29/03/2008, 21h19

Envoyé par dot-_-net

J’ai déclaré stdafx car je travaille sous visual c++ v6

So?

**Davidbrcz** · 29/03/2008, 22h36

stdafx.h est une horreur de VC++: un header pré compilé.
Un truc censé accélérer la compilation des projets. C'est bien sur non standard, non portable, ni rien du tout.
Ça devrait être banni des forums de programmation.

**dot-_-net** · 29/03/2008, 23h01

Envoyé par Davidbrcz

stdafx.h est une horreur de VC++: un header pré compilé.
Un truc censé accélérer la compilation des projets. C'est bien sur non standard, non portable, ni rien du tout.
Ça devrait être banni des forums de programmation.

Si je déclare pas cette biblio mon compilateur me signal erreur
Et pour quoi stdafx est une horreur ?

**corrector** · 29/03/2008, 23h47

Envoyé par dot-_-net

Si je déclare pas cette biblio mon compilateur me signal erreur
Et pour quoi stdafx est une horreur ?

Envoyé par Davidbrcz

stdafx.h est une horreur de VC++: un header pré compilé.
Un truc censé accélérer la compilation des projets. C'est bien sur non standard, non portable, ni rien du tout.
Ça devrait être banni des forums de programmation.

Qu'est-ce qu'il te faut?

**Médinoc** · 30/03/2008, 12h24

stdafx.h est un fichier d'en-tête du projet.
Si tu n'as pas de fichier d'en-tête nommé stdafx.h dans ton projet, alors tu n'as rien à inclure et c'est normal que ça foire.

PS: Je ne vois pas le code de ton constructeur de copie. De plus, tu dois aussi redéfinir l'opérateur d'affectation pour éviter des problèmes (de préférence en utilisant l'idiome copy-swap).

**ProgVal** · 30/03/2008, 17h50

stdafx est un en-tête obligatoire avec Visual Studio:

Voici son code:

Code :

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

1
2
3
4
5
6
7
8
// stdafx.h : fichier Include pour les fichiers Include système standard,
// ou les fichiers Include spécifiques aux projets qui sont utilisés fréquemment,
// et sont rarement modifiés
//
 
#pragma once
 
// TODO : faites référence ici aux en-têtes supplémentaires nécessaires au programme

**Médinoc** · 30/03/2008, 18h00

stdafx est un en-tête obligatoire avec Visual Studio

Cette phrase et fausse, pour plusieurs raisons:

Dans un projet sans en-tête précompilé, ou dans un fichier source pour lequel l'en-tête précompilé a été désactivé, il n'est pas nécessaire d'inclure stdafx.h
Du temps de VS2005, on peut choisir dès la création du projet si on utilise un fichier d'en-tête précompilé ou non.
Et on peut utiliser un autre nom que stdafx.h pour le fichier d'en-tête qui sera précompilé.

Sachant que le code posté est le code d'un stdafx.h vide. Théoriquement, on y met les inclusions des headers standards, et des diverses APIs qui n'ont pas tendance à être modifiées entre deux compilations (par opposition aux headers du projet qui ont de grandes chances d'être modifiés fréquemment).

screetch · 31/03/2008, 16h06

stdafx.h est tres pratique pour inserer des definitions (preprocesseur) qui doivent etre toujours definies dans chaque fichier c++. de plus, rien n'oblige a l'utiliser. enfin, c'est non portable un #include ? depuis quand ??? si on n'utilise pas stdafx.h, rien de special ne se produit.

**Davidbrcz** · 31/03/2008, 18h04

hum, #include <windows.h> c'est super portable non ?
Il en va de même pour stdafx.h

**Médinoc** · 31/03/2008, 18h06

Perdu.

windows.h est un header de l'API Windows.
stdafx.h est un header du projet.

screetch · 31/03/2008, 18h06

stdafx.h est crée par le programmeur. tu veux voir le mien ? il compile sous linux. par defaut il est vide.

ca me reloute les anti crosoft qui ont oublié de voir comment ca marchait...

**koala01** · 31/03/2008, 19h08

Salut,

Le problème de l'utilisation des en-têtes précompilés est à vrai dire un faux problème, et ne fait en rien approcher d'une solution. Je propose donc de recentrer le débat sur les problèmes réels de dot-_-net

Quitte à faire du C++, je te conseillerais de le faire correctement.

Les fichiers d'en-têtes <string.h> et <stdio.h> seraient agréablement remplacés par les fichiers d'en-tête <string> et <iostream>, ce qui te permettrait de remplacer avantageusement tes char* par la classe string, disponible dans l'espace de noms std.

On ne le répétera jamais assez, l'utilisation des chaines de caractères "C style" (c'est à dire sous la forme de tableaux de caractères terminés par un '\0') est très largement contestée en C++, et il est fortement conseillé de leur préférer la std::string car bien plus sécurisante, et permettant même, le cas échéant, de récupérer une chaine C style en cas de besoin.

Cela te permettra d'éviter tous les écueils du à l'utilisation de pointeurs, d'allocation dynamique et les différents problèmes que cela implique.

Ainsi, ta classe Client serait avantageusement modifiée sous la forme de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class client{
public:
	client();
	client(const client & evo);
	client(const std::string& nom,int age,int code);
	client::~client();
protected:
 
std::string _nom;
int age;
int code;
 
};

(il serait d'ailleurs sans doute intéressant de prévoir quelques accesseurs, mais on n'y est pas encore

)

Ensuite, il faut savoir que l'idéal reste toujours d'utiliser les listes d'initialisation dans les constructeurs, et leur implémentation deviendrait donc

Code :

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

1
2
3
4
5
6
7
8
9
Client::Client():_nom(""),age(0),code(0)
{
}
Client::Client(const Client& evo):_nom(evo._nom),age(evo.age),code(evo.code)
{
}
Client::Client(const std::string& nom, int age, int code):_nom(nom),age(age),code(code)
{
}

Le destructeur pouvant alors se contenter d'être implémenté en

Code :

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

1
2
3
Client::~Client()
{
}

Enfin, il me semble opportun de te rappeler, cher dot-_-net que le premier conseil que l'on peut donner à quelqu'un est

Aide nous à t'aider

En effet, tu dis que "ca bugue", mais tu ne nous indique absolument pas "ce qui bugue".

Si tu veux obtenir une aide rapide et efficace, il serait sans doute pas mal de commencer par nous indiquer en quoi ca bugue

(est-ce à la compilation

à l'édition de liens

à l'exécution

)

Fournis nous l'erreur obtenue, ou le comportement que tu juge comme anormal, et nous serons bien plus en mesure de t'aider