Valeur de retour de la surcharge operator[]

**reselsif** · 19/05/2013, 12h50

Bonjour !

Je rencontre un probl�me sur un code particulier, et le code minimal n'est pas plus aidant.
Il s'agit d'une classe (ici nomm�e objet) ayant pour seul attribut une std::string.
L'objectif �tant de pouvoir avoir acc�s � un caract�re de cette string via l'objet et son op�rateur [] : objet.string[i] <=> objet[i]

Seulement, l'utilisation de l'op�rateur [] pose probl�me dans le cadre des fonctions externes � l'objet.
En effet, si monObjet[i] marche tr�s bien dans le main, cela n'est pas valable pour la fonction objet2str, qui tente d'employer l'op�rateur [] et qui n'y arrive pas.

compilation :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
 
g++ -Wall -Werror  -c *.cpp
objet.cpp: In function ‘std::string objet2str(const objet*)’:
objet.cpp:41:33: error: no match for ‘operator+’ in ‘*(((const objet*)obj) + ((sizetype)(((long unsigned int)i) * 8ul))) + '0'’

La totalit� des fichiers minimaux sont ici :
main.cpp :

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
 
#include <iostream>
#include <cstdlib>
#include <fstream>
#include <sstream>
#include <string>
// LOCAL MODULES
#include "objet.h"
 
 
int main(int argc, char* argv[]) {
        objet obj("101010");
        std::cout << obj[0] << std::endl;
 
        return EXIT_SUCCESS;
}

objet.cpp :

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
31
 
#include "objet.h"
 
/***************************************************
 * CONSTRUCTOR
 ***************************************************/
objet::objet(const std::string sequence) {
        value = sequence;
}
 
/***************************************************
 * OPERATORS
 ***************************************************/
char objet::operator[](const unsigned int id) const {
        return value[id];
}
 
/***************************************************
 * OTHER FUNCTIONS 
 ***************************************************/
/*
 * CONVERSIONS
 */
std::string objet2str(const objet* const obj) {
        std::string str = "";
        for(unsigned int i = 0; i < obj->len(); i++) {
                str += (obj[i])+'0'; // ERREUR
                // Il semblerait que obj[i] renvois un objet, non un char !
        }
        return str;
}

objet.h:

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
31
32
#ifndef objet_H_INCLUDED
#define objet_H_INCLUDED
/*
 * LIBRARIES
 */
// STD LIB
#include <iostream>
#include <cstdlib>
#include <string>
 
/*
 * objet CLASS
 */
class objet {
	public:
	// CONSTRUCTOR
		objet(const std::string);
	// ACCESSORS
                unsigned int len() const { return value.size(); }
        // OPERATORS
                char operator[](const unsigned int) const;
	private:
	// ATTRIBUTES
                std::string value; 
};
 
/*
 * OTHER FUNCTIONS
 */
std::string objet2str(const objet*const);
 
#endif

Lorsque la ligne probl�matique est retir�e, le main fonctionne pour ainsi dire parfaitement. (obj[0] renvois bien 1)

En vous remerciant de votre aide.

PS: ce code est un code minimal. Les algorithmes mis en oeuvre sont donc souvent illogiques tels quels.

**JolyLoic** · 19/05/2013, 13h11

Dans ta fonction, obj est un pointeur sur object. Donc, quand tu �cris obj[3], il pense que ce pointeur pointe sur le premier �l�ment d'un tableau d'objets, et que tu demandes � ce qu'il retourne le troisi�me �l�ment de ce tableau.

donc deux solutions : Soit tu �cris (*obj)[i], soit tu modifie ta fonction pour qu'elle prenne en param�tre une r�f�rence � un obj, et non un pointeur (c'est probablement pr�f�rable).

**reselsif** · 19/05/2013, 13h32

Merci beaucoup JolyLoic !

Une erreur que j'aurais d� savoir relever moi-m�me...

Surcharger l'op�rateur [] pour qu'il prennent en charge les pointeurs d'objet est aussi une solution viable, non ?