Problème: modification d'une variable 'const'

Bonjour à tous,

je suis débutant en C++, et pour cela j'essaye de faire quelques exercices très simples... du moins c'est ce que je pensais :P

Car je suis tombé sur une erreur et je ne comprends pas du tout ce qu'il se passe!

Je souhaite implémenter le jeu du pendu, et donc je commence par initialiser le mot secret et la réponse:

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

void initAnswer(const int size, char answer[]) { for(int i = 0; i < size; i++) { answer[i] = '_'; } }   int main() {   const char secret[] = "automobile"; char answer[] = ""; int chance = 5; bool found = false; bool win = false;   cout << secret << endl; initAnswer(strlen(secret), answer); cout << secret << endl;   return 0; }

---

Je n'ai aucune erreur à la compilation, mais lorsque j'exécute j'ai les résultats suivants:

Citation:

---

automobile
_________e

---

Je ne comprends donc pas pourquoi la variable 'secret' a été modifiée ??? Je pense qu'il serait judicieux d'utiliser les pointeurs mais je n'en suis pas encore là... :roll:

Merci !!

Note: je suis sous Windows 7, avec Code::Blocks.

Bonjour,

A mon avis le comportement observé vient du fait que la variable secret à la malchance ... de se trouver juste avant la variable answer :aie:.
Le problème vient d'un débordement de tampon sans doute dû à une mauvaise interprétation de l'écriture char <variable> [].
Quand l'on écrit ceci

Code:

---

1 2	const char secret[] = "automobile"; char answer[] = "";

---

l'on déclare deux tableaux statiques, alloués sur la pile et dont la taille sera déduite à la compilation (à partir des chaînes). Le code est identique à

Code:

---

1 2	const char secret[11] = "automobile"; // 10 caratères + zéro terminal char answer[1] = "";

---

En revanche l'écriture

Code:

---

void initAnswer(const int size, char answer[])

---

est équivalente à

Code:

---

void initAnswer(const int size, char * answer)

---

Du coup l'on tente d'écrire 10 caractères dans un tableau de 1, qui déborde sur le tableau secret (écrasant les 9 premiers caractères) ; à noter que l'on rentre dans le cadre d'un comportement indéfini, qui pourrait aussi faire purement et simplement planter le programme.

Allouer un espace plus grand pour answer corrigerait le problème

Code:

---

1 2	const char secret[11] = "automobile"; // 10 caratères + zéro terminal char answer[11] = "";

---

mais pour un programme plus évolutif (avec secret pouvant être autre chose que "automobile" ;)), il vaudrait mieux utiliser des std::string gérant dynamiquement la mémoire et possédant beaucoup de fonctions très utiles.

Salut,

Lorsque tu fait

Code:

---

answer[i] = '_'

---

tu accèdes à des éléments de answer[] qui n'ont pas été initialisés (car la taille de answer à son initialisation est nulle).
Comme il n'y a pas de contrôle de dépassement des tableaux C-style, tu accèdes à un indice qui se trouve en dehors du tableau.
Il se trouve que (par hasard), lorsque tu débordes de answer, tu tombes sur secret -> tu modifies donc secret sans le savoir.
Mais tu pourrais très bien tomber sur n'importe quoi ou rien du tout, ton programme peut donc planter (Seg fault), ou avoir silencieusement un comportement non attendu (comme dans ton cas).

Comment éviter ça :
1. on est dans la section C++, faisons du C++ : les std::string sont faites pour gérer des chaînes de caractères, et supportent la modification dynamique de leur taille.
2. si vraiment on veut utiliser des tableaux, se rappeler de ce qu'ils sont : un pointeur sur le début du tableau. Faciles à utiliser pour une taille fixe, leur utilisation en taille dynamique demande de savoir jouer proprement avec les pointeurs.

edit : grillé :aie:, je vois qu'on est d'accord

Tu n'as pas alloué de mémoire pour tes caractères dans answer, donc tu écris dedans, tu écrase en fait de la mémoire qui appartient à d'autres variables (ici, secret, mais ça pourrait être n'importe quoi).

Plutôt qu'utiliser des tableaux o udes pointeurs, je te suggère d'utiliser des string (code non testé) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include <string> using namespace std;     int main() {   string const secret= "automobile"; string answer(secret.size(), '_'); int chance = 5; bool found = false; bool win = false;   cout << secret << endl; return 0; }

---

:ccool:

Ok, rien à dire!! Merci à tous les 3 pour vos réponses claires et précises. Ça marche tout de suite beaucoup mieux et j'ai bien tout compris :D

Merci encore !!