Discussion :

[-Gesicht-]

Bonjour,

J'écris un programme dans lequel j'ai une classe image template qui contient un pointeur vers un vecteur de T. Je voudrais surcharger l’opérateur [] mais je n'arrive pas à compiler.


classe Image

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
 
template<typename T>
class Image
{
public:
    Image(vector<T> &data, size_t width, size_t height, size_t depth);
    ~Image();
 
    vector<size_t> getSize() const {return m_size;}
 
     T& operator[](size_t const index) {return this->m_data[index];}
     T  operator[](size_t const index) const {return this->m_data[index];}
 
private:
    vector<T> *m_data;
    vector<size_t> m_size;

Je fais appel à l'opérateur [] dans une autre classe lorsque je veux lire/modifier la valeur:

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
 
template<typename T>
bool ConvFilter<T>::applyTo(Image::Image<T>& dest, const Image::Image<T> &src)
{
    if (dest.getSize() != src.getSize())
    {
        cerr << "input and output Images have not the same size" << endl;
        return false;
    }
 
    vector<size_t> size = src.getSize();
 
    for (size_t k=0,index=0; k<size[2]; ++k)
        for ( size_t j=0; j<size[1]; ++j )
            for ( size_t i=0; i<size[0]; ++i, ++index)
                cout << src[index] << endl;

Lors de l'appel à l'opérateur ligne16, ça va chercher l'opérateur ligne 12 de Image et ça me met:

Y:\programs\normalisationPIXEbySTIM\src\Image\Image.hpp:23(=ligne 12): erreur : cannot convert 'std::vector<float>' to 'float' in return

Pourtant il me semble que tel que je l'ai écris ma surcharge renvoie un Tet pas un vecto<T> non?

[Astraya]

Bonjour,

Code :

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

this->m_data[index];

Retourne un vector<T> et non pas un float car ton m_data est un pointeur, or si tu y accèdes de cette façon cela veux dire que tu accèdes à l'élément "index" de ton pointeur.
cf: arithmétique des pointeurs.
donc la réponse est :

Code :

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

this->(*m_data)[index];

Mais tu devrait t'assurer de pas faire un out of bound...

Cordialement,

[-Gesicht-]

Merci pour ta réponse,

je comprends mieux maintenant.

Mais tu devrait t'assurer de pas faire un out of bound...

Je préfère garder le comportement de l'operateur[] pour être un peu plus rapide. Il faudra juste faire attention

[Bousk]

Salut,

pourquoi faire un retour par copie dans le cas de la version const ?
Les const reférences sont là pour ça.

[Médinoc]

Code C++ :

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

1
2
3
Image(vector<T> &data, size_t width, size_t height, size_t depth);
...
vector<T> *m_data;

C'est vachement suspect, ça. J'espère que ce n'est pas ce dont ça a l'air...

[-Gesicht-]

Je vais sans doute poser une question bête mais :

C'est vachement suspect, ça. J'espère que ce n'est pas ce dont ça a l'air...

ça a l'air de quoi?

dans mon constructeur je fais :

Code :

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

1
2
 
m_data = &data;

[ternel]

C'est bien là le problème.
La variable donnée en référence peut ne plus exister et donc ton pointeur pointerait sur n'importe quoi…

[-Gesicht-]

C'est bien là le problème.
La variable donnée en référence peut ne plus exister et donc ton pointeur pointerait sur n'importe quoi…

Ha ok. Mais je m'assure que les données "ratachées" à data existe le temps dont j'en ai besoin. En fait je m'étais dit que comme les images que je traite sont susceptible d'être grosse, je ne voulais pas les passer en copie.

Par contre si il y un meilleur moyen je suis preneur :-)

[Médinoc]

C'est extrêmement sale, car une référence et un pointeur ont des sémantiques différentes pour l'utilisateur (connotations différentes au sujet de la durée de vie, etc.). Si ta classe mémorise un pointeur vers les data, son constructeur doit prendre directement en paramètre un pointeur vers les data.

L'alternative, c'est de mémoriser directement une référence dans la classe:

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
Image(vector<T> &data, size_t width, size_t height, size_t depth);
...
vector<T> &m_data;
 
...
 
Image::Image(vector<T> &data, size_t width, size_t height, size_t depth)
 : m_data(data)
{
	...
}

La décision est à prendre selon la durée de vie que tu attends de la classe Image, compte tenu des traditions de la programmation qui assurent une bonne compréhension du code.

• Si ton Image a vocation à être temporaire, allouée sur la pile, je conseille la référence de bout en bout.
• Si ton image a vocation à vivre plus longtemps que la fonction qui la crée, allouée sur le tas, je te conseille le pointeur de bout en bout.

[koala01]

Salut,

De plus, la durée de vie des données d'une image est, typiquement, celle de l'image elle-même :

Même si l'on peut envisager de les charger (d'une manière ou d'une autre) avant de créer l'image, elles seront, a priori, créées et détruites en même temps que l'image.

Chaque image devenant responsable de la durée de vie de ses propres données, cela facilite énormément le reste car il "suffit" de décider qu'une image n'est plus nécessaire pour que "tout ce qui s'y rattache" soit supprimé correctement, sans risquer qu'une image ne dispose plus des données dont elle a besoin ni que, pour une raison ou une autre, nous nous retrouvions à maintenir "quelque part" des données qui ne sont plus utilisées par aucune image