Problème avec la fonction getline

Version imprimable

Voir 40 message(s) de cette discussion en une page

27/07/2006, 17h17
iamor

Problème avec la fonction getline
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:

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
27/07/2006, 17h28
mnaulet

pourrais-tu mettre le bout de code qui ne marche pas et l'erreur qu'il te met?
27/07/2006, 17h30
homeostasie

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!
27/07/2006, 17h33
Eusebe

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

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:

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());

27/07/2006, 17h39
iamor

c'est pareil je ne vois pas la diiférence ou y en a t-il une ??
27/07/2006, 17h49
Eusebe

Citation:

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 ?

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:

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();
 
}

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:

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

27/07/2006, 18h15
iamor

Merci pour ta réponse
As tu des sorties parceque moi quand je compile, j'ai une fenêtre dos vide ???
27/07/2006, 18h17
iamor

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
27/07/2006, 18h22
iamor

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)??
27/07/2006, 18h23
Eusebe

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

Je t'envoie le code que j'ai

Code:

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

27/07/2006, 18h32
iamor

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!!!
27/07/2006, 18h44
iamor

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

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:

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; } }

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:

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?

28/07/2006, 10h56
Sachiel31
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:

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.

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:

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;
        }
    }
}

Voir 40 message(s) de cette discussion en une page