如何让'isalpha'遍历字符串的每个字符？

Question

我无法弄清楚如何检查关键字 (argv[1]) 中的每个字符。我知道我可能遗漏了一些非常明显的东西。 :(

我尝试将字符串保存到一个数组，声明一个新的 int，但还是同样的问题。

//check to make sure 2nd argument is fully alphabetic
string keyword = argv[1];

for(int i = 0, n = strlen(keyword); i < n; i++)
{
    if(isalpha(keyword[i]))
    {
        printf("Success! \n");
        return 0;
    }
    else
    {
        printf("Invalid key, must be fully alphabetic. \n");
        return 1;
    }
}

预期的输出应该是“无效的键，必须是全字母。”对于任何不完全按字母顺序排列的东西。相反，它只适用于开头字符，而不适用于整个关键字。

Answer 1

除非值是非字母的，否则不要短路（returning）；保存打印Success 并在整个循环完成时返回0， 由于非字母字符而退出：

for(int i = 0, n = strlen(keyword); i < n; i++)
{
    if(!isalpha(keyword[i]))
    {
        printf("Invalid key, must be fully alphabetic. \n");
        return 1;
    }
}
printf("Success! \n");
return 0;

Answer 2

两个问题

不要总是退出循环

@ShadowRanger

使用无符号字符值

isalpha(int x) 在 unsigned char 范围和 EOF 中为 x 定义。其他负面的char 值是未定义的行为。

// if(!isalpha(keyword[i]))
if(!isalpha((unsigned char) keyword[i]))

---

使用简化的循环 - strlen() 不需要。

int alexa_alpha_test(const char *keyword) {
  while (*keyword) {
    if(!isalpha((unsigned char) *keyword)) {
      printf("Invalid key, must be fully alphabetic. \n");
      return 1;
    }
    keyword++; 
  }
  printf("Success! \n");
  return 0;
}