probleme avec pointeur sur objet

probleme avec pointeur sur objet - C++ - Programmation

Marsh Posté le 25-03-2004 à 15:54:48    

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 :
  1. #include <iostream.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  5. typedef class Profile
  6. {
  7. public:
  8. char* nom;
  9. char* adresse;
  11. void init();
  12. }*pProfile;
  14. void Profile::init()
  15. {
  16. char nomp[100];
  17. cout<<"entrer le nom:";
  18. cin>>nomp;
  19. nom=(char *)malloc(strlen(nomp));
  20. strcpy(nom,nomp);
  22. char adressp[100];
  23. cout<<"entrer l'adresse:";
  24. cin>>nomp;
  25. adresse=(char *)malloc(strlen(adressp));
  26. strcpy(adresse,adressp);
  27. }
  29. void main(void)
  30. {
  31. pProfile profile;
  33. profile->init();
  35. cout<<profile->nom<<"  "<<profile->adresse<<endl;
  36. }

Reply

Marsh Posté le 25-03-2004 à 15:54:48   

Reply

Marsh Posté le 25-03-2004 à 16:05:55    

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
Reply

Marsh Posté le 25-03-2004 à 16:08:25    

pourquoi? c'est qui taz?


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

Marsh Posté le 25-03-2004 à 16:11:02    

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
Reply

Marsh Posté le 25-03-2004 à 16:12:09    

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

Reply

Marsh Posté le 25-03-2004 à 16:46:45    

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
Reply

Marsh Posté le 25-03-2004 à 16:47:37    

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 :
  1. #include <iostream>
  2. #include <string>
  4. using std::cin;
  5. using std::cout;
  6. using std::endl;
  7. using std::string;
  9. class User_profile
  10. {
  11. public:
  12.   string nom;
  13.   string adresse;
  14.   void init();
  15. };
  17. void User_profile::init()
  18. {
  19.     cout<<"entrer le nom:"<<endl;
  20.     cin>>nom;
  21.     cout<<"entrer l'adresse:"<<endl;
  22.     cin>>adresse;
  23. }
  25. int main()
  26. {
  27.   User_profile profile;
  29.   profile.init();
  30.   cout<<profile.nom<<"  "<<profile.adresse<<endl;
  31. }


 
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+,

Reply

Marsh Posté le 25-03-2004 à 17:00:25    

cris56 a écrit :

Code :
  1. void main(void)
  2. {
  3. pProfile profile;
  5. profile->init();
  7. cout<<profile->nom<<"  "<<profile->adresse<<endl;
  8. }




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

Reply

Marsh Posté le 25-03-2004 à 17:04:36    

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 :
  1. cout<<"entrer le nom:"<<flush; 
  2. cin>>nom; 
  3. cout<<"entrer l'adresse:"<<flush 
  4. cin>>adresse; 
  6. Profile prof(nom, addresse);
  8. prof.grincheux();

Reply

Marsh Posté le 25-03-2004 à 17:32:59    

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 :
  1. #include <iostream>
  2. #include <string>
  5. using std::cin;
  6. using std::cout;
  7. using std::endl;
  8. using std::string;
  9. using std::flush;
  11. class Profile
  12. {
  13. string nom;
  14. string adresse;
  16. public:
  19. Profile(const string& p_nom,const string& p_adresse)
  20. {
  21.  adresse=p_adresse;
  22.  nom=p_nom;
  23. };
  25. string getNom() const
  26. {
  27.  return nom;
  28. };
  30. string getAdresse() const
  31. {
  32.  return adresse;
  33. };
  34. };
  36. int main()
  37. {
  38. string nom,adresse;
  40. cout<<"entrer le nom:"<<flush; 
  41. cin>>nom; 
  42. cout<<"entrer l'adresse:"<<flush;
  43. cin>>adresse;
  45. Profile profile(nom,adresse);
  47. cout<<profile.getNom()<<"  "<<profile.getAdresse()<<endl;
  48. }


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

Marsh Posté le 25-03-2004 à 17:32:59   

Reply

Marsh Posté le 25-03-2004 à 17:36:01    

bon, allez, on va dire que ça va

Reply

Marsh Posté le 25-03-2004 à 17:39:54    

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
Reply

Marsh Posté le 25-03-2004 à 17:44:04    

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

Reply

Marsh Posté le 25-03-2004 à 17:49:41    

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

Reply

Marsh Posté le 25-03-2004 à 17:51:02    

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

Reply

Marsh Posté le 25-03-2004 à 17:54:25    

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

Reply

Marsh Posté le 25-03-2004 à 17:57:06    

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

Reply

Marsh Posté le 25-03-2004 à 18:01:54    

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

Reply

Marsh Posté le 25-03-2004 à 18:18:03    

> 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 :
  1. #include <iostream> 
  2. #include <string> 
  4. using std::cin;
  5. using std::cout;
  6. using std::endl;
  7. using std::flush;
  8. using std::string;
  10. class UserProfile 
  11. { 
  12. string nom; 
  13. string adresse;
  15. public:
  16. UserProfile(const string& n, const string& a) : nom(n), adresse(a) {}
  17. void print() const;
  18. }; 
  20. void UserProfile::print() const
  21. {
  22.   cout<<nom<<"  "<<adresse<<endl;
  23. }
  25. int main() 
  26. { 
  27. string nom;
  28. string adresse;
  30.   cout<<"entrer le nom:"<<flush; 
  31.   cin>>nom; 
  32.   cout<<endl;
  33.   cout<<"entrer l'adresse:"<<flush; 
  34.   cin>>adresse; 
  35.   cout<<endl;
  37.   UserProfile profile(nom, adresse);
  38.   profile.print(); 
  40. }


 
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
Reply

Marsh Posté le 25-03-2004 à 18:20:16    

la tu t'es planté gilou.  

Code :
  1. void UserProfile::print(ostream &os) const
  2.   {
  3.       os << nom << "  " << adresse << '\n';
  4.   }
  6. prof.print(cout);


 
après tu peux rajouter

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


 
cout << prof;
 

Reply

Marsh Posté le 25-03-2004 à 18:22:42    

> 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
Reply

Marsh Posté le 25-03-2004 à 18:23:56    

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

Reply

Marsh Posté le 25-03-2004 à 18:28:03    

moi j aurai penseé au const  :D
 
sinon je vait voir ca pour le cout<<prof

Reply

Marsh Posté le 25-03-2004 à 18:32:42    

Taz a écrit :

oui, et qu'il manquait un const

:o 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
Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed