Discussion :

[victor_gasgas]

Bonjour!

Pour expliquer mon problème je trouve qu'un code sera très parlant :
Je met à disposition des utilisateurs l'enum suivante :

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
 enum class PixelFormat
{
    R8U, ///< R 8 bits 8 unsigned
    RG8U, ///< RG 16 bits 88 unsigned
    RGB8U, ///< RGB 24 bits 888 unsigned
    RGBA8U, ///< RGBA 32 bits 8888 unsigned
};
 
 
unsigned int getBytesPerPixel(PixelFormat Format)
{
    switch(Format)
    {
        case PixelFormat::R8U: return 1;
        case PixelFormat::RG8U: return 2;
        case PixelFormat::RGB8U: return 3;
        case PixelFormat::RGBA8U: return 4;
    }
}

Seulement là j'ai besoin de rajouter soit de nouveaux format, soit de nouvelles propriétés (connaître le type du PixelFormat, savoir si on à affaire à un format compressé ou non, etc).
Et là je me rend compte que je vais vite avoir un code in-maintenable

Seulement je ne sais pas du tout par quoi remplacer. J'ai pensé à une classe ou l'utilisation de template mais rien ne me plaît vraiment..

Si vous avez une solution à me donner, une piste à suivre ou un design pattern je serais bien content!

Merci beaucoup à tous!

[oodini]

Code :

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

1
2
3
4
5
6
7
enum class PixelFormat
{
    R8U = 1, ///< R 8 bits 8 unsigned
    RG8U = 2, ///< RG 16 bits 88 unsigned
    RGB8U = 3, ///< RGB 24 bits 888 unsigned
    RGBA8U = 4, ///< RGBA 32 bits 8888 unsigned
};

[victor_gasgas]

Merci de ta réponse mais cela n'est valable que si la seule propriété que je veux connaître est le nombre de bits par format de pixel.

Seulement maintenant je cherche à étendre mes formats et je souhaiterais éviter les switch à répétitions pour avoir un code propre et surtout porté sur "l'avenir", comprendre par là qu'il soit facilement maintenable et extensible.

Merci beaucoup pour ta réponse néanmoins!

[oodini]

Ben tant qu'on ne sait pas comment tu veux les utiliser...

Le seul cas d'utilisation que tu as fourni était un switch.

[victor_gasgas]

J'ai une classe qui représente une Image et j'ai le code suivant (en raccourcis) :

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
class Image
{
    public :
        Image::Image(const uvec2& Size, PixelFormat Format, const unsigned char* Pixels);
 
    private :
        uvec2  m_Size;   ///< Dimensions de l'image
        PixelFormat m_Format; ///< Format de pixels
        std::vector<unsigned char> m_Pixels; ///< Tableau des pixels
};
 
 
// Dans le Image.cpp
// Constructeur
Image::Image(const uvec2& Size, PixelFormat Format, const unsigned char* Pixels) :
m_Size  (Size),
m_Format(Format),
m_Pixels(Pixels, Pixels + Size.x * Size.y * getBytesPerPixel(Format))
{
}

Imaginons maintenant que je crée une fonction qui à partir d'une image renvoi l'image ayant subi une symétrie verticale (ou tout autre transfo peu importe), j'aimerais ne l'appliquer que si le PixelFormat ne représente pas des donnés compressées :

Code :

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

1
2
3
4
5
6
7
void Image::flip()
{
    if( isCompressedFormat(m_Format) )
        throw std::runtime_error("Le PixelFormat ne doit pas être compréssé")
 
    // Le traitement
}

Et bien là déjà je me retrouve avec la fonction bool isCompressedFormat(PixelFormat), et donc un switch en plus à maintenir. Et si j'ajoute d'autres fonctions du genre ou d'autres PixelFormat je me retrouve avec une bouillie in-maintenable et difficilement extensible...

[gbdivers]

Bonjour

Classes de traits pour le format et politiques pour les fonctions + spécialisation template pour l'évolutivité. Lire Présentation des classes de Traits et de Politiques en C++