Discussion :

[sphere369]

Bonjour,

J'ai un problème sous VC++ 6 lorsque j'exécute le code ci-dessous avec les fichiers ci joints dans le meme repertoire que l'execute j'obtiens par exemple le resultat suivant :

traduisez le mot suivant : courir : to run(la seule réponse rentre au clavier)
Et ben non, la bonne réponse est : to run
traduisez le mot suivant : to do: Et ben non, la bonne reponse est : faire
traduisez le mot suivant : to do:

D'avance merci pour votre aide.

Sphere.

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
 #include <iostream>   // ça, tout le monde connaît !
#include <string>     // pour les chaînes de caractères
#include <fstream>    // pour lire dans les fichiers .txt
#include <vector>     // pour pouvoir utiliser les vecteurs
#include <ctime>      // pour les nombres aléatoires
 
using namespace std;
 
int aleatoire(const int min, const int max)
{
    return (int)(min+((float) rand()/RAND_MAX*(max - min + 1)));
}
 
void traduction(vector<string>& tabCherche, vector<string>& tabRes, int alea)
{
    string traduc;
    cout << "Traduisez le mot suivant : " << tabCherche[alea] << " : ";
    cin >> traduc;
 
    if(traduc == tabRes[alea])  //je vérifie la traduction
        cout << "Tres bien" << endl << endl;
    else if(traduc == "0000") // condition d'arrêt
    {
        cout << endl;
        cout << "Fin du test" << endl << endl;
        exit(0);
    }
    else
        cout << "Eh ben non, la bonne reponse est : " << tabRes[alea] << endl << endl;
}
 
int main()
{
    const int min = 0;
    const int max = 1;
 
    srand((unsigned) time(NULL));  // sert à initialiser le rand()
 
    vector<string> tabfr;  // les vecteurs se déclarent comme ceci !
    vector<string> taban;
    string mot;  // chaîne intermédiaire servant à stocker le mot dans le vecteur
 
    ifstream fichierAnglais("anglais.txt", ios::in); // ouverture lecture seule
    ifstream fichierFrancais("francais.txt", ios::in);
 
    if(fichierAnglais && fichierFrancais)
    {
        while(getline(fichierAnglais, mot)) // tant qu'on peut lire dans le fichier
        {
            taban.push_back(mot);  // pour rajouter dans un vecteur
            getline(fichierFrancais, mot);         
            tabfr.push_back(mot);
        }
        fichierAnglais.close();   // fermeture des fichiers
        fichierFrancais.close();
    }
    else
    {
        cerr << "Impossible d'ouvrir un des 2 fichiers !" << endl;
        exit(1);
    }
 
    for(int j = 0; j < 100; j++)
    {
        int alea2 = aleatoire(min, max);
        int alea = aleatoire(min, tabfr.size()-1);  // le -1 car dans un vecteur, on compte à partir de 0
 
        if(alea2 == 0) // si c'est 0, il demande un mot français
            traduction(taban, tabfr, alea);
        else // sinon, il demande un mot anglais
            traduction(tabfr, taban, alea);
    }
 
    return 0;
}

[Sylvain Togni]

Le problème vient de la façon de récupérer la réponse :

Code :

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

    cin >> traduc;

en effet l'opérateur >> s'arrête au premier espace, il extrait les mots un par un.

Pour récupérer l'expression entière, utilise getline() comme pour la lecture des fichiers.

[Médinoc]

Et aussi: Dans ces circonstances, quand tu dois déboguer ce genre d'erreurs, commence par afficher ce que tu as saisi. Tu aurais ainsi vu le problème plus vite.

Code C++ :

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
void traduction(vector<string>& tabCherche, vector<string>& tabRes, int alea)
{
    string traduc;
    cout << "Traduisez le mot suivant : " << tabCherche[alea] << " : ";
    cin >> traduc;
    cout << "Vous avez saisi : \"" << traduc << "\"" << endl; // <---- ICI
 
    if(traduc == tabRes[alea])  //je vérifie la traduction
        cout << "Tres bien" << endl << endl;
    else if(traduc == "0000") // condition d'arrêt
    {
        cout << endl;
        cout << "Fin du test" << endl << endl;
        exit(0);
    }
    else
        cout << "Eh ben non, la bonne reponse est : " << tabRes[alea] << endl << endl;
}

PS: Tes références sur les vecteurs devraient être const, au passage.

[sphere369]

Merci a tous pour immense aide.

Je pense que je me suis un peu emmêlé les pinceaux dans ma façon de debugage depuis que j'ai lu le chapitre sur le sujet dans "Trucs et Astuces pour les nuls en C++"....


getline fonctionne mais il m'a fallu définir la variable traduc, comme char [256] au lieu de string mais en plus avec 4 warnings:

Code :

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

1
2
3
4
 
    char traduc[256];
    cout << "Traduisez le mot suivant : " << tabCherche[alea] << " : ";
    cin.getline (traduc,256);

N'était il pas possible de garder la déclaration de traduc en string?
J'ai également défini mes vectors en const (sans vraiment comprendre la raison) :

Code :

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

1
2
 
void traduction(const vector<string>& tabCherche, const vector<string>& tabRes, int alea)

Encore merci pour tout.

Sphere

[Médinoc]

Code C++ :

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

getline(cin, traduc);

[sphere369]

Ca bug comme au tout debut....

Sphere