c - 为什么 isalpha 不起作用？

Question

我正在使用 C，我需要检查用户输入的第二个命令行参数 argv[1] 是否仅由字母字符组成，如果不是，则执行 else 循环内的操作。我使用了 is alpha 函数，但是当我编译和运行程序时，无论我的第二个命令行参数是什么(字母或其他)，它总是执行“else 循环”。我该如何解决这个问题？

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

int main(int argc, string argv[])
{
  int a  = argc;    

  if (a != 2)
  {
    return 1;    
  }

  string b = argv [1]; 
  int c = strlen(b);
  string m;

  for (int i = 0; i < c; i++)
  {
    if (isalpha(b[c]))
    { 
      m = GetString();    
    }
    else
    {
      printf("Please provide a valid keyword\n");
      return 1;
    }         
  }
}  

Answer 1

尝试替换

if (isalpha(b[c]))

与

if (isalpha(b[i]))

当前，您正在检查索引处的元素，这是在循环的每次迭代中 strlen(b) 的结果。因为数组索引在 C 中基于零，b[strlen(b)] 正在引用 '\0'，空终止符。

引用下面的 Keith Thompson 评论和对 this question 的回答您实际上应该将传递给 isalpha 的值转换为 unsigned char 以确保不会调用未定义的行为。

因此您应该将代码更改为

if (isalpha((unsigned char)b[i]))

确保没有UB