Discussion :

[tintin72]

Bonjour,

J'ai récupéré une fonction dont je ne suis pas bien sûr.
Pour simplifier la fonction ressemble en gros à ça:

Code :

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

1
2
3
4
5
6
7
8
9
 
unsigned int fonction()
{
  UneStructure *pointeur;
 
 ... //du code
 
  return (unsigned int)pointeur;
}

Puisque la valeur du pointeur est une adresse, le fait
de faire un cast ne risque t-il pas de corrompre cette valeur ?

Merci.

[Médinoc]

En théorie, ça risque : Le type de retour devrait être intptr_t ou uintptr_t à la place (et peut-être qu'il faut explicitement caster le pointeur en void* avant, en plus. Je ne connais pas trop ce point-là).

[valefor]

Pour ma part je dirait que cela dépend du type de pointeur de ton architecture.

Mais ce que tu fait reste du bricolage. As-tu une bonne raison de vouloir faire cela ???

Qu'est ce que tu peut faire avec un unsigned int que tu ne peux pas avec un pointeur ?

[tintin72]

Mais ce que tu fait reste du bricolage. As-tu une bonne raison de vouloir faire cela ???

Cette fonction n'est pas de moi je l'ai récupéré dans un bouquin, et
justement j'aimerais moi aussi savoir si son auteur a une bonne raison
de faire ça.

C'est une fonction qui utilise l'API Windows, le véritable type de retour est
un DWORD (un unsigned int donc).
Le principe de cette fonction est d'allouer de la mémoire
pour une structure et d'initialiser ses membres.
Je pense que le programmeur a voulu se simplifier la vie en faisant
une pierre 2 coups.
Si une erreur survient la fonction retourne le message
d'erreur Windows (qui est un DWORD) correspondant.
Si tout se passe bien la fonction retourne le pointeur via le cast.

Maintenant je me demande s'il ne vaudrait pas mieux modifier tout
ça et faire quelque chose d'un peu plus "safe".

[valefor]

Safe, je ne sais pas, mais lisible certainement !

[Emmanuel Delahaye]

J'ai récupéré une fonction dont je ne suis pas bien sûr.
Pour simplifier la fonction ressemble en gros à ça:

Code :

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

1
2
3
4
5
6
7
8
9
 
unsigned int fonction()
{
  UneStructure *pointeur;
 
 ... //du code
 
  return (unsigned int)pointeur;
}

Puisque la valeur du pointeur est une adresse, le fait
de faire un cast ne risque t-il pas de corrompre cette valeur ?

Horrible ! Rien ne dit que la valeur d'un pointeur puisse être représentée par un int (Par exemple, en x86 mode réel modèle de mémoire large, les pointeurs font 20-bit, sont codés sur 32-bit et les int font 16-bit). Faut pas faire n'importe quoi comme ça. Conception à revoir.

Si tu retrouves le codeur qui a fait ça, tu lui fous 2 claques de ma part, ça le calmera...

[tintin72]

Si tu retrouves le codeur qui a fait ça, tu lui fous 2 claques de ma part, ça le calmera...

Je n'y manquerai pas

Je tiens à préciser que ce bouquin (sorti il y a 10 ans environs) s'est
très bien vendu et a même reçu un Award.
Et après on s'étonne que les logiciels plantent...

[Emmanuel Delahaye]

Je tiens à préciser que ce bouquin (sorti il y a 10 ans environs) s'est
très bien vendu et a même reçu un Award.

Tu peux donner les références du bouquin ?
Tu es sûr qu'il n'y a pas eu de corrigendum ?

[tintin72]

Ce livre est uniquement disponible en anglais et je ne suis
pas sûr qu'on puisse encore le trouver.
http://www.maxmidi.com/
Il y a eu des erratums en effet mais pas au niveau du code apparement.
http://www.maxmidi.com/errata.html
Ceci dit c'est le seul livre (à ma connaissance) qui explique, et plutôt
bien, le principe du MIDI sous Windows et des fonctions multimedia
de l'API.
Il propose en plus le code détaillé et commenté d'un séquencer MIDI complet.
Je ne vais donc pas lui jeter la pierre, mais ça prouve qu'il faut
toujours être vigilant lorsqu'on récupère du code