Discussion :

[ilyes_ka]

bonjours,

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

Code :

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

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

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

[poukill]

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

[ilyes_ka]

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 :

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

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 public de la classe TS qui retourne un const unsigned char*

en complilant voilà ce que je reçoi

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 rien!!!!!
que puis je faire???

[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 :

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

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

Autres remarques :

Code :

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

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 :

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

1
2
 
payload=new u_char[184*entree.size()];

Et de le supprimer :

Code :

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

1
2
 
delete []payload;

Il est certainement plus judicieux de passer ton set par référence constante :

Code :

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

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

[ilyes_ka]

salut,

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

Code :

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

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 :

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

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’

[3DArchi]

Peux-tu montrer ta déclaration de TS ?

[ilyes_ka]

Peux-tu montrer ta déclaration de TS ?

voila le .h de TS

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
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 :

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

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