Pb avec le conteneur set

Version imprimable

bonjours,

voilà très rapidemant mon code!!!!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void Section :: set_payload(set<TS> entree){ int pos=0; //initaialiser la position à partir delaquelle on va copier les informations //dans payload payload=new u_char; for(int i=0; i<entree.size(); i++) { memcpy(payload+pos,(entree[i].getpayload()),(sizeof(entree[i].getpayload()))); pos+=sizeof(entree[i].getpayload()); } return payload; }
j'ai crée la classe TS que j'ai testé le bon fctnnement de ses methodes.
ma classe section est composé de TS pour ainsi créer son vrai payload à partir du payload de TS.(payload de TS c du unsigned char)

Néanmoins, en complilant voilà le msg qui vient me perturber :evilred:

section.cpp:114: error: no match for ‘operator[]’ in ‘entree[i]’

01/04/2009, 13h18
poukill

Bah c'est normal, l'opérateur [] n'existe pas pour std::set... :?

iterator sur set de classe et possibilite d'accès aux methose de cette classe

j'ai trouvé un problème pour accéder au contenu du conteneur "set" car l'pérateur [ ] n'existe pas.
en pensant aux iterateurs:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 void Section :: set_payload(set <TS> entree){ set<TS> :: iterator it; int pos=0; //initaialiser la position à partir delaquelle on va copier les informations dans payload payload=new u_char; for(it=entree.begin();it != entree.end(); it++) { memcpy(payload+pos,it->getpayload(),184); pos+=184; } }
avec " getpayload() " est une methode :Dpublic:mrgreen: de la classe TS qui retourne un const unsigned char*

en complilant voilà ce que je reçoi :evilred: :evilred: :evilred:

section.cpp: In member function ‘void Section::set_payload(std::set<TS, std::less<TS>, std::allocator<TS> >)’:
section.cpp:116: error: ‘const class TS’ has no member named ‘get_payload’
j'ai vérifier avec *it ... rien
j'ai vérifier l'hortographe:yaisse2: rien!!!!!
que puis je faire???8O

01/04/2009, 17h31
3DArchi
Salut,
Je présume que ce n'est pas l'intégralité de ton code. Vraisemblablement tu as déclaré une méthode getpayload mais celle dont il a besoin est la version const
Code:

1 2 3 4 5 6 class TS { public: unsigned char const *getpayload()const; }
Autres remarques :
Code:

1 2 payload=new u_char;
ceci n'alloue qu'un seul u_char. Si tu en as besoin de 184*entree.size(), alors il faut allouer la bonne taille :
Code:

1 2 payload=new u_char[184*entree.size()];
Et de le supprimer :
Code:

1 2 delete []payload;
Il est certainement plus judicieux de passer ton set par référence constante :

Code:

void Section :: set_payload(set <TS> const &entree)

Si tu débutes, évite l'utilisation des pointeurs : std::string pour les chaînes de caractères, std::vector pour les tableaux, etc...

Si payload est membre de Section : n'utilises pas de pointeur.
Ou alors, fais attention :pointeurs intelligents

Ajout des balises [CODE]. Merci d'y penser !!

salut,

j'ai modifié mon code de la façon que tu m'as indiqué :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void Section :: set_payload(set <TS> const & entree){ set<TS> :: iterator it; int pos=0; //initaialiser la position à partir delaquelle on va copier les informations dans payload payload=new u_char; for(it=entree.begin();it != entree.end(); it++) { /* memcpy(payload+pos,it.getpayload(),184); pos+=184;*/ it->get_payload(); } }
et dans la classe TS la methode :

Code:

unsigned char* TS :: getpayload() const

et l'erreur persiste comme meme:

section.cpp: In member function ‘void Section::set_payload(const std::set<TS, std::less<TS>, std::allocator<TS> >&)’:
section.cpp:116: error: ‘const class TS’ has no member named ‘get_payload’

01/04/2009, 18h06
3DArchi

Peux-tu montrer ta déclaration de TS ?

Citation:

Envoyé par 3DArchi

Peux-tu montrer ta déclaration de TS ?

voila le .h de TS

Code:

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
33
34
35
36
class TS{
 
    public:
        TS(){}
        TS(u_char* data){
                                    ts_h=new TS_header;
                                    memcpy(ts_h,data,4);
                                   // payload=new u_char[184];
                                    //memcpy(payload,data,7);
                                    }
        TS(u_char* Payload,int size){}
        virtual ~TS(){};
         // le pid du paquet TS
        u_int16 getpid();
        void setpid(u_int16 pid);
         //début d'une section
        bool getpayload_start();
        void setpayloadstart(bool start);
        // la valeur du continuity counter
        u_int8 getcoutinuity_counter() ;
        void setcountinuity_counter(u_int8 counter);
       //le vrai payload dans le paquet
        const u_int8* getpayload() const;
        void setpayload(u_int8* data);
        void setpayload(u_int8* data, int size);
       //l'existence d'un champ d'adaptation
        bool getadaptation_field();
        void setadaptation_field(bool adaptation);
        //cette methode retourne le table_id de la section si c le debut de la section sinon elle retourne une valeur qui ne correspond à une table
        u_int8 get_table_id();
     private:
           //les attributs
         TS_header* ts_h;
         u_char* payload;
 
};

et voilà celui de la methode getpayload:

Code:

1
2
3
4
const u_int8* TS :: getpayload() const {
 
    return payload;
}

:king:

01/04/2009, 20h27
3DArchi

Salut,
As-tu inclus le .h dans section ?
Le cas échéant, peux-tu montrer l'intégralité de cette classe (déclaration et définition) ?
02/04/2009, 00h11
IrmatDen

Salut,

getpayload() n'existe pas dans le .h que tu as posté.

Citation:

Envoyé par IrmatDen

Salut,

getpayload() n'existe pas dans le .h que tu as posté.

Si si, mais il est bien caché :

Code:

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
33
34
35
36
37
class TS{
 
    public:
        TS(){}
        TS(u_char* data){
                                    ts_h=new TS_header;
                                    memcpy(ts_h,data,4);
                                   // payload=new u_char[184];
                                    //memcpy(payload,data,7);
                                    }
        TS(u_char* Payload,int size){}
        virtual ~TS(){};
         // le pid du paquet TS
        u_int16 getpid();
        void setpid(u_int16 pid);
         //début d'une section
        bool getpayload_start();
        void setpayloadstart(bool start);
        // la valeur du continuity counter
        u_int8 getcoutinuity_counter() ;
        void setcountinuity_counter(u_int8 counter);
       //le vrai payload dans le paquet
        const u_int8* getpayload() const;
        void setpayload(u_int8* data);
        void setpayload(u_int8* data, int size);
       //l'existence d'un champ d'adaptation
        bool getadaptation_field();
        void setadaptation_field(bool adaptation);
        //cette methode retourne le table_id de la section si c le debut de la section sinon elle retourne une valeur qui ne correspond à une table
        u_int8 get_table_id();
     private:
           //les attributs
         TS_header* ts_h;
         u_char* payload;
        
};

salut,

j'ai tout vérifier!!!

bon tu as demandé le code de TS le voilà

Code:

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
#include "../include/section.h"
 
 
 
 
 
//******************les methodes de la classe TS**************************************//
 
 
 
 
//les methodes touchant le payload_strat_indicator
 
bool  TS :: getpayload_start()  {
 
    return (bool) ts_h->payload_strat_indicator;
}
void TS :: setpayloadstart(bool start){
 
  if(start)
 
      ts_h->payload_strat_indicator=1;
 
  else
 
       ts_h->payload_strat_indicator=0;
}
 
//les methodes pour verifier l'existance d'une partie adaptation dans le paquet TS
 
bool TS :: getadaptation_field()
        {
            return (bool) ts_h->adaptation_field;
        }
 
//les PID du paquet TS
 
u_int16 TS :: getpid(){
    u_int16 s=0;
    s=ts_h->PID_hi; 
    s<<=8;
    s |=ts_h->PID_lo;
    return s;
}
void TS :: setpid(u_int16 pid){
 
 
    ts_h->PID_lo=(u_int8) pid&0xFF;
    pid>>=8;
    ts_h->PID_hi=pid&0x1F;
}
 
//les methodes pour s'informer ou modifier la valeur du champ continuity counter
 
u_int8 TS :: getcoutinuity_counter() {
 
    return (u_int8) ts_h->continuity_counter;
 
}
void TS :: setcountinuity_counter(u_int8 counter){
 
    ts_h->continuity_counter =counter;
}
 
//les methodes touchant le vrai payload dans le paquet TS
 
const u_int8* TS :: getpayload() const {
 
    return payload;
}
void TS :: setpayload(u_int8* data){
 
 
    if (getpayload_start())
    {
        if(!getadaptation_field())
        {
            payload=new u_char;
            memcpy(payload,data+5+data[4],184-data[4]);
        }
    }
}
 
//cette methode n'a une signification que lors d'un debut de section
u_int8 TS :: get_table_id(){
    return payload[0];
}
 
 
 
//******************************************************************************************
//*****************************************************************************************
 
 
 
//******************les methodes de la classe Section**************************************//
 
 
//les composants TS
set <TS> Section :: get_composant_ts(){
    return composant_ts;
}
void Section :: set_composant(set <TS> value){
    composant_ts=value;
}
 
void Section :: set_payload(set <TS> const & entree){
set<TS> :: iterator it;
 
int pos=0;   //initaialiser la position à partir delaquelle on va copier les informations dans payload
payload=new u_char;
      for(it=entree.begin();it != entree.end(); it++)
        {
     /*   memcpy(payload+pos,it.getpayload(),184);
       pos+=184;*/
      u_char*p= (*it).get_payload();
 
    }
 
}
const u_char* Section ::get_payload(){
 
    return payload;
 
}

Code:

1
2
3
void Section :: set_payload(set <TS> const & entree){
set<TS> :: const_iterator it; // car set <TS> const & entree

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void Section :: set_payload(set <TS> const & entree){
set<TS> :: iterator it;
 
int pos=0;   //initaialiser la position à partir delaquelle on va copier les informations dans payload
payload=new u_char;
      for(it=entree.begin();it != entree.end(); it++)
        {
     /*   memcpy(payload+pos,it.getpayload(),184);
       pos+=184;*/
      u_char*p= (*it).get_payload();
       
    }
 
}

et :

Code:

1
2
3
4
5
class TS{
 
    public:
       //le vrai payload dans le paquet
        const u_int8* getpayload() const;

Et get_payload != getpayload

Citation:

Envoyé par ilyes_ka

j'ai vérifier l'hortographe:yaisse2: rien!!!!!

Pas assez, faut croire :mouarf:

02/04/2009, 11h15
ilyes_ka

fi licitation tu as une très bonne observation!
:king:
02/04/2009, 11h26
3DArchi

Citation:

Envoyé par ilyes_ka

fi licitation tu as une très bonne observation!
:king:

Non, j'ai un compilateur, je lis un message d'erreur et j'utilise la fonction 'rechercher' dans mon IDE. Après: :idea: