用C中的字符串计算字母出现次数[关闭]

Question

这个程序接收一个字母和字符串，当它在字符串中引入'FIM'时它会结束接收字符串。它保留在收到的信函中重复多次的字符串并将其全部打印出来。 它给了我一个错误，我不知道如何解决它（见内联评论）：

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

#define MAX 101

int conta(char frase[], char letr) {
    int n = 1, i;
    int nmax = 0;
    char primeiro = tolower(frase[0]), stringn[MAX];

    do {
        printf("introduce string: ");
        gets(frase);
        for (i = 1; frase[i] != '\0'; i++) {
            if (tolower(frase[i]) == primeiro)
                n++;
        }
        if (nmax < n) {
            nmax = n;
            stringn[] == frase[];   /*Error!!*/
        }
    } while (frase != 'FIM');

    printf("The string where %c surgiu %d appeared more time is: %c ",
           letr, nmax, stringn);
}

void main() {
    char frase[MAX];
    char letr;

    printf("give a letter:");
    scanf("%c", &letr);
    conta(frase, letr);
}

Answer 1

stringn[] == frase[] 是无效的 C 代码，要复制字符串，请使用 strcpy(stringn, frase)。

请注意，您的代码还有许多其他问题：

• frase != 'FIM' 也不正确，但编译器诊断将难以理解，因为您使用过时的 C 语法。要比较字符串，您应该使用strcmp。在这种情况下，strcmp(frase, "FIM") != 0。此外，您不应使用do / while 循环，因为在检查letr 的出现之前，您必须检查字符串是否匹配FIM。

• gets 已过时且存在风险。请改用fgets。请注意，fgets 读取的字符串如果输入，则末尾将包含'\n'。

• 您不应该将裸露的char 传递给tolower：如果char 类型是有符号的，您会为负字符值（例如非ASCII 字符）调用未定义的行为。使用以下语法：tolower((unsigned char)frase[0])。

• 您实际上计算了每行第一个字母的出现次数，而不是 letr 的出现次数。以这种方式初始化primeiro：primeiro = tolower((unsigned char)letr)。

• stringn 应初始化为空字符串，以防没有任何行出现 letr。使用strcpy(stringn, ""); 或*stringn = '\0'; 或更好的方式执行此操作，这样定义：char stringn[MAX] = "";