c - 以不区分大小写的方式计算字符在文件中出现的次数时出错

Question

请告诉我下面的代码有什么问题。执行后显示

"test.txt has 0 instances of letter 'r'"

`#include<stdio.h>
#include<stdlib.h>
#include<ctype.h>
int main()
{
  FILE *fp;
  char ch,cmp;
  char f[100];
  int count=0,tu1,tu2;
  printf("Enter the file name\n");
  scanf("%s",f);
  fp=fopen(f,"r");
  printf("Enter the character to be counted\n");
  scanf("%c",&ch);
  tu1=toupper((int)ch);
  while((cmp=fgetc(fp))!=EOF)
  {
    tu2=toupper((int)cmp);
    if(tu1==tu2)
    {
      count++;
    }
  }
  fclose(fp);
  printf("\nFile \'%s\' has %d instances of letter \'%c\'",f,count,ch);
  return 0;
}` 

Answer 1

第 1 点

在使用返回的文件指针之前，始终检查 fopen() 是否成功。在此(下方)代码行之后为 fp 添加 NULL 检查。如果为 NULL，则终止程序

 fp=fopen(f,"r");

第 2 点

改变

scanf("%c",&ch);

到

scanf(" %c",&ch); // mind the ' ' before c

避免之前按下 ENTER 键时存储的尾随换行符 (\n)。

第 3 点

根据 man page fgetc()，返回类型为int。将 cmp 的数据类型从 char 更改为 int。

注意:

1. main() 的推荐签名是int main(void)
2. 初始化局部变量始终是一个很好的做法。