Discussion :

[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

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]

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]

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?

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.

Ou se trouve l’erreur et merci d’avance

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

[dot-_-net]

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]

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

So?

[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.

[dot-_-net]

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 ?

[koala01]

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