Discussion :

[fahdijbeli]

bonjour ,
je fait une fonction qui possède un paramètre (générique) de type :

Code :

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

char *

je veux dire par générique car ce paramètre elle prend n'importe quel type comme le void * mais de moi je préfere le char * car pour le char je vais manipuler les octets , je montre le code pour comprendre :

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
typedef struct
{
  unsigned int IP_MULTICAST;
  unsigned int IP_LOCAL;
}IP_MREQTypeDef;
 
 
 
void test(const char * buff, int length)
{
    char s[100];
 
    memcpy(s,buff,length);
    printf("%s",s);
    printf("%d",strlen(s));
}
 
 
int main()
{ int i,j;
    IP_MREQTypeDef x;
    x.IP_MULTICAST=49;
    x.IP_LOCAL=50;
    test((char*)&x,sizeof(IP_MREQTypeDef));
       return 1;
}

voyez ici la fonction test elle possède un paramètre

Code :

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

const char *

c'est le paramètre générique . lors de l'appelle de cette fonction j'ai passé à ce paramètre une structure avec la cast bien sure . maintenant dans la fonction test je veux copier les 2 attributs de cette structure dans une chaîne de caractère juste je parle des octets donc copier les 4 octets de première attribut vers les 4 premiers octets de s et puis la deuxième , le problème ici lorsque j'exécute ce programme il m'affiche que 1 car le code ascii 49 avec un taille de s est 1 il a pas récupéré la deuxième valeur , je sais le problème c'est quoi car ici le première attribut sur 4 octet avec la valeur de 49 occupe un seul octet donc le reste des octets sont à NUL dans ce cas la je pense que la fonction memcpy lorsque elle trouve un NUL elle sarrette et l'autre attribut ne sera copié. j'ai posté ici pour me trouvé une solution pour ça ou bien une autre méthode . bon j'ai dit ce paramètre générique car je peux par exemple passer un int non un struct
et merci pour l'aide .

[Neckara]

Bonjour,

Tu passes à printf une chaîne de caractère donc celui-ci s'arrête au premier '\0' rencontré.

Je n'ai pas testé, mais pour ce que tu veux faire, je pense que cette solution serait plus adaptée :

Code :

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

1
2
3
4
5
6
void mon_print(const char * s, size_t taille)
{
       char format[1<<( sizeof(size_t) - 1) + 3]; // taille maximale
       sprintf(format, "%d%%c", taille);
       printf( format, s);
}

[Sve@r]

Salut
C'est un peu un gros fatras ton discours. Bon, sinon la réponse est facile
1) la fonction memcpy() ne s'arrête pas à un caractère précis ; ce n'est pas une copie de chaine. C'est d'ailleurs pour ça qu'on lui passe la taille de la zone. Donc je peux t'assurer que ta copie se fait intégralement
2) tu affiches ta zone avec un format "%s" comme si c'était une chaine de caractères (sous-entendu elle possède impérativement un '\0'). Or rien n'oblige cette zone a en avoir un. Toutefois, que se passe-t-il si elle en contient par hasard un par exemple en seconde position ? Je te laisse répondre tout seul à cette question (et c'est pareil avec strlen())
3) ton analyse de la structure est correct en théorie. Dans la réalité, il peut arriver que le compilo rajoute des octets "de bourrage". Et donc peut-être que IP_MULTICAST commence en position 2 et IP_LOCAL en position 7 (j'exagère bien sûr mais c'est pour illustrer)
4) que se passe-t-il si la zone à copier fait plus de 100 char ?

Accessoirement utiliser un char* au lieu d'un void* peut se justifier (après-tout, c'était comme ça avant) mais ça oblige l'appelant à faire un cast. Alors que si tu te forçais un peu à faire toi le cast depuis un void* cette fonction serait plus souple pour les autres ce qui est bien appréciable.

Ta fonction rectifiée

Code c :

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

1
2
3
4
5
6
7
8
void test(const char * buff, int length)
{
    char s[100];
 
    memcpy(s,buff,length);
    write(1, s, length);
    printf("%d",length);
}

