Discussion :

[donquichotte]

Bonjour à tous !

J'ai une fonction qui affiche des logs basée sur les ostringstream, celle-ci concatene plusieurs champs les un à la suite des autres et recupère la valeur de ces champs grace à des fonctions.

En gros en version simplifiée par rapport à la réalité

std::stringstream s;

s << "Field 1 " << getField1Value(); << std::endl;
s << "Field 2 " << getField2Value(); << std::endl;
...

Le code ci dessus est généré par macro est n'est pas vraiment facilement modifiable.

Jusqu'ici tout va bien, mais je rencontre un problème, certains de ces champs sont codés sur des uint8_t (unsigned char) et contienent des valeurs numériques non affichables, j'aimerais donc que ces valeurs soient affichées
en tant qu'entiers et non comme des caractères.

Il m'est impossible de caster ces valeurs vers des entiers comme :

s << "Field X " << (int) getFieldXValue(); << std::endl;

car j'ignore le type de retour de la fonction getFieldXValue(), il peut s'agir d'une string, d'un entier sur 1, 2, 4 ou 8 octets.

Je ne peux pas non plus changer le comportement des fonctions gets.


Je pensais donc surcharger l'opérateur << pour les uint8_t afin de faire le cast ici, mais je n'y arrive pas.

J'ai fait une classe qui dérive de ostringstream et fait une redefinition de l'opérateur, mais cela ne fonctionne pas :

Code :

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

1
2
3
4
5
6
7
8
struct Moss : public std::ostringstream
{
 
friend Hoss& operator<< (Moss& out, int c ) {static_cast<std::ostringstream &> (out) << c; return out;}
 
friend Hoss& operator<< (Moss& out, uint8_t c ) {out << (int) c; return out;}
 
};

J'ai rajouté la definition pour l'opérateur << et le int également, car sans cela, la ligne "out << (int) c" etait ambigue pour le compilateur.


Ensuite j'utilise ma classe Moss pour un ostringstream classique, mais les valeurs entières codées dans les uint8_t n'apparaissent toujours pas.


J'ai sans doute fait une betise, ou bien ce que je veux faire n'est pas possible, ou encore il existe des solutions plus simples, mais je pensais pouvoir surcharger l'opérateur << assez simplement...

Je serais ravi d'avoir des conseils là dessus, merci infiniment si vous pouvez m'aider !

[koala01]

Salut,

Y a-t-il la moindre raison pour que les entiers soient représentés sur huit bits par les fonctons getValue()

Il faut en effets savoir que si, par habitude, un char est codé sur 8bits, il ne revient souvent pas beaucoup plus cher au niveau de l'utilisation de la mémoire de coder les valeurs entières comme... des entiers (sur 32 bits, dirons nous par exemple )

Il existe en effet un phénomène d'alignement des données qui fait que, bien qu'un caractère ou un "short" soient codés sur moins de bits qu'un int, les trois nécessiteront exactement la place... d'un int pour leur représentation en mémoire (avec perte des possibilités d'accéder aux bits surnuméraires).

Bon, comme je suis conscient qu'il y a moyen de contourner ce phénomène, et que ce n'est pas tout à fait complet, je dis vite cela pour ne pas mentir trop longtemps, parce que je n'ai pas du tout envie de m'étendre sur le sujet

Mais toujours est-il que, en dehors de raisons particulières qui feraient que, il me semblerait particulièrement opportun de veiller à ce que les valeurs numériques soient toutes renvoyées avec de types clairement définis : (unsigned) int ou (unsigned) long( long) pour les valeur entières et float ou (long ) double pour les valeurs réelles, par exemple.

Je ne sais pas exactement quelles sont tes contraintes, mais, si c'est pour travailler sur PC (ou équivalent), il ne sert, à mon sens, pas à grand chose de vouloir faire autrement

Évidemment, si tu subi des contraintes dues à de l'embarqué vétuste (ou du moins... ancien dirons nous )...

[JolyLoic]

Pour avoir le droit de surcharger un opérateur, il faut que l'un au moins des types des paramètres soit un type défini par l'utilisateur. Ce n'est pas le cas ici.

Une solution est de passer par une indirection supplémentaire. Tu modifies ta macro pour qu'elle dise :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
s << "Field 1 " << prepareForDisplay(getField1Value()) << std::endl;
 
template<class T> T prepareForDisplay(T const &t)
{
  return t;
}
 
int prepareForDisplay(uint_8 u)
{
  return u;
}

[donquichotte]

Bonjour messieurs dames

@koala
Je suis d'accord, mais je ne peux pas modifier le type de ces variables car cela viendrait bouleverser l'alignement de ma structure : il est vital qu'aucun octet "filler" ne viennent s'intercaler entre mes champs.

@JolyLoic
Je pense que je vais suivre ta solution ! Avec les contraintes de mon problème c'est vraiment ce qui parait le mieux à faire..!

Merci beaucoup !