Discussion :

[grand's]

encore mon probleme de structure

je dois recuperer des donnees contenue dans des string se trouvant dans un ou plusieur fichiers
un string contient nom;prenom;adresse;....
je me dis que je vais creer une structure pour les mettre dedans et ayant plusieurs string a traiter, j'ai créé un vecteurde ma structure.
mais je ne vois pas comment inserer chaque terme a l'interieur.
ci-dessous mon 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
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
struct A
{
	string nom;
	string prenom;
	string rue;
	string num_rue;
	string ville;
	string pays;
 
	struct compte
	{
		string tab_compte[200];
	};
};
 
vector<A> vec_A;
 
int main(int argc, char *argv[])
{
	//recuperation noms de fichier et affichage
	cout << "Liste des fichiers" << endl;
 
	for (int i = 1; i < argc; i++)
	{
		cout << "Fichier " << i << " : " << argv[i] << endl;
	}
 
	for (i=1; i < argc ; i++)		//on examine tout les fichiers entres en ligne de commande
	{
		//ouverture fichier
		ifstream fichier(argv[i]);
		//controle existence du fichier
		if (!fichier) 
		{
			cerr << "Impossible d'ouvrir le fichier " << argv[i] << endl;
			exit(EXIT_FAILURE);
		}
 
		string s1,s2;
 
		while (!fichier.eof())		//on boucle tant qu'on est pas a la fin du fichier
		{
			//recupere la ligne
			getline(fichier,s1);
			int tmp = 0;
			for (size_t i = 0; i < s1.length(); i++)
			{
				if (s1[i] == ';')	//si onrencontre le separateur ;
				{
					s2 = s1.substr(tmp,(i-tmp));	//on copie le terme dans s2
					tmp = i+1;
					cout <<  s2 << " ";
				}
			}
			cout << endl;
		}
		fichier.close();
	}
 
 
 
	return 0;
}

j'arrive a extraire chaque terme mais comment l'inserer a la bonne place dans la structure??

[Laurent Gomila]

Si ta ligne est toujours composée du même nombre de champs distinct, il ne faut pas les extraire dans une boucle, il te faudra les récupérer un à un et les mettre ensuite dans la donnée membre de A correspondante.

Autre solution : stocker toutes tes données dans un tableau, et faire a.Champ[i] = s2 à chaque tour de boucle.

Encore une autre : utiliser un std::map<string, string> pour stocker tes membres, et un tableau contenant les nom de ces membres.
Genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
struct A
{
   map<string, string> Membres;
};
 
static const char* Champs[] = {"nom", "prenom", "rue", ...}
 
for (...)
{
   a.Membres[Champs[i]] = s2;
}

Astuce : tu peux utiliser getline avec comme 3eme paramètre ';' pour récupérer tes sous-chaines, au lieu de find + substr.

[grand's]

j'ai du faire une boucle car le nombre de champs peut varier
si les infos concernant la personne sont toujours pareils, j'ai des numero de compte a la fin avec un solde, et je peut en avoir un ou plusieurs.
je me base donc sur \n pour la fin de ligne

j'ai effectivement penser a mettre chaque terme dans un tableau (en fait un vecteur car je ne connais pas la taille a l'avance) mais je seche sur la facon de gerer le string suivant
comment le faire avancer sur le membre suivant (suis pas sur d'etre clair la)

[mtopoloff]

mais en fait ta string peut etre composée de x parametres dont le maximum est connu du genre

nom;prenom;adresse

mais aussi, parfois:

nom;prenom;adress;compte

Est-ce que tu peux influer sur la generation des string ?

du genre quand un parametre est sauté metre ;

nom;;;compte ???

Dans ce cas le parsing de ta string est simple...

[Aurelien.Regat-Barrel]

Quelques remarques:
- pourquoi n'utilises-tu pas un vector pour tab_compte ?
- le test

Code :

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

while (!fichier.eof())

n'est pas suffisant : la lecture peut échouer pour d'autres rainsons que la fin de fichier. remplace plutot cette ligne et la suivante par :

Code :

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

while ( getline(fichier,s1) )

voir la FAQ pour les explications
- il ne faut pas utiliser exit() dans les programmes C++. Fait un simple return.

Pour ton problème tu peux coder une fonction qui crée un objet A à partir d'une chaine et d'un position de début dedans. Du genre :

Code :

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

1
2
3
4
5
6
7
8
9
 
size_t ExtractA( const std::string & S, size_t pos, A & Result )
{
    // parcourir S à partir de pos
    // lire tous les champs en comptant le nombre de carcatères lus
    // remplir Result au fur et à mesure
    // renvoyer 0 si on est en fin de chaine et que tout n'est pas initialisé
    // sinon renvoyer le nombre de caractères traités
}

Ce peut être une fonction membre statique de struct A. Tu pourrais l'utiliser ainsi :

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
 
while ( getline(fichier,s) )
{
    size_t i = 0;
    A a;
    while ( i < s.length() )
    {
        size_t read = ExtractA( s, i, a );
        if ( read == 0 )
        {
            // erreur
        }
        vec_A.push_back( a );
        i += read;
    }
}

[grand's]

je recupere les string dans des fichiers txt
j'ai ma liste de champs et selon le string compte1, compte2, ou plus encore, mais pas forcement.
donc seul la fin du string change et je ne peut influer dessus.

[Aurelien.Regat-Barrel]

En fait comme cela ressemble à de la sérialisation, je pense que ce serait le bon endroit de mettre une fonction FromString et ToString dans ta classe A. Si tu modifies ta classes tu corriges le code de ces fonctions et tout le reste du code reste inchangé et marche avec la nouvelle classe.

[tut]

un moyen serait de créer un tableau de string, indicé par un enum.
L'enum te permets de dimenssionner ton tableau, d'incrémenter l'indice pour ta boucle, et d'accéder facilement à tes champs :

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
 
// déclaration 
enum EnIndice
{
   Nom,
   Prenom,
   Adresse, 
 
   EnIndiceMax
};
std::string Personne[EnIndiceMax];
 
// utilisation
Personne[Nom] = UnNomLu;
....
for (int i=0; i<EnIndiceMax; i++)
{
   std::cout << Personne[i] << std::endl;
}

c'est une idée, par forcément LA solution.
Je crois qu'il y aurait des trucs à faire avec un std::map.

[Luc Hermitte]

Si j'ai bien compris, tu cherche à faire ça, non ?

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
#include <vector>    // std::vector<>
#include <string>    // std::string
#include <sstream>   // std::istringstream
#include <algorithm> // std::copy
#include <iterator>  // std::ostream_iterator
#include <iostream>  
 
using namespace std;
 
struct A
{
    string nom;
    string prenom;
    string rue;
    string num_rue;
    string ville;
    string pays;
 
    vector <string> tab_comptes;
 
    void swap(A & other)
    {
        nom.swap(other.nom);
        prenom.swap(other.prenom);
        rue.swap(other.rue);
        num_rue.swap(other.num_rue);
        ville.swap(other.ville);
        pays.swap(other.pays);
        tab_comptes.swap(other.tab_comptes);
    }
}; 
 
std::ostream & operator<<(std::ostream & os, const A & v)
{
    os << v.nom << " " << v.prenom
        << "  habite au " << v.num_rue << ", " << v.rue
        << " " << v.ville << "(" << v.pays << ") --- "; 
    copy(v.tab_comptes.begin(),v.tab_comptes.end(),
            ostream_iterator<string>(os, ", "));
    return os;
}
 
istream & operator>>(istream & is, A & a)
{
    string ligne;
    if (getline(is,ligne)) {
        istringstream iss(ligne);
        A tmp;
        if (getline(iss,tmp.nom,';') && getline(iss,tmp.prenom,';')
                 && getline(iss,tmp.rue,';') && getline(iss,tmp.num_rue,';')
                 && getline(iss,tmp.ville,';') && getline(iss,tmp.pays,';'))
        {
            string cpt;
            while (getline(iss, cpt, ';')) {
                tmp.tab_comptes.push_back(cpt);
            }
            // éventuellement vérifier que les numéros de compte sont OK
            // et qu'il y a des comptes.
            a.swap(tmp);
        } else {
            std::cerr << "ERREUR: Entrée incorrecte: " << ligne << "\n";
            is.setstate(std::ios_base::failbit);
        }
    }
    return is;
}
 
 
void test(string const & fic)
{
    cout << fic << "\n";
    istringstream f(fic);
 
    A a;
    while (f >> a)
        std::cout << "---> " << a << "\n";
    if (!f.eof() && f.fail()) 
        std::cout << "Erreur!\n";
 
    cout << "\n------\n";
}
 
int main (int argc, char **argv)
{
    test("toto;titi;avenue truc;15;tlse;fr;1;2;42"
            "\ntata;tutu;rue chose;42;paris;fr;12;15;16");
    test("toto;titi;avenue truc;15;tlse;fr;1;2;42"
            "\ntata;tutu;rue chose;42;paris;fr;12;15;16\n");
    test("toto;titi;avenue truc;15;tlse;fr;1;2;42;5;6;8;9");
    test("toto;titi;avenue truc;15;tlse;fr;");
    test("toto;titi;avenue truc;15;tlse;fr");
    test("toto;titi;avenue truc;15;tlse;");
 
    std::cout << std::endl;
 
    return 0;
}

(on peut encore simplifier en sortant boost)

[tut]

ça sert à quoi le A::swap() ?
pourquoi ne pas utiliser le constructeur de recopie par défaut ?

[Aurelien.Regat-Barrel]

Parce que ça fait une recopie
swap ça échange le contenu, sans le copier.

[tut]

dans ce cas-là, deux std::string ont le même espace mémoire alors ?

[Luc Hermitte]

??
Les buffers internes sont échangés.

[tut]

[grand's]

bon bin on va etudier tout ca hein

ca me fait du boulot pour ce soir

[grand's]

d'abord, merci pour vos reponses
j'ai pu comprendre certaines choses et avancer ds mon code et aussi le rendre un peu plus propre

petite question toutefois concernant ton code Luc,

Code :

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

istringstream f(fic);

ca sert bien a recreer un flux a partir d'un string ?

a la compilation, j'ai une liste de warning qui apparaissent

• warning C4786: 'std::reverse_iterator<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const *,std::basic_string<char,std::char_traits<char>,std::allocator<char>
  >,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const *,int>' : identifier was truncated to '255' characters in the debug information

en fait j'ai 5 fois ce warning, mais je ne comprend pas pourquoi (j'ai eu le meme type de warning en utilisant une list pour stocker mes strings)

sinon merci pour ton exemple, ca m'a permit de comprendre et de programmer mon traitement (sans plaggia)

[Laurent Gomila]

petite question toutefois concernant ton code Luc,

Code :

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

istringstream f(fic);

ca sert bien a recreer un flux a partir d'un string ?

C'est ça oui.

warning C4786: 'std::reverse_iterator<std::basic_string<char,std::char_traits<char>,std::allocator<char> > const *,std::basic_string<char,std::char_traits<char>,std::allocator<char>
>,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const &,std::basic_string<char,std::char_traits<char>,std::allocator<char> > const *,int>' : identifier was truncated to '255' characters in the debug information

C'est juste que lorsque tu imbriques trop de templates, certains noms de symboles sont trop longs (> 255 caractères), donc il les tronque. Ca n'a absolument aucun effet, le mieux est d'ailleurs de désactiver ce warning à la noix.

[grand's]

ok merci