'N' 字符出现在字符串的末尾

Question

我是 C 新手，在尝试输入字符串时遇到了问题。更具体地说，当打印字符串时，字符“N”出现在它的末尾，因此，我无法测量确切的字符数。这是我的代码：

int main()
{
    int j = 0;
    char c[15];
    printf("Give surname: ");
    scanf(" %s", c);
    for (int i = 0; i<sizeof(c) / sizeof(c[1]); i++)
    {
        if (isalpha(c[i]))
        {
            printf("%c ", c[i]);
            j++;
        }
    }
    printf("\nNumber of characters: %d", j);

    return 0;
}

例如：如果输入是“john”，那么我会得到以下输出：

我做错了什么？

Answer 1

那是因为您在比实际字符串长的字符数组上使用了sizeof 运算符。 你可以使用string.h中的strlen，如上所述，或者简单地遍历字符直到\0，就像这样：

char * p = c;
while(*p++)
    if (isalpha(*p))
        printf("%c ", *p);

如果您对字符数感兴趣，您可以减去指针。

Answer 2

你的循环限制是错误的。它导致您评估不确定数据。特别错误的事情：

• 不检查scanf 是否成功
• 未指定长度受限的格式字符串以避免c[] 数组溢出。
• 如前所述，迭代整个数组，而不仅仅是您假设已成功读取的数据。

所有这些都可以修复，其中一些是微不足道的。

• 检查scanf 是否返回1
• 由于您的 c 数组是 15 个字符，并且必须为终止的 nullchar 考虑空格，因此您的格式字符串使用 %14s
• 要么使用strlen 作为 for 的限制（以某种方式，将其保存到临时或直接在条件子句中）或使用指针。

关于最后一项，我更喜欢后者，因为您只扫描字符串一次。 strlen 将扫描字符串以找到终止符，从而计算长度并返回它。然后在迭代期间再次扫描它。您可以通过简单地不使用 strlen 扫描来消除它，而是使用指针，并在发现终止符时停止扫描。

改写了，结果是这样的：

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

int main()
{
    char c[15];
    int j = 0;

    printf("Give surname: ");
    if (scanf("%14s", c) == 1)
    {
        for (const char *p = c; *p; ++p)
        {
            if (isalpha(*p))
            {
                printf("%c ", *p);
                ++j;
            }
        }

        printf("\nNumber of characters: %d", j);
    }

    return 0;
}

输入

123john456

输出

j o h n
Number of characters: 4

Answer 3

尝试改用strlen() 函数。它会检查实际字符串末尾的空终止字符（与完整数组相反，它可能在空值之后包含垃圾字符-终止字符）。

int length = strlen(c);
for (int i = 0; i < length; i++){

记得#include <string.h>

在这里查看它的实际效果： https://repl.it/repls/SugaryUnfoldedCleaninstall

如果您想知道为什么需要依赖函数来执行此操作，请查看 the source of strlen()。编写这样一个性能良好且在所有情况下都正确的函数比看起来更难。

进一步阅读
How to iterate over a string in C