Discussion :

[Chewbi]

Bonjour,

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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 ?

[Chewbi]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

- Que vaut currSize ?

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

- 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

[JolyLoic]

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 ?

[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.