Pb fonction strtok pour parser

**filipbou** · 26/01/2008, 23h55

Bonjour,

J'essaye de parser un fichier csv avec la fonction strtok cela fonctionne plutot bien. Seulement quand j'essaye de stocker les données dans une matrice mes données sont erronées ?

Voici mon fichier CSV :

Code :

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

Voici 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
int main(int argc, _TCHAR* argv[])
{
FILE *myfile;
char line1[1000];
char line2[1000];
char *stptr;
int lcount = 0; // Nombre de cellule
double**data; // tableau de stockage
int i,j;
double n=0;
double nb_colonnes=2;
double duree_historique=10;
 
//Allocation table data
	data=(double**)malloc(duree_historique*sizeof(double*));
		for (j = 0;j<duree_historique;j++)
               {
			data[j]=(double*)malloc(nb_colonnes*sizeof(double));
		}
	// Initialisation
	 for(i=0;i<duree_historique;i++){
                   for (j = 0;j<nb_colonnes;j++)
                  {
			data[i][j] = 0;
		  }
	 }
 
	while (fgets(line1,sizeof line1,myfile) != NULL){ // parcours le fichier
		lcount = 0;
		strcpy(line2,line1);
		stptr = strtok (line2,";");
		j=0;
		while (stptr != NULL){
			  cout<<"\ni = "<<i;
			  cout<<" j = "<<j<<"\n";
			  printf ("%s ",stptr);
			  stptr = strtok (NULL, ";");
			  n=atof(line2);
			  data[i][j]=n;
			  j++;
		}
		i++;
		j--;
	}
 
		fclose(myfile);
		//Affichage du tableau de stockage
		for(i=0;i<10;i++){
			for(j=0;j<2;j++){
				cout<<data[i][j]<<" ";
			}
			cout<<"\n";	
		}
		getch();
	return 0;
}

La sortie console avec le strtok me donne bien les bonnes valeurs.
Mais lorsque j'affiche ma matrice j'obtiens :

Code :

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

Une idée ?

**aoyou** · 27/01/2008, 00h35

Arg que c'est moche ce mélange de C et de C++...
On dispose en C++ de fonctions plus puissantes que le strok du C... mais veux-tu faire du C++ ou veux-tu faire du C mais dans ce cas-là tu n'est pas sur le bon forum.

**filipbou** · 27/01/2008, 00h56

Oui ce n'est peut etre pas tres propre ...
Je t'avoue peut importe que ca soit en c ou C++
Dois je plutot poster sur l'autre forum alors ?

**aoyou** · 27/01/2008, 01h01

Si tu veux faire du C, poste sur l'autre forum.

Je te donne ici l'équivalent en C++ de ton code.
On peut améliorer en utilisant une matrice unidimensionnelle.

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
 
#include <fstream>
#include <iostream>
#include <sstream>
 
using namespace std;
 
int main(int argc, char** argv[]) {
 
  double **data; // tableau de stockage
  int nb_colonnes=2;
  int duree_historique=10;
 
  //Allocation table data
  data = new double* [duree_historique];
  for (int j = 0 ; j < duree_historique ; j++) {
    data[j] = new double [nb_colonnes];
  }
 
  // Initialisation
  for(int i=0 ; i < duree_historique ; i++) {
    for (int j = 0 ; j < nb_colonnes ; j++) {
      data[i][j] = 0;
    }
  }
 
  ifstream myfile ("file.csv");
 
  if (myfile) {
 
    int count = 0;
    string line;
 
    while (getline (myfile, line)) {
      double d1, d2;
      char sep;
      istringstream iss (line);
      iss >> d1 >> sep >> d2;
      if (iss.fail () || sep != ';') {
	cerr << "Erreur à la ligne " << ++count << endl;
      } else {
	data[count][0] = d1;
	data[count][1] = d2;
	count++;
      }
    }
 
    myfile.close ();
 
  } else {
    cerr << "Ne peut pas ouvrir le fichier" << endl;
  }
 
  //Affichage du tableau de stockage
  for(int i = 0 ; i < 10 ; i++) {
    for(int j = 0 ; j < 2 ; j++) {
      cout<<data[i][j]<<" ";
    }
    cout<<"\n";	
  }
 
  return 0;
 
}

