Discussion :

[BaygonV]

Bonjour, j'ai un probleme de manipulation de stream.
Voilà une partie du code :

Pour le header (Driver.hh) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Driver{
//Plein de trucs....
// dont 
void setFileIn(const std::string& filein); // Fait ce qu'elle doit faire pour le fichier de données.
//.....
public :
 
std::string fileoutname;
std::ostream& outstream = std::cout;
 
Driver(const std::string& filein="",const std::string& fileout="");
 
};

Pour l'implementation (Driver.cc)

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
 
#include"Driver.hh"
//plein de trucs...
//......
 
Driver::Driver(const std::string& filein/*=""*/, const std::string& fileout/*=""*/)
{
  if(!filein.empty())
    {setFileIn(filein);}
  if(!fileout.empty())
    {
      std::ofstream tmp (fileout.c_str());
      tmp.swap(outstream);
    }
}

Je compile bien avec le standard C++11 justement pour swap :

Code :

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

1
2
 
g++ -c -std=c++11 Driver.cc

Et la première (mais pas la dernière !!) erreur du compilateur me dit :

Code :

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

1
2
3
4
5
 
In constructor ‘Driver::Driver(const string&, const string&)’:
Driver.cc:64:11: error: ‘std::ofstream’ has no member named ‘swap’
       tmp.swap(outstream);
           ^

Pourtant ici on me dit le contraire...

Une idée ? (sur ce problème... ou sur une autre facon de faire, je suis preneur !)

[Flob90]

Bonjour,

Quel version de GCC ? Il est probable que cette fonctionnalité n'ai pas été implémentée dans ta version, je ne sais plus où en est gcc, mais je crois que si niveau langage il est presque à 100% du C++11, niveau bibliothèque ce n'est pas le cas.

[BaygonV]

j ai la 4.8.1...
bon, j vais me reposer et j réfléchirai a ça plus tard.
Merci en tout cas, je pensais que les compilateurs étaient prêts dès la 'sortie' des standards...
Apparemment je n'ai pas bien compris comment les choses évoluent

[jo_link_noir]

Qu'à partir de gcc-5.0.

2 remarques:
- fstream peut prendre un std::string, plus besoin de c_str().
- swap s'applique sur un type équivalent. Ton code ne compilera pas car outstream est un ostream et tmp un ofstream.

Tu peux regarder vers std::ostream::rdbuf, cela suffit peut-être ?

[BaygonV]

En fait j imagine que vous comprenez que je veux que la sortie soit au choix la sortie standard ou un fichier sans pour autant changer la variable qui represente cette sortie.
J ai lu au sujet de rdbuf et ca a l air de pouvoir faire l affaire.
si vous voyez un autre moyen de le faire ou si vous trouvez ma maniere de faire ridicule dites le moi svp.

[ternel]

Changer le buffer peut se faire.
Il y a une autre possibilité, c'est d'utiliser un stream autre (genre BaygonV::out), que tu déclares comme tu en as besoin.

Dernière possibilité, faire une redirection du flux de sortie standard (vers le fichier) non pas dans le programme mais à l'appel.

./programme >fichier

Ainsi, le programme ne se pose pas la question, et si l'utilisateur veut intercaler un grep, un tee ou que sais-je encore, il peut le faire.

[BaygonV]

Merci pour vos réponses et je suis intéressé (depuis bien avant l'apparition de ce fil ici) par avoir mon propre stream à la manière (si j'ai bien compris) de ce que propose leternel ici :

Il y a une autre possibilité, c'est d'utiliser un stream autre (genre BaygonV::out), que tu déclares comme tu en as besoin.

Et en effet j'ai essayé ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Driver::Driver(const std::string& filein/*=""*/, const std::string& fileout/*=""*/)
{
  if(!filein.empty())
    {setFileIn(filein);}
  if(!fileout.empty())
    {
      outstream = *(new std::ofstream(fileout.c_str()));
 
    }
  else{outstream = std::cout; }
}

mais dans ce cas j'ai droit à un :

Code :

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

1
2
3
 
error: use of deleted function ‘std::basic_ostream<char>& std::basic_ostream<char>::operator=(const std::basic_ostream<char>&)’
       outstream = *(new std::ofstream(fileout.c_str()));

C'est l'impossibilité d'utiliser cet opérateur = pour affecter mon ostream qui m'a fait chercher un moyen de contournement...
J'ai pas du comprendre un (plusieurs ?) truc(s ?).

EDIT : Je pense me rendre compte que j'ai effectivement oublié comment fonctionnent les flux... je vais donc me faire des petits exemples. Cependant je suis preneur de toute remarque (pas trop méchante ).

[Bousk]

ios::rdbuff ?

[ternel]

