C++

[C++] problème avec delete []

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C++] problème avec delete []

Library

bonjour à tous

J'ai un petit problème, et je pense que c'est du à delete[]. Je n'ai ni erreur de compilation, ni erreur de segmentation, mais mon programme s'arrete (sans me redonner la main) au milieu d'une boucle. Voici la portion de code qui pose problème :

Code :

double * Reseau::Rcalcule_sortie (double * exemples)
{
// cette fonction s'occupe de calculer la sortie du réseau
cout << " a "<< flush ;
// initialisation de l'entrée
double * Rentree=new double[Rnb_entrees[0]];
for (int i=0;i<Rnb_entrees[0];i++)
{
Rentree[i]=exemples[i+Rdecalage]; // on remplit l'entrée avec les valeurs du vecteur d'entrée
}
double * entree=new double[Rnb_entrees[0]];
entree=Rentree; // entree est le vecteur d'entrée de la couche de neurones en cours
double * sortie=new double[Rcouche[0]]; // sortie est le vecteur de sortie de la couche de neurones en cours
cout << " b " << flush;
// calcul couche par couche de la sortie
int nprec=0;
for (int i=0;i<Rnbcouches;i++) // de la premiere à la derniere couche de neurones
{
cout << Rnbcouches << flush;
for (int j=0;j<Rcouche[i];j++) // du premier neurone de la couche au dernier
{
sortie[j]=neurone[nprec+j].Ncalcule_sortie(entree); // on calcule la sortie
// du neurone en cours et on la rentre dans le vecteur de sortie
}
nprec+=Rcouche[i]; // fait car les neurones sont ne sont pas indexés par couche
if (i!=Rnbcouches-1) // si on n'est pas sur la dernière couche de neurones
{
cout << "!" << flush;
// on veut ici dire que le vecteur de sortie de la couche en cours est le vecteur
// d'entrée de la couche suivante.
delete [] entree; // on efface entree, qui n'a plus d'utilité
entree=new double [Rcouche[i]]; // entree va etre de la taille de sortie
entree=sortie; // on met la sortie dans l'entrée
delete [] sortie; // on efface sortie, qui va etre de la taille du nombre de neurones
sortie=new double [Rcouche[i+1]]; // dans la couche suivante
cout << ";" << flush;
}
else // si la couche en cours est la derniere
{
Rsortie=sortie; // on sauvegarde la sortie dans la variable de la classe Reseau
cout << "-" << flush;
}
}
cout << "z" << flush;
return Rsortie; // et on renvoie la sortie
}

dans ce code, le probleme est entre le cout << "!" << flush; et le cout << ";" << flush; . Cette portion est exécutée environ 500 fois avant de s'arreter.
Vous auriez une idée de comment corriger ca ?

merci d'avance

Publicité

Taz

bisounours-codeur

entree=sortie; // on met la sortie dans l'entrée
delete [] sortie;

tu crois faire quoi là ? entree est dans le décor

Library

j'ai besoin de changer la taille de sortie, donc je pense qu'il faut le deleter avant non ?
ca marcherait si plutot que de faire entree=sortie, je copiais un à un les éléments de sortie dans entree ?

Taz

bisounours-codeur

je sais pas ce que t'as besoin de faire, je vois juste

entree=new double [Rcouche[i]]; // entree va etre de la taille de sortie
entree=sortie; // on met la sortie dans l'entrée

ça fuit comme un seau percé

entree=sortie; // on met la sortie dans l'entrée
delete [] sortie; // on efface sortie, qui va etre de la taille du nombre

corruption mémoire ...

Library

je vais prendre un exemple. On va dire qu'au début, entree est de dimension 5. Le programme va ensuite "transformer" ces entrées et va fournir, par exemple, 4 sorties (donc sortie est de dimension 4)
le truc, c'est que ces sorties vont être les futures entrées pour la couche d'après. Il faut donc que je sauvegarde sortie et que je le mette dans entree. C'est tout ce que je veux faire.
Mais pour ca, il faut bien que entree ait la meme taille que sortie non ? Et ensuite, sortie va devoir etre de dimension 3 (par exemple), donc je redimensionne sortie.

