probleme fichier lecture

Version imprimable

bonjour

tout d'abord voici le 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
ifstream ifs(filename);
    int ligne = 0;
    while(!ifs.eof())
    {
        char c = ifs.get();
        if(c == '\n')
        {
            ligne++;
        }
    }
    //cout << "ligne " << ligne << endl;
 
    srand(time(NULL));
    int lignechoisi  = 0;
    while(lignechoisi == 0)
    {
        lignechoisi = rand() % ligne;
    }
    cout << "ligne choisi " << lignechoisi << endl;
 
    ifs.seekg(0, ios::beg);
    int pos = ifs.tellg();
    cout << "position " << pos << endl;
    //lignechoisi--;
    while(lignechoisi > 0)
    {
        //cout << "entrer 2eme boucle" << endl;
        char c = ifs.get();
        cout << c << endl;
        if(c == '\n')
        {
            cout << c << " trouver" << endl;
            lignechoisi--;
            cout << "ligne " << lignechoisi << endl;
        }
    }

alors voila tout marche jusqu'a la deuxieme boucle ou il doit se placer a la ligne choisi il ne lis pas de caratere donc boucle infinie.

donc la question est pourquoi il fait cela ?

ps : je suis presque sur de mon code car deja utiliser une fois pour un jeu de mot mystere qui fonctionne encore

06/03/2013, 20h41
PilloBuenaGente
Bonsoir,
plutôt qu'entrer tes caractères les un après les autres utilise un "getline"
Code:

1 2 3 4 std::string name; int line(0); while( std::getline(ifs,name)) ligne++;
Tu parles de deuxième boucle, la ligne 15 ?
06/03/2013, 20h50
devdeb91

Pour le getline a voir.

non 3ème boucle ligne 25 désolé

Pour le getline c'est mile fois(au moins) plus facile !
(Je ne te vois à aucun moment où tu stockes un nom, un char stocke UNE lettre. Utilise les strings !! (tu peux jouer avec les mots sans te préoccuper de savoir comment sont gérées chaque lettre )
Un exemple pour te rendre compte de l’efficacité du getline et string :

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
#include <vector>
#include <string>
#include <iostream>
#include <fstream>
#include  <stdlib.h>
using namespace std;
 
int main(void) {
 
        vector < string > dico;
        ifstream ifs( "Dico.txt" );
        string nom;
 
        while( getline( ifs, nom ))//Tant qu'il y a des lignes
                dico.push_back( nom );//Stocke les strings dans un vecteur
 
        cout << "Taille du dico :" << dico.size() << endl;
 
        srand( time( NULL ));
        int lignechoisi = rand() % dico.size() + 1;
 
        cout << "Ligne choisi :" << lignechoisi << endl;
 
        cout << "Mot choisi : " << dico[ lignechoisi - 1 ] << endl;
 
        return 0;
}

06/03/2013, 21h47
devdeb91

Ouais mais dans mon cas je sais pas si cela servirait car je veux compter le nombre de ligne puis générer un nombre aléatoire de ces lignes qui sera la ligne choisi pour enfin parcourir jusqu’à se placer à la ligne.
06/03/2013, 21h55
PilloBuenaGente
Citation:

je veut compter le nombre de ligne puis generer un nombre aleatoire de ces lignes qui sera la ligne choisi pour enfin parcourir jusqu'a se placer a la ligne

, et recuperer le mot ?! Le code que je t'ai donné fait tout cela :

Nombre de lignes

Code:

dico.size()

Choix aléatoire
Code:

1 2 srand( time( NULL )); int lignechoisi = rand() % dico.size() + 1;
Récupération du mot à "lignechoisi"

Code:

dico[ lignechoisi - 1 ]
06/03/2013, 22h07
devdeb91

Sauf que dans mon cas mon fichier contient tout un tas d'information par ligne donc je me vois pas jouer avec le string pour couper en plusieurs string ou autres

Citation:

sauf que dans mon cas mon fichier contient tout un tas d'information par ligne donc je me vois pas jouer avec le string pour couper en plusieurs string ou autres

Ok :aie:
C'est sans doute pas le meilleur mais ferme la lecture de ton fstream puis ouvre en un second :

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
ifstream ifs(filename);
    int ligne = 0;
    while(!ifs.eof())
    {
        char c = ifs.get();
        if(c == '\n')
        {
            ligne++;
        }
    }
    ifs.close();
    //cout << "ligne " << ligne << endl;
 
    srand(time(NULL));
    int lignechoisi  = 0;
    while(lignechoisi == 0)
    {
        lignechoisi = rand() % ligne;
    }
    cout << "ligne choisi " << lignechoisi << endl;
 
ifstream ifs2(filename);//Créer un 2eme stream
 
    int pos = ifs2.tellg();
    cout << "position " << pos << endl;
    //lignechoisi--;
    while(lignechoisi > 0)
    {
        //cout << "entrer 2eme boucle" << endl;
        char c = ifs2.get();
        cout << c << endl;
        if(c == '\n')
        {
            cout << c << " trouver" << endl;
            lignechoisi--;
            cout << "ligne " << lignechoisi << endl;
        }
    }

06/03/2013, 23h08
devdeb91

non c'est bon c'est regler c'est juste le seekg qui placer mal donc j'ai remplacer par des ifs.close() puis ifs.open(filename) et voila le tour est jouer il se remet au debut et charge bien.

parcontre pour le rand() n'y a t'il pas moyen de generer aleatoirement un nombre sans que celui ci tombe presque a coup sur plusieur fois sur le meme d'affiler car pour ce que je veut faire ( chargement de monstre ) c'est bien d'avoir 2 ou 3 monstre pareil mais c'est mieu d'avoir plusieurs differents. je sais se que vous allai me dire faire en sorte de gerer les doublons mais je ne peut car on peu avoir des doublons.

donc y aurai t'il d'autres moyens de generer aléatoirement mieux que le rand() ?
07/03/2013, 00h11
Bousk

Citation:

Envoyé par devdeb91

sauf que dans mon cas mon fichier contient tout un tas d'information par ligne donc je me vois pas jouer avec le string pour couper en plusieurs string ou autres

Et tu crois qu'il est plus simple de lire le fichier par caractère pour découper sa ligne ?
Alors que std::string fournit un panel de méthodes qui permettent de maîtriser ça.

Cela dit, le seekg n'a pas l'air faux pourtant.
07/03/2013, 10h30
devdeb91

Et il ne l'est pas mais pourtant avec un seekg cela bug avec un close puis open cela passe sans problème allez savoir pourquoi
07/03/2013, 12h50
Emmanuel Deloget

Citation:

Envoyé par devdeb91

et il ne l'est pas mais pourtant avec un seekg cela bug avec un close puis open cela passe sans probleme aller savoir pourquoi

Parce que seekg() ne modifie pas l'état du flux, qui est en erreur puisque tu est allé jusqu'à eof(). Du coup, toute lecture subséquente est impossible. Ce n'est pas le seekg() le problème, c'est la lecture :)

