Discussion :

[LeFredBleu]

Bonjour,

J'ai deux méthodes, une servant à écrire un ostream à partir de mon objet, l'autre pour affecter des valeurs à mon objet à partir d'un istream formaté.

Je me demandais comment je pouvais tester si le fichier est bien ouvert dans le cas où le istream/ostream passé en paramètre est un ifstream/ofstream. Les méthodes .good() et .fail() me montrent bien si le fichier existe ou pas, mais je n'arrive pas à savoir si, dans le cas d'un fichier, celui-ci est ouvert ou pas. Bien sur, puisque les test sont fait sur des istream/ostream, je n'ai pas accès à la méthode is_open().

Si quelqu'un pouvais m'aider, je lui en serais très reconnaissant.

[yan]

Bonjour,
as tu essayer le if ?

Code :

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

1
2
3
4
5
void blabla(std::istream &is, std::ostream & os)
{
if (is) std::cout<<"flux in ouvert"<<std::endl;
if (os) std::cout<<"flux out ouvert"<<std::endl;
}

[LeFredBleu]

Je ne peux pas utiliser les if, puisqu'il s'agit d'un test de la précondition dans l'implantation de la théorie du contrat. J'ai essayé PRECONDITION(os) sans succès (où PRECONDITION(os) revient à un assert(os) plus ou moins).

Donc, en gros, cette solution ne fonctionne pas lorsque le fichier existe, mais qu'il est fermé.

[yan]

Je ne peux pas utiliser les if, puisqu'il s'agit d'un test de la précondition dans l'implantation de la théorie du contrat. J'ai essayé PRECONDITION(os) sans succès (où PRECONDITION(os) revient à un assert(os) plus ou moins).

Donc, en gros, cette solution ne fonctionne pas lorsque le fichier existe, mais qu'il est fermé.

???
as tu essayé???
sinon j'ai rien compris as ton explication
T'as peut être raison

[LeFredBleu]

Oui, oui, j'ai essayé. J'ai utilisé la méthode que tu proposais, mais en faisant le test avec une assertion plutôt qu'un if, et ça ne fonctionne pas. Voici un bout du code, si ça peux aider :

Code :

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

1
2
3
4
5
6
7
8
9
10
void CompteCheque::serialise (std::ostream& os) const
{
	PRECONDITION(!os.fail());
	PRECONDITION(os);
 
	Compte::serialise(os);
 
	os << reqTauxInteret() << std::endl << reqNombreTransactions() << std::endl;
	os << calculerInteret() << '\n';
}

La partie PRECONDITION(os) n'as pas marchée, et PRECONDITION n'est qu'une macro faisant une assertion venant de assert.h avec un message spécifique afin de mieux implanter la théorie du contrat (Design by Contract)

[yan]

avec un cast vers un ofstream?

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
#include <iostream>
#include <fstream>
using namespace std;
 
void test(std::ostream & os)
{
    try
        {
        ofstream & osf  = dynamic_cast<ofstream&>(os);
        if (osf.is_open()) std::cout<<"fichier ouvert"<<std::endl;
        else std::cout<<"fichier fermé"<<std::endl;
        }
    catch ( const  bad_cast &)
        {
            std::cout << "ceci n'est pas un fichier"<<std::endl;
        }
 
}
 
 
int main()
{
    ofstream f("essai.txt");
    test(f);
    f.close();
    test(f);
    test(cout);;
 
    return 0;
}

[EDIT]
ce qu'il faut mettre dans le catch

[LeFredBleu]

J'obtiens l'erreur suivante quand j'essaie de faire le dynamic_cast:

Code :

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

error C2680: 'class std::basic_ofstream<char,struct std::char_traits<char> > &' : invalid target type for dynamic_cast

Alors pas moyen de tester cette méthode non plus.

[yan]

pourtant ce code marche sous gcc..
Fait attention au std::
j'en ai pas mis partout à cause du using namespace std;

[LeFredBleu]

J'avais aussi mis un using namespace std. Par contre, c'est un travail pour un cours et on est obligé d'utiliser visual studio 6, avec son compilateur déprécié. L'erreur vient probablement de là ; les librairies qui me sont fournies par le professeur ne compilent même pas avec gcc... et un collègue m'a dit qu'elles ne compilaient pas non plus avec .Net.

