Problème d'exécution de mon programme

**LauraRL** · 13/03/2016, 18h09

Bonjour
J'aurais besoin d'aide pour mon programme. Donc dans mon programme je devais créer une classe mère de mon choix et ensuite créer une sous classe de cette même classe. Sauf que quand je souhaite exécuter mon programme cesse de fonctionner et ne veut pas m'afficher l'objet créer depuis la sous classe.
Mon programme:

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
#include <iostream>
#define TAXES 0.2
 
using namespace std;
 
class Article
{
protected:
    int number;
    int quantity;
    string name;
    float price; //it is the price without the taxes
 
public:
    Article(int number, int quantity, string name, float price);
    virtual void display();
    virtual float setPriceWithTaxes(); //it will give us the price with taxes
 
};
 
 
Article::Article(int aNumber, int aQuantity, string aName, float aPrice)
{
    number = aNumber;
    quantity = aQuantity;
    name = aName;
    price = aPrice;
}
 
void Article::display()
{
    cout << "The article number is " << number << endl;
    cout << "We have " << quantity << " of this article in our stock" << endl;
    cout << "The name of the article is " << name << endl;
    cout << "The price of this article without taxes is " << price << " euros" <<endl;
}
 
float Article::setPriceWithTaxes()
{
    float newPrice;
    newPrice = price+(price*TAXES);
    cout << "The price of this article with taxes is " << newPrice << " euros" << endl;
}
 
class Clothe : public Article
{
    string sizeClothe; //Size are from XS to XL
    string colorClothe;
 
public:
    Clothe(string sizeClothe, string colorClothe);
    virtual void display();
};
 
Clothe::Clothe(string aSizeClothe,string aColorClothe): Article(number,quantity,name,price)
{
    sizeClothe=aSizeClothe;
    colorClothe=aColorClothe;
}
 
void Clothe::display()
{
    Article::display();
    cout << "The size of the clothe is " << sizeClothe << endl;
    cout << "The color of the clothe is " << colorClothe << endl;
}
 
int main()
{
    Article femaleArticle = Article(1250369,10,"Maxi Dress",25.99);
    femaleArticle.display();
    femaleArticle.setPriceWithTaxes();
    Article maleArticle = Article(1236501,5,"Sweatshirt",35.99);
    maleArticle.display();
    Clothe a = Clothe("M","Red");
    a.display();
    return 0;
}

Est ce quelqu'un pourrait m'aider svp ?

**dalfab** · 14/03/2016, 00h05

Bonjour,
Je ne vois pas comment un code qui ne compile pas pourrait planter!
Ligne 55 : Clothe est un Article donc il doit s'initialiser au moins comme un article, il a un prix, un nom, ...

Code :

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

1
2
3
4
Clothe::Clothe(int number, int quantity, string name, float price,string aSizeClothe,string aColorClothe): Article(number,quantity,name,price) {
    sizeClothe=aSizeClothe;
    colorClothe=aColorClothe;
}

**LauraRL** · 14/03/2016, 01h45

Je ne pense pas que j'avais un problème de compilation puisque lorsque j'ai modifié mon code avec ce que vous m'aviez proposé mon fichier .exe plante toujours. Je ne vois pas d'où le problème provient. Je vous remercie tout de meme de m'avoir aider avoir l'erreur dans mon programme.

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
#include <iostream>
#include <string>
#define TAXES 0.2
 
using namespace std;
 
class Article
{
protected:
    int number;
    int quantity;
    string name;
    float price; //it is the price without the taxes
 
public:
    Article(int number, int quantity, string name, float price);
    virtual void display();
    virtual float setPriceWithTaxes(); //it will give us the price with taxes
 
};
 
 
Article::Article(int aNumber, int aQuantity, string aName, float aPrice)
{
    number = aNumber;
    quantity = aQuantity;
    name = aName;
    price = aPrice;
}
 
void Article::display()
{
    cout << "The article number is " << number << endl;
    cout << "We have " << quantity << " of this article in our stock" << endl;
    cout << "The name of the article is " << name << endl;
    cout << "The price of this article without taxes is " << price << " euros" <<endl;
}
 
float Article::setPriceWithTaxes()
{
    float newPrice;
    newPrice = price+(price*TAXES);
    cout << "The price of this article with taxes is " << newPrice << " euros" << endl;
}
 
class Clothe : public Article
{
    string sizeClothe; //Size are from XS to XL
    string colorClothe;
 
public:
    Clothe(int number,int quantity,string name,float price,string sizeClothe, string colorClothe);
    virtual void display();
};
 
Clothe::Clothe(int aNumber,int aQuantity,string aName,float aPrice,string aSizeClothe,string aColorClothe): Article(number,quantity,name,price)
{
    sizeClothe=aSizeClothe;
    colorClothe=aColorClothe;
}
 
void Clothe::display()
{
    Article::display();
    cout << "The size of the clothe is " << sizeClothe << endl;
    cout << "The color of the clothe is " << colorClothe << endl;
}
 