Un petit ifs.clear() avant le seekg() et hop, ça devrait marcher comme sur des roulettes :)
07/03/2013, 14h01
devdeb91

Citation:

Envoyé par Emmanuel Deloget

Parce que seekg() ne modifie pas l'état du flux, qui est en erreur puisque tu est allé jusqu'à eof(). Du coup, toute lecture subséquente est impossible. Ce n'est pas le seekg() le problème, c'est la lecture :)

Un petit ifs.clear() avant le seekg() et hop, ça devrait marcher comme sur des roulettes :)

bravo c'est exactement sa
07/03/2013, 14h21
devdeb91

Citation:

Envoyé par devdeb91

pour le rand() n'y a t'il pas moyen de generer aleatoirement un nombre sans que celui ci tombe presque a coup sur plusieur fois sur le meme d'affiler car pour ce que je veut faire ( chargement de monstre ) c'est bien d'avoir 2 ou 3 monstre pareil mais c'est mieu d'avoir plusieurs differents. je sais se que vous allai me dire faire en sorte de gerer les doublons mais je ne peut car on peu avoir des doublons.

y aurai t'il d'autres moyens de generer aléatoirement mieux que le rand() ?
07/03/2013, 14h48
ternel

Il y a boost.random et son jumeau std::<random>, ce dernier n'étant disponible qu'en C++11
07/03/2013, 15h37
devdeb91

Citation:

Envoyé par leternel

Il y a boost.random et son jumeau std::<random>, ce dernier n'étant disponible qu'en C++11

pourriez vous me donner quelque exemple basic car malgrés les recherches et tout le reste ces deux proposition reste trés flou mais vraiment flou je n'y comprend quasiment rien
07/03/2013, 16h22
ternel

Oui: lis la documentation disponible chez boost.
C'est assez clair, normalement.
07/03/2013, 18h16
devdeb91

Pour boost que dois je prendre sachant que je suis sur code blocks sous windows ?
07/03/2013, 18h19
PilloBuenaGente