[Neckara]

Code c :

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

1
2
memcpy(s,buff,length);
write(1, s, length);

Pour des raisons de performances et de portabilité, ne devrait-on pas utiliser fwrite (ainsi C) à la place de write ?

Sinon, 1 correspond à la sortie standard, mais il n'y a pas un define existant pour éviter d'écrire un nombre magique (nombre écrit en dur dans le programme) ?

[edgarjacobs]

Salut
Ta fonction rectifiée

Code c :

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

1
2
3
4
5
6
7
8
void test(const char * buff, int length)
{
    char s[100];
 
    memcpy(s,buff,length);
    write(1, s, length);
    printf("%d",length);
}

Si la fonction est rectifiée comme tu le dis, on peut encore la simplifier

Code c :

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

1
2
3
4
5
void test(const char * buff, int length)
{
    write(1, buff, length);
    printf("%d",length);
}

Je n'ai pas vraiment regardé le reste....

[fahdijbeli]

merci les amis

[Sve@r]

Pour des raisons de performances et de portabilité, ne devrait-on pas utiliser fwrite (ainsi C) à la place de write ?

Pas savoir. Perso je préfère les fonctions bas niveau (c'est toi qui gère le buffer) plutôt que les fonctions bufferisées.

Sinon, 1 correspond à la sortie standard, mais il n'y a pas un define existant pour éviter d'écrire un nombre magique (nombre écrit en dur dans le programme) ?

Euh oui, je crois que c'est STDOUT_FILENO...

Si la fonction est rectifiée comme tu le dis, on peut encore la simplifier

Code c :

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

1
2
3
4
5
void test(const char * buff, int length)
{
    write(1, buff, length);
    printf("%d",length);
}

Ben comme j'ai dit, son discours était un peu fouilli et donc j'ai lu en diagonale. J'ai vu qu'il parlait de memcpy() et j'ai cru que c'était son but (le printf n'étant qu'un affichage de debug).
En relisant mieux je vois qu'en fait, il voulait juste afficher le contenu de sa structure et que le memcpy() n'était qu'un moyen (moyen évidemment bien inutile)

[Médinoc]

je veux dire par générique car ce paramètre elle prend n'importe quel type comme le void * mais de moi je préfere le char * car pour le char je vais manipuler les octets

Je parle d'expérience, une telle fonction c'est de la merde à utiliser (il n'y a qu'à voir les versions Windows de send() et recv() qui exigent des casts vers char* sans aucune raison). Même si tu bosses en interne avec des char*, il faut que l'interface "exposée" par tes fonctions accepte des pointeurs void.

[fahdijbeli]

Je parle d'expérience, une telle fonction c'est de la merde à utiliser (il n'y a qu'à voir les versions Windows de send() et recv() qui exigent des casts vers char* sans aucune raison). Même si tu bosses en interne avec des char*, il faut que l'interface "exposée" par tes fonctions accepte des pointeurs void.

oui effectivement je dois faire un driver wifi pou une carte électronique dans ce cas la je suis entrain de faire une bib comme socket.h de donc je suis entrain de faire des fonction send() et recv() . mais j'ai pas bien compris qu'est ce que ta dit , si j'ai compris je dois changer l'attribut vers void * ?

[Sve@r]

mais j'ai pas bien compris qu'est ce que ta dit , si j'ai compris je dois changer l'attribut vers void * ?

Oui. Faut toujours te mettre dans l'hypothèse que tu ne travailles pas que pour toi mais aussi pour les autres. Ca rendra ton code plus souple, plus modulaire, mieux commenté, bref ça ne peut que l'améliorer.
Et dans cette hypothèse, vaut mieux que ta fonction soit vraiment générique (accepter un void*) et que les casts éventuels soient faits en interne plutôt que de forcer les autres à faire un cast pour utiliser ta fonction.

[fahdijbeli]

OK, d'accord je vais le changer et merci pour l'aide autre fois .