Discussion :

[onet]

Hello,

Voila, la situation initale:

J'ai un struct que je passe en référence à une fonction. Cette fonction à pour but de recevoir ce struct, et d'en faire une copie dans un tableau de struct. Comment je le fais? Je n'y arrive pas!

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
 
struct msg {
    int    cmd_id;
    int    lenght;
    char * params;
    bool urgent;
 
};
 
msg list_msg[MSG_NBR_MAX];
 
bool Queue::send_msg( msg * message ){
 
    int pos;
 
    if( this->end_list + 1 != this->start_list ){
        if(this->end_list != MSG_NBR_MAX - 1)
            pos = this->end_list + 1;
        else
            pos = 0;
 
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
 
        this->end_list++;
    }
 
    return true;
}

la ou y a les XXXXXXXXXXX, je dois copier le contenu de mon message (le contenu, pas sa référence) dans mon tableau de struct. J'ai essayé:
this->list_msg[pos].cmd_id = message.cmd_id;

Sans succès (avec *message.cmd_id, &message.cmd_id aussi) . Mes bases de C++ étant relativement lointaine, je nage un peu... Si qqun a une piste pour m'aiguiller?

merci.
Onet

[onet]

C'est bon, résolu!

Code :

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

message->cmd_id

Merci, et désolé du bruit!

Onet

[onet]

mmm...

Petite précision... Y a pas moyen de faire plus "propre":

Code :

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

1
2
3
4
        this->list_msg[pos].cmd_id = message->cmd_id;
        this->list_msg[pos].urgent = message->urgent;
        for(int i = 0; i < NBR_PARAMS_MSG; i++)
            this->list_msg[pos].params[i] = message->params[i];

Sachant que ma struct est:

Code :

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

1
2
3
4
5
6
struct msg {
    int    cmd_id;
    char params[NBR_PARAMS_MSG];
    bool urgent;
 
};

???

Onet

[ram-0000]

Code :

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

this->list_msg[pos] = *message;

ou

Code :

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

memcpy(this->list_msg[pos], message, sizeof(this->list_msg[pos]));

[Goten]

Le this est n'est pas obligatoire.

Soit dit en passant tu devrais regarder du côté de std::vector<>

[vikki]

Hello,
Tu pourrais surcharger l'opérateur d'affectation de ta structure:

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
 
 
struct msg 
{
    int    cmd_id;
    std::string params;
    bool urgent;
 
    msg & operator=(const msg & right)
    {
         cmd_id = right.cmd_id;
         params = right.params;
         urgent = right.urgent;
    }
 
};

A enjoliver ensuite en ajoutant un constructeur par copie.
Dans tous les cas, je te déconseilles d'utiliser l'opérateur d'affectation sans le surcharger en gardant un char* pour params : tu feras une copie de pointeur (chaque params pointera vers le même emplacement mémoire), pas une copie du contenu (à moins que cela ne soit ce que tu souhaite).
C'est vrai qu'un std::vector ou une std::list va te faciliter la vie dans ce cas.