Discussion :

[MathG]

Bonjour tout le monde,

Depuis quelques heures je suis confronté à un problème de "segmentation fault" lors d'un appel de sprintf dans une des méthode de mon objet.

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
 
void FortranFile::write(double * d,long size)
{
	unsigned short i = (sizeof(double)*size)%(0xFFFF+1);
	unsigned short e = (sizeof(double)*size)/(0xFFFF);
	char * str;
	if ( !fileIsOpened() ) open();
/*....*/
	else if ( format == FortranFileFormat::TEXT )
	{
		for ( long i = 0 ; i < size ; i++ )
		{
                        /*Le probleme intervient ici */
			sprintf(str,"%f",*(d+i));
			cout << str << endl;
		}
	}
 
}

et j'ai les deux méthodes suivantes qui appellent ce code :

Code :

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

1
2
3
4
5
6
7
8
9
void FortranFile::write(double * d)
{
	write(d,(long)1);
 
}
void FortranFile::write(double d)
{
	write(&d,(long)1);
}

Et voici l'appel effectué avec le programme principal utilisé pour tester ma classe :

Code :

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

1
2
3
4
5
6
7
8
 
int main(char * argv[],int argc)
{
	double d = 5.0;
	FortranFile f("nom de fichier","w",FortranFileFormat::TEXT);	
	f.write(d);
	f.close();	
}

le programme affiche bien 5.0000 qu'on effectue f.write(d) ou f.write(&d) alors que quand j'utilise un code du meme genre que ci-dessous il fonctionne parfaitement.

Code :

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

1
2
3
4
5
6
7
8
 
int main(char * argv[],int argc)
{
	double  d[] = {5.0,6.0};
	FortranFile f("nom de fichier","w",FortranFileFormat::TEXT);	
	f.write(d,2);
	f.close();	
}

J'ai beau chercher je ne comprend absolument pas où est le problème dans ce code... si quelqu'un a une idée je suis vraiment preneur.

Un grand merci d'avance à tout ceux qui jetterons un coup d'oeil par ici.

[koala01]

Salut,

Je suis peut etre buté sur ce point, mais, pourquoi d'enquiquiner avec des fonctions et des type venant du C, alors que tu pourrais tout aussi bien utiliser les classes et méthodes éprouvées propre au C++

apres tout:

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
void FortranFile::write(double * d,long size)
{
	unsigned short i = (sizeof(double)*size)%(0xFFFF+1);
	unsigned short e = (sizeof(double)*size)/(0xFFFF);
	std::string str;
	if ( !fileIsOpened() ) open();
/*....*/
	else if ( format == FortranFileFormat::TEXT )
	{
		for ( long i = 0 ; i < size ; i++ )
		{
                        std::istringstream iss(str);
                        iss>>*(d+1);
			cout << str << endl;
		}
	}
 
}

devrait, si j'a bien analysé, donner le meme résultat...

D'un autre coté, qui dit segmentation vault, dit, bien souvent, acces à un emplacement de la memoire auquel on n'aurait pas du accéder...

De la meme manière, le transtypage du genre de

Code :

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

write(d,(long)1);

me semble particulièrement hasardeux...

[Volune]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
void FortranFile::write(double * d,long size)
{
    /*....*/
	char * str;
	if ( !fileIsOpened() ) open();
    /*....*/
}

Est-ce que ta variable str est initialisée quelque part ? Sinon c'est probablement l'oriogine de ton problème ...

Code :

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

1
2
3
	write(d,(long)1);
// peut etre mieux :
	write(d,1L);

Concernant le code proposé par koala01, je pense qu'un ostringstream serait plus propice a fournir le même résultat :

Code :

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

1
2
3
4
5
6
7
8
/*...*/
		for ( long i = 0 ; i < size ; i++ )
		{
                        std::ostringstream oss("");
                        oss << d[i];
			cout << oss.str() << endl;
		}
/*...*/

[David Fleury]

mais à quoi peut bien servir le ostrinstream ici ?
autant directement la variable dans cout sans passer par oss ou un sprintf.

[MathG]

le cout n'est là que pour vérifier que ma variable est bien lue, le reste du code n'as pas encore été effectué, voilà donc la petite raison de ce placement là

Pour ce qui est des stream à utiliser, oui effectivement ce serait une plus belle solution que d'utiliser ce qui est disponible en c++ et bien plus simple, mais c'est surtout parce que mes collègues ne sont pas habitués au c++ ( déjà leur mettre des objets dans les mains ... ils ont du mal lol donc j'reste avec ce qu'ils connaissent le mieux )

Pour ce qui est de la résolution du problème j'ai effectué (un peu par hasard)

Code :

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

1
2
3
4
5
6
7
 
for ( long i = 0 ; i < size ; i++ )
{
char * str
sprintf(str,"%f",*(d+i));
delete str;
}

et là, plus de segmentation fault alors que dans le code d'avant tout était initialisé correctement... à n'y rien comprendre, si quelqu'un à une explication j'suis vraiment preneur ( on apprend toujours de ses erreurs... en tout cas on espère)

[Luc Hermitte]

Franchement, si vous utilisez un compilo de C++, ils pourraient faire un effort. Visiblement tu ne sais pas encore gérer la mémoire et il faut bien qu'ils comprennent qu'ils auront d'autres stagiaires et autres débutants tout aussi peu initiés -- accessoirement, s'ils sont si rompus au C que cela, ils auraient du avoir vu ce qui clochait en même pas 5 secondes. Dans le cas contraire .. enfin bon je ne dirai rien.

1- man sprintf
Que lis-tu dans la doc sur la chaine entrante ?

2- à supposer que cette fonction de la libc réserve de la mémoire pour toi.
a- cela se ferait avec une fonction de la famille des *alloc (new n'existant pas en C). Donc on libère avec ?
b- combien même elle aurait utilisé un opérateur d'allocation du C++, cela aurait été new[] et non new. Avec quoi on libère ces tableaux dynamiques ?


Dernier truc, une segmentation fault, c'est pas déterministe. Ca peut sembler marcher, comme péter. Bref, en 1- et 2-, je parlais bien évidemment du dernier bout de code que tu as extrait. Bout de code qui est incorrect, et qui est parfaitement représentatif de ton problème que nous tu as initialement soumis.