问题正是您的编译器告诉您的;您不允许初始化 VLA。Zack 在评论中给出了一个明显的解决方案:删除初始化。您将在此答案中找到工作示例,其中一些允许初始化,而另一些则不允许。您可以在评论中找到更多相关信息。以下示例从最合理(恕我直言)到最不合理(涉及使用malloc
)为表示数字的十进制数字序列分配存储。
我建议使用相同的技巧来确定将int
值存储为十进制数字所需的字节数,就像您用于八进制一样:将 an 中的总位数除以int
3 并添加任何符号和 NUL 终止。digit_count
可以写成预处理器宏,如下所示:
#include <limits.h>
#include <stddef.h>
#include <stdio.h>
#define digit_count(num) (1 /* sign */ \
+ sizeof (num) * CHAR_BIT / 3 /* digits */ \
+ (sizeof (num) * CHAR_BIT % 3 > 0)/* remaining digit */ \
+ 1) /* NUL terminator */
int main(void) {
short short_number = -32767;
int int_number = 32767;
char short_buffer[digit_count(short_number)] = { 0 }; /* initialisation permitted here */
char int_buffer[digit_count(int_number)];
sprintf(short_buffer, "%d", short_number);
sprintf(int_buffer, "%d", int_number);
}
如您所见,这里的一个强大好处是digit_count
无需修改即可用于任何类型的整数:char
, short
, int
, long
,long long
和相应的unsigned
类型。
相比之下,一个小缺点是您浪费了几个字节的存储空间,尤其是对于像1
. 在许多情况下,该解决方案的简单性足以弥补这一点;在运行时计算十进制数字所需的代码将占用更多的内存空间,而不是这里浪费的空间。
如果您准备放弃上述代码的简单性和通用性,并且您真的想计算小数位数,那么 Zacks 建议适用:删除初始化。这是一个例子:
#include <stddef.h>
#include <stdio.h>
size_t digit_count(int num) {
return snprintf(NULL, 0, "%d", num) + 1;
}
int main(void) {
int number = 32767;
char buffer[digit_count(number)]; /* Erroneous initialisation removed as per Zacks advice */
sprintf(buffer, "%d", number);
}
回应malloc
建议:解决这个问题的最不可怕的方法是避免不必要的代码(例如,调用malloc
和稍后free
)。如果您不必从函数返回对象,请不要使用malloc
! 否则,考虑存储到调用者(通过参数)提供的缓冲区中,以便调用者可以选择要使用的存储类型。这不是使用malloc
.
但是,如果您确实决定使用malloc
andfree
为此,请以最不可怕的方式进行操作。避免对 of 的返回值进行类型转换malloc
和乘以 by sizeof (char)
(始终为 1)。以下代码是一个示例。使用上述任一方法计算长度:
char *buffer = malloc(digit_count(number)); /* Initialisation of malloc bytes not possible */
sprintf(buffer, "%d", number);
free(buffer);
...当你完成它时不要忘记。