最近在合流一些功能，然后有一部分功能的代码需要重构。
需要重构的代码如下，没有重构之前是可以正确执行的。
unsigned int Hex2Int (char c) {
if (c >= '0' && c <= '9') {
return (unsigned int) (c - '0');
} else {
return (unsigned int)(c - 'a'+10)；
}
}这是我代码里面使用的一个转换函数，我刚开始没仔细看，觉得用atoi就能完成我的功能，结果在实际测试的时候挂逼了，因为没有正确的数值返回导致我的芯片还异常工作了。
#include <stdio.h>
#include <stdlib.h>
unsigned int Hex2Int (char c) {
if (c >= '0' && c <= '9') {
return (unsigned int) (c - '0');
} else {
return (unsigned int)(c - 'a'+10);
}
}
int main() {
unsigned int val = 0;
char buffer[5] = {"1234"};
const unsigned int totalread = 3;
printf("%x %d\n", atoi(buffer), atoi(buffer));
val = Hex2Int(buffer[totalread - 3]) * 0x1000;
val += Hex2Int(buffer[totalread - 2]) * 0x0100;
val += Hex2Int(buffer[totalread - 1]) * 0x0010;
val += Hex2Int(buffer [totalread]);
printf("%x ,%d\n", val, val);
return 0;
}函数输出：
(base) C++_and_C\> gcc 23.c && ./a.out
4d2 1234
1234 ,4660
(base) C++_and_C\>然后，我刚开始还是比较坚持用我自己写的函数，毕竟已经测试很久了，但是和同事讨论后，我觉得他说的更有道理，我们应该无条件的优先相信库函数。
所以我们使用了另外的库函数strtol来实现这个功能。
#include <stdio.h>
#include <stdlib.h>
unsigned int Hex2Int(char c) {
if (c >= '0' && c <= '9') {
return (unsigned int) (c - '0');
} else {
return (unsigned int)(c - 'a'+10);
}
}
int main() {
unsigned int val = 0;
char buffer[5] = {"1234"};
const unsigned int totalread = 3;
printf("%x %d\n", atoi(buffer), atoi(buffer));
val = Hex2Int(buffer[totalread - 3]) * 0x1000;
val += Hex2Int(buffer[totalread - 2]) * 0x0100;
val += Hex2Int(buffer[totalread - 1]) * 0x0010;
val += Hex2Int(buffer [totalread]);
printf("%x ,%d\n", val, val);
char* p_end;
val = strtol(&buffer, &p_end, 16);
if (*p_end != '\0') {
printf("strtol error\n");
return -1;
}
printf("%x ,%d\n", val, val);
return 0;
}函数输出：
4d2 1234
1234 ,4660
1234 ,4660
(base) C++_and_C\>可以看到最后两个结果是一样的。
这里面需要讨论的情况是，是不是应该要把这段代码改写
val = Hex2Int(buffer[totalread - 3]) * 0x1000;
val += Hex2Int(buffer[totalread - 2]) * 0x0100;
val += Hex2Int(buffer[totalread - 1]) * 0x0010;
val += Hex2Int(buffer[totalread]);如果不改写，石斛也没有什么问题，毕竟已经是跑了很久的功能代码，但是不改写，这部分代码看起来就有点奇怪，Hex2Int名字似乎也不太对，这里实际上实现的应该是CharToHex。
