Discussion :

[olivierace]

Bonjour,

Je voudrais mettre le caractère x2 après le résultat de la chaine reçu qui est ici ps ou pu8String.
Après de multiples tentatives, je demande votre aide.
Je vous remercie d'avance.

O.B

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
37
38
39
40
41
 
/****************************************************************************
 *
 * NAME: vSerial_Init
 *
 * DESCRIPTION:
 *
 * PARAMETERS:      Name            RW  Usage
 * None.
 *
 * RETURNS:
 * None.
 *
 * NOTES:
 * None.
 ****************************************************************************/
PUBLIC void vSerialRxString(uint8 *ps)
{
    uint8 *pu8String;
    int16 i16Chr;
    int16 X;
    int16 str;
 
    /* Copy the received string from the receive queue */
    /*str = (char)ps; */
    /*strcat (str,"x2") ;*/
   /* ps = (uint8)str;*/
    for(pu8String = ps; ((i16Chr = i16Serial_RxChar()) != (int16)RX_STRING_END_CHAR); pu8String++)
    {
         /* X = strcat ((uint8)i16Chr,"x2") ; */
 
        *pu8String = (uint8)i16Chr;
       /*   X =strcat (*pu8String,"x2") ; */
 
    }
 
/* *pu8String = (uint8)'x'; /* Append x2 character to the end of the string */
   /* *pu8String = (uint8)'2'; /* Append x2 character to the end of the string */
   /* *pu8String = (uint8)'x2'; */ /* Append x2 character to the end of the string */
    *pu8String = (uint8)'\0'; /* Append NULL character to the end of the string */
}

[Neckara]

Bonjour,



Tu parles de multiples tentatives mais comment sommes-nous censé savoir ce qui te bloque?

Je présume que tu as une erreur de segmentation (?)

En C++ on utilisera plutôt un std::string pour stocker les chaînes de caractères.
Sinon, il faudrait savoir comment ps est alloué :
- automatiquement (= variable locale) : il faudra donc que ta fonction retourne une nouvelle chaîne que tu auras alloué à moins que tu ais prévu suffisamment de place dans ta chaîne
- avec new : tu n'a pas tellement de choix, il va falloir copier (cf std::copy ?)
- avec malloc : il faut alors faire un realloc

Pour positionner le dernier caractère une fois que tu es sûr que tu as suffisamment de place allouée reste assez simple :

Code :

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

1
2
3
for(uint8 * tmp = ps; *tmp != 0; ++tmp);
//tmp pointera sur le '\0', tmp - ps donnera la taille de ta chaîne sans compter le '\0' final ou tmp - ps + 1 pour le compter
*++tmp = 0x02;

[Bousk]

Bonjour,

'x2' c'est 2 caractères et non 1 seul.

[Neckara]

Bonjour,

'x2' c'est 2 caractères et non 1 seul.

Je pense qu'il voulait dire 0x02 (2 en hexadécimal) soit STX (start of Text) ?

[olivierace]

Bonjour,

Je vous remercie dans un premier de votre aide car ce n'est pas simple pour moi qui débute en langage c++.

Je voulais dire la caractère '*' et le caractère '2'.

J'espère que cela est clair, sinon, n'hésitez pas à me la faire savoir.

J'essaie de programmer ce que ma proposer 'Neckara' comme solution.

Merci

[ternel]

quoi que la notation me semble un peu barbare, il faut remplacer son *++tmp=0x02; par *++tmp='*';*++tmp='2';

[Bousk]

Mais je crains que la mémoire allouée ne soit pas suffisante.

Pourquoi ne pas utiliser une simple std::string ?

[ternel]

peut-être parce que son code ressemble à une lib à l'interface en C, rédigée en C, et qu'il se serait trompé de forum…

sinon, la string me parait tout indiquée.
Je dirai même que toute gestion manuelle d'une chaine brute me semble dangereuse.

[phi1981]

peut-être parce que son code ressemble à une lib à l'interface en C, rédigée en C, et qu'il se serait trompé de forum…

sinon, la string me parait tout indiquée.
Je dirai même que toute gestion manuelle d'une chaine brute me semble dangereuse.

Je pense que t'as raison. Ce code ressemble en particulier à FreeRTOS, donc du C pur et dur.

[olivierace]

Bonjour,

Je vous remercie de votre aide, ceci est du c++ et non du c.

Je n'arrive toujours pas à afficher les caractères x et 2 après la chaîne. Ce qui me préoccupe pour l'instant est qu'au moment de la compuilation j'ai 3 erreurs : assignment of read-only location <unnamed>::g_namesmap.

