如果是你，会不会修改这段代码？

最近在合流一些功能，然后有一部分功能的代码需要重构。

需要重构的代码如下，没有重构之前是可以正确执行的。

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。