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