Comment faire ma propre classe d'erreur d'exécution personnalisée?

Question

J'essaie de faire un runtime_error personnalisé simple. Je définis la classe:

#include <string> 
#include <stdexcept> 


namespace utils{ 

class FileNotFoundException: public std::runtime_error{ 
    public: 
    FileNotFoundException():runtime_error("File not found"){} 
    FileNotFoundException(std::string msg):runtime_error(msg.c_str()){} 
}; 

}; 

Je jette l'erreur:

bool checkFileExistence(std::string fileName) 
{ 
    boost::filesystem::path full_path = boost::filesystem::system_complete(boost::filesystem::path(fileName)); 
    if (!boost::filesystem::exists(full_path)) 
    { 
    char msg[500]; 
    _snprintf(msg,500,"File %s doesn't exist",fileName.c_str()); 
    throw new FileNotFoundException(msg); 
    } 
} 

Et j'utiliser un bloc try/catch

try{ 
      checkFileExistence(fileName); 
    } 
    catch(utils::FileNotFoundException& fnfe) 
     { 
      std::cout << fnfe.what() << std::endl; 
    } 

Erreur d'exécution est lancée correctement comme FileNotFoundException mais la ligne avec std :: cout n'est jamais atteint et aucune ligne n'est écrite sur la console.

Toutes les idées sont les bienvenues. Merci!

Answer 1

C'est parce que vous lancez un pointeur. Faites simplement: throw FileNotFoundException(msg);.

Chaque fois que vous utilisez un pointeur, à moins de le placer dans un conteneur/wrapper, vous ne faites probablement pas la bonne chose.

Answer 2

Vous écrivez throw new FileNotFoundException(msg), il devrait être 'throw FileNotFoundException (msg)'. La règle est lancer par valeur, attraper par référence.

Answer 3

Vous lancez actuellement un pointeur sur un objet affecté par le tas (FileNotFoundException *) afin que les types ne correspondent pas. Généralement, lancer par valeur et attraper par référence (rule 73). BTW, avec la déclaration suivante, vous créez une copie de la chaîne msg.

Answer 4

FileNotFoundException(std::string msg):runtime_error(msg.c_str()){} 

Write "const std :: string & msg" au lieu. Il mettra juste une référence sur une pile. Alors que maintenant vous mettez toute la chaîne sur une pile.