Discussion :

[mahboub]

Salut 'a vous,

je me demande c'est quoi la difference entre

Code :

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

1
2
3
 
chaine(char *ch2);// parametre 
  chaine(char & ch2);// constructuer par copie

pour pouvoir dire que le premier est constructueur par parametre et le deuxieme par copie. finalement pourquoi il ya une difference entre char * et char & puisuqe les deux renvoi une addresse
voic mon code [CODE]

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
 
 
#ifndf CHAINE
#define CHAINE
#include <iostream>
#include <cstring>
using namespace std;
 
class chaine
{
  char *ch_;
  int longueur_;
  int lmax_;
  static int quantite_;
 
 public:
  chaine(int n=100);
  chaine(char *ch2);// parametre 
  chaine(char & ch2);// constructuer par copie
  ~chaine();
  void affiche(){cout<<ch_<<endl;}
  static int combien(){return quantite_;}
 
};
#endif
/CODE]

[Goten]

Si tu parles du constructeur de copie alors non la bonne forme est :

Code :

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

1
2
 
chaine(const chaine&);

[mahboub]

Si tu parles du constructeur de copie alors non la bonne forme est :

Code :

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

1
2
 
chaine(const chaine&);

tu as raison je viens de voir mon erreur hehe

cependant c'est quioi la difference char & et char*

[koala01]

Salut,
char* est ... un pointeur sur caractères (autrement dit, même si le raccourcis est un peu rapide, l'équivalent à une chaine de caractères "C style"), alors que char & est, tout simplement, une référence sur un caractère unique

Ceci dit, pourquoi diable te faire ch à créer ta propre classe de type "chaine de caractères", alors qu'il existe la classe string, disponible (comme tout ce qui est issu du standard) dans l'espace de noms std, par simple inclusion du fichier d'en-tête <string>

[mahboub]

merci pour ta reponse je comprends mieux.
en fait je debute et j'ai trouve l'idee de cree une calss string bonne pour deux raisons
familiarise avec la surcharge d'operateur et manipuler destchaines de caratere et des pointeur
il n'y a pas mieux
meme si c'est casse pied je l'avoue

[koala01]

Je comprend l'attrait "didactique" de réinventer la roue, mais, le gros problème est qu'il est souvent synonyme de prise de mauvaises habitudes...

Celle de manipuler des pointeurs "à tout va", sans penser aux possibilités fournies par le standard en est une, et il y a surement des exercices plus corrects à envisager pour s'habituer à leur utilisation

Si le seul but est de t'habituer à manipuler des pointeurs, tu devrais, par exemple, envisager un petit projet où tu mettrais en oeuvre les méthodes qui nécessitent vraiment leur utilisation (héritage, polymorphisme et autres en font plus ou moins partie )

D'un autre coté, tu devrais très rapidement prendre l'habitude de travailler à coup de std::string, std::vector et autres conteneurs, car une mauvaise habitude est d'autant plus difficile à perdre au profit d'une bonne que la mauvaise aura eu le temps de s'installer

[mahboub]

salut,
merci pour tes conseils.
j'ai un progblem avec le resutat ca compile bien ca affiche le resultat
mais ca affiche un tres grand ... ou
merci

bash-3.2$ g++ main.cpp chaine.cpp chaine.h -o chain
bash-3.2$ ./chain
delem alikomm hahhaha
*** glibc detected *** ./chain: double free or corruption (out): 0xbfdd5d68 ***
======= Backtrace: =========
/lib/libc.so.6[0x2b5ac1]
/lib/libc.so.6(cfree+0x90)[0x2b90f0]
/usr/lib/libstdc++.so.6(_ZdlPv+0x21)[0x1df111]
/usr/lib/libstdc++.so.6(_ZdaPv+0x1d)[0x1df16d]
./chain(__gxx_personality_v0+0x2ff)[0x804899f]
./chain(__gxx_personality_v0+0x1d6)[0x8048876]
/lib/libc.so.6(__libc_start_main+0xe0)[0x262390]
./chain(__gxx_personality_v0+0x41)[0x80486e1]
======= Memory map: ========
00110000-0012b000 r-xp 00000000 08:03 648970 /lib/ld-2.7.so
0012b000-0012c000 r--p 0001a000 08:03 648970 /lib/ld-2.7.so
0012c000-0012d000 rw-p 0001b000 08:03 648970 /lib/ld-2.7.so
0012d000-0012e000 r-xp 0012d000 00:00 0 [vdso]
0012e000-0020c000 r-xp 00000000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
0020c000-0020f000 r--p 000dd000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
0020f000-00211000 rw-p 000e0000 08:03 111844 /usr/lib/libstdc++.so.6.0.8
00211000-00217000 rw-p 00211000 00:00 0
00217000-0023e000 r-xp 00000000 08:03 645918 /lib/libm-2.7.so
0023e000-0023f000 r--p 00026000 08:03 645918 /lib/libm-2.7.so
0023f000-00240000 rw-p 00027000 08:03 645918 /lib/libm-2.7.so
00240000-0024b000 r-xp 00000000 08:03 648978 /lib/libgcc_s-4.1.2-20070925.so.1
0024b000-0024c000 rw-p 0000a000 08:03 648978 /lib/libgcc_s-4.1.2-20070925.so.1
0024c000-0039f000 r-xp 00000000 08:03 648853 /lib/libc-2.7.so
0039f000-003a1000 r--p 00153000 08:03 648853 /lib/libc-2.7.so
003a1000-003a2000 rw-p 00155000 08:03 648853 /lib/libc-2.7.so
003a2000-003a5000 rw-p 003a2000 00:00 0
08048000-0804a000 r-xp 00000000 00:19 136314967 /usagers/hamaa/Desktop/temp/string/chain
0804a000-0804b000 rw-p 00001000 00:19 136314967 /usagers/hamaa/Desktop/temp/string/chain
0897e000-0899f000 rw-p 0897e000 00:00 0 [heap]
b7e00000-b7e21000 rw-p b7e00000 00:00 0
b7e21000-b7f00000 ---p b7e21000 00:00 0
b7faf000-b7fb1000 rw-p b7faf000 00:00 0
b7fd6000-b7fd7000 rw-p b7fd6000 00:00 0
bfdc2000-bfdd7000 rw-p bffeb000 00:00 0 [stack]
delem alikomm hahhahaAbandon

Code :

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

1
2
3
4
5
6
7
8
9
10
11
nclude "chaine.h"
 
int main()
{
 
  chaine c(10);
  chaine  a("delem alikomm hahhaha");
  a.affiche();
  cout<< a.tostring();
  return 0;
}

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
 
 
 
#ifndef CHAINE
#define CHAINE
#include <iostream>
#include <cstring>
using namespace std;
 
class chaine
{
  char *ch_;
  int longueur_;
  int lmax_;
  //static int quantite_;
 
 public:
  chaine(int n=100);
  chaine(char *ch2);// parametre 
  chaine(chaine & ch2);// constructuer par copie
  int size(){ return  strlen(ch_);}
  ~chaine();
  char* tostring(){return ch_;};
  void affiche(){cout<<ch_<<endl;}
  //static int combien(){return quantite_;}
 
};
#endif

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
 
#include "chaine.h"
 
chaine:: chaine(int n)
{
  if(n>0)
    {
      ch_= new char[n+1]
	;
       longueur_ =n ;
       //   	quantite_++;
    }
  else
    {
      ch_= new char[101];
        longueur_ = 100;  
	//	quantite_++;
    }
 
  lmax_=100;
 
 
}
chaine::chaine(char *ch2)
{
  if(ch2)
    {
      if (strlen(ch2)-1>lmax_)
	{
	  ch_ = new char[lmax_];
	  ch_=strcpy(ch_,ch2);
	}
      else
	{
	  ch_=strncpy(ch_,ch2,100);
	}
 
 
    }
  else
    {
 
    }
 
}// parametre 
 
chaine::chaine(chaine  & ch2)
{
  ch_= strcpy(new char[ch2.size()+1],ch2.tostring());
  longueur_ =strlen(ch_) ;
 
}
// constructuer par copie
chaine::~chaine()
{
  if(ch_)
    delete [] ch_;
  // quantite_--;
}

[mahboub]

c'est cette instruction qui me fait defaut

Code :

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

 chaine  a("delem alikomm hahhaha");

merci pour votre aide

[Arzar]

Ben faut investir dans un debuggeur !
J'ai juste copié collé ton code, lancé et boum, il s'est planté ici :

Code :

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

1
2
3
4
 else
{
   ch_=strncpy(ch_,ch2,100);
}

Sur l'appel de

Code :

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

chaine  a("delem alikomm hahhaha");

Juste deux remarques :
(1) Les nombres magiques c'est mal. Pourquoi 100 ? Si tu ne connais pas la taille de la chaine alors la convention est plutôt de construire une chaine vide.
(2) Vouloir rattraper une erreur grave sans prévenir l'utilisateur c'est mal. Si l'utilisateur a donné un nombre négatif pour le constructeur de la chaine c'est qu'il a une grave erreur de logique dans son programme. Autant que ça plante directement plutôt qu'allouer une chaine de 100 caractères silencieusement.

[mahboub]

Ben faut investir dans un debuggeur !
J'ai juste copié collé ton code, lancé et boum, il s'est planté ici :

Code :

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

1
2
3
4
 else
{
   ch_=strncpy(ch_,ch2,100);
}

Sur l'appel de

Code :

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

chaine  a("delem alikomm hahhaha");

Juste deux remarques :
(1) Les nombres magiques c'est mal. Pourquoi 100 ? Si tu ne connais pas la taille de la chaine alors la convention est plutôt de construire une chaine vide.
(2) Vouloir rattraper une erreur grave sans prévenir l'utilisateur c'est mal. Si l'utilisateur a donné un nombre négatif pour le constructeur de la chaine c'est qu'il a une grave erreur de logique dans son programme. Autant que ça plante directement plutôt qu'allouer une chaine de 100 caractères silencieusement.

Merci j'ai vu l'erreure copy sans allocation de memoire
merci je vais tester