Problème avec la fonction getline

**iamor** · 27/07/2006, 17h17

Bonjour,

Je viens de débuter en C++, et je me pose donc forcement des questions sur un nombre de choses elementaires. Je suis entrain de construire un petit programme poour lire une matrice à partir d'un texte et je suis bloqués devant l'utilisation de la fonction getline.

Voilà

Code :

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

1
2
3
4
 
ligne = new char [longueur]; 
	fichierEntree.getline(ligne,longueur);
	cout << ligne << endl;

Quand longeur est une constante et est égale par exemple à 100, ça marche trés bien. Tandis que lorsque je mets cette fonction dans une boucle et que je met longueur = 8 * i (par exemple) ça compile pas.
Pouvez vous me dire pourquoi ça ne marche pas quand on longueur en fonction de i

Merci de votre aide

**mnaulet** · 27/07/2006, 17h28

pourrais-tu mettre le bout de code qui ne marche pas et l'erreur qu'il te met?

**homeostasie** · 27/07/2006, 17h30

Alors je n'ai pas regardé les paramètres de la fonction getline mais je crois que "longueur" doit être un string.

Dans ton cas avec boucle, longueur est surement de type entier. Donc il te faudra convertir en string.

Je vais tout de même rejeté un coup d'oeil si c'est bien cela!

**Eusebe** · 27/07/2006, 17h33

Pourquoi ne pas plutôt utiliser std::getline comme proposé dans la FAQ ?
http://c.developpez.com/faq/cpp/?pag...ture_par_ligne

**iamor** · 27/07/2006, 17h38

Merci pour votre réponse

Voilà le code en entier, en fait longueur est bien de nature int puisqu'il envoie vers le nombre de caractère à lire dans le string c'est une méthode de istream

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
 