Ou l'autre exemple, qu'on trouve sur cppreference: basic_ios::rdbuf
Il est un peu plus C++11-esque. (et je préfère le style de la description de fonction, mais c'est mon goût)

[BaygonV]

Merci j ai lu ces deux exemples déjà mais j aime moyen le fait que toute information envoyee sur cout finisse dans un fichier par inadvertance(meme si c est l inadvertance qu il faudrait chasser). Bref j ai fait autre chose :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Driver {
 std::ostream* poutstream;
 bool ostreamdelete;
Driver(const std::string& i="", const std::string& o="");
virtual~Driver(){
if(ostreamdelete) delete poutstream;}
};
 
Driver::Driver(const std::string& i/*=""*/, const std::string& o/*=""*/){
if(!i.empty()){yyin=setInFile(i);} //setInFile s arrange avec l existance de i.
if(o.empty()){poutstream=&cout; ostreamdelete=false;}
else{poutstream=new std::ofstream(o);ostreamdelete=true;}
}

J en suis pas tres fier mais je l ai fait.

[ternel]

au moins, code le bien.
Je n'arrive même pas à lire ton code.

La même chose mieux présentée

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
class Driver {
private:
    std::ostream* out;
    bool out_needs_delete;
public:
    Driver(const std::string& i="", const std::string& o="");
    virtual ~Driver(){
        if (out_needs_delete) delete out;
    }
};
 
//la création, c'est délicat, il faut le faire dans la création
Driver::Driver(const std::string& i, const std::string& o) :
    yyin ( (!i.empty())? setInFile(i) : 0),
    out(o.empty() ? &cout : new std::ofstream(o)),//je me permets un new ici, car c'est le seul.
    out_needs_delete(!o.empty())
{}

Et c'est quand même maladroit.

[BaygonV]

Désolé pour la présentation, j aurais du patienter et ne pas écrire le code sur mon téléphone...

Peux tu me dire en quoi c est maladroit ?
Je n'ai pour ma part que l'impression de maladresse sans arriver a me donner des arguments...

Tu ne pouvais pas savoir (et j'aurais mieux fait de ne pas le mettre...) mais le yyin n'est pas un membre de la classe Driver, c'est une variable globale qui est définie par un scanner généré par flex.

[Bousk]

Merci j ai lu ces deux exemples déjà mais j aime moyen le fait que toute information envoyee sur cout finisse dans un fichier par inadvertance(meme si c est l inadvertance qu il faudrait chasser)

Faut savoir : tu veux rediriger un flux ou non ?
Si tu veux rediriger un flux, tu le rediriges entièrement. Je vois pas où est l'inadvertance là-dedans ?

[BaygonV]

Faut savoir : tu veux rediriger un flux ou non ?

Alors alors.... la réponse à cette question est en fait : j'en sais rien car je me suis construit une définition de flux qui n'est peut-être pas la bonne...

Pour moi un flux c'est un moyen d'écrire ou lire des informations quelque part sans avoir à "mettre les mains dans le cambouis" et laisser la responsabilité de la gestion de la lecture/écriture et autres buffers à des gens plus doués que moi.
Si cette définition n'est pas trop loin de la réalité alors oui je veux rediriger un flux.

Sinon, je rentre un peu plus dans les détails de ce qui m'a amené à vouloir faire ça :

Je veux écrire un scanner (généré par flex) qui doit envoyer des informations vers divers endroits (fichiers ou autres processus).
Les divers destinataires sont liés a des paramètres qui lui sont parfois inconnus et qui sont maitrisés par ma classe Driver.
De plus, lorsqu'il s'agit d'informations à distribuer dans des fichiers, il s'agit ni plus ni moins que de recopier le caractère lu par le scanner.

Pour arriver à faire ça, j'ai donc :

• Redéfini la macro (flex) YY_DECL en :
  

  Code :

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

  #define YY_DECL int yylex(Driver& driver)

• Et redéfini la macro (flex encore) ECHO en
  

  Code :

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

  1
  2
  3
  4
  #if defined ECHO
  #undef ECHO
  #endif
  #define ECHO { (*driver.poutstream)<<yytext;}

L'important pour moi est que le scanner n'ait pas à ce soucier réellement de l'endroit où il doit faire ses ECHO.
Ce sont donc les informations recueillies par ce flux pointé par driver.poutstream que je dois rediriger ensuite, selon le contexte, dans (un, plusieurs ou aucun) fichier(s) ou sur la sortie standard.

Si tu veux rediriger un flux, tu le rediriges entièrement. Je vois pas où est l'inadvertance là-dedans ?

Si j'utilise des rdbuf et que je crois écrire à droite alors que j'écris effectivement à gauche, ça me pose problème...
mais encore une fois c'est peut être parce que je n'ai pas correctement compris ce qu'est un flux.

[Bousk]

Ce sont donc les informations recueillies par ce flux pointé par driver.poutstream que je dois rediriger ensuite, selon le contexte, dans (un, plusieurs ou aucun) fichier(s) ou sur la sortie standard.

Dans ce cas c'est pas du tout de la redirection, c'est un simple dispatch.
Tu crées une classe qui gère les différentes sorties et fait elle-même le dispatch dans l'une ou l'autre.

[ternel]

Ca ressemble à du log, et tu peux peut-être t'inspirer de la structure de bibliothèque comme log4cpp.

[BaygonV]

Ok je vais voir ça mais du coup.... bon je vais surement poser une question bête dans un nouveau fil après m'être renseigné un peu.
Par contre ci vous pouviez me dire en quoi le dernier code que j'ai proposé est maladroit je pense que ca me ferait pas mal progresser.
Personnellement dans ce code, je n aime pas du tout l'idée que je ne sache pas a coup sur si mon poustream est un truc à détruire ou pas et la présence du bool a tester pour le savoir me semble être la preuve de la maladresse. Y a t il un moyen de l’éviter ? Ou est ce que "c'est la vie" finalement ?