有没有更简单的方法来编写这个C代码？

Question

我对C很新，所以我想要一些建议。

此代码正在查看单词是anagrams还是no。代码威胁大写输入与小写输入相同，并且忽略输入字符不是字母。最后它应该和它显示的是单词anagrams或no。

我想知道有没有一种更简单的方法来编写这段代码，或者这是非常多的吗？

int alphabet[26] = {0}, sum = 0; 
char first[20], second[20]; 
int i = 0; 

printf("Enter the first word: "); 
do 
{ 
    first[i] = getchar(); 
    if(isalpha(first[i])) 
    alphabet[toupper(first[i]) - 'A'] += 1 ; 
    i++; 

}while(first[i - 1] != '\n'); 

printf("Enter the second word: "); 
i = 0; 
do 
{ 
    second[i] = getchar(); 

    if(isalpha(second[i]) && alphabet[toupper(second[i]) - 'A'] > 0) 
    { 
     alphabet[toupper(second[i]) - 'A'] -= 1; 
    } 
    i++; 

}while(second[i - 1] != '\n'); 

for(i = 0; i <= 26 - 1; i++) 
{ 
    sum += alphabet[i]; 
} 
if (sum == 0) 
    printf("Anagrams\n"); 
if (sum != 0) 
    printf("Not anagrams\n"); 

我没有一个编辑，在第二输入我拿出的，如果条件之一，现在它看起来像这样

do 
{ 
    second[i] = getchar(); 

    if(isalpha(second[i])) 
    { 
     alphabet[toupper(second[i]) - 'A'] -= 1; 
    } 
    i++; 

Answer 1

您的代码是不正确的：它认为“a”和“ ac“作为anagrams，由于错误的计数逻辑。我固定的，并有利于存储只是最近输入字符去掉整个first和second词的存储：

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

int main(void) 
{ 
    int alphabet[26] = {0}; 
    int ch; 
    int i; 

    printf("Enter the first word: "); 
    do 
    { 
    ch = getchar(); 
    if(isalpha(ch)) 
     alphabet[toupper(ch) - 'A']++; 
    }while(ch != '\n'); 

    printf("Enter the second word: "); 
    do 
    { 
    ch = getchar(); 
    if(isalpha(ch)) 
     alphabet[toupper(ch) - 'A']--; 
    }while(ch != '\n'); 

    for(i = 0; i < 26; i++) 
    { 
    if (alphabet[i]) 
    { 
     printf("Not anagrams\n"); 
     return 0; 
    } 
    } 

    printf("Anagrams\n"); 
    return 0; 
}