文件中的字符重复 - 不区分大小写

Question

我编写了一个程序来计算文件中的字符数。我打算让程序不区分大小写。但我得到一个错误打印如下。

#include <stdio.h>
#include <string.h>

int main() {
    FILE *fp1;
    char c[100], f[20], k;
    int count = 0;

    printf("Enter the file name\n");
    scanf("%s", f);

    printf("Enter the character to be counted\n");
    scanf(" %c", &k);

    fp1 = fopen(f, "r");

    while (fscanf(fp1, "%c", c) != EOF) {
        if (strcmpi(c, k) == 0)
            count++;
    }

    fclose(fp1);

    printf("File '%s' has %d instances of letter %c", f, count, k);
    return 0;
}

在这个程序中我收到一个错误提示

warning: passing argument 2 of 'strcmpi' makes pointer from integer without a cast [-Wint-conversion]

我能做些什么来纠正它？

Answer 1

您的代码存在多个问题：

• char f[20]; 对于许多文件名来说可能太短了。
• 您应该通过提供要存储到数组中的最大字符数来防止scanf() 中的缓冲区溢出：scanf("%19s", f);
• 您应该测试scanf() 返回值以避免在转换失败时出现未定义的行为。
• 您应该测试fopen 是否成功。

• 不应将char 值传递给非标准函数strcmpi，而应通过标准函数tolower 从<ctype.h> 将字符转换为小写：

  if (tolower((unsigned char)k) == tolower((unsigned char)c[0]))
      count++;`
  

  tolower 的 char 参数被转换为 (unsigned char) 以避免传递行为未定义的潜在负值。

这是一个更正的版本：

#include <ctype.h>
#include <stdio.h>

int main(void) {
    FILE *fp;
    char f[1024], k;
    int lower_k, c;
    int count = 0;

    printf("Enter the file name: ");
    if (scanf("%1023s", f) != 1)
        return 1;

    printf("Enter the character to be counted: ");
    if (scanf(" %c", &k)) != 1)
        return 1;

    fp = fopen(f, "r");
    if (fp == NULL) {
        printf("cannot open file '%s'\n", f);
        return 1;
    }

    lower_k = tolower((unsigned char)k);
    while ((c = getc(fp)) != EOF) {
        if (tolower(c) == lower_k)
            count++;
    }

    fclose(fp);

    printf("File '%s' has %d instances of letter %c\n", f, count, k);
    return 0;
}

注意事项：

• 文件名f不能包含空格。
• 此程序无法计算空白字符，因为scanf(" %c", &k) 会跳过它们，直到您键入非空白字符。

考虑使用fgets() 来读取输入并修复这些缺点。

编辑如果你想要或必须使用函数strcmpi，你应该将字符读入第二个字节为空字节的2字节数组中，以使这些数组正确的字符串传递给@ 987654339@，但是这个解决方案效率很低：

#include <stdio.h>
#include <string.h>

int main(void) {
    FILE *fp1;
    char f[100], c[2] = "", k[2] = "";
    int count = 0;

    printf("Enter the file name\n");
    if (scanf("%99s", f) != 1) return 1;

    printf("Enter the character to be counted\n");
    if (scanf(" %c", k) != 1) return 1;

    fp1 = fopen(f, "r");
    if (fp1 != NULL) {
        while (fscanf(fp1, "%c", c) == 1) {
            if (strcmpi(c, k) == 0)
                count++;
        }
        fclose(fp1);

        printf("File '%s' has %d instances of letter %s\n", f, count, k);
    }
    return 0;
}

Answer 2

要不区分大小写地比较字符，只需使用tolower() 或toupper() 将它们都转换为相同的大小写。

#include<stdio.h>
#include<ctype.h>

int main()
{
    FILE *fp1;
    char f[20], k;
    int c;
    int count = 0;

    printf("Enter the file name\n");
    scanf("%s",f);

    printf("Enter the character to be counted\n");
    scanf(" %c",&k);
    int k_int = tolower((unsigned char)k);        

    fp1 = fopen(f,"r");

    while((c = fgetc(fp1)) != EOF)
    {
        if(tolower(c) == k_int) {
            count++;
        }
    }

    fclose(fp1);

    printf("File '%s' has %d instances of letter %c\n", f, count, k);
    return 0;
}

Answer 3

strcmpi 需要两个指向字符串的指针。但是您按值传递 k 。那是char。

char c[100], f[20], k;
// ...
    if(strcmpi(c,k)==0)  count++;
//...
// fix:
    if(strcmpi(c,&k)==0)  count++;

您或许应该考虑使用符合 POSIX 的 _stricmp 而不是 strcmpi。

Answer 4

您甚至不需要使用 strcmpi() 来表示您可以直接比较两个字符的字符：

if(tolower(c) == tolower(k))  
  count++;