Discussion :

[clairetj]

Bonjour a toutes et tous

J'ai un problème donc je viens solliciter votre aide. Je bosse actuellement sur un programme rédigé en c++ que je dois passer en c. Dans l'ensemble je n'ai pas trop de problème sauf sur un point: je ne trouve pas l'équivalence d'une classe c++ en c (si cela existe). Donc avez-vous des idées à me suggérer ???

La classe que je souhaite convertir en c est super simple, c'est pas une classe très étoffé (je vous mets le code en dessous). Tellement simple que beaucoup d'entre vous vont certainement me dire que j'en ai pas besoin, mais à priori cette classe est très importante dans le programme en c++ et j'aimerai garder la même "forme" entre mon programme c++ et sa conversion en c.

tableau.h

Code :

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

1
2
3
4
5
6
7
8
class Tableau
{
public:
    Tableau(int const& taille);//: longueur(taille), elements(new char [taille]){}
 
    int longueur;
    char *elements;
};

tableau.cpp

Code :

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

1
2
3
4
5
6
7
8
#include "tableau.h"
/*=========================================
 * Description : Création d'un tableau de char
 *=========================================*/
Tableau::Tableau(int const& taille): longueur(taille), elements(new char [taille])
{
 
}

Je vous mets aussi un exemple de son utilisation dans une fonction pour comprendre en quoi la classe est importante pour le programme

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
void sendTrameWriteModBus(char adrSlave, char CodeCommande, int adresseData, int nbrData, Tableau *tableau)
{
    int variableLocale;
    qint64 nombreOctetsEcris;                                                   // variable de vérification pour l'écriture sur le port
    Tableau *bufferEmissionModBus = new Tableau((tableau->longueur * 2) + 9);   // + 9 car : 7 octets d'entêtes MODBUS, 2 octets de CRC
 
    bufferEmissionModBus->elements[0] = adrSlave;
    bufferEmissionModBus->elements[1] = CodeCommande;
    bufferEmissionModBus->elements[2] = adresseData / 256;
    bufferEmissionModBus->elements[3] = adresseData & 255;
    bufferEmissionModBus->elements[4] = nbrData / 256;
    bufferEmissionModBus->elements[5] = nbrData & 255;
    bufferEmissionModBus->elements[6] = nbrData * 2;
 
    variableLocale = 7;
    for(int i = 0; i < tableau->longueur; i++)
    {
        bufferEmissionModBus->elements[variableLocale] = tableau->elements[i] / 256;
        variableLocale++;
        bufferEmissionModBus->elements[variableLocale] = tableau->elements[i] & 255;
        variableLocale++;
    }
 
    addCRC16(bufferEmissionModBus,bufferEmissionModBus->longueur - 2);          // ajout du CRC sur les deux derniers octets du tableau 'bufferEmissionModBUS'
 
    nombreOctetsEcris = serialPort->write(bufferEmissionModBus->elements, bufferEmissionModBus->longueur);  // écriture sur le port série
}

[Bktero]

Bonjour,

Tu as réussi à passer le reste de ton projet mais pas cette classe ? Perso, je pense qu'une structure avec un tableau et une taille fera l'affaire. La fonction de création demande une petite boucle si tu veux dupliquer les éléments.

[clairetj]

