Problème avec les strings

Version imprimable

Bonjour à tous;

Je viens tout récemment de me mettre au C++. Hors je rencontre plusieurs problèmes avec le type string.

Voila mon code:

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
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
 
#include <iostream>
#include <string>
#include <ctime>
#include <cstdlib>
#include "main.h"
 
using namespace std;
 
int main()
{
    srand(time(0));
 
    string mot_mystere(""), mot_entre(""), mot_melange("");
 
    cout << "Saisissez un mot: " << endl;
    cin >> mot_mystere;
 
    mot_melange = melangerLettres(mot_mystere);
 
    cout << mot_melange << endl;
 
    do
    {
 
        cout << endl << "Quel est ce mot ? " << endl;
        cin >> mot_entre;
 
        if(mot_entre != mot_mystere)
        {
            cout << "Ce n'est pas le bon mot ! "<< endl;
        }
        else
        {
            cout << "Bravo !" << endl;
        }
 
    }while(mot_entre != mot_mystere);
 
 
 
    return 0;
}
 
string melangerLettres(string mot_mystere)
{
 
    long taille_mot(0), i, position(0);
    taille_mot = mot_mystere.size();
 
    string mot_melange;
 
    for(i=0;i<taille_mot;i++)
    {
        position = rand() % mot_mystere.size();
        mot_melange[i] = mot_mystere[position];
        mot_mystere.erase(position, 1);
    }
 
    for(i=0;i<taille_mot;i++)
    {
 
        cout << mot_melange[i];
 
    }
 
    return mot_melange;
}

Dans ce code ma fonction mélanger fonctionne bien puisque:

Code:

1
2
3
4
5
6
for(i=0;i<taille_mot;i++)
    {
 
        cout << mot_melange[i];
 
    }

m'afficher le mot avec les lettres mélangées.

Hors dans ma fonction main:

Code:

cout << mot_melange << endl;

le mot ne s'affiche pas du tout :? j'ai l'impression qu'il est vide.

J'ai beau tenté plusieurs trucs je n'arrive vraiment pas à régler ce problème. Auriez vous des explications à me fournir afin que je puisse régler tout seul ce problème s'il vous plait ?

Merci d'avance.

Le problème vient de la taille de mot_melange.
Citation:
Envoyé par Menudelux
Code:

1 2 3 4 5 6 7 8 string mot_melange; for(i=0;i<taille_mot;i++) { position = rand() % mot_mystere.size(); mot_melange[i] = mot_mystere[position]; mot_mystere.erase(position, 1); }
En effet dans ce code, la taille de mot_melange est nulle, et tu y accèdes comme s'il faisait la taille de mot_mystere.

Lors de l'affichage manuel avec la boucle for et la variable taille_mot (qui n'a rien à voir avec la taille de mot_melange) (il y a deux mauvaises pratiques : ne pas utiliser mot_melange.size() et ne pas utiliser std::cout << mot_melange << std::endl (tout simplement)); tu n'as pas de chance et tu arrives à utiliser la mémoire derrière mot_melange qui n'appartient à personne.

La solution est donc de déclarer mot_melange avec la bonne taille.

Pour trouver "automatiquement" ce genre de dépassement, tu peux utiliser des programmes comme valgrind qui permettent de détecter beaucoup d'erreurs, lors de l'exécution du programme, comme le fait d'utiliser de la mémoire qui ne t'appartient pas.

Merci je comprends d’où vient l'erreur.

Voila ma nouvelle fonction mélanger:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 string melangerLettres(string mot_mystere) { long taille_mot, i, position; taille_mot = mot_mystere.size(); string mot_melange[taille_mot]; for(i=0;i<taille_mot;i++) { position = rand() % mot_mystere.size(); mot_melange[i] = mot_mystere[position]; mot_mystere.erase(position, 1); } return mot_melange; }
Mais elle plante maintenant au retour :s j'ai vraiment du mal avec les strings...

L'erreur du compilateur:

||=== Entrainement en C++, Debug ===|
C++\main.cpp|125|error: conversion from 'std::string*' to non-scalar type 'std::string' requested|
||=== Build finished: 1 errors, 1 warnings ===|

03/11/2012, 17h07
Bousk

Tel que declare, mot_mélange est un tableau de string. Et tu veux une string seule.
03/11/2012, 21h09
I_believe_in_code

La bonne solution n'est-elle donc pas d'écrire :

string mot_melange (taille_mot, ' ');

Ce qui initialise mot_melange comme une chaîne de longueur taille_mot, ne contenant que des espaces. Bien sûr on peut initialiser avec un autre caractère que celui d'espacement pour ce que Menudelux veut faire.

PS : cela fait longtemps que je n'ai pas fait de cplusplus mais je me suis renseigné ici : http://www.cplusplus.com/reference/s...string/string/

Une autre solution consiste à utiliser les algorithmes de la librairie standard :
Code:

1 2 3 4 5 6 7 8 9 #include <algorithm> std::string melange(const std::string& mot_mystere) { std::string mot_melange(mot_mystere); std::random_shuffle(mot_melange.first(), mot_melange.last()); return mot_melange; }
Ca a le mérite d'avoir été testé, testé et re-testé des millions de fois :)

05/11/2012, 17h26
Menudelux

Merci bien pour votre aide. Faut dire aussi que dans le cours du C++ tout n'est pas dit.

Sinon pour la bibliothèque standard en faite je ne l'ai pas encore vu mais merci pour l'information.
05/11/2012, 23h54
Emmanuel Deloget

Citation:

Envoyé par Menudelux

Merci bien pour votre aide. Faut dire aussi que dans le cours du C++ tout n'est pas dit.

Sinon pour la bibliothèque standard en faite je ne l'ai pas encore vu mais merci pour l'information.

La bonne chose, c'est que cppreference donne des infos sur comment cette fonction est implémentée, donc tu peux t'en inspirer si, pour cet exercice, tu ne souhaite pas utiliser la fonction.

http://en.cppreference.com/w/cpp/alg...random_shuffle