Discussion :

[globule71]

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
bool WriteLog(std::string strdata, std::string strpath)
{
		char* tmpdate = new char[256];
		char buffer [80];
		struct tm *gm;
		struct tm *lm;
 
		time_t maintenant;
		time(&maintenant);
 
		gm=gmtime(&maintenant);
		//lm=localtime(&maintenant);
		gm->tm_hour=gm->tm_hour;
		sprintf(tmpdate,"%02d/%02d/%02d %02d:%02d:%02d  GMT : ",
		gm->tm_mday, gm->tm_mon + 1, gm->tm_year % 100,
		gm->tm_hour, gm->tm_min, gm->tm_sec);
		std::string strdate = std::string(tmpdate);
 
 
		std::fstream fileLOG(strpath.c_str(),std::ios::app);
		fileLOG.exceptions ( std::ifstream::eofbit | std::ifstream::failbit | std::ifstream::badbit );
			try
			{
				fileLOG.write((strdate+strdata).c_str(),(strdate+strdata).length());
				fileLOG.close();
			return true;
			}
			catch(std::fstream::failure e)
			{
				std::cout <<"exception de fichier non geree"<<std::endl;
				return false;
			}
 
}

Qu'est ce que tu penses de ma fonction log ?
Enfin je trouve quand même bizarre que ça plante alors qu'elle est même pas encore appelée !! C'est sa definition qui me fait tout planter ...


edit : Mon pb se passe quand j'appele cette fonction de ma fonction log :

Code :

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

std::fstream fileLOG(strpath.c_str(),std::ios::app);

[mchk0123]

Pas le temps ce soir de regarder + précisément ta fonction.

Mais en attendant :

Code :

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

1
2
3
4
   catch(std::fstream::failure e) {
       std::cout <<"exception de fichier non geree"<<std::endl;
       return false;
    }

C'est pas terrible. Si un pb. quelconque arrive dans la couche C++ de gestion de des flux (donc émission d'une exception std::fstream::failure), il ce peut que ton programme n'arrive plus à faire le "cout <<".

Je préfèrerais écrire :

Code :

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

1
2
3
4
   catch(std::fstream::failure e) {
       printf("exception de fichier non geree\n");
       return false;
    }

[globule71]

Voilà, je crois que je ne peux pas faire mieux niveau défrichage de code...
Pourquoi est ce que l'instance d'un fstream ferait-elle planter cout à l'execution ?
Je ne sais pas trop comment le compilateur fait son boulot mais ceci fait le même type d'erreur quand j'instancie un fstream dans une définition de fonction (hors du main).
Je pense que lors de la compilation y'a un amalgamme entre des membres des deux fonctions ou un truc du genre...

Code :

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

1
2
#include <iostream>
#include <fstream>

Code :

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

1
2
3
4
5
6
nt _tmain(int argc, TCHAR* argv[], TCHAR* envp[])
{
	std::fstream fileLOG;
	int nRetCode = 0;
	std::cout<<"Attention ça plante encore:/"<<std::endl;
...

Code :

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

1
2
3
4
5
nt _tmain(int argc, TCHAR* argv[], TCHAR* envp[])
{
	int nRetCode = 0;
	std::cout<<"Par contre là ça plante plus !"<<std::endl;
...

edit : ok je regarderai ça demain je vais arrêter pour ce soir lol. En tout cas merci pour le conseil concernant le try/catch.

[globule71]

Bon j'ai pas trouvé d'ou pouvait provenir le bug... alors pour l'instant je remplace tous les cout par des printf... mais bon c'est pas très pratique j'ai un tas de conversion à faire du coup !

[mchk0123]

Est si tu ne compiles que ça : (pas d'autre code autour)

Code :

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

1
2
3
4
5
6
int _tmain(int argc, TCHAR* argv[], TCHAR* envp[])
{
	std::fstream fs("fichier.txt", std::fstream::out);
	int nRetCode = 0;
	std::cout<<"Attention ça plante encore:/"<<std::endl;
...

Qu'est-ce qu'il se passe ?

[globule71]

il aime pas trop ce style de déclaration, bizarre pourtant je l'avais déjà déclaré comme ça sans problème... mais l'inconvénient ici de cette façon c'est qu'on ne peut pas mettre l'ouverture du fichier dans le bloc try/catch.

1>.\xxxxxxxx_MFC.cpp(139) : error C2653: 'fstream' : n'est pas un nom de classe ni d'espace de noms

edit : au temps pour moi j'ai oublié le std:: devant mais ça ne change rien au pb, ça continue de planter au même endroit ...

[mchk0123]

Dslé, petit erreur dans mon code. Corrigé ci-dessus.

[globule71]

Je sais pas si les "edits" comptent pour des up mais voilà c'était pour dire qu'on retrouvait le même pb que la déclaration précédente

[globule71]

Bon simplement pour dire que j'ai pas réussi à résoudre mon problème... Alors en attendant j'ai tout fait à coup de printf (enfin presque parce que y'a des conversions assez ch*** à faire pour certaines trucs...) donc voilà si quelqu'un d'autre à une idée... En attendant je laisse le post en suspens...

[mchk0123]

Tu y vas un peu à la "bourine" en remplaçant tout en prinf ...

Déjà dans ton code tu utilise les TCHAR sans englober tes chaînes de caractères par les macros _TEXT.

Bien que cela n'est pas directement lié à ton pb, c'est vraiment pas propre.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <tchar.h>
 
#include <string>
#include <iostream>
#include <fstream>
 
int _tmain(int argc, TCHAR* argv[], TCHAR* envp[])
{
	std::fstream fs(_TEXT("fichier.txt"), std::fstream::out);
	int nRetCode = 0;
	std::cout<<_TEXT("Attention ça plante encore:/")<<std::endl;
 
        return 0;
}

[Médinoc]

L'unicode et les TCHARs ne servent à rien dans les streams ouverts par défaut, le format des données d'entrée ne décide pas de l'encodage en sortie.

[mchk0123]

En compilant et exécutant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <string>
#include <iostream>
#include <fstream>
 
int main(int argc, char* argv[])
{
	std::fstream fs("fichier.txt", std::fstream::out);
	int nRetCode = 0;
	std::cout<<"Attention ça plante encore:/"<<std::endl;
 
        return 0;
}

Tu as encore une erreur à l'exécution ? Si c'est pas le cas, ton problème provient des caractères MBCS.

[globule71]

j'ai remplacé par les char* mais visiblement ça change rien

[mchk0123]

Je test aujourd'hui sur un ordi avec VC++ 2005 Express. Pour voir si j'ai le même pb., je te dis ça rapidement.

[globule71]

C'est fort possible que ça passe de ton coté, car la première fois que ça m'a fait ça avec des streams, j'ai refait le projet et ça a marché... Donc à voir peut être un pb de configuration du compilateur...

[mchk0123]

Bon , cet aprés-midi pour avoir compilé l'exemple copié-collé sous VS2005.
Aucun plantage ...

Vraiment bizarre.