allocation mémoire dans une classe

Bonjour,
voilà je suis bloquée alors je vais mettre l'énoncé de tout ce que je veux faire. Je suis débutante.

Voilà, je souhaite écrire une classe Chaine qui me permet de manipuler et de créer une chaine de caractères.

Dans cette classe Chaine j'ai deux propriétes:

- longueur de ma chaine (entier)
- adresse d'une zone allouée dynamiquement(pointeur sur caractères)

Ensuite, voici mes méthodes:

-chaine() qui initialise une chaine à vide
-chaine(char *) qui initialise la chiane passée en argument
-operator =
-operator ==
-operator + qui permet de concaténer deux chaines de caractères
-operator[] qui permet d'accéder à un caractère de rang donné par une chaine de caractère
-affiche () qui permet l'affichage d'une chaine de caractères

Voici ce que j'ai commencé en prenant note des critiques:

#include <iostream>

class Chaine
{
private :

int m_longueur;
char *m_allouee;

public :

//constructeurs et destructeurs
Chaine();
Chaine( char * texte );
~Chaine();

//opérateurs
void operator=( Chaine & ch );
int operator== (Chaine );
Chaine operator+ (Chaine ch );
char operator[](int i );
void affiche();


};

//constructeur par défaut
Chaine::Chaine()
{
m_longueur = 0; // taille de base
m_allouee = new char[ m_longueur ];
}


//Chaine::Chaine(char *texte)//constructeur2
//{
//
//
//}

void Chaine::operator = (Chaine &ch)
{
m_longueur = ch.m_longueur;
m_allouee = ch.m_allouee;

}

int Chaine::operator == (Chaine ch)
{
m_longueur = ch.m_longueur;
m_allouee = ch.m_allouee;
}

Chaine Chaine::operator +(Chaine ch)
{
Chaine resultat; // création de l'objet résultat
resultat.m_longueur = m_longueur + ch.m_longueur;
resultat.m_allouee = ch.m_allouee;
return resultat;
}

char Chaine::operator [] (int i)
{
}


//destructeur
Chaine::~Chaine()
{
delete[] m_allouee;
}


void Chaine::affiche()
{
cout <<"La longueur de la chaine est de :" << m_longueur << endl;
}


int main()
{

Chaine a ("Bonjour"), b("Maria") , c,d ("Bonjour"),e;

if(a==b) cout <<"Gagne !\n";
else cout <<"Perdu ! \n";
if(a==d) cout <<"Gagne !\n";
else cout <<"Perdu ! \n";

cout <<"a:";
a.affiche();
cout <<"b:";
b.affiche();
cout <<"d:";
d.affiche();
c = a+b;
cout <<"c:";
c.affiche();

for(int i=0; c !='\0'; i++)

cout << c;
system("pause");
return 0;

}


Le problème c'est que je n'arrive pas à remplir mon deuxième constructeur. Je ne sais pas s'il est bon ou pas

Ensuite pour l'operator [](int i) je ne vois pas comment procéder ni pour void Chaine::operator = (Chaine &ch)

Merci de prendre le temps de m'aider.

donc ton 2er constructeur donnerai

CChaine::CChaine(char pChaine)
{
    int iSize = strlen(pChaine) // ou creer ta propre methode pour le pas utilisé de fonction externe
    if (iSize > 0)
   {
          m_longueur = iSize ;
          m_allouee = new char[iSize];

          for (int i=0; i < iSize; i++)
          {
                   m_allouee[i] = pChaine[i];
           }
    }
}

void CChaine::operator = (CChaine &ch)
devrais etre
CChaine & CChaine::operator = (const CChaine &ch);
tu retourne simplement (*this)

ton operator == selon la norme parle d',operateur binaire soit est egual à , donc devrai retourné un bool et dans ta mthode elle devrais detecter si la chaine est pareil à un autre

Bonjour,
merci de m'aider. Comme je débute, j'aimerais savoir dans le 2ème constructeur à quoi correspond pChaine. Est-ce un pointeur ? J'ai donc fait cela avec comme paramètre *texte. Mais je ne sais pas si cela est bon :

Chaine::Chaine(char *texte)
{
    int iSize = strlen(texte);
    if (iSize > 0)
   {
          m_longueur = iSize ;
          m_allouee = new char[iSize];

          for (int i=0; i < iSize; i++)
          {
                   m_allouee[i] = texte[i];
           }
    }
}

Je retravaille mes autres méthodes et je reposte après pour une vérification.Merci encore.

Il manquera le 0 de fin dans chaine.
J'aurais fais quelque chose comme ceci:

Chaine::Chaine(char *texte)
{
    int iSize = strlen(texte);
    if (iSize > 0)
   {
          m_longueur = iSize ;
          if((m_allouee = new char[iSize+1])) {
              char *c = m_allouee;
              while((*c = *texte)) {
                 c++;
                 texte++;
              }
          }
    }
}

Au passage, il faut TOUJOURS vérifier que la mémoire a bien été allouée. Aussi, il faut une place supplémentaire dans ton buffer pour le 0 de fin de chaine.
Finalement, quand on met des classes en paramètre, il faut prendre l'habitude de passer une référence vers cette classe. Sinon, chaque appel à la méthode génèrera un appel au constructeur de copie.

Pour ton l'operateur [](int i), suffis de prendre i comme index de ton tableau et retourner le char correspondant sous forme de référence ou non, à toi de voir.

L'opérateur == devrait servir à comparer les chaines. strcmp.

Tes opérateurs + et = causeront bien des problèmes. En effet, tu ne fais que faire pointer l'attribut this.m_allouee sur sont correspondant dans l'autre classe.
Quand l'autre classe libèrera sa mémoire, les pointeurs deviendrons invalides. De plus, les classes qui pointent désormais sur des zones invalides libèrerons aussi leur mémoire... crash en perspective.

Pour ton opérateur =, peut-être faudrait il faire, à la place, un constructeur de copie.

Pour ton opérateur +, il faudra concaténer le contenue de ch.m_allouee dans this.m_allouee. Pour ça, faudra faire un nouveau buffer ayant la taille de this.m_longueure + ch.m_longueure puis copier le contenue de l'ancien buffer dans le nouveau buffer avant de détruire l'ancien. Grosse perte de temps mais c'est comme ça avec l'objet.

C++ (@++)

Bonjour,

 Je vais retravaillais en prenant note de ces commentaires. C'est sympa de prendre le temps de m'expliquer.
Merci infiniment