Discussion :

[theory23]

Bonjour à tous!

Je débute dans le C++. Je viens de réaliser une fonction de conversion ASCII -> Hexadecimal.

Cette fonction prend comme paramètre d'entrée un tableau de 8 entiers et doit retourner un tableau de 2 chars.

Code :

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

1
2
3
4
5
6
7
8
9
char* hexa(int *tab)
{
 
char *tab_hex = new char[2];
 
//remplissage du tableau en fonction des 8 bits de tab (à base de if).
 
return ((char*) tab_hex);
}

et à partir de là, je n'arrive pas à afficher les deux valeurs du tableau dans une autre fonction...

Comment faut-il faire?

Merci d'avance!!

(je ne peux copier coller tous les if car je développe sur un PC qui n'a pas internet, désolé...)

[Quatre_Couleurs]

Lu,
Il me semble que le cast au niveau du return est inutile puisque tu as deja un char*.
Sinon est-ce que t'arrives a afficher le tableau dans ta fonction,essaie juste avant le return.

[theory23]

oui quand je fais printf juste avant le return mais avec des tableaux définis par moi-même dans la fonction.

C'est quand j'appelle la fonction que ça ne marche plus...

[ram-0000]

On peut voir plus de code s'il te plait ?

[Quatre_Couleurs]

Oui montre nous un peu plus de code

oui quand je fais printf juste avant le return mais avec des tableaux définis par moi-même dans la fonction.

Qu'est ce que t'entends par "définis par moi-même dans la fonction."?

[theory23]

Bonjour,

Désolé d'avoir mis tant de temps à répondre, petit souci de santé...
Mais ça va mieux

alors voila mon code:

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
char* FenetrePrincipale::hexadecimal(int *tab3){
 
char *tab_hex = new char [2];
 
//int tab3[8] ={0,1,1,1,0,0,1,1};
 
int k=0;
 
if(tab3[k] == 0 && tab3[k+1] == 0 && tab3[k+2] == 0 && tab3[k+3] == 0){
    tab_hex[0] = 0;
    //printf("%d \n",tab_hex[0]);
    }
 
if(tab3[k] == 1 && tab3[k+1] == 0 && tab3[k+2] == 0 && tab3[k+3] == 0){
    tab_hex[0] = 1;
    }
 
....
 
//printf("%d%d \n\n",tab_hex[0], tab_hex[1]);
 
return ((char*)tab_hex);
 
}

et je cherche à appeler cette fonction :

Code :

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

1
2
3
4
5
6
7
8
void FenetrePrincipale::genererCode()
{
int tab_xor6[8]={0,1,1,1,0,0,1,1};
 
char* result = hexadecimal(tab_xor6);
 
printf("%s \n",result);
}

et là rien ne s'affiche...

[Quatre_Couleurs]

Le tableau que tu passe en parametres ne verifie pas les confitions.
int tab_xor6[8]={0,1,1,1,0,0,1,1};
Les quatre premieres cases ne sont ni egales a {0, 0, 0, 0} ni a {1, 0, 0, 0}.

En plus: je ne vois toujours pas a quoi sert le cast et ta variable k semble obsolete.

[3DArchi]

Bonjour,
Effectivement, %s spécifie que c'est une chaîne de caractère à la C qui doit être affichée. Or, une chaîne de caractère C-style est une suite de char terminée par un '\0'. Ensuite, tout dépend de ce que tu veux afficher. Si ce sont les valeurs (en tant qu'entier) alors tu dois utiliser les format %d (format de spécifications) :

Code :

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

1
2
 
printf("%d %d\n",static_cast<int>(result[0]),static_cast<int>(result[1]));

Dans l'ordre, je me permets quelques remarques :
1/ Tu ne gère pas l'échec d'allocation dans hexadecimal,
2/ Tu ne détruit pas le pointeur dans genererCode,
3/ En C++, on préfère l'utilisation des conteneurs (dans ton cas un std::vector) plutôt que les tableaux dynamiques pour éviter justement ces soucis.
4/ En C++, on utilise les flux et non les fonctions printf.
5/ En C++, on n'utilise pas les cast à la C, mais les opérateurs dédiés
Ce qui pourrait donner :

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
 
std::vector<char> FenetrePrincipale::hexadecimal(std::vector<int>const & tab3) const
{
 
   std::vector<char> tab_hex;
   tab_hex.resize(2);
 
   int k=0;
 
   if(tab3[k] == 0 && tab3[k+1] == 0 && tab3[k+2] == 0 && tab3[k+3] == 0){
       tab_hex[0] = 0;
    }
 
   if(tab3[k] == 1 && tab3[k+1] == 0 && tab3[k+2] == 0 && tab3[k+3] == 0){
       tab_hex[0] = 1;
    }
//   ....
 
   return tab_hex;
}

et

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
void FenetrePrincipale::genererCode() const
{
   std::vector<int> tab_xor6;
   tab_xor6.reserve(8);
   tab_xor6.push_back(0);
   tab_xor6.push_back(1);
   tab_xor6.push_back(1);
   tab_xor6.push_back(1);
   tab_xor6.push_back(0);
   tab_xor6.push_back(0);
   tab_xor6.push_back(1);
   tab_xor6.push_back(1);
 
   std::vector<char> result = hexadecimal(tab_xor6);
 
   std::cout<<static_cast<int>(result[0])<<" "<<static_cast<int>(result[1])<<std::endl;
   // ou :
   std::copy(
      result.begin(),
      result.end(),
      std::ostream_iterator<int>(std::cout," ")
   );
   std::cout<<std::endl;
}

Et tu auras besoin des en-têtes :

Code :

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

1
2
3
4
 
#include <iostream> // pour les std::cout
#include <iterator> // pour les std::ostream_iterator
#include <vector> // pour les std::vector

[Quatre_Couleurs]

Ah oui j'avais pas vu le changement de C (premier bout de code) vers C++.

[theory23]

Merci!!
C'est vrai que j'avais mélangé C et C++ donc ça faisait un beau bazar.

J'ai pu modifier cela et ça a marché.

Merci encore