Question: est-ce que cette classe, dans l'utilisation qui en est faite peut être remplacer par un tableau classique en C du style char toto[8] par exemple parce que c'est ce que j'ai commencé à faire, j'obtiens ce résultat:

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
void sendTrameWriteModBus(char adrSlave, char CodeCommande, int adresseData, int nbrData, char tableau)
{
/*=========================================
 * Description : Formate la trame ModBus et envoi la trame creee
 *=========================================*/
    int variableLocale;
    long nombreOctetsEcris;                                                   // variable de vérification pour l'écriture sur le port
    char bufferEmissionModBus[(sizeof(tableau) * 2) + 9];   					// + 9 car : 7 octets d'entêtes MODBUS, 2 octets de CRC
 
    bufferEmissionModBus[0] = adrSlave;
    bufferEmissionModBus[1] = CodeCommande;
    bufferEmissionModBus[2] = adresseData / 256;
    bufferEmissionModBus[3] = adresseData & 255;
    bufferEmissionModBus[4] = nbrData / 256;
    bufferEmissionModBus[5] = nbrData & 255;
    bufferEmissionModBus[6] = nbrData * 2;
 
    variableLocale = 7;
    for(int i = 0; i < sizeof(tableau); i++)
    {
        bufferEmissionModBus[variableLocale] = tableau[i] / 256;
        variableLocale++;
        bufferEmissionModBus[variableLocale] = tableau[i] & 255;
        variableLocale++;
    }
 
    addCRC16(bufferEmissionModBus,sizeof(bufferEmissionModBus) - 2);          // ajout du CRC sur les deux derniers octets du tableau 'bufferEmissionModBUS'
 
    nombreOctetsEcris = write(bufferEmissionModBus, sizeof(bufferEmissionModBus));  // écriture sur le port série
}

Mais je ne sais pas si cela peut être viable

[gangsoleil]

Bonjour,

Comme le dit Bktero, c'est une structure qu'il faut que tu utilises, pas un "simple tableau". Et c'est ta structure qui contiendra un entier pour longueur et un pointeur pour elements (attention a bien penser a allouer la zone memoire).

[clairetj]

donc un truc du style:

Code :

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

1
2
3
4
5
6
 
struct tableau
{
     int longueur
     char *elements
}

[ManusDei]

Oui.

[phi1981]

Il est pas beau ce code C++ ....

Je crois même que j'ai repéré une fuite mémoire :

Code :

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

1
2
 
Tableau *bufferEmissionModBus = new Tableau((tableau->longueur * 2) + 9);   // + 9 car : 7 octets d'entêtes MODBUS, 2 octets de CRC

Je vois pas où pourrait se trouver le delete qui correspond à ce new.

Comme dit Bktero, tu peux utiliser un tableau et une longueur, ça suffira.

Toutefois, un conseil, en ModBus on manipule des mots de 16 bits, donc perso plutôt que de manipuler un bête tableau de chars, je manipulerais un tableau de shorts.
Cela devrait simplifier l'écriture de l'adresse et de la taille (avec un byte-swap si nécessaire).
Mais bon je connais pas les autres contraintes.

[clairetj]

Bonjour,

Tu as réussi à passer le reste de ton projet mais pas cette classe ? Perso, je pense qu'une structure avec un tableau et une taille fera l'affaire. La fonction de création demande une petite boucle si tu veux dupliquer les éléments.

Par contre j'ai pas trop compris l'histoire de la boucle pour la duplication, est-ce que l'on peut m'expliquer svp

[Bktero]

En C, tu ne peux pas copier des tableaux avec l'opérateur =. Tu dois donc faire une boucle (ou utiliser memcpy d'ailleurs ). En fait, j'ai lu un peu vite et j'ai cru que tu avais un constructeur par copie mais ce ne semble pas être le cas.

Si on manipule des données de 16 bits, autant utiliser (u)int16_t que short, histoire d'être totalement certain de la portabilité et parce que ça indique clairement qu'on souhaite exactement 16 bits.

[clairetj]

autre question: dans ma fonction sendTrameWriteModBus, je passe en paramètre un tableau. Du coup quand je défini ma fonction est-ce cela marche ainsi avec les structure:

Code :

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

sendTrameWriteModBus(char adrSlave, char CodeCommande, int adresseData, int nbrData, struct tableau)

?

[Bktero]

Code :

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

sendTrameWriteModBus(char adrSlave, char CodeCommande, int adresseData, int nbrData, struct tableau unTableau)

Ton écriture est valable lors d'une déclaration (prototype sans corps de fonction) mais lors de la définition (avec le corps de la fonction), il te faut donner un nom au paramètre de type struct tableau.