0

菜鸟问题，可能。我查看了其他答案，并尝试了他们所做的事情。但我仍然每次都收到错误。这是我的凯撒密码程序的第一个片段。

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

int main()
{
    char message[81];
    char cipher[81];
    
    do
    {
        printf("Enter a message: ");
        scanf("%[^\n]", &message); 
      //scanf("%s", &message) = error as well
        printf("Test"); //to see if it prints. doesn't print, so seg fault must be on above line
        if (!isalpha(message))
        {
            printf("Invalid input. Enter only letters.\n");
        }
        else
        {
            valid = 1;
        }
    } while (valid == 0);

// ...

作为输入，我将输入任何内容 - 一个字符串、一个字符、一个数字等，但仍然会出现段错误。

我正在使用 -W -Wall -ansi -pedantic 编译我的代码，并且没有收到与段错误相关的错误或警告。

4

2 回答 2

2

对于初学者，此调用中的参数类型

scanf("%[^\n]", &message);

是无效的。你需要写

scanf("%[^\n]", message);

您还需要在下次调用 scanf 之前从输入缓冲区中删除换行符 '\n'。

该函数isalpha需要一个类型为 的对象char。但是您传递的是 type 的表达式char *。

if (!isalpha(message))

调用未定义的行为。

声明变量的代码片段也会出现问题valid。

如果要检查字符串是否包含字母符号或/和空格，则需要检查输入字符串的所有符号。

这是一个演示程序。

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

int all_letters( const char *s )
{
    while ( *s && isalpha( ( unsigned char )*s  ) ) ++s;
    
    return *s == '\0';
}

int main(void) 
{
    char message[81];
    int valid = 0;
    
    do
    {
        printf( "Enter a message: " );
        scanf( "%80[^\n]", message );
        while ( getchar() != '\n' );
        
        valid = all_letters( message );
        
        if ( !valid )
        {
            printf( "Invalid input. Enter only letters.\n" );
        }
    } while ( !valid );
    
    return 0;
}

它的输出可能看起来像

Enter a message: 123
Invalid input. Enter only letters.
Enter a message: Hello
于 2021-05-31T19:51:22.553 回答
0

我在下面运行了这段代码valgrind，错误实际上在这一行：

if (!isalpha(message))

请记住，它isalpha需要一个char（技术上是一个int，但它应该只包含一个字符）。通过传入message，您将传入 a char [81]，它不是正确的类型。

令我感到困惑的是，为什么即使将警告级别提高到最大值，它也能编译和运行，因为没有从 achar [81]到 single的隐式转换char。

要解决此问题，您需要更改您在此处编写的代码，以便更明确地测试输入的所有字符都是字母。您可能还需要检查返回值，scanf以确保它不会失败，并在失败时清除无效字符stdin。

于 2021-05-31T19:52:45.863 回答