Tu as le choix entre windows et unix donc tu prends windows.
07/03/2013, 18h22
devdeb91

Oui mais d’après ce que je sais code blocks et un utilitaire linux qui utilise des composant linux

dans l'incapacité d'utiliser boost je me suis tourner vers std::random mais celui ci me genere des nombre enorme alors que je veut un chiffre moins grand

exemple
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 i = 0; while(i != 10) { // boost::random::mt19937 rng; // produces randomness out of thin air // see pseudo-random number generators std::mt19937 x(100); //boost::random::uniform_int_distribution <> six(1,5); // distribution that maps to 1..6 // see random number distributions //int x = six(rng); cout << x << endl; i++; // simulate rolling a die }
je cherche a etablir un test de 10 valeur entre 0 et 100 mais lui me retourne des valeurs largement superieur a 100 .

j'ai chercher un peu des tuto et dans cpp reference mais pas trop d'exemple concret.

auriez vous un exemple concret pour pouvoir faire se que je veut

07/03/2013, 23h44
Ehonn

J'avais posté un petit bout de code pour jouer avec random.
http://www.developpez.net/forums/d12...s/#post6581714

Sinon, tu peux regarder ici.
http://www.cplusplus.com/reference/r..._distribution/

std::mt19937 generator(seed); est en fait un générateur qui prend en paramètre la graine.
std::uniform_int_distribution<int> distribution(0, 100) est une distribution qui prend en paramètre le min et le max.

Pour avoir un nombre pseudo-aléatoire, on donne à manger le générateur à la distribution.

Code:

int n = distribution(generator);

Moi aussi je me suis amusé depuis mon post et j'en suis arrivé à la

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 <fstream>
#include <ctime>
#include <cstdlib>
#include <random>
#include <tr1/random>
//#include <boost/random/mersenne_twister.hpp>
//#include <boost/random/uniform_int_distribution.hpp>
 
using namespace std;
 
int main()
{
    srand(time(0));
    int x = 0;
    while(x == 0)
    {
         x = rand() % 5;
    }
    int i = 0;
    srand(time(0));
    while(i != 10)
    {
        int y = 0;
        while(y == 0)
             y = (rand() % 100);
 
        cout << y << endl;
        i++;
    }
 
    cout << endl << endl;
 
    i = 0;
    std::tr1::mt19937 gen;
    std::tr1::uniform_int <int> a(0,100);
    while(i != 10)
    {
       // boost::random::mt19937 rng;         // produces randomness out of thin air
                                // see pseudo-random number generators
        //std::mt19937 x(100);
        //std::tr1::variate_generator<;
        //std::tr1::minstd_rand&;
        //std::tr1::minstd_rand gen;
        //std::tr1::mt19937 gen;
        //std::tr1::uniform_int <int> x(0,100);
        //gen.seed((unsigned int)time(NULL));
        //boost::random::uniform_int_distribution <> six(1,5);
                                    // distribution that maps to 1..6
         //int gen;                           // see random number distributions
        //int x = six(rng);
        //cout << gen () % 10 << endl;
        cout << a(gen) << endl;
 
        i++;                 // simulate rolling a die
    }
 
 
   /* std::mt19937 x(100);
    cout << endl;
    cout << endl;
      cout << x() << endl;
 
      cout << endl;
    cout << endl;
      cout << x.max() << " et " << x.min() << endl;*/
    return 0;
}

le problème est que la série de "a" après un première fois fait ne se réinitialise pas donc pas de génération aléatoire

07/03/2013, 23h56
PilloBuenaGente

Quand tu trouve la réponse merci le dire, ça évite la perte de temps ^^
08/03/2013, 00h01
Ehonn

std::tr1::mt19937 gen(time(0)); ?
08/03/2013, 00h01
devdeb91

Citation:

Envoyé par PilloBuenaGente

Quand tu trouve la réponse merci le dire, ça évite la perte de temps ^^

désoler mais je n'est rien dis car je n'est pas encore trouver la réponse car comme je dis et comme tu peut voir c'est pas la meme facon de faire et mes resultat bug encore donc j'ai laisser au cas ou on m'apporterais mieux.
08/03/2013, 00h02
devdeb91

Citation:

Envoyé par Ehonn

std::tr1::mt19937 gen(time(0)); ?

ah bah voila avec cela sa marche mieux
08/03/2013, 00h07
Ehonn

Que voulais-tu dire par :

Citation:

le probleme est que la serie de "a" aprés un premiere fois fait ne se reintilise pas donc pas de génération aléatoire

Et par hasard, le fait de changer la graine entre les exécution (ma ligne de code) ne règle-t-il pas le problème ?
(Lorsque tu rajoute ma ligne de code, tu as bien un générateur de nombre pseudo aléatoire entre 0 et 100.)

Edit: j'ai répondu quand il y avait encore «??» dans ton message ^^

Bon alors voila j'ai toujours un gros problème voici le code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 else if(Map.tileAt(joueur.x(), joueur.y()) == 'M')
           {
               srand(time(NULL));
               int nbmonstre = 0;
               while(nbmonstre == 0)
               {
                   nbmonstre = rand() % 5;
               }
               string filename = Map.title + "_monstre" + ".txt";
               cout << "nombre monstre " << nbmonstre << " " << "nom fichier " << filename << endl;
               for(int i = 0; i < nbmonstre; i++)
               {
                   chargemonstre(Map, filename, i);
               }

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
void chargemonstre(SMap & Map, string filename, int index)
{
    cout << "chargement monstre" << endl;
 
    ifstream ifs(filename);
    int ligne = 0;
    while(!ifs.eof())
    {
        char c = ifs.get();
        if(c == '\n')
        {
            ligne++;
        }
    }
    //cout << "ligne " << ligne << endl;
 
    std::tr1::mt19937 gen(time(0));
    std::tr1::uniform_int <int> a(1,ligne);
    int lignechoisi = a(gen);
    /*srand(time(0));
    int lignechoisi  = 0;
    while(lignechoisi == 0)
    {
        lignechoisi = (rand() % ligne);
    }*/
    //cout << "ligne choisi " << lignechoisi << endl;
   /* ifs.clear();
    ifs.seekg(0, ios::beg);
    int pos = ifs.tellg();
    cout << "position " << pos << endl;*/
    ifs.close();
    ifs.open(filename);
//    ifstream ifs(filename);
    lignechoisi--;
    while(lignechoisi > 0)
    {
        //cout << "entrer 2eme boucle" << endl;
        char c = ifs.get();
        //cout << c << endl;
        if(c == '\n')
        {
            //cout << c << " trouver" << endl;
            lignechoisi--;
            //cout << "ligne " << lignechoisi << endl;
        }
        /*string nom;
        getline(ifs, nom);
        lignechoisi--;
        cout << nom << endl;*/
    }
 
    string nom;
    int hp;
    int mp;
    string type;
    int exp;
    int gold;
    //cout << "remplissage" <<  endl;
    ifs>>nom>>hp>>mp>>type>>exp>>gold;
 
    //cout << nom << " " << hp << " " << mp << " " <<
    //        type << " " << exp << " " << gold << endl;
    //cout << "fin remplissage" << endl;
    //Map.monstre.insert(std::make_pair(Position(joueur.x(), joueur.y()), SMonstre(nom, hp, mp, type, exp, gold)));
    Map.monstre.insert(std::make_pair(int(index), SMonstre(nom, hp, mp, type, exp, gold)));
}

il me genere toujour le meme nombre de la ligne choisi dans la fonction il change rarement voir jamais peu importe que sa soit avec

Code:

1
2
3
std::tr1::mt19937 gen(time(0));
    std::tr1::uniform_int <int> a(1,ligne);
    int lignechoisi = a(gen);

ou avec

Code:

1
2
3
4
5
6
srand(time(0));
    int lignechoisi  = 0;
    while(lignechoisi == 0)
    {
        lignechoisi = (rand() % ligne);
    }

je ne comprends pas pourquoi malgré les test ailleurs dans des petits programmes très concluants quand c'est ici ça fonctionne à moitié.

auriez-vous une idée de pourquoi

08/03/2013, 00h25
Ehonn

Avec la solution en C, c'est à dire avec srand et rand; srand permet d'initialiser la graine du générateur pseudo aléatoire. Il ne faut l'initialiser qu'une seule fois (au début du main par exemple).

C'est le même principe en C++11, il ne faut créer qu'une seule fois le générateur (et le passer en référence à la fonction).
08/03/2013, 00h30
devdeb91

Et avec rand() alors comment suis je censé faire j'ai juste a retirer le srand() ou il faut rajouter quelque chose
08/03/2013, 00h35
Ehonn

Il faut appeler srand(time(0)) qu'une seule fois, tu peux le mettre à la première ligne de la fonction int main().

Grosso modo, un générateur pseudo aléatoire donne toujours la même suite de nombre. La graine permet de commencer à des endroits différents.
08/03/2013, 00h43
devdeb91

Ok bah merci pour ta patiente et pour tous ces renseignements ça m'a beaucoup aidé :)