[Débutant] trouver les erreurs dans un code

Bonjour,

J'ai un code en C++ dans lequel je dois trouver les erreurs, le voici :

Code:

---

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

typedef double T; class A{ public: int n; T *v; A(int nn=0):n(nn),v(n>0 ? new T[n]:0) {} T& operator [] (int i) { assert(v && (i>=0) && (i<n) ); return v[i]; }   ~A() { if (v) delete [] v; }   }   int main( int argv, const char *[]) { A a(10), b(10); for (int i=0;i<10;i++) a[i]=i; b=a; A c(b); assert( c[4]==4); return 0; }

---

La seule erreur que j'ai trouvée pour l'instant c'est le ';' qui manque après l'accolade de fin de classe.
Dans le constructeur par exemple, je comprends pas ce que signifie cette ligne :

Code:

---

v(n>0 ? new T[n]:0)

---

Je vois bien que c'est pour construire un tableau de type T et qui a nn éléments, mais c'est quoi le '?' ?
Pareil dans le main, j'ai jamais vu des const char *[] je me dis que l'erreur vient peut-etre de là sachant que j'ai toujours vu des main du type :

Code:

---

int main ( int argc, char *argv[])

---

Merci de bien vouloir m'aiguiller :D

Déjà un premier conseil: indente le code correctement, tu verras certainement des erreurs apparaitre :)

as tu essayé de compiler le code et de voir ce que disait le compilateur ?
as tu essayé de lancer le programme ?

la syntaxe test ? resutat1 : resultat2 est un operateur qui, selon le test, renvoie la premiere valeur ou bien la seconde. donc

Code:

---

v(n>0 ? new T[n]:0)

---

signifie qu'on initialise v avec le resultat de

Code:

---

n>0 ? new T[n]:0

---

cela signifie que si n est superieur a 0 alors on initialisera v avec new T[n] ou sinon a 0.

const char * ca ne change pas grand chose, cela signifie que tu ne peux pas modifier la ligne de commande. dans la mesure ou tu ne l'utilises pas, ca ne doit pas psoer trop de probleme.

il y a un probleme bien plus important, c'est que la classe A ne definit pas de constructeur de copie mais que la fonction main appelle une copie. Le constructeur de copie (et l'operateur de recopie) sont donc definis par defaut, et le code compile mais est incorrect.

Qu'est ce que ça veut dire idente ? :mrgreen:

la touche des scores dans counter strike

Merci Screetch.
Effectivement j'avais remarqué le constructeur par copie n'est pas défini.
J'ai en fait avec l'exercices les erreurs de compilation de données, mais j'essaye avant de trouver sans regarder.

Citation:

---

Envoyé par rouliane

Qu'est ce que ça veut dire idente ? :mrgreen:

---

Oui pardon, faute de frappe: indente.

Indenter le code c'est le mettre dans la bonne forme (avec les tabulations).
Un exemple de code non indenté:

Code:

---

1 2 3 4 5 6 7

class Toto { public: Toto(); private: int membre; };

---

Un exemple de code indenté:

Code:

---

1 2 3 4 5 6 7 8

class Toto { public: Toto();   private: int membre; };

---

ben en fait c'est la seule erreur de syntaxe, et le fait que assert ne soit pas defini (manque un #include)

apres ca l'autre probleme est bien plus important

Merci pour la precision sur indente, j'ai "indenté" mon code :mrgreen:

Effectivement, j'ai rajouté cassert en include et mis le ; je n'ai plus d'erreur.
Je vais essayer de régler le problème du constructeur par copie.

Merci à vous en tout cas.

Alors j'ai fait le constructeur par copie suivant, apparemment y'a pas d'erreur de syntaxe ni de compilation, mais je ne sais pas si c'est juste :

Code:

---

1 2 3 4 5

A(const A & source) { n=source.n; v= new T[n]; for (int i=0; i<n; i++) v[i]=source.v[i]; }

---

Qu'en pensez-vous ?

Attention rouliane, il vaut mieux respecter la règle décrite dans le premier constructeur, et qui dit que si le tableau n'a pas d'élément, il n'a pas non plus de tableau v ("new T[0]" ne renverra pas un pointeur nul). Dans ta version, la copie aura une différence avec l'original, ce qui n'est pas nécessaire et peut amener pas mal de petites confusions.

Et aussi petit "truc": delete[] 0; ne fait rien, tu peux dont te passer du if dans ton destructeur.

Cheers,

Carl

Merci !