Discussion :

[Menudelux]

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 :

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

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

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 :

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

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.

[Ehonn]

Le problème vient de la taille de mot_melange.

Code :

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

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.

[Menudelux]

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

Voila ma nouvelle fonction mélanger:

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

[Bousk]

Tel que declare, mot_mélange est un tableau de string. Et tu veux une string seule.

[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/

[Emmanuel Deloget]

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

Code :

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

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