Discussion :

[devdeb91]

bonjour

tout d'abord voici le 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
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

[PilloBuenaGente]

Bonsoir,
plutôt qu'entrer tes caractères les un après les autres utilise un "getline"

Code :

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

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 ?

[devdeb91]

Pour le getline a voir.

non 3ème boucle ligne 25 désolé

[PilloBuenaGente]

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 :

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

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

[PilloBuenaGente]

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 :

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

dico.size()

Choix aléatoire

Code :

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

1
2
srand( time( NULL ));
        int lignechoisi = rand() % dico.size() + 1;

Récupération du mot à "lignechoisi"

Code :

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

dico[ lignechoisi - 1 ]

[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

[PilloBuenaGente]

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
C'est sans doute pas le meilleur mais ferme la lecture de ton fstream puis ouvre en un second :

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

[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() ?

[Bousk]

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.

[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

[Emmanuel Deloget]

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

[devdeb91]

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

[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() ?

[ternel]

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

[devdeb91]

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

[ternel]

Oui: lis la documentation disponible chez boost.
C'est assez clair, normalement.

[devdeb91]

Pour boost que dois je prendre sachant que je suis sur code blocks sous windows ?

[PilloBuenaGente]

Tu as le choix entre windows et unix donc tu prends windows.

[devdeb91]

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