Je garde quand même ta méthode avec le dynamic_cast en tête, ça pourra éventuellement être utile dans un cadre de développement plus "sérieux" et c'était très "propre". Quoi que ça laisse un try-catch dans le code qui ne sera pas désactivé en phase release, mais bon, on ne peut pas tout avoir

[yan]

Je garde quand même ta méthode avec le dynamic_cast en tête, ça pourra éventuellement être utile dans un cadre de développement plus "sérieux" et c'était très "propre". Quoi que ça laisse un try-catch dans le code qui ne sera pas désactivé en phase release, mais bon, on ne peut pas tout avoir

Try catch désactivé????
euh la ca m'étonnerai.....

[yan]

as tu activé le rtti dans visual 6???
je sait plus ou est l'option

[LeFredBleu]

Non, non, je me suis mal exprimé. Je voulais seulement dire qu'en ayant à faire un test avec un try-catch qui n'était pas basé seulement sur une macro de théorie du contrat, on ne pourrait PAS le désactiver en mode release :p

[LeFredBleu]

Nope, je ne l'ai pas activé. En fait, je ne sais mêm pas c'est quoi le rtti. Mais de toute façon, il ne faut pas que la compilation de mon code dépende sur les "settings" de VC++, c'est un plan pour que ça ne compile pas pour le correcteur et qu'il saute immédiatement à la pénalité de 50%...

[LeFredBleu]

Enfin, ne te casse pas la tête à trouver comment changer le rtti. Je peux faire une recherche pour savoir ce qu'est le rtti et franchement, je ne peux pas le changer dans VC++ pour le cours et je doutes d'avoir à me resservir de Visual Studio 6.0 en dehors du contexte de mes cours...

[yan]

Enfin, ne te casse pas la tête à trouver comment changer le rtti. Je peux faire une recherche pour savoir ce qu'est le rtti et franchement, je ne peux pas le changer dans VC++ pour le cours et je doutes d'avoir à me resservir de Visual Studio 6.0 en dehors du contexte de mes cours...

pas si sur il existe encore des projet en visual 6...
Le rtti c'est ce qui permet de rajouter des choses internes au class avec visual 6....
ca veut run time et je sais plus
Malheureusement, sans, typeinfo, dynamic_cast ... ne fonctionneront pas.
Bon courage.

[LeFredBleu]

Bien merci.

Finalement, il s'agit plus de problèmes de limitations du cours que de manque de solution. Je n'implanterai probablement cette vérification dans mon travail, mais je retire quand même quelque chose de ce thread

[yan]

J'ai essayer deux second avec visual 6.
Juste pour info. Ce code compile.

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
#include <iostream>
#include <fstream>
using namespace std;
 
void test(std::ostream & os)
{
    try
        {
        ofstream & osf  = dynamic_cast<ofstream&>(os);
        if (osf.is_open()) std::cout<<"fichier ouvert"<<std::endl;
        else std::cout<<"fichier fermé"<<std::endl;
        }
    catch (...)
        {
            std::cout << "ceci n'est pas un fichier"<<std::endl;
        }
 
}
 
 
int main()
{
    ofstream f("essai.txt");
    test(f);
    f.close();
    test(f);
    test(cout);;
 system("pause");
    return 0;
}

Sans le rtti , dynamic_cast<ofstream&> produit toujours une exeption
Avec rtti. ca marche.
Par contre je n'ai pas trouvé quoi mettre dans le catch...

[LeFredBleu]

Et bien, encore merci.

[JolyLoic]

rtti = runtime type information. Désactivé par défaut en VC6.

[koala01]

Salut,

<snip>
Par contre je n'ai pas trouvé quoi mettre dans le catch...

L'exception lancée devrait de toutes manière être une dérivée de std::exception, et donc tu peux très bien la récupérer sous cette forme.

Et, pour être précis et sauf erreur de ma part, l'exception lancée est, tout simplement de type bad_cast

Cela signifie que le catch devrait prendre la forme de

Code :

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

1
2
3
4
catch (std::exception& e)
{
    /*....*/
}

ou de

Code :

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

1
2
3
4
catch(std::bad_cast& e)
{
 
}

[EDIT]la première nécessitant l'inclusion du fichier <exception> et la seconde nécessitant l'inclusion du fichier <typeinfo>