[Debutant] memoire corrompu?

Bonjour,
mon programme a un comportement bizzare concernant le contenu de certaines variables à l'execution. voici un extrait du main:

Code:

---

1 2 3 4 5 6 7 8 9

int main(int argc,char**argv) { eoParser parser (argc, argv);   Q3APBenchmarkParser &fParser=do_make_instance(parser);   cout<<"1-appel N="<<fParser.getN()<<endl<<flush; cout<<"2-appel N="<<fParser.getN()<<endl<<flush;

---

voici l'affichage:

Code:

---

1 2	1- appel N=9 2- appel N=5

---

la variable en question est un entier retourné par la fonction fParser.getN()
et getN ne modifie pas la valeur de N, mais j'obtiens des resultats diff le premier est juste et le suivant faut.

Code:

---

1 2 3 4

int getN () { return N; }

---

y a t'il une explication logique à ça? genre probleme avec le compilateur par exp ?
merci.

Ca semble étrange a priori. Quelques hypothèses en vrac :

- Le vrai getN, qui n'est visiblement pas celui que tu nous montres, contient une subtilité qui t'a échappée
- L'affichage modifie le comportement du parser, car il est lié à std::cout d'une manière ou d'une autre
- La mémoire est corrompue par un bug ailleurs dans le programme, mais qui ne se manifeste que maintenant
- Le compilateur a un problème.

La dernière hypothèse me semble la moins probables des 4.

Citation:

---

Code:

---

cout << ... << endl << flush;

---

---

petit détail qui ne résoudra pas ton problème : pas besoin du flush quand on fait endl, c'est inclus dans le prix :D.

De plus déclare (si ce n'est pas deja le cas) la fonction getN comme const, ça aidera à éliminer de possibles causes d'erreur (getN qui modifie sans faire exprès une variable...)

On peut voir le code de la fonction do_make_instance ?

Bonjour, merci à tous, voici la fonction do_make_instance. sachant que j'ai rajouté à la fin un autre affichage dans cette fonction utilisant la même fonction getN dans fParser.
la vrai valeur pour N est 9.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

Q3APBenchmarkParser & do_make_instance(eoParser & _parser){   // benchmark file name string benchmarkFileName = _parser.getORcreateParam (string (), "BenchmarkFile", "Benchmark file name (benchmarks are available at " + BENCHMARKS_WEB_SITE + ")", 'B', "Representation", true).value ();   if (benchmarkFileName == "") { std::string stmp = "*** Missing name of the benchmark file\n"; stmp += " Type '-B=the_benchmark_file_name' or '--BenchmarkFile=the_benchmark_file_name'\n"; stmp += " Benchmarks files are available at " + BENCHMARKS_WEB_SITE; throw std::runtime_error (stmp.c_str ()); } // reading of the parameters contained in the benchmark file Q3APBenchmarkParser fParser (benchmarkFileName); cout<<"dans make instance,N="<<fParser.getN()<<endl; return fParser; }

---

voici maintenant la sortie d'execution:

Code:

---

1 2 3 4

dans make instance,N=9 1- appel N=9 2- appel N=4229940

---

sinon, j'ai changé le nom de la fonction getN en getNN pour s'assurer que c'est bien celle que je vous ai montré, et j'ai ajouté const à sa declaration mais tjrs la même chose.
et pour le compilateur, j'ai changé de pc et c tjrs pareil aussi.
merci

Bonjour,
tu retourne une référence sur un objet détruit a la fin de la fonction. d'où ton bug

Code:

---

1 2 3 4 5 6 7

Q3APBenchmarkParser & do_make_instance(eoParser & _parser){ ... Q3APBenchmarkParser fParser (benchmarkFileName); cout<<"dans make instance,N="<<fParser.getN()<<endl; return fParser; }

---

si Q3APBenchmarkParser a un constructeur par recopie, enlève le & du retour de t'as fonction ou, si Q3APBenchmarkParser as une fonction open(), passe le en référence dans les variables de ta fonction

oui je pense que c'est cela (le probléme est dans le return de fParser que j'ai mis dans Q3APBenchmarkParser. car j'ai recopié le contenu de la fonction do_make_instance et je l'ai mis directement dans main (donc plus besoin de return) et ça marche :D

merci à tous.