C++

probleme avec pointeur sur objet

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : probleme avec pointeur sur objet

cris56

voila ca fait une journée que je cherche ou est l'erreur mais rien a faire, le code ci dessous compile mais plante a l'execution

Code :

#include <iostream.h>
#include <string.h>
#include <stdlib.h>
typedef class Profile
{
public:
char* nom;
char* adresse;
void init();
}*pProfile;
void Profile::init()
{
char nomp[100];
cout<<"entrer le nom:";
cin>>nomp;
nom=(char *)malloc(strlen(nomp));
strcpy(nom,nomp);
char adressp[100];
cout<<"entrer l'adresse:";
cin>>nomp;
adresse=(char *)malloc(strlen(adressp));
strcpy(adresse,adressp);
}
void main(void)
{
pProfile profile;
profile->init();
cout<<profile->nom<<" "<<profile->adresse<<endl;
}

Publicité

ffluff

Challenge Everything

pitié, faites que Taz ne passe pas par là ... paix à son âme.

---------------
«Le succès consiste à aller d'échecs en échecs sans jamais perdre son enthousiasme» - Churchill

cris56

pourquoi? c'est qui taz?

Message édité par cris56 le 25-03-2004 à 16:09:38

ffluff

Challenge Everything

dés qu'il seras arrivé ... ton pire ennemi.

La tu fais du C ... tu remplace class par struct, cout par printf et cin par scanf et opla t as du C.

---------------
«Le succès consiste à aller d'échecs en échecs sans jamais perdre son enthousiasme» - Churchill

cris56

ben non c'est du c++, ya une classe, on peut pas faire ca en c?

SomeBugsInMe

life suxx

cris56 a écrit :

void Profile::init()
{
char nomp[100];
cout<<"entrer le nom:";
cin>>nomp;
nom=(char *)malloc(strlen(nomp)+1);
strcpy(nom,nomp);

char adressp[100];
cout<<"entrer l'adresse:";
cin>>adressp;
adresse=(char *)malloc(strlen(adressp)+1);
strcpy(adresse,adressp);
}

mais bon ton code est très laid.
On peut pas appeler ça du C++...

Message édité par SomeBugsInMe le 25-03-2004 à 16:48:49

gilou

Modérateur
Modzilla

Sinon, pour ton pb:
> nom=(char *)malloc(strlen(nomp));

Ca serait pas
nom=(char *)malloc(strlen(nomp)+1);
:whistle:

Maintenant, si tu veux vraiment que ce soit du c++, vaudrait mieux faire un truc de ce genre:

Code :

#include <iostream>
#include <string>
using std::cin;
using std::cout;
using std::endl;
using std::string;
class User_profile
{
public:
string nom;
string adresse;
void init();
};
void User_profile::init()
{
cout<<"entrer le nom:"<<endl;
cin>>nom;
cout<<"entrer l'adresse:"<<endl;
cin>>adresse;
}
int main()
{
User_profile profile;
profile.init();
cout<<profile.nom<<" "<<profile.adresse<<endl;
}

J'ai pas de compilo c++ sous la main pour verifier que ca passe correctement, mais ca te donnera une idée du style a employer...
A+,

pascal_

cris56 a écrit :

Code :

void main(void)
{
pProfile profile;
profile->init();
cout<<profile->nom<<" "<<profile->adresse<<endl;
}

En plus dans le main, profile n'est jamais initialisé !
Fallait déjà écrire un truc genre
pProfile profile = new Profile;

Taz

bisounours-codeur

cris56 a écrit :

ben non c'est du c++, ya une classe, on peut pas faire ca en c?

ton code est du C (et du mauvais) à part les cout...

1 bon point pour gilou. seulement moi, seulement c'est plus propre de séparer E/S et stockage de données.

donc le mieux, est de

Code :

cout<<"entrer le nom:"<<flush;
cin>>nom;
cout<<"entrer l'adresse:"<<flush
cin>>adresse;
Profile prof(nom, addresse);
prof.grincheux();

cris56

merci a tous pour vos reponse, j'ai arrange ca en suivant vos conseil mais du coup je suis plus du tout sur, je connais pas tout ca

Code :

#include <iostream>
#include <string>
using std::cin;
using std::cout;
using std::endl;
using std::string;
using std::flush;
class Profile
{
string nom;
string adresse;
public:
Profile(const string& p_nom,const string& p_adresse)
{
adresse=p_adresse;
nom=p_nom;
};
string getNom() const
{
return nom;
};
string getAdresse() const
{
return adresse;
};
};
int main()
{
string nom,adresse;
cout<<"entrer le nom:"<<flush;
cin>>nom;
cout<<"entrer l'adresse:"<<flush;
cin>>adresse;
Profile profile(nom,adresse);
cout<<profile.getNom()<<" "<<profile.getAdresse()<<endl;
}

Message édité par cris56 le 25-03-2004 à 17:42:08

Publicité

Taz

bisounours-codeur

bon, allez, on va dire que ça va

cris56

merci,
flush c'est pour vider le tampon, donc tu fait ca avant une saisie, c'est ca?

