Question

我尝试使用函数填充一些字符串，但字符串似乎没有正确填充。 print语句只有4个空行。但是如果我取消注释char ** pls行，它会正确打印所有三个字符串，即使我从不在任何地方使用变量pls。它也可以在调试模式下正常运行，而不存在pls变量。我不完全确定我做了什么，并不高兴。

char * dataFile = (char *) calloc(64, sizeof(char));
//char ** pls = &dataFile;
char * queryFile = (char *) calloc(64, sizeof(char));
char * outFile = (char *) calloc(64, sizeof(char));
for(i = 1; i <argc; ++i)
{
    char command[3];
    char * iterator = argv[i];
    command[0] = *iterator;
    ++iterator;
    command[1] = *iterator;
    ++iterator;
    command[2] = *iterator;

    if(strcmp(command, "df=") == 0)
        determineFileString(iterator, &dataFile);
    else if(strcmp(command, "if=") == 0)
        determineFileString(iterator, &queryFile);
    else if(strcmp(command, "of=") == 0) 
        determineFileString(iterator, &outFile);
}
printf("%s\n%s\n%s\n", dataFile, queryFile, outFile);

void determineFileString(char * iterator, char ** file)
{
    char * p = *file;
    ++iterator;
    while(*iterator != '\0')
    {
        *p = *iterator;
        ++p;
        ++iterator;
    }
    *p = '\0';
}

Answer 1

您正在调用strcmp，但第一个操作数未指向字符串。字符串定义为一些字符后跟一个空终止符。

如果argv[i]字符串短于2个字符，您的代码也会导致未定义的行为，因为您总是从中复制3个字符。

要解决此问题，请将command放大并在末尾添加空终止符，或使用memcmp代替strcmp。（但要注意memcmp，因为如果两个对象的长度都不如大小那么大，它也会导致UB。

这是一个可能的解决方法：

for(i = 1; i <argc; ++i)
{
    if ( strlen(argv[i]) < 3 )
         continue;

    if ( memcmp(argv[i], "df=", 3) == 0 )
        determineFileString(argv[i] + 3, &dataFile);

    else if    // etc.
}

BTW，determineFileString函数不进行任何缓冲区大小检查（它可能缓冲溢出）。我建议重新设计这个功能;也许它可以进行长度检查并在函数内部调用realloc。

C字符串未在函数调用中填充

1 个答案: