Allocation dynamique de char* ....

Question

Bonjour à tous.... Avant d'acheter un flingue pour de bon, je me suis dit qu'il pourrait etre judicieux de demander de l'aide quelque part... voila, je vais vous exposer le problême. Voici tout d'abord le fragment de code correspondant : /* Constructeur qui marche bien :) */ String::String(char* chaine) { sz = new char[lstrlen(chaine) + 1]; lstrcpy(sz, chaine); } /* LE problême vient d'ici !!*/ String DLLEXPORT String::operator=(char* chaine) { Alloc_Buffer(this->sz, lstrlen(chaine) + 1); lstrcpy(this->sz, chaine); return *this; } /* Peut-etre ce code vous serez utile... */ void String::Alloc_Buffer(char* szBuffer, UINT uSize) { char* chaine = new char[lstrlen(szBuffer) + 1]; lstrcpy(chaine, szBuffer); delete szBuffer; szBuffer = new char[uSize]; lstrcpy(szBuffer, chaine); delete chaine; } >Voila, donc j'explique maintenant le souci...alors quand j'instancie mon objet de type String avec un char* (fonction 1), et que je l'affiche genre avec cout, tout marche nikel..mais quand je déclare d'abord la variable à blanc (j'ai un constructeur aussi pour ça) puis je lui affecte le char*, ça affiche NADA... code de traduction : /* Marche */ String x = "loll"; cout << x.sz << endl; /* Marche pas!! */ String x; x = "looll"; cout << x.sz << endl; voila, merci d'avance...soit de m'aider, soit de confirmer la necessité du flingue -_-"

fredcl · Answer

Bonjour,

void String::Alloc_Buffer(char** szBuffer, UINT uSize) {
  
  char* chaine = new char[lstrlen(*szBuffer) + 1];
  lstrcpy(chaine, *szBuffer);
  
  delete [] *szBuffer;
  
  *szBuffer = new char[uSize];
  lstrcpy(*szBuffer, chaine);
  
  delete chaine;                 
 }

Bon j'ai pas le temps de compiler pour v&#233;rifier, mais un truc dans ce genre devrais aller mieux.

Cordialement

Fred Cailleau-Lepetit (http://cfred.free.fr)

Gendal67 · Answer

d&#233;sol&#233; d'etre nul mais...interet d'avoir un char** plutot qu'un char* ?? je ne calle pas...qu'est-ce qui &#233;tait faux dans mon code et que tu semblerais avoir rectifi&#233; ? Explique moi je suis nul....

vecchio56 · Answer

Parce que tu modifies la valeur du pointeur, tu ne peux donc pas le passer par valeur

Gendal67 · Answer

Jcroyais qu'un passage de char* se faisait toujours par r&#233;f&#233;rence et de mani&#232;re automatique... jvais m&#233;diter la dessus...merci...

fredcl · Answer

Bon pour que tu comprennes oubli que c'est un pointeur imagine que c'est un entier int

tu as ta fonction
void mafonction(int mavaleur)
{
mavaleur = 3;
}

si tu fait
a = 2;
mafonction(a);
cout << a ;

cela affichera 2, car la modification de mavaleur n'as pas d&#233;pass&#233; la port&#233; de la fonction.

si tu as 

void mafonction(int* mavaleur)

{
*mavaleur = 3;

}

l'appel de mafonction(&a); modifiera la valeur de a;

maintenant si tu remplace int par char* tu obtiens void mafonction(char** mavaleur)

et donc tu pourras modifier l'adresse contenue dans le pointeur (et aussi ce qui se trouve &#224; l'adresse indiqu&#233; par le pointeur)

J'esp&#232;re avoir &#233;t&#233; clair, la notion de pointeur est toujours assez difficile &#224; comprendre au d&#233;but. Mais avec un peu de pers&#233;v&#233;rence tu finiras par comprendre.

Cordialement

Fred Cailleau-Lepetit (http://cfred.free.fr)

vecchio56 · Answer

Tu n'a qu'a prendre un exemple simple:

void f(int* x)
{
  // x est une variable locale
  x = 0;
}

int main()
{
  int* x = new int;
  f(x);
  // ici tu vois bien que x ne vaut pas 0
}

Gendal67 · Answer

Bonsoir

Je vous remercie pour ces explications! fredcl, merci, ton code marche et a r&#233;solu le probl&#234;me! Par contre, je ne pensais pas &#234;tre d&#233;butant avec les pointeurs...je connaissais le systeme d'appel par valeur et par r&#233;f&#233;rence avec des types... mais je ne savais pas avec char*... merci &#224; vous deux en tout cas :)

NitRic · Answer

/* LE probl&#234;me vient d'ici !!*/
String DLLEXPORT String::operator=(char* chaine) {
  
  Alloc_Buffer(this->sz, lstrlen(chaine) + 1);
  lstrcpy(this->sz, chaine);
  
  return *this;      
 }



...



String & DLLEXPORT String::operator = (const char * chaine)

{

  ...

  return *this;

}



avec ton &#171; String DLLEXPORT ... &#187; et ton &#171; return *this &#187; tu vas causer

une copie de ta String, retourne une r&#233;f&#233;rence, pas une copie, parce que

tu dois retourner &#171; *this &#187; et non une copie de &#171; *this &#187; ...



tu devrais aussi te faire des m&#233;thodes priv&#233;es pour manager ton buffer

et non l'envoyer un peu n'importe o&#249; pour le manager ...



String & DLLEXPORT String::operator = ( const char * la_chaine )

{

  size_t len = strlen( la_chaine );



  if ( this->_grow( len ) )

  {

    // copie &#171; la_chaine &#187; dans ton &#171; this->sz &#187; ...

  }

  return *this;

}



String & DLLEXPORT String::operator = ( const String & str )

{

  if ( this->_grow( str.Length() ) )

  {

    // copie &#171; str &#187; dans ton &#171; this->sz &#187; ...

  }

  return *this;

}

//...



bool DLLEXPORT String::_grow( size_t new_size )

{

  if ( new_size >= this->current_size ) // this->current_size repr&#233;sente l'espace m&#233;moire occup&#233; par this->sz

  {

    char * new_buff = new char[new_size + EXTRA_SIZE]; // disont que EXTRA_SIZE vaut 8



    if ( this->current_size > 0 )

    {

      if ( this->taille_de_sz > 0 ) // this->taille_de_sz repr&#233;sente le nombre de caract&#232;re dans this->sz

      {

        memcpy(new_buff, this->sz, taille_de_sz);

      }

      delete [] this->sz;

    }

    this->sz = new_buff;

    this->current_size = new_size;

  }

  return true;

}



ce n'est qu'un exemple, faut pas faire de copy/paste ...

et n'utilise pas les attributes de ta classe directement!



class String

{

public:

  String();

  ~String();

  // ...

  const char * getBuffer() const

  {

    return m_buffer;

  }

  size_t getLength() const

  {

    return m_length;

  }

  // ...

private:

  char * m_buffer;

  size_t m_length;

  // ...

};




~(.:: NitRic ::.)~

Allocation dynamique de char* ....

8 réponses

Votre réponse

Discussions similaires