comportement non attendu avec memcpy

Bonjour,

J aimerai copier une variable dans une chaine de caractere.

Par exemple copier l entier int i = 123 dans la chaine de caractere char *s.
Lorsque je dis copier, j aimerai que les 4 octets de i soient recopier dans les 4 premier octets de s.

Voici mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

1 #include <stdio.h> 2 #include <stdlib.h> 3 #include <string.h> 4 5 6 7 int main(void) 8 { 9 char *s = calloc(5, sizeof(char)); // j alloue la taille d'un int + 1 pour le caractere de fin de chaine. 10 const int i = 134423; 11 12 memcpy(s, &i, 4); 13 printf("%d\n", *s); // j affiche les 4 premier octets. 14 15 return 0; 16 }

---

Cela affiche :

Code:

---

23

---

Ou ai je puis faire une erreur ???

Merci

Que souhaite tu réaliser?

Si tu veux convertir un int en char *, il existe d'autres solutions plus propres.

Oh my god...

1) pour ecrire un entier dans une chaine, c'est
sprintf (zone, "%d", entier)
2) pas besoin d'allocation dynamique, un tableau de caractere de taille fixe suffit
3) il manque le free

Au fait, arretez de croire que "char *" = "chaine de caractere"
On est pas en VB, ni PHP
Il faut raisonner par "zones memoire"

Code:

---

char *s = calloc(5, sizeof(char));

---

plutôt:

Code:

---

char *s = calloc(sizeof(int)+1, sizeof(char));

---

et:

Code:

---

memcpy(s, &i, sizeof(int));

---

pour la portabilité.

Citation:

---

1) pour ecrire un entier dans une chaine, c'est
sprintf (zone, "%d", entier)

---

Pas ici:

Citation:

---

Lorsque je dis copier, j aimerai que les 4 octets de i soient recopier dans les 4 premier octets de s.

---

Code:

---

1 2	printf("%d\n", *s); // j affiche les 4 premier octets.

---

Pourquoi "*s" ?:roll:

Bon et pour repondre au probleme

Code:

---

printf("%d\n", *((int *)s));

---

sauf que c'est debile de passer par un char *, si tu veux l'utiliser comme un int *

Citation:

---

Envoyé par seriousme

Citation:

---

Lorsque je dis copier, j aimerai que les 4 octets de i soient recopier dans les 4 premier octets de s.

---

---

Oué, ok, désolé
Mais, je vois pas trop l'utilité
A la limite dans une zone de type unsigned char, dans le but de serialiser la donnée, pour envoyer sur le reseau ou dans un fichier
Mais ça peut se faire directement par fwrite

Citation:

---

Envoyé par Gruik

1) pour ecrire un entier dans une chaine, c'est
sprintf (zone, "%d", entier)

---

Ce n est pas ce que je veux faire.
J ai envoie de copier les 4 bytes de l entier dans les 4 byte de s.
Or avec sprintf la taille de s est pas egale a 4 mais egale au nombre de chiffre qu a l entier i.

par exemple i = 255

alors i (en bas 2) = 00000000 00000000 0000000 11111111

j ai envie que
s[0] (en base 2) = 00000000
s[1] (en base 2) = 00000000
s[2] (en base 2) = 00000000
s[3] (en base 2) = 11111111

Citation:

---

Envoyé par Gruik

2) pas besoin d'allocation dynamique, un tableau de caractere de taille fixe suffit

---

Certes, mais ce ce que je fais n est pas faux non plus.

Citation:

---

Envoyé par Gruik

3) il manque le free

---

Alors la je n ai rien a redire :P .

tout a fait ca je doit envoyer sur un reseau plusieurs information.
Ce que je voulais faire c etait de creer une chaine de caracteres, comme ca je met les informations voulues, .... et j envoie :)

Dans ce cas la...

Code:

---

1 2 3

int a = 0x7f00ff80; char * s = (char *)&a;

---

A priori, ça devrait donner
s[0] (en base 2) = 01111111
s[1] (en base 2) = 00000000
s[2] (en base 2) = 11111111
s[3] (en base 2) = 10000000

..mais ya des histoires de big endian / little endian, y se peut que l'ordre soit l'inverse..

Citation:

---

Envoyé par berg

tout a fait ca je doit envoyer sur un reseau plusieurs information.
Ce que je voulais faire c etait de creer une chaine de caracteres, comme ca je met les informations voulues, .... et j envoie :)

---

Dans ce cas, il faut vraiment faire attention à l'ordre des octets quand on envoie et reconstitue l'entier...
Yavait un fonction dans la librairie reseau pour convertir big endian en little endian et vice versa.

http://man.developpez.com/man3/htons.3.php

ok merci pour voter aide !!!

:D

Si tu veux que ça marche correctement, il faut faire comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

#include <stdio.h> #include <stdlib.h> #include <string.h> int main(void) { char *s = calloc(1, sizeof(int)); // j alloue la taille d'un int const int i = 134423;   memcpy(s, &i, sizeof(int)); printf("%d\n", *(int *)s); // j affiche l'entier free(s);   return 0; }

---

Le %d suppose que l'argument passé est un entier, donc il faut lui passer un entier. s est un pointeur vers une chaîne de caractères à l'origine, tu as "détourné" ce "pointeur de caractères" en pointeur vers un entier, donc il faut explicitement écrire le processus de transformation dans le printf.