How do I write exception safty code?

[dragon9]

I want to write some small programs with exception safety, but I don't how
to write. Because my programs wrap C-api function( example: win32 api). I
don't find my direction. I wish any hot heart man to help me. My english is
too bad. I am sorry! Thanks!

#include "windows.h"
#include <string>
//
#ifndef FILE1_H
#define FILE1_H
namespace wukexin{
//
std::string Fileattrib_to_s tr( const DWORD attrib );
//
std::string Filetime_to_str ( const FILETIME* x );
//
class Cfile_informati on
{
public:
explicit Cfile_informati on(std::string fname){initiali zer(fname);};
//~Cfile_informat ion(){destroyer ();};
void print();
protected:
//friend ostream& operator<< (std::ostream& os, const Cfile_informati on&
file);
private:
void initializer(std ::string fname);
void destroyer();
bool destroySuccess_ ;
HANDLE hfile_;
std::string fname_;
WIN32_FIND_DATA file_;
};
}
#endif //file1.h
////////////////
file.cpp
//**
void Cfile_informati on::initializer (std::string fname){
WIN32_FIND_DATA file;
//std::cout<<fnam e<<"construct success!\n";
hfile_=FindFirs tFile(fname.c_s tr(),&file_);
if(INVALID_HAND LE_VALUE == hfile_){
//std::cout<<"Fin dFirstFile(fnam e,&file) run error!\n"<<std: :endl;
throw <<"FindFirstFil e(fname,&file) run error!\n";
}
destroySuccess_ =false;
}
//
Cfile_informati on::~Cfile_info rmation(){
if( !destroySuccess _ ){
try{
destroyer();
}
catch(){
// codes
}
};
//**

[asterisc]

On Sep 12, 3:34 pm, "dragon9" <maigre_dra...@ 126.comwrote:

I want to write some small programs with exception safety, but I don't how
to write. Because my programs wrap C-api function( example: win32 api). I
don't find my direction. I wish any hot heart man to help me. My english is
too bad. I am sorry! Thanks!

#include "windows.h"
#include <string>
//
#ifndef FILE1_H
#define FILE1_H
namespace wukexin{
//
std::string Fileattrib_to_s tr( const DWORD attrib );
//
std::string Filetime_to_str ( const FILETIME* x );
//
class Cfile_informati on
{
public:
explicit Cfile_informati on(std::string fname){initiali zer(fname);};
//~Cfile_informat ion(){destroyer ();};
void print();
protected:
//friend ostream& operator<< (std::ostream& os, const Cfile_informati on&
file);
private:
void initializer(std ::string fname);
void destroyer();
bool destroySuccess_ ;
HANDLE hfile_;
std::string fname_;
WIN32_FIND_DATA file_;
};}

#endif //file1.h
////////////////
file.cpp
//**
void Cfile_informati on::initializer (std::string fname){
WIN32_FIND_DATA file;
//std::cout<<fnam e<<"construct success!\n";
hfile_=FindFirs tFile(fname.c_s tr(),&file_);
if(INVALID_HAND LE_VALUE == hfile_){
//std::cout<<"Fin dFirstFile(fnam e,&file) run error!\n"<<std: :endl;
throw <<"FindFirstFil e(fname,&file) run error!\n";
}
destroySuccess_ =false;
}
//
Cfile_informati on::~Cfile_info rmation(){
if( !destroySuccess _ ){
try{
destroyer();
}
catch(){
// codes
}
};
//**

Firstly, it's not a good practice to send non-pointer non-reference
types by const, because a copy it will be used anyway.
Another idea is to return an error code from the function and pass
through references your data, for example:

std::string Fileattrib_to_s tr( const DWORD attrib );

bool Fileattrib_to_s tr( DWORD attrib, std::string& retVal );

Exceptions should be used in exceptional cases only, everything else
should be treated as errors.
Exception mechanism is very slow as well.

I think you should refactor your code.

Use it carefully!

[AnonMail2005]

On Sep 12, 8:34 am, "dragon9" <maigre_dra...@ 126.comwrote:

I want to write some small programs with exception safety, but I don't how
to write. Because my programs wrap C-api function( example: win32 api). I
don't find my direction. I wish any hot heart man to help me. My english is
too bad. I am sorry! Thanks!

#include "windows.h"
#include <string>
//
#ifndef FILE1_H
#define FILE1_H
namespace wukexin{
//
std::string Fileattrib_to_s tr( const DWORD attrib );
//
std::string Filetime_to_str ( const FILETIME* x );
//
class Cfile_informati on
{
public:
explicit Cfile_informati on(std::string fname){initiali zer(fname);};
//~Cfile_informat ion(){destroyer ();};
void print();
protected:
//friend ostream& operator<< (std::ostream& os, const Cfile_informati on&
file);
private:
void initializer(std ::string fname);
void destroyer();
bool destroySuccess_ ;
HANDLE hfile_;
std::string fname_;
WIN32_FIND_DATA file_;
};}

