Échange d'octets en C (endianness)

Question

J'essaye d'échanger des octets pour une affectation, et j'ai pensé que ce serait la façon de le faire. P pointe vers l'objet scalaire à inverser size est le nombre d'octets de l'objet.

void *ReverseEndian(void *p, size_t size) 
{ 
    char *head = (char *)&p; 
    char *tail = head + size - 1; 

    for (; tail > head; --tail, ++head) { 
     char temp = *head; 
     *head = *tail; 
     *tail = temp; 
    } 
    return p; 
} 

Y a-t-il quelque chose qui ne va pas de soi? Je reçois une erreur avec mon code de test du professeur:

#define TestIt(a, b) TestReverse((void *)&(a),\ 
        ReverseEndian((void *)&(b), sizeof(b)), sizeof(b)) 


int main() 
char ch = 0x01, ch1 = ch; 
    short sh = 0x0123, sh1 = sh; 
    long lo = 0x, lo1 = lo; 
    float fl = 1234.567e27F, fl1 = fl; 
    double db = 123456.567890, db1 = db; 
    long double ld = 987654.321053e-204L, ld1 = ld; 
    void *vp = (void *)0x0123, *vp1 = vp; 
    char *cp = (char *)0x4567, *cp1 = cp; 
    char *ip = (char *)0x89AB, *ip1 = ip; 

    TestIt(ch1, ch); 
    TestIt(sh1, sh); 
    TestIt(lo1, lo); 
    TestIt(fl1, fl); 
    TestIt(db1, db); 
    TestIt(ld1, ld); 
    TestIt(vp1, vp); 
    TestIt(cp1, cp); 
    TestIt(ip1, ip); 

    printf("ReverseEndian succeeded!\n"); 

    return EXIT_SUCCESS; 
} 

void TestReverse(const void *before, const void *after, size_t size) 
{ 
    const char *cpBfore = (const char *)before; 
    const char *cpAfter = (const char *)after; 
    const char *tail; 

    for (tail = cpBfore + (size - 1); size; --size) 
     if (*tail-- != *cpAfter++) 
     { 
     fprintf(stderr, "ReverseEndian failed!\n"); 
     exit(EXIT_FAILURE); 
     } 
} 

Answer 1

Ici

char *head = (char *)&p; 

vous définissez un pointeur qui pointe vers un autre pointeur. Donc vous avez un pointeur -> pointeur -> octets qui n'est évidemment pas ce que vous voulez. Heureusement, le programme ne s'est pas écrasé.

Il doit être char *head = (char *)p; afin que chaque octet soit lu à partir de la mémoire pointée par p.

Answer 2

char *head = (char *)&p devrait être char *head = (char *)p

Answer 3

p est une variable de pointeur (c.-à-une variable qui est titulaire d'une adresse mémoire de quelque chose), et dans la ligne suivante:

char *head = (char *)&p; 

... vous utilisez &p, qui vous donne l'adresse de la variable de pointeur elle-même, plutôt que la valeur d'adresse de mémoire que la variable pointeur contient (pointe vers).

Vouliez-vous dire à faire à la place?

char *head = (char *)p; 

---

Par ailleurs, votre code source semble inverser l'ordre d'une chaîne entière, par exemple "Bonjour" -> "olleH" (sauf si je le lis mal)

Ce n'est pas la même chose que l'inversion endianness. Le terme endian fait généralement référence à une valeur multi-octets qui est stockée avec l'octet le plus ou le moins significatif en premier.

A titre d'exemple, la valeur décimale 43981 est exprimée en hexadécimal sous la forme ABCD. Dans big-endian, l'octet le plus significatif (AB) serait stocké dans l'emplacement de mémoire inférieur et l'octet le moins significatif (CD) serait stocké dans l'emplacement de mémoire supérieur.

Dans little-endian, l'ordre est inversé; moins significatif suivi de plus significatif: CDAB.

Answer 4

Vous ne devriez pas prendre l'adresse de p

Cela devrait fonctionner:

void *ReverseEndian(void *p, size_t size) 
{ 
    char *head = (char *)p; // here 
    char *tail = head + size - 1; 

    for (; tail > head; --tail, ++head) { 
     char temp = *head; 
     *head = *tail; 
     *tail = temp; 
    } 
    return p; 
}