Discussion :

[baiphoa]

bonjours, voilà mon probleme: j'ai crée une petite fonction qui à pour rôle d'afficher les login et password qui se trouvent dans un fichier sous cette forme:
login|password
login2|password2
mon probleme est que ça ne m'affiche pas du tout ce que je souhaite...

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
using namespace std;
bool loadAdmin()
{
    string data;
    char stop = 124; // correspond au code ASCII de ce caractère "|"
    char impair=true;
    ifstream file("Data/player/admin.txt", ios::in);
    if (file == false)
    {
        cout << "! echec de l'ouverture du fichier" << endl;
    }
    else
    while (getline(file,data))
    {
        if(impair == true)
        {
            getline(file, data, stop);
            cout << data;
            impair=false;
        }
        if(impair == false)
        {
            getline(file, data);
            cout << " "+data << endl;
            impair=true;
        }
    }
    file.close();
    cout << " chargement de la liste des administrateurs effectuee!" << endl;
    return true;
}

[Terry-Slipknot]

Déjà, je vois 4 problèmes dans ton code ...

1) Évite de mettre des valeurs numérique à des "char"
2) Ensuite, c'est quoi cette valeur booléenne à un char ?
3) Met des parenthèses à ton bloc else, parce du coup, si l'ouverture ne réussi pas, il va t'afficher que ça n'a pas réussi et après que ça a réussi...
4) Pourquoi tu mes une valeur de retour si elle sera toujours vrai ?

[thomas41]

Bonjour,

Je pense que tu as commis une erreur dans l'utilisation de la fonction getline.
Il faut que tu récupère tout d'abord une ligne:

string line;
while (getline(file,line)) {

ensuite tu peux stocker le contenu de ta ligne dans un flux stringstream afin de pouvoir travailler dessus:

stringstream ss;
ss << line;

pour récupérer le nom et le mdp, tu utilises ta fonction getline deux fois:
string nom, mdp;
getline(ss, nom, '|');
getline(ss, mdp, '|');

Tu réalise cette opération autant de fois que cela est nécéssaire.

++

[Steph_ng8]

Deux autres remarques…

Tu commences par lire une ligne de ton fichier au début de la boucle « while », pour vérifier que le flux est toujours valide.
Sauf que tu n'en fais rien !
Du coup, tu perds une ligne à chaque itération…

Ensuite, ton « booléen »…
Tu l'initialises à « true ». Soit.
Au début de la boucle « while », on teste s'il vaut « true » ; comme c'est le cas, on rentre dans le bloc « if », et à la fin on lui affecte « false ».
Deuxième test de la boucle, on vérifie s'il vaut « false » ; c'est le cas, alors on rentre dans le bloc « if », et à la fin on lui affecte « true ».
Fin de la boucle, itération suivante.
On teste s'il vaut « true »…

Au final, on rentre systématiquement dans tous les blocs « if » de la boucle.
Donc ces tests sont inutiles.
Mais alors, quelle est l'utilité de ce « booléen » ?

PS: Puisque toutes les lignes ont un format et un nombre de tokens bien définis, tu n'es pas obligé de passer par un std::stringstream pour analyser le contenu des lignes.
Mais dans le cas général, c'est la bonne méthode.

[baiphoa]

@terry
1) Évite de mettre des valeurs numérique à des "char"
Le compilateur, ne voulais pas prendre en compte char stop "|"; j'ai donc donc du le remplacer par sa valeur ASCII

2) Ensuite, c'est quoi cette valeur booléenne à un char ?
Une erreur de ma part, je voulais effectivement mettre bool, mais pour je ne sais quel raison j'ai mit char.

3) Met des parenthèses à ton bloc else, parce du coup, si l'ouverture ne réussi pas, il va t'afficher que ça n'a pas réussi et après que ça a réussi...
Un simple et bette oublie... honte à moi

4) Pourquoi tu mes une valeur de retour si elle sera toujours vrai ?
ça s'est temporaire, c'était pour les test, mais ça va viré, et être remplacer dès lors que le chargement des infos du fichier sera correctement mit en place.