Et si je mettais entree et sortie de taille maximale (la taille maximum qu'ils peuvent prendre) et que je ne les delete pas, tu penses que ca marcherait ?

Taz

bisounours-codeur

je te lis pas, je m'intéresse pas à ton algo.

t'as rien bité aux pointeurs

forme toi ou passe à std::vector

Library

j'ai une connaissance assez limitée des pointeurs
merci quand meme

Taz

bisounours-codeur

ben alors les utilises pas

tu te rends compte de ce que tu fais quand tu écris

entree=new double [Rcouche[i]];
entree=sortie;
delete [] sortie;

Library

y a pas trop le choix quand meme

dans le code, là, d'abord je déclare que entree a une certaine taille, ensuite je dis que l'adresse mémoire d'entree vaut l'adresse mémoire de sortie, mais la troisieme ligne je sais pas trop comment elle est gérée. Soit ca supprime l'adresse mémoire de sortie, et donc celle d'entree, soit ca fait que supprimer la liaison entre sortie et son adresse mémoire, c'est ca que j'ignore.
Mais vu que ca seg fault pas, j'ai pensé que c'etait la deuxieme solution la bonne

Taz

bisounours-codeur

mais tu me parles de quoi là ?

d'abord je déclare que entree a une certaine taille, ensuite je dis que l'adresse mémoire d'entree vaut l'adresse mémoire de sortie

t'es dans l'incompréhension totale, t'es pas cohérent du tout en plus ... pour toi l'affectation à deux sens différents ...

allez utilise std::vector

Publicité

Library

utiliser std vector sera pas plus simple pour moi
je vais d'abord essayer ce que j'ai mis plus haut, en esperant que ca marche.

Taz

bisounours-codeur

ça marchera pas, au mieux, ça fuira. t'es incapable de gérer la mémoire, c'est trop complexe pour toi. utilise std::vector et après, si tu y tiens vraiment, tu pourras retourner aux pointeurs : comme ton algo fonctionne et ne s'occupe pas de la mémoire, ça sera plus facile. concentre toi sur ton algo, tu n'es pas capable de gérer la mémoire, et c'est de toute façon déconseillé de le faire de maniè-re brute

cricri_

Là suis franchement d'accord avec Taz, j'espère que c'est pour un truc perso qui ne sera jamais divulgué ...

belgique

Cricri_ a écrit :

Là suis franchement d'accord avec Taz, j'espère que c'est pour un truc perso qui ne sera jamais divulgué ...

C'est pour le code de PyGTK. Le code est disponible

Taz

bisounours-codeur

belgique a écrit :

C'est pour le code de PyGTK. Le code est disponible

heink ?

Library

Cricri_ a écrit :

Là suis franchement d'accord avec Taz, j'espère que c'est pour un truc perso qui ne sera jamais divulgué ...

bien sûr, c'est perso
j'ai pas la prétention de vouloir faire quelque chose de professionnel, loin de là.

belgique

Taz a écrit :

heink ?

Humour foireux :whistle:

Message édité par belgique le 08-05-2004 à 22:01:09

Taz

bisounours-codeur

Library a écrit :

bien sûr, c'est perso
j'ai pas la prétention de vouloir faire quelque chose de professionnel, loin de là.

fais quelque chose qui marche

Library

crois bien que c'est là mon seul but
que ce soit optimisé et propre, je m'en fous completement

Taz

bisounours-codeur

mais souvent optimisation et propreté sont voisines
et quelque chose qui marche vraiment doit être propre

Library

je veux quelque chose qui marchotte, qui est plus lent que ce que ca devrait etre, moins fonctionnel aussi, mais qui fais ce que je veux.

Taz

bisounours-codeur

:pfff:

Library

on a pas la meme conception de la programmation, c'est tout

Taz

bisounours-codeur

ça c'est clair, tu bricoles

gilou

Modérateur
Modosaurus Rex

Library a écrit :

on a pas la meme conception de la programmation, c'est tout

Vaudrait peut etre pas employer ce terme, vu ta conception.
A+,

blackgoddess

vive le troll !

entree=new double [Rcouche[i]];
entree=sortie;
delete [] sortie;

=> comment se tirer dans le pied en 3 lignes ^^

---------------
-( BlackGoddess )-

Publicité

FORUM HardWare.fr

Programmation

C++

[C++] problème avec delete []

Sujets relatifs
Problème avec les index	MySQL: problème avec une requête multitable
probleme de creation d un compte sous sql serveru via php	probleme pour creer des compte sql via php
Probléme media	Problème d'utilisation de PEAR::DB
petit probleme de recursivité	probleme de declaration de variable
Probleme d'utilisation des variables je crois (PHP + SVG)	[C] problème de pointeur
Plus de sujets relatifs à : [C++] problème avec delete []

Page générée en 0.081 secondes