Discussion :

[Bash01]

Bonjour,

J'obtiens le warning :

Code :

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

warning: deprecated conversion from string constant to ‘char*’

En fait, j'utilise les exceptions et j'ai un code de ce type :

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
#ifndef MONEXCEPTION_H
#define MONEXCEPTION_H
 
#include <stdexcept>
 
class MonException : public std::exception{
 
	private :
	char* texte;
 
	public :
	MonException();
	MonException(char* texte);
	const char* what() const throw() {return this->texte;}
};
 
#endif

Du coup, lorsque je fais "throw MonException("Un message"), j'obtiens ce warning, et je ne sais pas comment m'en débarrasser.
Que dois-je faire ? (Je veux que le code soit propre en corrigeant ce warning, pas donner des options au compilateur pour que ce dernier soit ignoré).

Merci.

[Laurent Gomila]

Les chaînes littérales du type "mon message" sont en lecture seule, elles ne sont pas censées être modifiées. Du coup leur type est non pas char*, mais const char*. C'est donc ce type que ta classe doit manipuler.

D'une manière générale il faut toujours bien veiller à utiliser le mot-clé const autant que nécessaire, c'est-à-dire à chaque fois qu'une variable ne va pas être modifiée par le code qui l'utilise.

[Invité]

Vu qu'il n'est pas logique de modifier la chaine de caractère dans ton exception, tu peux rajouter un const :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class MonException : public std::exception{
 
	private :
	const char* texte;
 
	public :
	MonException();
	MonException(const char* texte);
	const char* what() const throw() {return this->texte;}
};

[Bash01]

Merci pour vos réponses.