将未知长度字符串转换为小写问题

Question

所以我不太擅长 C，但我正在设计一个 GLUT 应用程序，它读取一个不区分大小写的文件。为了更容易，我想将我的字符串转换为所有小写。我创建了一个函数 makeLower 来修改通过引用传入的变量。

我在方法 makeLower 中有一个 While 循环，它似乎通过了 while 循环的第一次迭代的一部分，然后 EXE 崩溃了。任何提示都会很棒，谢谢!

输出:

C:\Users\Mark\Documents\Visual Studio 2010\Projects\Project 1\Debug>"Project 1.e
xe" ez.txt

Line is #draw a diamond ring

Character is #

然后出现错误“project 1.exe has stopped working”

代码:

void makeLower(char *input[]){
    int i = 0;
    printf("Line is %s\n", *input);

    while(input[i] != "\0"){
        printf("Character is %c\n", *input[i]); 
        if(*input[i] >= 'A' && *input[i] <= 'Z'){
            *input[i] = tolower(*input[i]);
        }
        i++;
    }

}

int main(int argc, char *argv[]) {
    FILE *file = fopen(argv[1], "r");
    char linebyline [50], *lineStr = linebyline;
    char test;

    glutInit(&argc, argv);

    while(!feof(file) && file != NULL){
        fgets(lineStr , 100, file);
        makeLower(&lineStr);
        printf("%s",lineStr);

        //directFile();

    }
    fclose(file);


    glutMainLoop();
}

Answer 1

我现在看到更多问题，所以我将我的评论扩展到一个答案:

您分配了一个包含 50 个字符的数组，但告诉 fgets 获取最多 100 个字符，这可能是致命的，因为 fgets 将覆盖不在字符串中的内存。

将C字符串传递给函数时，不必将指针的地址传递给字符串(&lineStr)，实际的指针或数组就可以了。这意味着您可以将 makeLower 函数更改为 void makeLower(char *input) 或 void makeLower(char input[])。现在，makeLower 的参数被声明为数组或 char 指针，而不是指向 char 数组的指针。

在我上面提出的新的 makeLower 中，您可以将单个字符作为数组 (input[i]) 或作为指针加偏移量 ( *(input + i)。正如我在评论中所说，如果您使用第一个版本，编译器可能会创建最后一个版本。但第一个版本更具可读性，所以我建议这样做。

同样在 makeLower 中，您与 "\0" 进行了比较，它是一个字符串而不是一个字符。这实际上几乎是正确的:您应该使用 input[i] != '\0'。

最后这就是我将如何实现它:

void makeLower(char *input)
{
    while (*input != '\0')  /* "while (*input)" would also work */
    {
        *input = tolower(*input);
        input++;
    }
}

关于函数的一些解释:

• 所有字符数组都可以转换为字符指针，但反之则不行。传递 char 指针实际上是传递字符串的最常见方式，正如您可以从所有接受字符串的标准函数(如 strlen 或 strcpy)中看到的那样。
• 表达式 *input 取消引用(即获取指针指向的值)字符串。它与 *(input + 0) 相同，因此获取字符串中第一个字符的值。
• 虽然字符串中的第一个字符不是 '\0'(从技术上讲这是一个正常的零)，但我们将循环。
• 获取字符串的第一个字符并将其传递给tolower 函数。无论字符是什么，这都会起作用，tolower 只会将大写字符转换为小写字符，所有其他字符将按原样返回。
• tolower 的结果复制到第一个字符上。这是可行的，因为赋值的右侧必须在赋值之前执行，所以不会有任何错误或问题。
• 最后我们将指针加一。这将使 input 指向字符串中的下一个字符。这是有效的，因为 input 是一个局部变量，所以对 pointer 的操作不会影响调用函数中的任何内容。

这个函数现在可以这样调用:

char input[100];
fgets(input, sizeof(input), stdin);
printf("before: \"%s\"\n", input);
makeLower(input);
printf("after : \"%s\"\n", input);