**filipbou** · 27/01/2008, 01h14

Merci pour ta réponse, effectivement c'est nettement plus propre !

Juste une question à quoi sert ton test :

Code :

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

1
2
3
if (myfile.fail ()) {
      cerr << "Erreur à la ligne " << count + 1 << endl;
}

J'ai l'erreur des que je suis à ligne 4 en supposant que j'ai 3 lignes de données dans mon csv.

**filipbou** · 27/01/2008, 01h24

Mille excuses, effectivement c'était le probleme de la ligne vide. J'ai parlé trop vite.

J'ai un autre petit souci, les données du csv proviennent d'excel et certaines valeurs peuvent etre à virgules
ex: 2;10,3

Ca pose probleme avec le C++ qui voudrait plutot : 2;10.3

**aoyou** · 27/01/2008, 01h26

Envoyé par filipbou

Merci pour ta réponse, effectivement c'est nettement plus propre !

Juste une question à quoi sert ton test :

Code :

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

1
2
3
if (myfile.fail ()) {
      cerr << "Erreur à la ligne " << count + 1 << endl;
}

J'ai l'erreur des que je suis à ligne 4 en supposant que j'ai 3 lignes de données dans mon csv.

J'ai corrigé le code. Cf. mon post précédent.
La ligne devient donc iss.fail() et non plus myfile.fail().
Je vérifie juste que la ligne est bien formatée suivant double;double.
Oui, il y a une erreur à la ligne 4. Améliore le code pour qu'une ligne vide ne provoque pas une erreur.

**coyotte507** · 27/01/2008, 01h20

Voilà le code après correction:

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
#include <cstdio>
#include <cstdlib>
#include <conio.h>
#include <iostream>
 
using namespace std;
 
int main(int argc, char* argv[])
{
    FILE *myfile=fopen("text.txt", "r");
    char line1[1000];
    char line2[1000];
    char *stptr;
    int lcount = 0; // Nombre de cellule
    double**data; // tableau de stockage
    int i,j;
    double n=0;
    double nb_colonnes=2;
    double duree_historique=10;
 
  //Allocation table data
    data=(double**)malloc((int)(duree_historique)*sizeof(double*));
    for (j = 0;j<duree_historique;j++)
    {
        data[j]=(double*)malloc((int)(nb_colonnes)*sizeof(double));
    }
    // Initialisation
    for (i=0;i<duree_historique;i++){
        for (j = 0;j<nb_colonnes;j++)
        {
            data[i][j] = 0;
        }
    }
    i=0;
    while (fgets(line1,sizeof line1,myfile) != NULL){ // parcours le fichier
        lcount = 0;
        strcpy(line2,line1);
        stptr = strtok (line2,";");
        j=0;
        while (stptr != NULL){
            cout<<"\ni = "<<i;
            cout<<" j = "<<j<< endl;
            printf ("%s \n",stptr);
            n=atof(stptr);
            stptr = strtok (NULL, ";");
            data[i][j]=n;
            j++;
        }
        i++;
        j--;
    }
 
    fclose(myfile);
    //Affichage du tableau de stockage
    for (i=0;i<10;i++){
        for (j=0;j<2;j++){
            cout<<data[i][j]<<" ";
        }
        cout<<endl;
    }
    getch();
    return 0;
}

D'abord j'ai mis des (int) dans le malloc sinon le compilo était pas d'accord.
Ensuite, j'ai rajouté un "i=0" avant le while sinon j'avait une segfault
Sinon, j'ai remplacé les cout << "n" par des cout << endl.
Sinon, tu utlises <iostream> ou <iostream.h>?

Enfin, pour ton problème:
Tu mets le strcpy avant ta deuxième boucle!!
Ce qui veut dire que line2 n'est pas changée dans ta boucle, c'est pour ça que tu as deux fois les mêmes chiffres.

Enfin c'est corrigé

Code :

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