Discussion :

[pacolito]

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

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 :

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
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:

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

Merci !!

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

[gb_68]

Bonjour,

A mon avis le comportement observé vient du fait que la variable secret à la malchance ... de se trouver juste avant la variable answer .
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 :

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

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 :

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

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

En revanche l'écriture

Code :

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

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

est équivalente à

Code :

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

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 :

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

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.

[valAa]

Salut,

Lorsque tu fait

Code :

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

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é , je vois qu'on est d'accord

[JolyLoic]

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 :

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
#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;
}

[pacolito]

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

Merci encore !!