Les 3 erreurs pointent sur les lignes :
*pu8String = (uint8)'x'; /* Append 'x' character to the end of the string */
*pu8String = (uint8)'2'; /* Append '2' character to the end of the string */
*pu8String = (uint8)'\0'; /* Append NULL character to the end of the string */


PUBLIC void vSerial_TxString(const uint8 *ps)
{
const uint8 *pu8String;
uint8 *tmp;

for ( *tmp = ps; *tmp != 0 ; ++ tmp ) ; //tmp pointera sur le '\0',
//tmp - ps donne la taille de la chaîne sans compter le '\0' final ou tmp - ps + 1 pour le compter
*++ tmp = 0x02 ; //+1 à la longueur de la chaine
/* créér et allouer la chaîne pu8String avec une taille de ps-tmp+3 */
pu8String = (uint8*)malloc((tmp-ps+3) *sizeof(uint8)); /* ou alors: pu8String = (uint8 *)malloc(tmps-ps+3); */

*pu8String = (uint8)'x'; /* Append 'x' character to the end of the string */
*pu8String = (uint8)'2'; /* Append '2' character to the end of the string */
*pu8String = (uint8)'\0'; /* Append NULL character to the end of the string */

for(pu8String = ps; (!bSerialQ_Full(TX_QUEUE) && (*pu8String != TX_STRING_END_CHAR)); pu8String++)
{
vSerialQ_AddItem(TX_QUEUE, *pu8String); /* Add character to the transmit queue. */
}


vUART_StartTx(); /* Start the tx process if it has stalled */
}



Si vous avez des idées , merci beaucoup car là je suis coincé.

Merci d'avance.

Olivier

[phi1981]

Le compilateur ne fait que son devoir. Tu déclares un pointeur vers des données constantes :

Code :

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

1
2
 
const uint8 *pu8String;

puis tu demandes à modifier ces données :

Code :

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

1
2
3
4
 
*pu8String = (uint8)'x'; /* Append 'x' character to the end of the string */
*pu8String = (uint8)'2'; /* Append '2' character to the end of the string */
*pu8String = (uint8)'\0'; /* Append NULL character to the end of the string */

Si tu vires le qualificateur const, ton erreur de compilation disparaitra.

Mais ce code n'a pas que des erreurs de compilation. Je soupçonne que tu manques un peu d'expérience dans la gestion des chaines de caractères en C, et la manipulation des pointeurs en général.

[olivierace]

Bonsoir,

effectivement, après avoir enlever le 'const' dans les déclarations, je n'ai plus d'erreur.

Malheureusement, les caractères 'x' et '2' ne n'apparaissent toujours pas derrière la chaîne.

Si une personne d'entre vous vois une erreur, je lui en serait très reconnaissant.

Merci bien.

Olivier

[koala01]

Salut,

Bonsoir,

effectivement, après avoir enlever le 'const' dans les déclarations, je n'ai plus d'erreur.

Malheureusement, les caractères 'x' et '2' ne n'apparaissent toujours pas derrière la chaîne.

Si une personne d'entre vous vois une erreur, je lui en serait très reconnaissant.

Merci bien.

Olivier

et pour cause, si le code est réellement

Code :

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

1
2
3
*pu8String = (uint8)'x'; /* Append 'x' character to the end of the string */
*pu8String = (uint8)'2'; /* Append '2' character to the end of the string */
*pu8String = (uint8)'\0'; /* Append NULL character to the end of t

Tel qu'il est là, tu place le caraxtère 'x' à l'adresse mémoire représentée par py8String, puis, à la ligne de code suivante, tu places le caractère '2' à l'adresse mémoire représentée par pu8String sans avoir changé la valeur de cette adresse mémoire, puis, enfin, tu places le '\0', touours à la même adresse mémoire... Cherchez l'erreur

Aller, un petite indice : une adresse mémoire particulière déclarée comme devant représenter un type donné ne peut jamais représenter qu'une et une seule valeur, or, '2' doit se trouver après 'x' et '\0' doit se trouver après '2'

[Bousk]

Les erreurs ont été citées depuis le début:
- gérer des chaînes de caractères n'est pas trivial, il faut savoir ce qu'on fait et ce n'est donc pas donné à un débutant
- utiliser std::string
- la taille allouée risque d'être insuffisante
- utiliser std::string
- les pointeurs sont mal utilisés, arithmétique des pointeurs à revoir
- utiliser std::string