Recopie de tableau

Version imprimable

Recopie de tableau

Bonjour,

Je ne vois vraiment pas où pourrait être le bug, mais bug il y a sûrement ici:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 // initialisation effectuée qu'une seule fois: vect=new SignEval[allocSize]; // plus loin, répété un nombre arbitraire de fois: allocSize=2*allocSize+1; // * SignEval* aux=vect; // pour se rappeler de l'adresse de vect vect=new SignEval[allocSize]; // boucle pour récupérer les anciennes valeurs; currSize<allocSize depuis * for (int i=currSize-1; i>=0; i--) vect[i]=copie(aux[i]); // enfin on libère l'espace mémoire devenu inutile delete [] aux; // puis on ajoute le nouvel élément vect[currSize++]=copie(parametre_de_la_fonction)
Le constructeur de recopie est OK d'après mes tests précédents sur une autre version du programme, et j'alloue bien le bon espace mémoire. J'en déduis qu'il y a un problème avec delete [] aux ou SignEval* aux=vect, mais je ne vois vraiment pas pourquoi :-/

J'obtiens une segFault avec l'utilisation de cette classe, et tout se passe bien en utilisant la STL (wiwiwi je sais utiliser la STL c'est bien mieux, mais je voulais la spécialiser pour ce dont j'ai besoin).

Merci d'avance,
Benjamin.

16/12/2006, 22h26
JolyLoic

Dans le bout de code en question, je ne vois pas de problèmes, mais il y a plein de non dits :
- Es-tu vraiment certain de tes constructeur de copie et opérateur= et destructeurs ? Ils peuvent passer certans tests et pas d'autres.
- Que fait la fonction copie ?
- Que vaut currSize ?
- Question de stye : Pourquoi remplis-tu la copie à l'envers ?
16/12/2006, 23h58
Chewbi
Citation:

Envoyé par JolyLoic

Dans le bout de code en question, je ne vois pas de problèmes, mais il y a plein de non dits :
- Es-tu vraiment certain de tes constructeur de copie et opérateur= et destructeurs ? Ils peuvent passer certans tests et pas d'autres.
- Que fait la fonction copie ?

Voici le constructeur de recopie:
Code:

1 2 3 4 5 6 7 SignEval (int pf, float ev, unsigned long cle) { prof=pf; eval=ev; clef=cle; }
Citation:

- Que vaut currSize ?

Initialisé à zéro, reste toujours <=allocSize et est donc <allocSize après la ligne commentée "// *".

Citation:

- Question de stye : Pourquoi remplis-tu la copie à l'envers ?

Car la comparaison à 0 est plus rapide, même si le compilo s'en chargerait sûrement sans moi :)
17/12/2006, 01h49
JolyLoic

Citation:

Envoyé par Chewbi

Voici le constructeur de recopie:

Ce que tu me montres n'est pas un constructuer de copie, mais un simple constructeur.

Et la fonction copie, elle est écrite comment ?
20/12/2006, 23h19
Chewbi

en fait si, car je zappe l'appel au constructeur de recopie (new appelant le constructeur basique) pour effectuer des assignations "prof=pf; eval=ev; clef=cle;" plus rapides qu'un appel de fonction; donc c'est une sorte de recopie :-)

Désolé de ne pas avoir répondu plus tôt, je préparais mon retour en France (j'étudie au Québec et viens de rentrer pour les vacances).

Sinon, j'ai résolu mon problème bêtement: il suffisait d'initialiser le pointeur vect à 0 dans le constructeur, évitant les appels de delete [] avec des valeurs bidons. J'ai un peu honte :-/

Bonnes fêtes de noël,
Benjamin.