Co jest nie tak z moim algorytmem?

Question

W porządku, przygotowałem trochę kodu, aby odwrócić szesnastkowe znaki w ramach zabawy, którą wymyśliłem.

Oto co mam w tej chwili:

#include <stdio.h> 
int main() { 
    char a,b,c; 
    while (1) { 
     c = getchar(); 
     if (!feof(stdin)) { 
      a = c % 16; 
      b = (c - a)/16; 
      c = (a*16) + b; 
      putchar(c); 
     }else{break;} 
    } 
return 0; 
} 

To działa dobrze dla większości wartości. Na przykład 0xA0 staje się 0x0A itd ...

Jednak nie gra się dobrze z wartościami zaczynającymi się na "F".

0xF1 staje 0x10
0xFF staje 0xF0
etc ...

Czy ktoś może wskazać mi w dobrym kierunku?

Answer 1

Używasz podpisanego (na komputerze) typu danych. Przełącz na niepodpisany i powinien działać poprawnie.

Answer 2

Jeśli znak jest podpisany w systemie, to gdy górny przekrój c jest f, c jest ujemny, a c% 16 daje wynik ujemny.

Answer 3

Nie wiem, dlaczego ktoś robi *, /, operacje%, podczas gdy proste operacje bitowe mogą robić takie rzeczy.

A = (C & 0x0F) < < 4;
b = (c & 0xF0) >> 4;
c = a | b;

Answer 4

getchar i putchar powrotu i podjęcia int s. Nawet lepiej niż oni używają wartości char rzutowanej na unsigned char, co oznacza, że ​​dla wszystkich ważnych znaków putchar zwróci wartość dodatnią. Jest to potrzebne dla twojego algorytmu, ponieważ używasz % i w przeciwnym razie będziesz musiał polegać na zdefiniowanych przez implementację zachowaniach.

Po przypisaniu wartości getchar do int można sprawdzić, czy odczyt nie powiódł się z jakiegokolwiek powodu (nie tylko z końca strumienia) przez porównanie z EOF. Używanie feof nie jest wtedy konieczne - wcześniej nie było wystarczające.

E.g.

int main(void) { 
    int c; 
    while ((c = getchar()) != EOF) { 
     /* algorithm goes here */ 
     putchar(c); 
    } 
    return 0; 
}