snprintf，用于 C 中整数到字符串的转换

Question

我有一小段代码可以将 c 中的整数转换为字符串。 该代码必须在 32 位和 64 位平台上运行。

我正在循环中解析参数，因此我需要 malloc 来创建缓冲区

int tmp_integer = /*read as integer*/
char* tmp_string = malloc(20* sizeof(char)); /*enough room for the biggest integer*/
snprintf(tmp_string, 20,"%d",tmp_integer); /*can I avoid using 20 here*/
a[i - 1] = tmp_string; /*save the parsed argument for final usage*/

我的问题是： 有什么办法可以让它变得很好使用 snprintf，或者我应该回退到 sprintf。

我认为使用 snprintf 不是正确的做法，因为最初我想防止缓冲区溢出，但由于整数的大小已知，我认为它是无用的。我仍然想知道这里的最佳实践是什么。

Answer 1

在这种情况下你是对的。您知道您的数字不大于 64 位，并且您知道这意味着它的位数不会超过 20 位。因此，您实际上并不需要使用snprintf。

Bug：最大的无符号 64 位数字是 18,446,744,073,709,551,615，即 20 位数字。但是，每个字符串末尾都有一个 '\0' (NUL) 字符（因此称为 nul 终止字符串术语）。因此，您应该为数组分配 21 字节，而不是 20。

Answer 2

如果动态分配内存，则可以使用 log10 来计算字符串中所需的位置数：

int tmp_integer = /*read as integer*/ 
int signpadding = 0;
if tmpinteger < 0 then signpadding = 1;
int digitcount = (integer)floor(log10(abs(value)))+ 1 + signpadding;
char* tmp_string = malloc(digitcount * sizeof(char)); 
snprintf(tmp_string, digitcount,"%d",tmp_integer);

Answer 3

问题是那个神奇的“20”从何而来。由于它很神奇，因此它应该表示为符号常量，而不是代码中重复的整数文字。使用符号常量还具有让编译器为您进行错误检查的好处：（

#define MAX_INTEGER_DIGITS (20)

int value = /* ... */
char* tmp_string = malloc(MAX_INTEGER_DIGITS);
snprintf(tmp_string, MAX_INTEGER_DIGITS, "%d", value);

另请注意我如何删除 sizeof (char) 内容，因为它完全多余并且（imo）非常混乱。）

从性能的角度来看，您可能可以取消字符串格式化程序的受保护变体，但由于您无论如何都在这里调用 malloc() （不便宜），因此它可能不是一个很好的选择赢来删除它。

Answer 4

您可以在临时缓冲区中使用 sprintf ，然后分配合适的大小（随着 sprintf 的返回），然后将临时值复制到缓冲区中。

int tmp_integer = /*read as integer*/
static char tmp_string[20];
int size = sprintf(tmp_string,"%d",tmp_integer); 
char *myValueString = malloc((size+1)*sizeof(char));
a[i - 1] = strcpy(myValueString,tmp_string); 

Answer 5

这是另一种（可能不是那么漂亮）基于 sum1stolemyname 代码片段进行转换的方法：

char *convert_int_to_string(int value)
{
    int digitcount;
    char * tmp_string;
    int increment = 2; // one for rounding, one for '\0' terminator
    if(value <0){
            increment += 1; // make room for sign 
    }
    if(0 == value){
            tmp_string = malloc(2* sizeof(char));
            sprintf(tmp_string, "%u", 0);
    }
    else{
            digitcount = (int)floor(log10((double)abs(value)))+ increment;
            tmp_string = malloc(digitcount * sizeof(char));
            sprintf(tmp_string, "%d", value);
    }
    return tmp_string;
}

Answer 6

解决此问题的一个简单、可移植（跨不同位数、平台和 CPU）的解决方案是：

int tmp_integer = /*read as integer*/
const size_t len = 4 * sizeof(int);
char tmp_string[len];
snprintf(tmp_string, len, "%d", tmp_integer);