Probleme d'executions

**Html** · 11/03/2006, 13h20

Bonjour a tous voila je n'ai aucune erreur a la compilation ni au link mais a l'execution ca me sort une vielle erreur du genre Debug assertion failed...

Voila si qql peut maider sur mon erreur...voici le code et tous le dossier si qql veut bien l'executer...http://cagnesumadre.rockweb.org/TP3.rarMerci Beaucoups!

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
//ARTICLE.H
#include <iostream.h>
#include <string.h>
#define DIM 10
 
class CArticle
{
private:
		unsigned int m_uRef;
		char* m_pNom;
		float m_fPrixHT;
		unsigned int m_uQte;
public:
	CArticle(unsigned int,char*,float);						//On connait pas la quantité dans ce contructeur;
	CArticle(unsigned int,char*,float,unsigned int);		//On connait  la quantité dans ce contructeur;
	CArticle(const CArticle & C,char *);							//Constructeur de Copie
	~CArticle();											//Destructeur
	void Ajouter(unsigned int);								//Ajouter un nombre à la quantité
	void Retirer(unsigned int);								//Retirer un nombre a la quantité
	float CalCulerPrix();									//Calcule du prix TTC
	//void Afficher();										//Fonction d'affichage du prix
	friend ostream& operator<<(ostream&,CArticle);			//Surchage de l'operateur <<
};

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
 
//Article.CPP
 
 
 
#include "ARTICLE.H"
//On connait pas la quantité dans ce contructeur;
 
 
CArticle::CArticle(unsigned int uRef,char* pNom,float fPrixHT)
{
	 m_uRef=uRef;
	 m_pNom=new char[DIM];
	 strcpy(m_pNom,pNom);
	 m_fPrixHT=fPrixHT;
	 m_uQte=0;
 
 
}
 
CArticle::CArticle(const CArticle & C,char* pNom)
{
	m_pNom=new char[DIM];
	strcpy(m_pNom,pNom);
	m_uRef=C.m_uRef;
	m_uQte=C.m_uQte;
	m_fPrixHT=C.m_fPrixHT;
 
}
 
//On connait  la quantité dans ce contructeur;
 
CArticle::CArticle(unsigned int uRef,char* pNom,float fPrixHT,unsigned int uQte)
{
	m_uRef=uRef;
 
	strcpy(m_pNom,pNom);
	m_pNom=new char[DIM];
	m_fPrixHT=fPrixHT;
	//CArticle(uRef,pNom,fPrixHT);
	m_uQte=uQte;
}
 
CArticle::~CArticle()
{
	delete m_pNom;
}
//Ajouter un nombre à la quantité
void CArticle::Ajouter(unsigned int Nb=0)
{
	m_uQte=m_uQte+Nb;
}
 
 
//Retourne le stock quand on ajoute un nombre à la quantite
 
void CArticle::Retirer(unsigned int Nb=0)
{
	m_uQte=m_uQte-Nb;
}
 
 
//Retourne le stock enlever un nombre à la quantite
 
float CArticle::CalCulerPrix()
{
		const float TVA=19.6f;
		return (m_uQte*(m_fPrixHT+(m_fPrixHT*(TVA/100))));
}
 
 
 //Fonction d'affichage du prix
 
/*void CArticle::Afficher()
{
	cout<<"Nom :"<<m_pNom<<endl;
	cout<<"Quantité :" <<m_uQte<<endl;
	cout<<"Prix TTC :"<<CalCulerPrix()<<endl;
 
 
 
}
*/	
ostream& operator<<(ostream& cS,CArticle article)
{
    cS<<"Nom :"<<article.m_pNom<<endl;
	cS<<"Quantite :" <<article.m_uQte<<endl;
	cS<<"Prix TTC :"<<article.CalCulerPrix()<<endl;
	return cS;
}

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
 
//T_Arcticle
 
 
#include "ARTICLE.H"
 
 
 
 
void main()
{	
cout<<"-----------------------------"<<endl;
cout<<"|Test de la classe CArticle |"<<endl;
cout<<"-----------------------------"<<endl;
 
CArticle cArt(1,"Art",12.1f);
 
CArticle cArt2(cArt);
 
CArticle cArt3(1,"Foot",12.1f);
//cArt.Afficher();
cout<<cArt2;
cArt2.~CArticle();
 
}

**JolyLoic** · 11/03/2006, 13h54

Je n'ai passé qu'une dizaine de secondes à survoler ton code, j'ai donc pu laisser passer des choses...

Tu utilises des char*. Utiliser std::string à la place te simplifiera grandement la tâche. D'ailleur, tu gères mal ces char* : Tu les alloues par new xx[nn]; mais les désalloue par delete xx; au lieu de les désallouer par delete[] xx;

Un constructeur à deux arguments n'est pas un constructeur de copie.

cArt2 est construit par copie de cArt, avec donc le constructeur de copie par défaut, qui ne fait pas ce qu'il faut (il le ferait, si tu utilisais std::string...).

Vouloir spécifier le constructeur de copie sans spécifier l'opérateur d'affectation est presque toujours une erreur.

**Html** · 11/03/2006, 14h56

D'accord mais je change quoi je comprend pas... tu peux me montrer stp...

**Pragmateek** · 12/03/2006, 12h05

Salut!

A quel endroit du test ça plante?

**JolyLoic** · 12/03/2006, 12h21

Qu'est-ce que tu ne comprends pas ?

**Rafy** · 12/03/2006, 13h37

La dernière ligne du main :

Code :

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

cArt2.~CArticle();

