Problème de gestion des exceptions

Version imprimable

Bonjour à tous voici un bout de code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
void CSLEngine::EndScene()
{
	try
	{
		m_pVideoMode->Display();
		throw CException(__LINE__, "Impossible de terminer le rendu de scène");
	}
	catch(const CException &e)
	{
		CLogger::LogToFile(CLogger::ITError, e.what(), "CSLEngine");
	}
}

Mon problème c'est que dans mon loggeur il me dit toujours "Impossible de terminer le rendu de scène", alors qu'après de multiple test cette méthode semblent fonctionner correctement.

Y'a-t-il une erreur dans cette méthode?

Voici le code de la classe loggeur si besoin est

CLogger.h

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
#ifndef __CLogger_h__
#define __CLogger_h__
 
#include <iostream>
#include <fstream>
#include <string>
 
namespace SLE
{
	class CLogger
	{
	public:
		enum InfoType
		{
			ITInfo,
			ITWarning,
			ITError
		};
 
		static void SetFileName(const std::string strFileName = "LogFile.txt");
 
		static void LogToFile(InfoType ITType, const std::string &strMessage, const std::string &strEntry);
 
	private:
		CLogger();
		virtual ~CLogger();
 
		//Flux du fichier log
		static std::string m_strFileName;
	};
}
 
#endif

CLogger.cpp

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
#include "CLogger.h"
 
using namespace SLE;
 
std::string CLogger::m_strFileName;
 
CLogger::CLogger()
{}
 
CLogger::~CLogger()
{}
 
void CLogger::SetFileName(const std::string strFileName)
{
	m_strFileName = strFileName;
 
	//Vide le fichier de log existant
	std::ofstream ofsLogFile(m_strFileName.c_str(),std::ios::trunc);
	ofsLogFile.close();
}
 
void CLogger::LogToFile(CLogger::InfoType ITType, const std::string &strMessage, const std::string &strEntry)
{
	std::ofstream ofsLogFile(m_strFileName.c_str(),std::ios::app);
 
	std::string strInfo;
	switch (ITType)
	{
	case ITInfo:
		strInfo = "Info";
		break;
	case ITWarning:
		strInfo = "Warning";
		break;
	case ITError:
		strInfo = "Error";
		break;
	}
	ofsLogFile << strInfo << " -- " << strMessage << " -- " << strEntry << std::endl;
	ofsLogFile.close();
}

01/11/2007, 12h26
Mat007

Salut,

Citation:

Envoyé par trecks

Mon problème c'est que dans mon loggeur il me dit toujours "Impossible de terminer le rendu de scène"

En même temps c'est normal vu que c'est ce que tu lui demandes de faire : throw lève une exception, c'est donc ce qui se produit juste après l'appel à m_pVideoMode->Display()...

MAT.
01/11/2007, 12h38
trecks

comment je dois faire pour qu'il ne m'affiche que lorsque m_pVideoMode->Display() ne fonctionne pas, j'ai essayé en testant cette méthode mais elle est de type void
01/11/2007, 12h45
Mat007

Je ne sais pas moi, est-ce qu'elle peut vraiment échouer cette méthode ?
Elle est supposé faire quoi en cas d'erreur (regarde le code et/ou la documentation qui l'accompagne) ?
Sur quel(s) critère(s) tu te bases pour dire si elle fonctionne ou non ?

MAT.
01/11/2007, 12h47
NiamorH

Si tu met throw ici, il va lever l'exception à chaque fois, après un rendu.

Si c'est toi qui a codé Display(), il faut que tu mettes un throw dans cette méthode, dès que tu détectes une erreur.

Si tu ne l'a pas codé et quelle ne renvoie pas de code d'erreur, elle lève peut-être déjà elle-même une exception, ou il existe peut-être une méthode getLastError(), sinon tu ne peut rien faire de plus.

Mais de toutes façons, ton try catch me semble mal placé dans EndScene(). Généralement j'en met un dans le main et c'est tout. Là, tu risques de continuer à rendre ta scène comme si rien ne s'était passé. Il faudrait au moins que tu relance l'exception dans le catch si tu veux le laisser dans EndScene().