#include "essai.h"
#include <iostream>
#include <fstream>
#include <iomanip>
using namespace std;
const int LGMAX = 20;
void main()
{
  // Ouverture du fichier
 
	ifstream fichierEntree("c:\\test1.txt", ios::in);
 
  // Test d'ouverture du fichier
 
  if (!fichierEntree) {
    cerr << "Problème d'ouverture de fichier" << endl;
    exit(1);
  }
 
 
  char* ligne;
  int i = 0; 
  char * err = NULL;
 
ligne = new char[10];
fichierEntree.getline(ligne, 4);
int conv1 = strtoul(ligne,0,10);
ArrayLengthSpecifier dimension(conv1);
cout << conv1 << endl;
SymmetricMatrix covariance(dimension);
int longueur;
do
	{
	i = i + 1;
	longueur = 9 * i;
	ligne = new char [longueur];//format "0.00E+00" 
	//donc 8 caractères; 9i=8i+ (i-1) espaces + élt nul terminal 
	fichierEntree.getline(ligne,longueur);
	cout << ligne << endl; 
 
	/*	for (int j=1;j<=i;j++)
		{
			char temp[8];
			for (int k=1;k<=8;k++)
			{
				temp[k-1] = ligne[(9*j)+ k - 1-1];//attention dimension de 0 à n-1
				cout << temp[k] << endl;
			}
			const char* caractere=temp;
			covariance(i,j) = strtod(caractere, &err); 
 
		}
	delete ligne;	
 
	//z = strtod(ligne, &err);
 
}
 
	while (!fichierEntree.eof());

**iamor** · 27/07/2006, 17h39

c'est pareil je ne vois pas la diiférence ou y en a t-il une ??

**Eusebe** · 27/07/2006, 17h49

Envoyé par iamor

c'est pareil je ne vois pas la diiférence ou y en a t-il une ??

Si tu parles de std::getline, tu as regardé le lien que je t'ai donné ? c'est expliqué !
getline() d'un std::istream demande en paramètre un pointeur sur un tableau de char.
std::getline() renvoie un std::string... Et pose donc moins de problèmes d'allocation mémoire

Edit : tu es sûr que ton code est complet ? J'ai l'impression qu'il y a un "while (!fichierEntree.eof());" qui se balade tout seul à la fin. Et quel est le message d'erreur ?

**iamor** · 27/07/2006, 17h57

Il manquer juste deux petites lignes mais voilà le code complet.
Je suis d'accord sur le fait que la fonction getline demande en entré un pointeur, mais dans ce cas comment faire pour le lui dire qu'elle doit lire à chaque fois un nombre differents de caractères suivant ma boucle en fait le fichier .txt à lire est le suivant:

3
1.23E+00
5.13E-02 1.02E+00
8.21E-03 2.36E-02 1.36E+00

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
 
 
#include "essai.h"
#include <iostream>
#include <fstream>
#include <iomanip>
using namespace std;
const int LGMAX = 20;
void main()
{
  // Ouverture du fichier
 
	ifstream fichierEntree("c:\\test1.txt", ios::in);
 
  // Test d'ouverture du fichier
 
  if (!fichierEntree) {
    cerr << "Problème d'ouverture de fichier" << endl;
    exit(1);
  }
 
 
  char* ligne;
  int i = 0; 
  char * err = NULL;
 
ligne = new char[10];
fichierEntree.getline(ligne, 4);
int conv1 = strtoul(ligne,0,10);
ArrayLengthSpecifier dimension(conv1);
cout << conv1 << endl;
SymmetricMatrix covariance(dimension);
int longueur;
do
	{
	i = i + 1;
	longueur = 9 * i;
	ligne = new char [longueur];//format "0.00E+00" 
	//donc 8 caractères; 9i=8i+ (i-1) espaces + élt nul terminal 
	fichierEntree.getline(ligne,longueur);
	cout << ligne << endl; 
 
	/*	for (int j=1;j<=i;j++)
		{
			char temp[8];
			for (int k=1;k<=8;k++)
			{
				temp[k-1] = ligne[(9*j)+ k - 1-1];//attention dimension de 0 à n-1
				cout << temp[k] << endl;
			}
			const char* caractere=temp;
			covariance(i,j) = strtod(caractere, &err); 
 
		}
	delete ligne;	
 
	//z = strtod(ligne, &err);
 
}
 
	while (!fichierEntree.eof());
 
 fichierEntree.close();
 
}

**Eusebe** · 27/07/2006, 18h08

Après quelques corrections je n'ai pas d'erreur...
main doit renvoyer un int
Pour désallouer ligne, il faut que tu utilises delete[]

mon code minimal qui compile et fonctionne :

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
#include <iostream>
#include <fstream>
 
using namespace std;
 
int main()
{
  // Ouverture du fichier
	ifstream fichierEntree("c:\\test1.txt", ios::in);
 
  // Test d'ouverture du fichier
    if (!fichierEntree) {
        cerr << "Problème d'ouverture de fichier" << endl;
        exit(1);
    }
 
    char* ligne;
    int i = 0;
 
    ligne = new char[10];
    fichierEntree.getline(ligne, 4);
    int conv1 = strtoul(ligne,0,10);
 
    int longueur;
 
    do
	{
	i = i + 1;
	longueur = 9 * i;
	ligne = new char [longueur];//format "0.00E+00"
	//donc 8 caractères; 9i=8i+ (i-1) espaces + élt nul terminal
	fichierEntree.getline(ligne,longueur);
	cout << ligne << endl;
 
	delete[] ligne;
    }
	while (!fichierEntree.eof());
 
    fichierEntree.close();
}

Mais en C++, utilises plutôt des std::string, et pour convertir, essaie plutôt avec istringstream : http://c.developpez.com/faq/cpp/?pag...NGS_convert_to

**iamor** · 27/07/2006, 18h15

Merci pour ta réponse
As tu des sorties parceque moi quand je compile, j'ai une fenêtre dos vide ???

**iamor** · 27/07/2006, 18h17

non en fait ça marche mais ça continue à generer des lignes vides et dinc je ne vois pas le résultat à moins que je fasse une impression écran dès l'execution

Je cherche l'origine et je te tiens au courant

**iamor** · 27/07/2006, 18h22

je ne vois ce que ça peut être pourquoi il continue à executer (en sortant des lignes vides après les valeurs imprimés à l'écran)??

**Eusebe** · 27/07/2006, 18h23

Bizarre... Chez moi les lignes du fichier texte s'affichent bien...

**iamor** · 27/07/2006, 18h25

Je t'envoie le code que j'ai

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
 
 
#include "essai.h"
#include <iostream>
#include <fstream>
#include <iomanip>
using namespace std;
const int LGMAX = 20;
int main()
{
  // Ouverture du fichier
 
	ifstream fichierEntree("c:\\test1.txt", ios::in);
 
  // Test d'ouverture du fichier
 
  if (!fichierEntree) {
    cerr << "Problème d'ouverture de fichier" << endl;
    exit(1);
  }
 
 
  char* ligne;
  int i = 0; 
  char * err = NULL;
ligne = new char[10];
fichierEntree.getline(ligne, 4);
int conv1 = strtoul(ligne,0,10);
ArrayLengthSpecifier dimension(conv1);
cout << conv1 << std::endl;
SymmetricMatrix covariance(dimension);
 
do
	{
	i++;
	ligne = new char [9 * i];//format "0.00E+00" 
	//donc 8 caractères; 9i=8i+ (i-1) espaces + élt nul terminal 
	fichierEntree.getline(ligne, 9 * i);
	cout << ligne << endl; 
	delete[] ligne;
}
 
	while (!fichierEntree.eof());
 
 
  // Fermeture du fichier
 fichierEntree.close();
 
}

Est ce que tu peux le tester stp merci

**iamor** · 27/07/2006, 18h32

J'ai teste ton code et ça me fait la même chose (il continue à executer an mettant des lignes vides après les nombres)
Je le lancer avec générer la solution, puis avec débugger, puis avec executer sans debugger et ça continue à calculer
J'ai visual studio.net 2003
Bizarrrrrrre!!!

**iamor** · 27/07/2006, 18h44

Est ce que ça marche chez toi le code que je t'ai envoyé?

**Eusebe** · 27/07/2006, 19h45

Il fonctionne si je commente #include "essai.h" (que je n'ai pas) et les 3 lignes :
ArrayLengthSpecifier dimension(conv1);
cout << conv1 << std::endl;
SymmetricMatrix covariance(dimension);

Parce que je ne connais pas ArrayLengthSpecifier et SymmetricMatrix.

Mais mon compilateur est GCC, alors que toi si j'ai bien compris tu fais du C++ managé (sous .net). Et là, je n'y connais rien...

Mais de toutes façons, en C++, le code ci dessous est préférable :

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
#include <iostream>
#include <fstream>
#include <sstream>
 
using namespace std;
 
int main()
{
    // Ouverture du fichier
    ifstream fichierEntree( "c:\\test1.txt" );
 
    // Test d'ouverture du fichier
 
    if (!fichierEntree) {
        cerr << "Problème d'ouverture de fichier" << endl;
        exit(1);
    }
 
    string ligne; // variable contenant chaque ligne lue
 
    getline( fichierEntree, ligne );
 
    int conv1;
 
    istringstream(ligne) >> conv1;
 
    cout << "Nombre de lignes : " << conv1 << endl;
 
    // cette boucle s'arrête dès qu'une erreur de lecture survient
    while ( getline( fichierEntree, ligne ) )
    {
        // afficher la ligne à l'écran
        cout << ligne << endl;
    }
 
}

**iamor** · 28/07/2006, 09h51

BOnjour,

Bon voilà j'ai réussi à faire un truc qui tourne pas mal mais qui bute sur l'extraction des nombres à partir du ligne,
Si tu veux essayer le code suivant tu fais un copier coller dans un nouveau projet et tu le lance le fichier bloc notes qui va avec est le suivant :

3
0.000001
0.000001 0.000002
0.000001 0.000002 0.000003

Le code est le suivant :

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
 
 
#include "essai.h"
#include <iostream>
#include <fstream>
#include <iomanip>
using namespace std;
 
 
using namespace std;
 
void main()
{
  // Ouverture du fichier
		 ifstream fichierEntree("c:\\test1.txt", ios::in);
 
  // Test d'ouverture du fichier
    if (!fichierEntree) {
        cerr << "Problème d'ouverture de fichier" << endl;
        exit(1);
    }
 
    char* ligne;
    int i = 0;
	char * err = NULL;
	//double z; 
 
    ligne = new char[10];
    fichierEntree.getline(ligne, 4);
    int conv1 = strtoul(ligne,0,10);
	ArrayLengthSpecifier dimension(conv1);
	cout << conv1 << std::endl;
	SymmetricMatrix covariance(dimension);
 
    int longueur;
 
    do
		 {
		 i = i + 1;
		 longueur = 9 * i + 1;
		 ligne = new char [longueur];//format "0.00E+00"
		 //donc 8 caractères; 9i=8i+ (i-1) espaces + élt nul terminal
		 fichierEntree.getline(ligne,longueur);
		 cout << ligne << endl;
 
 
		 for (int j=1;j<=i;j++)
		{
			char temp[8];
			for (int k=0;k<=7;k++)
			{
				temp[k] = ligne[(9*j)+ k - 2];//attention dimension de 0 à n-1
 
			}
			const char* caractere=temp;
			cout << caractere << endl;
			covariance(i,j) = strtod(caractere, &err); 
 
		}
		delete[] ligne;	
 
		 }
		 while (!fichierEntree.eof());
 
    fichierEntree.close();
}

En sortie j'ai un problème il extrait mal les nombres on dirait qu'il prend pos en compte le premier nombre l'erreur vient surement de mes boucle k et j. Mais je n'arrive pas à voir l'origine t'as une idée la dessus?

**Sachiel31** · 28/07/2006, 10h56

Bonjour,

J'ai essayé ton code et voilà ce que j'ai obtenu :
(voir image jointe)

Par contre j'ai du mettre les lignes suivantes en commentaire puisque je ne connais ni ArrayLengthSpecifier ni SymmetricMatrix :

Code :

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

1
2
3
4
5
6
7
// #include "essai.h"
 
// ArrayLengthSpecifier dimension(conv1);
// cout << conv1 << std::endl;
// SymmetricMatrix covariance(dimension);
 
// covariance(i,j) = strtod(caractere, &err);

Est-ce que ça correspond à ce que tu souhaite ou pas???

P.S.: je ne pense pas que tu veuilles les 4 dernières lignes (qui sont bizarres), donc il se pourrait bien que tu aies des erreurs dans tes boucles for.

**Eusebe** · 28/07/2006, 11h21

Pourquoi ne pas utiliser le C++ ?
Ca serait tellement plus simple...
Essaie ce code et dis-nous si c'est ce que tu veux :

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
#include <iostream>
#include <fstream>
#include <sstream>
 
using namespace std;
 
int main()
{
    // Ouverture du fichier
    ifstream fichierEntree("c:\\test1.txt");
 
    // Test d'ouverture du fichier
    if (!fichierEntree) {
        cerr << "Problème d'ouverture de fichier" << endl;
        exit(1);
    }
 
    string ligne;
    int i = 0;
 
    getline( fichierEntree, ligne );
 
    int conv1;
    istringstream(ligne) >> conv1;
 
    cout << conv1 << endl;
 
    while ( getline( fichierEntree, ligne ) )
    {
        i = i + 1;
        cout << ligne << endl;
 
        istringstream iss(ligne);
        string temp;
 
        while (getline(iss, temp, ' '))
        {
            float MonDouble;
            istringstream(temp) >> MonDouble;
            cout << MonDouble << endl;
        }
    }
}

Problème avec la fonction getline

C++

Discussions similaires

Partager

Partager