C 可以推测我想用哪个数组来存储我的字符吗？

Question

我正在编写代码来检查数组是否是回文:

Write a program that reads a message, then checks whether it's a palindrome
(the letters in the message are the same from left to right as from right to left):

Enter a message: He lived as a devil, eh?
Palindrome

Enter a message: Madam, I am Adam.
Not a palindrome

当我进入时他像魔鬼一样生活，是吗？,
它给了我输出不是回文，
但真正的输出应该是回文。

下面的代码是我迄今为止尝试过的。

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

#define MAX_LEN 100

int main(void) {

    char message[MAX_LEN];
    char c, *p = message, *q;

    printf("Enter a message: ");

    while ((c = toupper(getchar())) != '\n' & p < message + MAX_LEN) {
        if (isalpha(c))
            *p++ = c;
    }
    p--;

    for (q = message; q < p; q++, p--) {
        if (*p != *q) {
            printf("Not a palindrome\n");
            return 0;
        }
    }
    printf("Palindrome\n");
    return 0;
}

Answer 1

对于初学者，您应该将变量 c 声明为具有 int 类型。用户可以中断输入过程，在这种情况下，函数 getchar 返回整数值 EOF，您应该检查是否发生这种情况。

char *p = message, *q;
int c;

while 语句的条件存在错误

while ((c = toupper(getchar())) != '\n' & p < message + MAX_LEN) {

您必须使用逻辑 AND 运算符 &&，而不是按位运算符 &。

正如我已经说过的，您应该检查 while 语句的条件是否用户中断了输入。例如

while (  p < message + MAX_LEN && ( c = toupper(getchar())) != EOF && c != '\n') {
    if (isalpha(c))
        *p++ = c;
}

调用toupper或isalpha的参数应转换为unsigned char类型。否则，一般来说，如果没有强制转换，此类调用可能会调用未定义的行为。

最好不要从输入的字符串中排除数字。因此，最好至少调用函数 isalnum 而不是函数 isalpha。

用户可以输入一个空字符串，在这种情况下是指针的减量

p--;

还可以调用未定义的行为。

当程序有一点退出时，效果会更好。

程序可以如下所示

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

#define MAX_LEN 100

int main(void) 
{
    char message[MAX_LEN];

    printf( "Enter a message: " );

    char *p = message;

    for ( int c; p < message + MAX_LEN &&  ( c = getchar() ) != EOF && c != '\n';   )
    {
        if( isalnum( ( unsigned char )c ) )
        {
            *p++ = toupper( ( unsigned char )c );
        }           
    }

    int palindrome = 1;

    if ( p != message )
    {
        for ( char *q = message; palindrome && q < --p; ++q )
        {
            palindrome = *q == *p;
        }           
    }

    printf( "The entered message is %spalindrome\n", 
            palindrome ? "" : "not " );

    return 0;
}

它的输出可能看起来像这样

Enter a message: He lived as a devil, eh?
The entered message is palindrome

或者类似

Enter a message: Madam, I am Adam
The entered message is not palindrome

请注意，您可以仅使用函数 fgets 的一次调用，而不是使用多次调用函数 getchar 的循环

fgets( message, sizeof( message ), stdin );

或

if ( fgets( message, sizeof( message ), stdin ) != NULL )
{
    // check whether the entered string is a palindrome
}