@steph:
la variable impair joue un rôle important, cela peut semblé inutile au premier regard, mais son rôle est réel:
la première fois dans la boucle while, impaire vaut true, et donc on rentre dans le premier if, ici on utiliser getline(file, data, stop), donc on enregistrera dans data les information du début du fichier jusqu'au caractère stop qui ici est "|" ==> donc on lit le login. puis impair vaut false
donc la seconde fois dans la boucle while, on entre dans le seconde if, et ici on ne li pas jusqu'au caractère stop, mais jusqu'au caractère "/n" fin de ligne. grace à getline(file, data) (remarque, il n'y a pas de 3eme paramètre, donc la lecture se fin jusqu'à la fin de ligne) ==> donc on li le mot de pass.
sinon je vois ce que tu veut dire. on li le premier et ensuite le second...oui j'ai effectivement zaper ça...

[baiphoa]

stringstream ss=line; //error, std::stringstream has initializer but incomplete type

[koala01]

Salut,

Une petite remarque de départ: le paramètre ios::in est implicite pour les ifstream, étant donné que c'est la classe qui s'occupe... des fichier entrant (Input File STREAM).

Sinon, il existe une quantité non négligeable de solution simples et efficace pour traiter ton problème.

En voici quelques unes:

La fonction getline présente également une signature prenant trois paramètres: istream& getline ( istream& is, string& str, char delim ) où "delim" représente le caractère qui doit être utilisé comme délimiteur (en plus du retour à la ligne).

Nous pourrions donc parfaitement envisager de lire directement deux chaines de caractères avec deux appels successifs à getline sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
std::ifstream ifs("fichicer.txt");
if(ifs)
{
    std::string login;
    std::string pass;
    while(std::getline(ifs,login,'|') && std::getline(ifs,pass))
    {
        /* nous n'avons plus qu'à gérer login et pass */
    }
}
else
{
    std::cout<<"impossible d'ouvrir le fichier"<<std::endl;
}

Nous pourrions également décider d'utiliser les fonctions find et substr de la classe string.

La première permet de trouver un caractère donné dans la chaine, la seconde de récupérer une souschaine:

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
std::ifstream ifs("fichicer.txt");
if(ifs)
{
    std::string in;
    while(std::getline(ifs,in))
    {
        size_t found=in.find('|');
        if(found !=std::string::npos)
        {
            std::string login=in.substring(0,found);
            std::string pass=in.substring(found+1);
            /* nous n'avons plus qu'à gérer login et pass */
        }
        else
        {
            throw BadFileFormat(); // exception à définir toi-même
        }
    }
 
}
else
{
    std::cout<<"impossible d'ouvrir le fichier"<<std::endl;
}

Et il serait bien sur possible de trouver quantité d'autres solutions à coup de masturbation mentale, mais, la première ligne de ma signature devrait être une ligne directrice dans tout développement

[Steph_ng8]

stringstream ss=line; //error, std::stringstream has initializer but incomplete type

Et si tu essayais d'inclure le fichier d'en-tête correspondant ?
Il s'agit de « sstream », pour information.

[baiphoa]

merci pour votre aide!

@steph: oui maintenant que je sais que cela nécessitai un au autre include, je comprend cette erreur xD encore merci.

@koala010: merci de m'avoir éclairé de ta lanterne, j'ai finalement opté pour ta première solution, simple et efficace, maintenant il ne me reste pulque à entré ça sous forme d'un array

[baiphoa]

j'ai une autre question qui me trote dans la tête, connaissez vous une fonction ayant le même rôle que atoi, mais pour les double en Cpp?
car enfaite parfois je doit récuperé des string qui ne contiennent rien d'autre que des double.

PS: oui j'ai bien dit double, car j'ai besoin de très grand nombres

[Steph_ng8]

Non, je ne pense pas qu'une telle fonction existe.
Par contre : http://cpp.developpez.com/faq/cpp/?p...RINGS_strtonum
Bon, il y a une petite manipulation à faire, mais je pense que tu sauras trouver laquelle…