Discussion :

[Nemesis007]

Voici le code de la structure:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
typedef struct Trame
{
	char AdresseDuModule[2];
	char Fonction[2];
	char AdresseDuRegistre[4];
	char EtatDuVentilateur[4];
	char Checksum[4];
 
}st_Trame;

Voici le main :

Code :

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

1
2
3
4
5
6
7
8
9
 
void main()
{
	char ChaineRecue[17] = "030600020001E828";
	st_Trame* MaTrame;
	MaTrame = (st_Trame*) ChaineRecue;
       cout << "%s" << MaTrame->AdresseDuModule << endl;
 
}

Je sais, c'est un petit programme tout bête mais j'utilise tout cela dans un programme beaucoup plus gros.

Bref, mon problème est, quand j'affiche AdresseDuModule, cette variable contient toute la chaine (030600020001E828) et non ce que je veux (03).

Comment puis-je régler ce petit problème ?

Merci

[thewho]

Bonjour,

Et le 0 teminal de tes chaines, il se met tout seul ?

[Nemesis007]

Ah ... je viens de comprendre ^^

Mais pourtant, j'ai déjà utilisé ce code est il n'y avait aucun problème (enfin c'est un vague souvenir)

mais comment je peux le mettre le '\0' à la fin de chaque chaine ?

EDIT : Voir commentaire dans le 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
25
 
typedef struct Trame
{
	char AdresseDuModule[2];
	char Fonction[2];
	char AdresseDuRegistre[4];
	char EtatDuVentilateur[4];
	char Checksum[4];
 
}st_Trame;
 
void main()
{
	char ChaineRecue[17] = "030600020001E828";
	st_Trame* MaTrame;
	MaTrame = (st_Trame*) ChaineRecue;
 
	cout << "\n" << MaTrame->AdresseDuModule << endl; // doit contenir "03"
	cout << "\n" << MaTrame->Fonction << endl; // doit contenir "06"
	cout << "\n" << MaTrame->AdresseDuRegistre << endl; //doit contenir "0002"
	cout << "\n" << MaTrame->EtatDuVentilateur << endl; // doit contenir "0001"
	cout << "\n" << MaTrame->Checksum << endl; //doit contenir "E828"
 
 
}

Résultat obtenu :

030600020001E828

0600020001E828

00020001E828

0001E828

E828

[nikko34]

à part le std::cout, c'est 100% C et non C++ ton code

[Nemesis007]

à part le std::cout, c'est 100% C et non C++ ton code

Je me suis surement trompé de section mais personne n'a de solution(s) pour ce problème qui est tout bête je suis sûr mais que j'arrive pas à résoudre ?

[nikko34]

Je me suis surement trompé de section mais personne n'a de solution(s) pour ce problème qui est tout bête je suis sûr mais que j'arrive pas à résoudre ?

à mon avis, ne fait pas comme ça mais utilise les fonctions de string.h pour découper ta chaîne en morceaux et les mettre où il faut. Il faut qu'à la fin de chaque chaîne de charactère, il y ait une place libre pour mettre un "\0".

[Nemesis007]

Ok merci, je vais tester cette solution

[gelam]

Bonjour,
Pour ma part je pense que si on passe au C++ on doit éviter les char du C et utiliser les string du C++.
Tant qu'à faire on peut faire une classe qui emballe tout ça.

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
 
#include <iostream>
#include <string>
using namespace std ;
 
class Trame{
public:
	Trame( string chainerecue ){
		adresseDuModule = chainerecue.substr(0,2) ;
		fonction = chainerecue.substr(2,2) ;
		adresseDuRegistre = chainerecue.substr(4,4) ;
		etatDuVentilateur = chainerecue.substr(8,4) ;
		checksum = chainerecue.substr(12,4) ;
	}
 
	string getAdresseDuModule(){return adresseDuModule;}
	string getFonction(){return fonction;}
	string getAdresseDuRegistre(){return adresseDuRegistre;}
	string getEtatDuVentilateur(){return etatDuVentilateur;}
	string getChecksum(){return checksum;}
 
 
private:
	string adresseDuModule;
	string fonction;
	string adresseDuRegistre;
	string etatDuVentilateur;
	string checksum;
};
 
int main()
{
	Trame trame("030600020001E828");
 
	cout << trame.getAdresseDuModule() << endl ;
	cout << trame.getFonction() << endl ;
	cout << trame.getAdresseDuRegistre() << endl ;
	cout << trame.getEtatDuVentilateur() << endl ;
	cout << trame.getChecksum() << endl ;
	return 0 ;
}

[nikko34]

entièrement d'accord gelam mais il n'a jamais dit qu'il voulait passer au C++..

sinon pour tes get tu peux renvoyer des const std::string &. Et mettre les fonctions const.

Code :

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

1
2
3
4
5
 
const std::string & getChecksum() const
{
  return checksum;
}

et le param du constructeur aussi, comme ça :

Code :

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

1
2
3
4
5
 
Trame( const std::string & chainerecue )
{
...
}

[Nemesis007]

entièrement d'accord gelam mais il n'a jamais dit qu'il voulait passer au C++..

sinon pour tes get tu peux renvoyer des const std::string &. Et mettre les fonctions const.

Code :

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

1
2
3
4
5
 
const std::string & getChecksum() const
{
  return checksum;
}

et le param du constructeur aussi, comme ça :

Code :

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

1
2
3
4
5
 
Trame( const std::string & chainerecue )
{
...
}

Je vais essayer vos 2 solutions en espérant que cela marche (ce qui ne m'étonnerait pas )