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]

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