Có gì sai với thuật toán của tôi?

Question

Được rồi, tôi đã nấu một số mã để đảo ngược các ký tự hex xung quanh như một phần của bài tập vui nhộn mà tôi đã tạo.

Dưới đây là những gì tôi có vào lúc này:

#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; 
} 

Nó hoạt động tốt cho hầu hết các giá trị. Ví dụ: 0xA0 trở thành 0x0A v.v ...

Tuy nhiên, nó không hoạt động tốt với các giá trị bắt đầu bằng 'F'.

0xF1 trở thành 0x10
0xFF trở thành 0xF0
vv ...

ai đó có thể điểm tôi vào đúng hướng?

Answer 1

Bạn đang sử dụng loại dữ liệu đã ký (trên máy của bạn). Chuyển nó thành unsigned và nó sẽ hoạt động đúng.

Answer 2

Nếu char được ký trên hệ thống của bạn, thì khi nibble trên của c là f, c là âm và c% 16 sẽ cho kết quả âm.

Answer 3

Tôi không biết tại sao một người nào đó đang thực hiện thao tác *, /,% trong khi thao tác bitwise đơn giản có thể thực hiện những việc như vậy.

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

Answer 4

getchar và putchar trả lại và mất int s. Thậm chí tốt hơn điều này, họ sử dụng giá trị của char truyền đến một unsigned char có nghĩa là đối với tất cả các ký tự hợp lệ putchar sẽ trả lại giá trị dương. Điều này là cần thiết cho thuật toán của bạn khi bạn sử dụng % và nếu không bạn sẽ cần phải dựa vào hành vi được xác định thực hiện.

Nếu bạn chỉ định giá trị getchar thành int thì bạn có thể kiểm tra xem đọc có thành công không vì bất kỳ lý do gì (không chỉ kết thúc luồng) bằng cách so sánh với EOF. Sử dụng feof là không cần thiết - trước đây chưa đủ.

Ví dụ:

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