Discussion :

[vikki]

Bonjour à tous,
J'implémente actuellement un petit wrapper c++ de ffmpeg, en m'inspirant des streams c++, sauf qu'au lieu d'extraire des caractère j'extrais des frames (image rgb ou audio). J'ai implémenté une classe inspirée de std::ios_base (dont hérite ma classe ffmpeg_istream) avec les méthodes eof(),fail(),rdstate(), ect... Ma question est la suivant: puisque que tout ceci est déjà implémenté dans ios_base, serait-il possible d'hériter directement de ios_base ou est-ce une très mauvaise idée (si oui pourquoi)?

[koala01]

Salut,

De manière générale, on ne peut sincèrement pas estimer qu'un wrapper est - sémantiquement parlant - un flux de données.

Un wrapper est une classe qui permet certaines opérations sur des données, dont celles qui consistent à envoyer ou à recevoir ces données vers/ au départ d'un flux

Il ne semble donc pas opportun, ne serait-ce que parce que le principe de substitution de liskov ne sera vraiment pas respecté, d'envisager de faire hériter ton wrapper d'une quelconque classe de gestion de flux

Je me demande même s'il est opportun de donner la responsabilité à ton wrapper de s'occuper du rdstate

En outre, les différents flux n'ont me semble-t-il pas vocation à servir pour l'héritage.

Bref, tu l'auras compris, pour ma part je te déconseille ce point de vue

[Arzar]

serait-il possible d'hériter directement de ios_base ou est-ce une très mauvaise idée (si oui pourquoi)?

C'est une très mauvaise idée.











Ah oui, pourquoi ?
Déjà parce que ios_base déclare tout une pelletée de flag (boolalpha, dec, fixed, hex, internal, left, oct, right, scientifc, left, oct, right, showbase .... etc) et de fonctions (setf(), precision(), width(), unsetf(), imbue(), getloc()) dont tu n'as pas forcément besoin mais surtout parce que:

Responsability for error handling
One might except that the error handling would be contained in ios_base because error indication is charactere-independant. However, the functionality for access to the stream state and the exception mask is located in class basic_ios<class charT, class Traits>. Class ios_base contains only the definition of all stream state flags, such as basbit, eofbit, etc. Access to the stream state and the exception mask is via member function of class basic_ios<class charT, class Traits>, by invoking functions like exception(), good(), bad(), setstate(). [...]

Code :

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

1
2
3
4
5
6
7
void foo(ios_base& str)
{
   int idx = str.xalloc(); // might set badbit in case of memory shortage
 
   if( str.fail() )  // does not work; no access to the stream state here.
     ...
}

[vikki]

Désolé pour la réponse tardive, pas mal de boulot en ce moment. Bon, effectivement j'ai pas besoin de tous les flags et fonctions definies dans ios_base, donc on va oublier. Je vais me contenter de m'inspirer de l'architecture des flux de la stl sans les utiliser. Merci pour vos réponses claires