Discussion :

[kochfet]

Salut à tous.

J'ai un fichier test.txt. Ce fichier est de la forme suivante :
C_1 10 14 12
C_2 1 3 12

Je veut récupérer son contenu dans des variables.

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
 
void recuperer()
{
         int card1;
	 int tObj1;
	 int tIdObj1;	
         string c;
	 ifstream fichier("test.txt", ios::in);  
         if(fichier)  
         {	
	    string ligne;  
	    while(getline(fichier, ligne))  
	    {
               if(ligne.find("C_1")==0)
		{
		fichier >> c >> card1 >> tObj1 >> tIdObj1 ;  
		fichier.close();
		}
	    }
		    fichier.close();   
        }
		else
		{
		  cerr << "Impossible d'ouvrir le fichier !" << endl;
		}
}

A l’exécution, j'obtiens les variables avec les valeurs de la deuxième ligne càd la ligne qui correspond à la condition de la recherche avec find(string) a été sauté. svp aidez moi à connaitre ou réside le problème.

Merci d'avance.

[Ehonn]

Bonjour

Lorsque tu as trouvé la ligne qui t'intéresse, c'est ligne qu'il faut lire et pas fichier.

Pour utiliser un string comme un flux, utilise istringstream.

[moham104]

Je crois que tu utilise mal la fonction find() de string

Le meilleur manière de coder ce problème ça serait

Code :

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

1
2
3
4
5
6
7
8
 
std::string str("C_1");
std::size_t trouve = ligne.find(str);
if (trouve != std::string::npos )
{
    fichier >> c >> card1 >> tObj1 >> tIdObj1 ;  
    fichier.close();
}

Je dirais que c'est risqué de faire

Code :

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

1
2
 
if(ligne.find("C_1")==0)

Puisque il se peut qu'il y ai un espace au début du fichier ou autre autre caractère. Donc la fonction find pourrais te retourner un autre indice que le zero.

De plus si tu sais que le "C_1" se trouve toujours à l'indice 0 (ce qui est rarement le cas parce que souvent on ne sait pas dans quel ordre l'information est organisé dans une fichier) donc pas besoin de le chercher avec find() tu peux directement vérifier si les trois premier caractères sont le "C", le "_" et le "1" sinon tu passe à autre instruction.

[Ehonn]

Oui, il probablement utiliser != std::string::npos (sauf si on veut absolument que ça soit en début de ligne).
Par contre les variables temporaires str et trouve (qui sont même pas const et utilisés qu'une seule fois) dégradent clairement la lisibilité.

[Bousk]

Bonjour

Lorsque tu as trouvé la ligne qui t'intéresse, c'est ligne qu'il faut lire et pas fichier.

Pour utiliser un string comme un flux, utilise istringstream.

this

Ton code lit bien la ligne du fichier, mais il agit ensuite sur le fichier, dont le curseur de lecture se trouve sur la ligne suivante.
Pour parser la ligne lue, il faut... parser la ligne lue et non le fichier.

[ctxnop]

Salut,
Je suis assez étonné que personne n'ai donné une solution utilisant boost.spirit :

Code c++ :

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
 
template <typename Iterator>
bool parse_line(Iterator first, Iterator last, std::string& s, int& i1, int& i2, int& i3)
{
	using namespace boost::spirit;
	namespace phx = boost::phoenix;
 
	bool r = qi::phrase_parse(first, last,
		(
			(
				qi::lexeme[+(qi::char_ - ' ')  [phx::ref(s) += qi::_1]] >>
				qi::int_[phx::ref(i1) = qi::_1] >>
				qi::int_[phx::ref(i2) = qi::_1] >>
				qi::int_[phx::ref(i3) = qi::_1]
			)
		), ascii::blank);
 
	if(!r || first != last) return false;
	return r;
}
 
void f()
{
	std::string input = "C_1 10 14 12";
	std::string o;
	int i1, i2, i3;
	bool r = parse_line(input.begin(), input.end(), o, i1, i2, i3);
	// si r == true alors o == "C_1" && i1 == 10 && i2 == 14 && i3 == 12
}

[Davidbrcz]

Salut,
Je suis assez étonné que personne n'ai donné une solution utilisant boost.spirit :

C'est pas du tout overkill !

[ctxnop]

Qui à dit le contraire ? C'est une solution comme une autre...
C'est tout de même assez adapté au problème : parser des lignes de texte.
En plus on ne peut pas dire que je suis un maître de spirit, il y a probablement plus simple.
Je m'étonnais juste qu'il n'y ai personne à proposer cette solution alors qu'on la sort généralement à tout bout de champ.

[Neckara]

Bonjour,

Je pense que kochfet doit débuter.

Lui proposer une solution avec boost.spirit reviendrait à devoir lui expliquer :
- ce qu'est une bibliothèque ;
- comment compiler/installer une bibliothèque ;
- comment utiliser une bibliothèque ;
- comment fonctionne boost.spirit ;
- les templates ;
- les using namespace ;
- les itérateurs ;

S'il n'a pas déjà fuit en voyant le code

Ne mettons donc pas la charrue avant les bœuf


Après, j'aime bien boost, mais je ne pense pas que ce soit une solution adapté à ce cas là.
On ajoute pas de dépendances aussi "gros" pour l'utiliser à un seul endroit où on aurait même très bien pu s'en passer.

[Ehonn]

Il n'y a pas que kochfet qui lit le forum.

[therwald]

Mais il n'y a pas que kochfet qui doive éviter d'utiliser un missile atomique pour tuer une mouche...