int main()
{
    Article femaleArticle = Article(1250369,10,"Maxi Dress",25.99);
    femaleArticle.display();
    femaleArticle.setPriceWithTaxes();
    Article maleArticle = Article(1236501,5,"Sweatshirt",35.99);
    maleArticle.display();
    Clothe dress = Clothe(1265879,2,"Wedding Dress",1250.90,"S","White");
    dress.display();
    return 0;
}

Du coup lorsque j’exécute mon programme cette erreur s'affiche avez vous une autre idée ?

Nom : assignment4.PNG
Affichages : 109
Taille : 17,7 Ko

**foetus** · 14/03/2016, 02h10

Cette ligne est suspicieuse

: Clothe dress = Clothe(1265879,2,"Wedding Dress",1250.90,"S","White");.

Si je ne dis pas de bêtises

tu crées un objet temporaire, et ensuite tu appelles le constructeur (opérateur ???) de recopie.
Et comme il n'est pas codé (c'est une copie binaire), alors ton attribut name est indéfini [... comme les autres attributs, mais celui ci n'est pas un type simple/ P.O.D.] et plante.

Essaye

Clothe dress(1265879,2,"Wedding Dress",1250.90,"S","White");.

**LauraRL** · 14/03/2016, 02h19

Mon programme ne plante plus avec votre solution mais le problème du coup c'est que je ne peux pas utiliser ma fonction display pour afficher ce que j'avais mis dans mon construteur Clothe. Comment puis je le faire afficher autrement tout en me servant de ma fonction display.

**Bousk** · 14/03/2016, 09h35

Euh non il me semble que l'opérateur= (ou constructeur par copie) par défaut ne fera qu'appeler l'opérateur= (ou constructeur par copie) sur chaque variable membre, dans l'ordre où elles sont indiquées dans le .h
Du coup, sans aucune allocation dynamique ni rien, je vois pas où ce programme peut planter

As-tu lancé avec le debugger ? Parce qu'il te dira directement où ça plante.
Un std::string par défaut est très bien connu, c'est une chaîne vide.

Parmi les bonnes pratiques, il convient de déclarer le destructeur virtual si tu comptes faire de l'héritage. Là ça va t'as aucune allocation dynamique, donc aucune fuite mémoire, mais bon.

le problème du coup c'est que je ne peux pas utiliser ma fonction display pour afficher ce que j'avais mis dans mon construteur Clothe. Comment puis je le faire afficher autrement tout en me servant de ma fonction display.

Kamoulox, j'ai rien compris. Pourquoi tu peux pas faire dress.display(); ?

**dalfab** · 14/03/2016, 10h50

Bonjour,

En effet, il n'y a rien ici qui pourrait provoquer un plantage.
Un doute m'habite. Puisque le premier code ne pouvait pas compiler et que le plantage existait déjà! *
@Laura, n'es-tu pas en train d'appeler une ancienne version de l'executable au lieu de celle que tu dois compiler?

**LauraRL** · 14/03/2016, 13h29

Normalement je recompile avant de faire exécuter mon programme du coup je ne pense pas exécuter une ancienne version...
@Bousk en faite je me suis trompée j'avais mal lu la réponse de @foetus du coup je peux appeler ma fonction display.
De plus mon programme plante dès que je fais appel à mon constructuer du coup le problème doit venir de la mais je ne vois pas le problème...

**Bousk** · 14/03/2016, 14h58

Je ne vois absolument rien dans le code montré, si tenté que ce soit le seul code que tu aies, qui puisse faire planter quoi que ce soit.
Lance le code dans le debugger et regarde/montre-nous la callstack.

**stephane78l** · 14/03/2016, 15h18

@Laura, ton constructeur de Clothe tel que tu l'as mis n'est pas correct car tu as fait un copier coller bete:
remplace ceci :

Code :

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

1
2
3
4
5
Clothe::Clothe(int aNumber,int aQuantity,string aName,float aPrice,string aSizeClothe,string aColorClothe): Article(number,quantity,name,price)
{
    sizeClothe=aSizeClothe;
    colorClothe=aColorClothe;
}

par :

Code :

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

1
2
3
4
5
Clothe::Clothe(int aNumber,int aQuantity,string aName,float aPrice,string aSizeClothe,string aColorClothe): Article(aNumber,aQuantity,aName,aPrice)
{
    sizeClothe=aSizeClothe;
    colorClothe=aColorClothe;
}

cela devrait mieux marcher

**ternel** · 14/03/2016, 16h11

Et ceci serait beaucoup mieux:

Code :

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

1
2
3
4
5
Clothe::Clothe(int aNumber,int aQuantity,string aName,float aPrice,string aSizeClothe,string aColorClothe):
    Article(aNumber,aQuantity,aName,aPrice),
    sizeClothe(aSizeClothe),
    colorClothe(aColorClothe)
{}

De plus, sizeClothe et colorClothe sont deux membres de la classe Clothe, il n'y a a pas d'utilité à les suffixer par Clothe.
En bonus, dans le cas d'un constructeur, dans la liste d'initialisation, il n'y a pas d'ambiguité à nommer un parametre de la même manière que le champ.
Ainsi, tu pourrais écrire:

Code :

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

1
2
3
4
5
Clothe::Clothe(int number, int quantity,string name,float price,string size,string color):
    Article(number, quantity, name, price),
    size(size),
    color(color)
{}