#endif //file1.h
////////////////
file.cpp
//**
void Cfile_informati on::initializer (std::string fname){
WIN32_FIND_DATA file;
//std::cout<<fnam e<<"construct success!\n";
hfile_=FindFirs tFile(fname.c_s tr(),&file_);
if(INVALID_HAND LE_VALUE == hfile_){
//std::cout<<"Fin dFirstFile(fnam e,&file) run error!\n"<<std: :endl;
throw <<"FindFirstFil e(fname,&file) run error!\n";
}
destroySuccess_ =false;
}
//
Cfile_informati on::~Cfile_info rmation(){
if( !destroySuccess _ ){
try{
destroyer();
}
catch(){
// codes
}
};
//**

Read Sutter's Exceptional C++. In it, there's a series of examples
that
systematically walks the reader through the implementation of a stack
container and teaches the fundamentals of exception safety along the
way.

A must read for any serious C++ programmer IMHO.

[dragon9]

Thanks very much, my question is how to wrap C-api functions with exception
safety.
example:
FindFirstFile() ,etc.

[BobR]

dragon9 <ma***********@ 126.comwrote in message...

Thanks very much, my question is how to wrap C-api functions with

exception

safety.
example:
FindFirstFile() ,etc.

FindFirstFile() is windows, so, your best bet on that is to try a windows
NG. This group is for C++ (standard) language.
#include <iostream>
#include <exception>
#include <stdexcept>

class MyError : public std::runtime_er ror { public:
MyError( std::string const &msg = "") : std::runtime_er ror(msg){}
};

void SomeFunc(){
// you find an error that you can not recover from here, so you do:
throw MyError( "Rats, I really hate when that happens!!");
return; // it won't get here
}

int main() try {

// - your code here -

SomeFunc(); // for test

std::cout<< "main() finished"<<std: :endl;
return 0;
} // main()

catch( MyError const &Me ){
std::cout<< "main() caught MyError: "
<<Me.what()<<st d::endl;
return 1;
}
catch( std::exception const &Se) {
std::cout<< "main() caught exception: "
<<Se.what()<<st d::endl;
return 1;
}
catch( ... ) {
std::cout<<"cau ght unknown"<<std:: endl;
return 1;
}
See "Thinking in C++" vol.2
"Part 1: Building Stable Systems, 1: Exception Handling.

Get "Thinking in C++", 2nd ed. Volume 1&2 by Bruce Eckel
(available for free here. You can buy it in hardcopy too.):
http://www.mindview.net/Books/TICPP/...ngInCPP2e.html
FAQ http://www.parashift.com/c++-faq-lite

--
Bob R
POVrookie

[James Kanze]

On Sep 13, 1:29 am, "dragon9" <maigre_dra...@ 126.comwrote:

Thanks very much, my question is how to wrap C-api functions with exception
safety.
example:
FindFirstFile() ,etc.

If it's a function from a C API, you don't need to do anything
with regards to exception safety, since the function will never
throw. Otherwise, it couldn't be used from C. In particular,
both the Windows API and Posix are C, and none of the functions
in these API's ever raise an exception. (I think. I'm 100%
sure about Posix, but I think Windows is the same.)

If you want the function to throw for certain types of errors,
you'll have to wrap it, check the return status, and manually
throw the exception you want for the types of errors you want.
Regardless of whether you want to throw or not, I would strongly
recommend wrapping all of the system API in some sort of
platform neutral C++ interface; I use something roughly similar
to java.io.File, for example, with different implementations for
Windows and for Posix/Linux. And some of the functions in this
class do throw, for certain types of errors.

--
James Kanze (GABI Software) email:ja******* **@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientier ter Datenverarbeitu ng
9 place Sémard, 78210 St.-Cyr-l'École, France, [phone]

[dragon9]

Thank everyone. Now I haven't any good done, but I simple use
std::runtime_er ror to wrap it.

--
James Kanze (GABI Software) email:ja******* **@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientier ter Datenverarbeitu ng
9 place Sémard, 78210 St.-Cyr-l'École, France, [phone]

[=?ISO-8859-1?Q?Erik_Wikstr=F6m?=]

On 2007-09-14 15:10, dragon9 wrote:

Thank everyone. Now I haven't any good done, but I simple use
std::runtime_er ror to wrap it.

--
James Kanze (GABI Software) email:ja******* **@gmail.com
Conseils en informatique orientée objet/
Beratung in objektorientier ter Datenverarbeitu ng
9 place Sémard, 78210 St.-Cyr-l'École, France, [phone]

Please, do not use other peoples signatures, it really confuses people.

--
Erik Wikström

[dragon9]

Sorry, I knows.