Discussion :

[Linu6]

Voila
ce code se compile correctement, mais à la fin l'invite de commande gèle:

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
#include<iostream>
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<strings.h>
 
 
 
using namespace std;
 
class Personne
{
private:
    char * nom;
    char * pnom;
    int age;
public:
    Personne();
    Personne(char * n, char * p, int a);
    ~Personne();
    //void Modif(Personne& p);
    void Affich();
};
 
Personne::Personne()
{
    nom = new char[6];
    nom = "userN";
    pnom = new char[5];
    pnom = "userP";
    age = 10;
}
 
Personne::Personne(char * n, char * p, int a)
{
    nom = new char[strlen(n)];
	strcpy(nom, n);
 
    pnom = new char[strlen(p)];
	strcpy(pnom, p);	
	age = a;
 
}
 
Personne::~Personne()
{
	delete[] nom;
	delete[] pnom;
}
void Personne::Affich()
{
    cout << "\nNom: " << nom << endl;
    cout << "Prenom: " << pnom << endl;
    cout << "Age: " << age << endl;
}
int main()
{
	char *uname;
	uname = new char[12];
        cin >> uname;
 
    Personne p(uname, "toto", 24);
    Personne a;
 
    a.Affich();
    p.Affich();
 
	delete[] uname;
    getchar();
}

d'où ça peut venir?

merci

[droggo]

Lae,

Sans avoir tout lu:

Code :

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

1
2
3
4
5
6
7
8
9
10
Personne::Personne(char * n, char * p, int a)
{
    nom = new char[strlen(n)];
	strcpy(nom, n);
 
    pnom = new char[strlen(p)];
	strcpy(pnom, p);	
	age = a;
 
}

il faut réserver strlen(n) +1 pour la nouvelle chaîne (ne jamais oublier le 0 final ).

[Davidbrcz]

d'où l'utilité d'utiliser des std::string !

[droggo]

Gao,

d'où l'utilité d'utiliser des std::string !

Évidemment

[coyotte507]

Salut,

pour moi ça marche mais ça doit être un coup de chance.

En effet, ici:

Code :

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

1
2
3
4
5
    nom = new char[6];
    nom = "userN";
    pnom = new char[5];
    pnom = "userP";
    age = 10;

Apparemment tu n'as pas tout saisi, désolé.
Pour copier une chaine dans une autre, ça ferait:

Code :

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

1
2
3
4
5
    nom = new char[6];
    strcpy(nom, "userN");
    pnom = new char[6];
    strcpy(pnom, "userP");
    age = 10;

Ce que tu faisais, c'est que tu mettais l'équivalent d'un const char * dans ton pointeur, et donc ton pointeur pointe vers une autre zone maintenant, celle où il y a écrit "userN". Du coup plus rien ne pointe vers la mémoire allouée dynamiquement, et quand tu fais un delete [] sur un const char * ça a de grandes chances de planter...

Je ne suis peut-être pas très clair, mais regarde dans ton bouquin les const char *, et la recopie d'une chaîne dans une autre, il expliquera ça mieux que moi.

Sinon les autres remarques sont valides, (strlen+1 au lieu de strlen), et il manque un 'return 0' à la fin de ton main.

Une fois tout ça fait, tout va bien:

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
#include<iostream>
#include<string.h>
 
using namespace std;
 
class Personne
{
private:
    char * nom;
    char * pnom;
    int age;
public:
    Personne();
    Personne(char * n, char * p, int a);
    ~Personne();
    //void Modif(Personne& p);
    void Affich();
};
 
Personne::Personne()
{
    nom = new char[6];
    strcpy(nom, "userN");
    pnom = new char[6];
    strcpy(pnom, "userP");
    age = 10;
}
 
Personne::Personne(char * n, char * p, int a)
{
    nom = new char[strlen(n)+1];
	strcpy(nom, n);
 
    pnom = new char[strlen(p)+1];
	strcpy(pnom, p);
	age = a;
 
}
 
Personne::~Personne()
{
	delete[] nom;
	delete[] pnom;
}
void Personne::Affich()
{
    cout << "\nNom: " << nom << endl;
    cout << "Prenom: " << pnom << endl;
    cout << "Age: " << age << endl;
}
int main()
{
    cout << "Entrez votre nom: ";
 
	char *uname;
	uname = new char[12];
        cin >> uname;
 
    Personne p(uname, "toto", 24);
    Personne a;
 
    a.Affich();
    p.Affich();
 
	delete[] uname;
    getchar();
 
    return 0;
}

[droggo]

Meo,

pour moi ça marche mais ça doit être un coup de chance.

Moi, j'appelle ça de la malchance, car c'est un programme buggé, au comportement indéfini, et qui semble fonctionner normalement, alors que...

[Linu6]

Code :

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

1
2
3
4
5
    nom = new char[6];
    nom = "userN";
    pnom = new char[5];
    pnom = "userP";
    age = 10;

Apparemment tu n'as pas tout saisi, désolé.
Pour copier une chaine dans une autre, ça ferait:

Code :

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

1
2
3
4
5
    nom = new char[6];
    strcpy(nom, "userN");
    pnom = new char[6];
    strcpy(pnom, "userP");
    age = 10;

Ah oui, j'avais confondu la syntaxe de char avec celle de string..
Quoi qu'il arrive merci à vous tous! ;-)