sinon pour le constructeur, j'ai vu qu'on pouvais faire ca :

Profile(string p_nom,string p_adresse): adresse(p_adresse),nom(p_nom){};

quel est l'avantage ?

j'ai passer les string par reference dans le constructeur (voir maj de l'autre post), c'est mieux ou inutile?

Message édité par cris56 le 25-03-2004 à 17:43:18

Taz

bisounours-codeur

cris56 a écrit :

merci,
flush c'est pour vider le tampon, donc tu fait ca avant une saisie, c'est ca?

oui. cout est généralement bufferisé en mode ligne, donc quand tu fais un cout sans '\n' suivi d'un cin, il vaut mieux flusher pour que l'affichage se fasse bien avant la saisie

cris56 a écrit :

Profile(const string &p_nom, const string &p_adresse): adresse(p_adresse),nom(p_nom){};

quel est l'avantage ?

1) les const & évite la copie inutile des paramètres
2) c'est la synthaxe d'initialization. sinon, chaque champ est d'abord construit implicitement (comme si tu écrivait adresse(), puis operator=, etc ... la syntaxe d'initialisation est simple, logique, et 3x fois plus rapide

cris56

a ouai d'accord, c'est bon a savoir ca

sinon encore une question (desole, c'est la derniere)

pour les accesseur je retourne un objet string par valeur, mais je devrai plutot retourner par reference aussi, non?

en tout cas je te remerci pour tout ces eclaircissement

Taz

bisounours-codeur

si tu peux. à ce moment là, retourne une const&

cris56

le const a quel utilite a ce moment la? pourquoi il vaut mieux le mettre en retour?
(c'est peut etre con ce que je dit mais sur la plupart des sources que je vois le const est omis pour une valeur retour par reference...)

Taz

bisounours-codeur

ben si tu renvoie une & seule, tu permets aussi l'accès en écriture (à moins que ce ne soit ce que tu veux)

cris56

ok, oui en effet je ne desir pas proteger par un const pour le moment

gilou

Modérateur
Modzilla

> c'est plus propre de séparer E/S et stockage de données

J'allais pas tout lui faire non plus...

Sinon j'aurais fait quelque chose comme:

Code :

#include <iostream>
#include <string>
using std::cin;
using std::cout;
using std::endl;
using std::flush;
using std::string;
class UserProfile
{
string nom;
string adresse;
public:
UserProfile(const string& n, const string& a) : nom(n), adresse(a) {}
void print() const;
};
void UserProfile::print() const
{
cout<<nom<<" "<<adresse<<endl;
}
int main()
{
string nom;
string adresse;
cout<<"entrer le nom:"<<flush;
cin>>nom;
cout<<endl;
cout<<"entrer l'adresse:"<<flush;
cin>>adresse;
cout<<endl;
UserProfile profile(nom, adresse);
profile.print();
}

EDIT: Ah je vois que c'est a peu pres a quoi le topic est arrive pendant mon absence (pour bechage de poireaux)

A+,

Message édité par gilou le 25-03-2004 à 18:57:26

Taz

bisounours-codeur

la tu t'es planté gilou.

Code :

void UserProfile::print(ostream &os) const
{
os << nom << " " << adresse << '\n';
}
prof.print(cout);

après tu peux rajouter

Code :

inline ostream& operator<<(ostream &os, const Profile &p)
{ p.print(os); return os; }

cout << prof;

gilou

Modérateur
Modzilla

> la tu t'es planté gilou.

Au sens que je l'ai pas fait assez generique pour ton gout?? J'ai pas essaye de sortir de son pb initial, sinon il y aurait aussi des accesseurs par champ dans la classe.
A+,

Message édité par gilou le 25-03-2004 à 18:23:53

Taz

bisounours-codeur

gilou a écrit :

> la tu t'es planté gilou.

Au sens que je l'ai pas fait assez generique pour ton gout??
A+,

oui, et qu'il manquait un const

cris56

moi j aurai penseé au const

sinon je vait voir ca pour le cout<<prof

gilou

Modérateur
Modzilla

Taz a écrit :

oui, et qu'il manquait un const

Argh! I was Tazed!
C'est un oubli dont je suis coutumier quand je me relis pas celui là.
Je vas editer mon post original.
(Pour etre honnete, pour le print, j'avais bien pense a une fonction generique avec un ostream lors de mon premier post, mais j'avais eu la flemme de la taper, :whistle: )
A+,

Message édité par gilou le 25-03-2004 à 18:59:18

Publicité

FORUM HardWare.fr

Programmation

C++

probleme avec pointeur sur objet

Sujets relatifs
[C] Problème pour obtenir l'@ IP	Probleme html
Un probleme avec unserialize( )	Problème pour signer une applet : pb avec jarsigner
probleme avec la couleur de fond d'un DIV...	Probleme avec cette merd... de easy php/ myadmin
probleme avec iostream.h dans visual.net	Help : Probleme PHP : tableaux et variables!
Probleme au niveau des classes...	Problème de connection à une base de données Oracle
Plus de sujets relatifs à : probleme avec pointeur sur objet

Page générée en 0.080 secondes