Discussion :

[jane40]

Bonjour à tous,
Je cherche à écrire une fonction me permettant de lire un fichier mot par mot. J'ai déjà le code me permettant de lire un fichier ligne par ligne mais je ne trouve pas d'info pour lire mot par mot.
Voici le code dont je dispose :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
    std::ifstream fichier( "staff.txt" );
    if ( fichier ) // ce test échoue si le fichier n'est pas ouvert
    {
	string ligne; // variable contenant chaque ligne lue
	cout<<i;
	i++;
        while ( std::getline( fichier, ligne ) )
        {
           std::cout << ligne << std::endl;
        }
    }

Pourriez-vous me donner une piste pour arriver à ce que je veux?

Je souhaite écrire cette fonction car je compte m'en servir pour ensuite trouver un mot dans un texte en utilisant strcpm(mon mot,chaque mot du texte). Est-ce réalisable en faisant comme ça ou dois-je chercher complètement autre chose?

Merci pour vos idées

[FunK92]

Bonsoir,

En supposant qu'entre chaque mot il y ai un espace, il suffit de rajouter un paramètre a std::getline();

Il s'agit du delimiteur qui, s'il n'est pas spécifier lors de l'appel de la fonction est par défaut '\n'. D'ou ta lecture ligne par ligne.

donc utilise std::getline comme ceci :

Code :

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

std::getline( fichier, ligne , " " );

[jane40]

Ok, merci.
Maintenant, j'essaie d'écrire la fonction qui cherche l'occurence d'un nom dans un fichier.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void stafflist::SearchStaffIdentification(){
string u;
 std::ifstream fich( "staff.txt" );
 
if ( fich ) 
	{
	string ligne; 
    	while ( u=std::getline( fich, ligne, ' ' )) {        	
            	if (strcmp("Dupont",u) == 0)
           	cout << "yes";
        	}
	}
}

Mais j'obtiens :

erreur: cannot convert ‘std::string’ to ‘const char*’ for argument ‘2’ to ‘int strcmp(const char*, const char*)’

Je pense que ma façon de stocker les mots au fur et à mesure : u=std::getline( fich, ligne, ' ' ) ne va pas et je ne trouve pas d'info sur la manière de comparer un mot avec une variable. Est ce que strcmp("Dupont",u) est juste?
Encore une fois, j'en appelle vos lumières

[loufoque]

Quelle est ta définition de "mot" ?
S'il s'agit de vrais mots, faut sortir l'artillerie lourde : Unicode a un algorithme pour ça. (attention, pour le thailandais, il faut un dictionnaire)

[koala01]

Salut,

C'est normal, la classe std::string n'est pas une chaine de caractères "C style" (ce n'est pas un tableau de caractères terminé par '\0', même si la classe std::string contient quelque chose de similaire )

On ne peut donc pas utiliser strcmp pour comparer deux std::string (ou assimilable à).

Par contre, l'opérateur d'égalité (au même titre que tous les opérateurs de comparaion, d'ailleurs) est défini pour cette classe.

En outre, tu ne semble pas vraiment avoir compris le principe de std::getline...

Lorsque tu écris le code

Code :

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

std::getline(fichier, chaine /*, séparateur */)

tu demande de placer dans... chaine la suite du contenu de fichier jusqu'au séparateur éventuellement donné.

Pour comparer vérifier si une chaine lue dans le fichier, tu dois donc tester l'égalité entre... la chaine lue et... la chaine de référence

Cela donnerait donc un code proche de

Code :

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

1
2
3
4
5
std::getline(fichier, chaine /*, separateur */);
if(chaine=="Dupont")
{
    /* ce qui doit etre fait */
}

Ceci dit, si tu sais que tu n'aura de toutes manière que des chaines à récupérer, tu peux envisager un code proche de

Code :

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

1
2
3
4
5
6
7
while(fichier>> chaine)
{
    if(chaine == "Dupont")
    {
        /*...*/
    }
}

En effet, traditionnellement, la fonction getline est destinée à prendre d'un seul coup tout ce qui se trouve avant le retour à la ligne suivant (ou assimilé), le plus souvent de manière à être en mesure de "parser" cette chaine

[jane40]

Merci bcp, ça marche bien maintenant
Je mets le code, si ça peut aider quelqu'un

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
 
if ( fich ) 
{
   while ( (fich>>chaine) && (chaine != "Dupond") ) 
   {
 
   }
 
   if(chaine=="Dupond")
   {
      cout<<chaine;
      cout << "The staff  is in the list\n";
   }
   else
      cout<<"The staff  is not in the list\n";
   }
}

Ah, dernière chose, j'ai rien à mettre dans ma boucle while, à priori ça ne me met pas d'erreurs, mais ça ne peut pas causer des problèmes?

[ram-0000]

Si tu avais indenté ton code correctement, tu verrais qu'il y a une accolade fermante de trop.

Telle qu'elle est faite, ta boucle while semble être une boucle sans fin, je ne comprend pas trop la condition de sortie

[koala01]

Si tu avais indenté ton code correctement, tu verrais qu'il y a une accolade fermante de trop.

oui, mais...

Telle qu'elle est faite, ta boucle while semble être une boucle sans fin, je ne comprend pas trop la condition de sortie

On boucle:

• tant qu'on peut lire quelque chose dans le fichier ET
• tant que ce que l'on a lu n'est pas égal à Dupont

Comme Jane40 a utilisé l'opérateur ET optimisé, la deuxième condition n'est vérifiée que si la première est évaluée à vrai (que la lecture a fonctionné )

[Goten]

Ben oui mais alors pourquoi un if() qui re-teste la condition déjà testé pour sortir du while?

[Sylvain Togni]

Le if() est là pour savoir laquelle des deux conditions du while() a échoué.

Décidément, pour une fois qu'on a un code correct...

La seule chose, si on veut pinailler, se serait de rajouter une gestion d'erreur :

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
while ( (fich>>chaine) && (chaine != "Dupond") ) 
{
}
 
if (!fich && !fich.eof())
{
   cerr << "Reading error!\n";
}
else if(chaine=="Dupond")
{
   cout<<chaine;
   cout << "The staff  is in the list\n";
}
else
   cout<<"The staff  is not in the list\n";

[Goten]

Toute mes confuses... on devrait jamais posté a jeun ... toutes mes excuses au posteur original.

[ram-0000]

OK, je comprend mieux alors la condition de sortie, je ne l'avais pas lu comme cela, en fait, je n'avais pas compris que fich était un objet "fichier"
comme quoi un commentaire bien placé vaut 10 explications

[jane40]

Toute mes confuses... on devrait jamais posté a jeun ... toutes mes excuses au posteur original.

pas de souci
Merci a tous pour votre aide