Discussion :

[Invité]

Bonjour à la communauté,

J'ai ci-dessous une fonction que me sert à convertir une chaine hexadécimale (stringName) en chaine de caractère (o_cData).

0x00115660AEFF ===> "00115660AEFF"

Les caractères de la chaine héxa sont appelé deux à deux avant de remplir la nouvelle chaine avec un sscanf, et c'est la que ça coince.
Peut-être qu'un sprintf serait plus approprié ?
Pourriez-vous m'aider sur ce point ?

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
 
int fonction() {
  char *stringName = "0x00115660AEFF";  //Chaine de caractère d'entrée
  unsigned int nbDataSimuFile = 15; //Nombre de caractère de la chaine en comptant le caractère de fin
  int o_iNbDatas;  //Nombre de caractère de la chaine de sortie
  unsigned char *o_cData; //Chaine de caractère de sortie
 
 
  int j = 0;  //Permet de sélectionner l'emplacement dans la chaine de caractère de sortie
 
  o_cData = malloc (sizeof(char) * nbDataSimuFile - 2));  //Permet d'allouer de la mémoire à la chaine de sortie sans les deux premiers caractères
 
  if('0' == stringName[0] && 'x' == stringName[1]) { //Vérification que la chaine d'entrée soit bien en hexadécimale
    *o_iNbDatas = (nbDataSimuFile - 3) / 2;  //Calcul du nombre de caractère de sortie
    o_cData = malloc (sizeof(char) * (*o_NbDatas));  //Nouvelle allocation mémoire pour la chaine de sortie
 
    for (int i; i < nbDataSimuFile - 1; i += 2) {  //Boucle for pour cibler 2 termes par termes
      char temp[3];
      temp[0] = stringName[i];  //Décomposition de l'information
      temp[1] = stringName[i + 1];
      temp[2] = '\0';
 
      sscanf((char*) o_cData[j], "%c%c", temp); //Remplissage de la chaine de sortie
 
      j += 2;
 
    }
 
    return 1;
 
  } else {
 
    return 0;
 
  }
}

Cordialement
Bastien

[Bousk]

C'est vraiment le code ?
Que vaut i au début dans for (int i; i < nbDataSimuFile - 1; i += 2) ?
Sinon y'a sûrement plus simple avec juste des strncpy et memcpy.
Ou encore parce que ta chaîne de sortie est juste la chaîne d'entrée privée des 2 premiers caractères. Donc c'est juste un décallage de la chaîne d'entrée. out = input + 2; et aucune copie n'est nécessaire.
Si tu veux absolument une copie, strncpy ou memcpy. Et aucunement besoin de boucle.

[Invité]

Oui, pardon, j'ai fait une erreur en recopiant, c'est int i = 2 dans la boucle for.

[Bousk]

Ok mais cette boucle n'a de toutes façons rien à faire.
Tout ce dont tu as besoin c'est memcpy(o_cData, stringName + 2, nbDataSimuFile - 2);
Btw, vu que ton o_cData est interne et fait une allocation, ta fonction est juste un puit à fuite mémoire telle quelle.
Pourrait-on voir la vraie fonction ?

[Invité]

Nous ne pouvons pas décaler de deux car la chaine de caractère doit être en hexa.
Je m’explique :
Si la chaine d’entrée est 0x4147 soit un char [6]
La chaine de sortie doit être 4147 soit un char[4]
Pour que s’il l’on fasse un printf ou un sprintf de la chaine de sortie, on obtient AG.

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
42
43
44
45
46
47
48
49
 
int fonction(int i_portNumber, int *o_iNbDatas, unsigned char *o_cData) {
  int functionReturn = 0;
  char *stringName = 0;  //Chaine de caractère d'entrée
  unsigned int nbDataSimuFile = 0; //Nombre de caractère de la chaine en comptant le caractère de fin
  int o_iNbDatas;  //Nombre de caractère de la chaine de sortie
  int status = STATUS_NO_ERREUR;
  char tmpString[MAX_LONG_MSG_SIZE];  //Chaine caractère temporaire
 
 
  int j = 0;  //Permet de sélectionner l'emplacement dans la chaine de caractère de sortie
 
  //On récupère la donnée d'un fichier
  sprintf(StrTexte, "%s%d", (char*)__func__, i_portNumber);  //StrText est globale : char strTexte[LONG_MAX_MSG_SIZE] = "";
  functionReturn = SIMU_GenerateString((char*)PLG_SIMU_NAME, StrTexte, &nbDataSimuFile, &stringName);  //PLG_SIMU_NAME est globale : const char PLG_SIMU_NAME = "PLG";
 
  if (0 == functionReturn && 0 != nbDataSimuFile)
  {
    o_cData = malloc (sizeof(char) * nbDataSimuFile - 2));  //Permet d'allouer de la mémoire à la chaine de sortie sans les deux premiers caractères
 
    if('0' == stringName[0] && 'x' == stringName[1]) { //Vérification que la chaine d'entrée soit bien en hexadécimale
 
      status = STATUS_NO_ERROR;
      *o_iNbDatas = (nbDataSimuFile - 3) / 2;  //Calcul du nombre de caractère de sortie
      o_cData = malloc (sizeof(char) * (*o_NbDatas));  //Nouvelle allocation mémoire pour la chaine de sortie
 
        for (int i; i < nbDataSimuFile - 1; i += 2) {  //Boucle for pour cibler 2 termes par termes
        char temp[3];
        temp[0] = stringName[i];  //Décomposition de l'information
        temp[1] = stringName[i + 1];
        temp[2] = '\0';
 
        sscanf((char*) o_cData[j], "%c%c", temp); //Remplissage de la chaine de sortie
 
        j += 2;
 
      } else {
 
      status = STATUS_ERREUR;
 
  } else {
 
    status = STATUS_NO_ERREUR;
 
  }
 
  return status;
 
}

Cordialement

[Médinoc]

Donc to objectif n'est pas de convertir 0xYYZZ en "YYZZ" comme tu l'annonces dans ton premier post, mais de convertir "0xYYZZ" en une chaîne qui s'échapperait en "\xYY\xZZ" ? De sorte que "0x414243" donne "ABC"?