Ton objet est construit statiquement, donc le constructeur sera apellé tout seul, il n'y a pas besoin de cette ligne...
Comme le disait JolyLoic, utilise des std::string (tu inclues l'entête en plus) ça sera beaucoup plus simple et empécherait des erreurs..
Dans ce code :

Code :

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

strcpy(m_pNom,pNom);

Si pNom est plus long que DIM - 1, il va se passer des bricoles
Et je crois même que strcpy ne met pas de '\0' en fin de chaine...
Problèmes évités en utilisant des std::string
Bref, les char* ça ne devrait plus exister dans un code en C++ pour stocker et utiliser des chaînes de caractères.

**Rafy** · 12/03/2006, 13h45

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
//ARTICLE.H
#include <iostream>
#include <string>
 
class CArticle
{
private:
		unsigned int m_uRef;
		std::string m_Nom;
		float m_fPrixHT;
		unsigned int m_uQte;
public:
	CArticle(unsigned int,const std::string&,float);						//On connait pas la quantité dans ce contructeur;
	CArticle(unsigned int,const std::string&,float,unsigned int);		//On connait  la quantité dans ce contructeur;
	CArticle(const CArticle & C,const std::string&);							//Constructeur de Copie
	~CArticle();											//Destructeur
	void Ajouter(unsigned int);								//Ajouter un nombre à la quantité
	void Retirer(unsigned int);								//Retirer un nombre a la quantité
	float CalCulerPrix();									//Calcule du prix TTC
	//void Afficher();										//Fonction d'affichage du prix
	friend std::ostream& operator<<(std::ostream&,CArticle);			//Surchage de l'operateur <<
};

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
 
//Article.CPP
 
 
 
#include "ARTICLE.H"
//On connait pas la quantité dans ce contructeur;
 
 
CArticle::CArticle(unsigned int uRef,const std::string& Nom,float fPrixHT)
{
	 m_uRef=uRef;
	 m_Nom = Nom;
	 m_fPrixHT=fPrixHT;
	 m_uQte=0;
 
 
}
 
CArticle::CArticle(const CArticle & C,const std::string& Nom)
{
	m_Nom = Nom;
	m_uRef=C.m_uRef;
	m_uQte=C.m_uQte;
	m_fPrixHT=C.m_fPrixHT;
 
}
 
//On connait  la quantité dans ce contructeur;
 
CArticle::CArticle(unsigned int uRef,const std::string& Nom,float fPrixHT,unsigned int uQte)
{
	m_uRef=uRef;
	m_Nom = Nom;
	m_fPrixHT=fPrixHT;
	//CArticle(uRef,pNom,fPrixHT);
	m_uQte=uQte;
}
 
CArticle::~CArticle()
{
 
}
//Ajouter un nombre à la quantité
void CArticle::Ajouter(unsigned int Nb=0)
{
	m_uQte=m_uQte+Nb;
}
 
 
//Retourne le stock quand on ajoute un nombre à la quantite
 
void CArticle::Retirer(unsigned int Nb=0)
{
	m_uQte=m_uQte-Nb;
}
 
 
//Retourne le stock enlever un nombre à la quantite
 
float CArticle::CalCulerPrix()
{
		const float TVA=19.6f;
		return (m_uQte*(m_fPrixHT+(m_fPrixHT*(TVA/100))));
}
 
 
 //Fonction d'affichage du prix
 
/*void CArticle::Afficher()
{
	std::cout << "Nom :" << m_Nom << std::endl;
	std::cout << "Quantité :" << m_uQte << std::endl;
	std::cout << "Prix TTC :" << CalCulerPrix() << std::endl;
 
 
 
}
*/	
std::ostream& operator<<(std::ostream& cS,CArticle article)
{
    cS << "Nom : " <<article.m_Nom << std::endl;
	cS << "Quantite : " << article.m_uQte << std::endl;
	cS << "Prix TTC : "<< article.CalCulerPrix() << std::endl;
	return cS;
}

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
 
//T_Arcticle
 
 
#include "ARTICLE.H"
 
 
 
 
void main()
{	
        std::cout <<"-----------------------------" std::<<endl;
        std::cout <<"|Test de la classe CArticle |" std::<<endl;
        std::cout <<"-----------------------------" std::<<endl;
 
        CArticle cArt(1,"Art",12.1f);
 
        CArticle cArt2(cArt);
 
        CArticle cArt3(1,"Foot",12.1f);
        //cArt.Afficher();
        cout<<cArt2;
 
}

Voila ton code transformé avec des std::string...
Je crois que j'ai trouvé ce qui n'allait pas dans ton code d'ailleurs :

Code :

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

1
2
3
4
5
6
7
8
9
10
CArticle::CArticle(unsigned int uRef,char* pNom,float fPrixHT,unsigned int uQte) 
{ 
   m_uRef=uRef; 
 
   strcpy(m_pNom,pNom);     // Ligne 1
   m_pNom=new char[DIM]; // Ligne 2
   m_fPrixHT=fPrixHT; 
   //CArticle(uRef,pNom,fPrixHT); 
   m_uQte=uQte; 
}

Dans la ligne 1 tu copie la chaine, mais tu n'alloue la mémoire qu'a la ligne 2...
Donc je pense que quand strcpy test ses argument il trouve m_pNom qui pointe sur rien....
[édit]Normalement c'est correct maintenant[\édit]

**Rafy** · 12/03/2006, 13h49

Je viens de voir encore un truc :
Il faut inclure non pas

Code :

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

1
2
#include <iostream.h> 
#include <string.h>

mais plutôt

Code :

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

1
2
#include <iostream> 
#include <string>

et d'utiliser soit

Code :

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

using namespace std;

ou alors de mettre devant tout ce qui vient de la bibliothèque standard des

Code :

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

std::

Donc
cout deviendrai std::cout
endl deviendai std::endl
etc...
J'édite mon message du dessus pour que ça soit prit en compte

Probleme d'executions

C++

Vue hybride

Discussions similaires

Partager

Partager