Initialisation dynamique d'un tableau

Bonjour,

Je cherche à initialiser un tableau de caractères (contenu dans une classe) de façon dynamique.
J'ai donc écris le code suivant :

class Coord
{
  public :
     
         char *m_Nom;
         //autres données...         
};


int main(int argc, char *argv[])
{
    char *pConteneur = new char[100];

    cout<<"Entrez votre nom :"<<endl; 
    gets(pConteneur);
   
    longueur = strlen(pConteneur);
   
    Coord personne;
    personne.m_Nom = new char[longueur + 1]; //allocation de mémoire
   
   
    for(int i = 0; i < longueur; i++) //initialisation
    {
      personne.m_Nom[i] = *(pConteneur + i); 
    }  
  
 
    cout<<personne.m_Nom<<endl;
   
    delete [] pConteneur; //désallocation
    delete [] personne.m_Nom;
 
  system("PAUSE"); 
  return 0;
}

Le code fonctionne, toutefois ne l'ayant pas testé dans toutes les contextes possibles je voudrais savoir :

1) si l'allocation et la désallocation de mémoire du membre m_Nom sont correctes ?
2) si l'initialisation de ce membre est correcte ?
3) Est il possible d'utiliser ce code sans risquer un plantage (à cause des nombreux pointeurs) ou d'autres    problèmes plus subtiles.

Merci

Tintin 72

Belle entrée avec un joli risque de buffer overflow.
Et si le prénom dépasse 100 caractères hein ?

Stef
[ Lien ] LE site pour les bassistes

Oui bien sûr :-) le 100 c'est arbitraire, je pourrai mettre 1000 par ex,
mais je voulais savoir ce qu'il en est pour le reste du code.

"Et si le prénom dépasse 100 caractères hein ?"
Le gars peut faire un procés à ses parents ;-)

nb: de toute façon on est obligé de mettre une valeur arbitraire à un moment donné, on peut pas faire du total dynamique en C/C++, non ?

Tintin 72

Le principe c'est que si tu connais la limite (100), l'allocation dynamique ne sert à rien, autant faire char pConteneur[100].
On utilise surtout de l'allocation dynamique quand la taille n'est pas connue à la compilation:
int n;
n=...;
char* pConteneur = new char[n];

Sinon ton code m'a l'air correct

Sisi on doit pouvoir avec un vecteur...

Bon en admettant les 100 caractères, voilà ma correction ^^

1.Version marrante
int main(int argc, char *argv[])
{
    Coord personne;
    personne.m_Nom = new char[100]; 

    cout<<"Entrez votre nom :"<<endl; 
    gets(personne.m_Nom);    
    cout<<personne.m_Nom<<endl;
    
    delete [] personne.m_Nom;
 
  system("PAUSE"); 
  return 0;
}

2.Plus approprié pour ton souci
int main(int argc, char *argv[])
{
    char *pConteneur = new char[100];

    cout<<"Entrez votre nom :"<<endl; 
    gets(pConteneur);
    
    //longueur = strlen(pConteneur);
    //Coord personne;
    //personne.m_Nom = new char[longueur + 1]; //allocation de mémoire
    //for(int i = 0; i < longueur; i++) //initialisation
    //{
    // personne.m_Nom[i] = *(pConteneur + i); 
    ///}  
    //
    // Tout cette partie au dessu est sans intéret
    // à remplacer par

    personne.m_Nom = new char[strlen(pConteneur)+ 1];
    strcpy(personne.m_Nom,pConteneur);
  
    cout<<personne.m_Nom<<endl;
   
    delete [] pConteneur; //désallocation
    delete [] personne.m_Nom;
 
  system("PAUSE"); 
  return 0;
}


[ Lien ] LE site pour les bassistes

Effectivement ça va + vite comme ça
Tu dis :.
"Sisi on doit pouvoir avec un vecteur..."
c'est quoi exactement ?

Tintin 72

Tu en sais déjà trop ! Tu verras les vecteurs plus tard, après avoir passé les premières étapes.
Je dis pas ça pour t'embêter mais trop d'infos en même temps, c'est perturbant quand on débute.
Dis toi que le vecteur est un tableau magique où tu peux ranger tout et n'importe quoi, et qu'il s'agrandit au fur et à mesure sans que tu n'ai besoin de manipuler la mémoire.

Stef

[ Lien ] LE site pour les bassistes

Ok, merci beaucoup

Tintin 72