字符串不会存储超过一个单词 (C)

Question

这是我编写的程序的第一步。

目标：用户需要写一个句子，我的程序需要将它存储在一个数组中。这可以是他们喜欢的短或长。

我的程序：运行没有技术错误

问题：当我运行程序时，我无法提交超过 1 个单词的内容。 'Hello' 提交并且程序执行。除此之外的任何东西 - 新单词、空格、标点符号和数字，当我按 Enter 键时 - 我只是不断跳到终端中的新行，程序不会进入下一步。

这是我正在使用的代码

1. 提示用户输入文本和
2. 存储答案。

#include <cs50.h>
#include <stdio.h>
#include <string.h>
#include <math.h>
#include <ctype.h>
    
int main(void)
{
    string z = get_string("Text: \n");
    
    //  int n = strlen(z);
    
    int a = 0;
    int d = 1;
    int e = 0;
    
    while (z[a] != '\0')
    {
        while (isalpha(z[a]))
        {
            a++;
        }
    
        while (z[a] ==' ')
        {
            d++;
        }
    
        //Count the sentences
    
        while (z[a] == '.' || z[a] == '!' || z[a] == '?' || z[a] == '/')
        {
            e++;
        }
    }
    
    // printf("%i letter(s)\n", a);
    // printf("%i word(s)\n", d);
    // printf("%i sentence(s)\n", e);
    
    //Coleman-Liau index formula
    
    int m = (0.0588 * (100 * a / d) - 0.206 * (100 * e / d) - 15.8);
    
    if (m <= 1)
    {
        printf("Before Grade 1\n");
    }
    
    if (m >= 16)
    {
        printf("Grade 16+\n");
    }
    
    if (m < 16)
    {
        printf("Grade %i\n", m);
    }
}

非常新的编码和任何帮助将不胜感激！

我的终端截图

Answer 1

问题在于计算单词和句子的while循环永远不会增加a，这是z的索引。所以他们一遍又一遍地测试同一个角色，你就陷入了无限循环。

它适用于单个单词，因为这些测试从一开始就不会成功。

您也不应该使用将字母计数为数组索引的变量。

#include <stdio.h>
#include <string.h>
#include <math.h>
#include <ctype.h>
    
int main(void)
{
    
    char *z = get_string("Text: \n");
    
    //  int n = strlen(z);
    
    int i = 0;
    int a = 0;
    int d = 1;
    int e = 0;
    
    while (z[i] != '\0')
    {
        int counted = 0;
        while (isalpha(z[i]))
        {
            a++;
            i++;
            counted = 1;
        }
    
        while (z[i] ==' ')
        {
            d++;
            i++;
            counted = 1;
        }
    
        //Count the sentences
    
        while (z[i] == '.' || z[i] == '!' || z[i] == '?' || z[i] == '/')
        {
            e++;
            i++;
            counted = 1;
        }
        if (!counted) {
            i++;
        }
    }
    
    // printf("%i letter(s)\n", a);
    // printf("%i word(s)\n", d);
    // printf("%i sentence(s)\n", e);
    
    //Coleman-Liau index formula
    
    int m = (0.0588 * (100 * a / d) - 0.206 * (100 * e / d) - 15.8);
    
    if (m <= 1)
    {
        printf("Before Grade 1\n");
    }
    
    if (m >= 16)
    {
        printf("Grade 16+\n");
    }
    
    if (m < 16)
    {
        printf("Grade %i\n", m);
    }
}

Answer 2

以下建议的代码：

1. 干净地编译使用：gcc -Wall -Wextra -Wconversion -pedantic -std=gnu11 -c "untitled2.c" -o "untitled2.o"
2. 执行所需的功能
3. 使用有意义的变量名
4. 使用唯一的变量名来单步调试文本
5. 不包括那些内容没有使用的头文件
6. 使用for() 循环遍历文本
7. 使用size_t 而不是int 变量，因为这些变量的内容永远不会是

现在建议的代码：

#include <cs50.h>
#include <stdio.h>
//#include <string.h>
//#include <math.h>
#include <ctype.h>
    
int main(void)
{
    string z = get_string("Text: \n");
    
    size_t spaces = 0;
    size_t sentences = 0;
    size_t letters = 0;
    
    for( size_t i = 0; z[i]; i++ )
    { 
        if( z[i] ==' ' )
        {
            spaces++;
        }
        
        if( isalpha( z[i] ) )
        {
            letters++;
        }
    
        //Count the sentences
    
        if( z[i] == '.' || z[i] == '!' || z[i] == '?' || z[i] == '/')
        {
            sentences++;
        }
    }
    
    // printf("%i letter(s)\n", a);
    // printf("%i word(s)\n", d);
    // printf("%i sentence(s)\n", e);
    
    //Coleman-Liau index formula
    
    int m = (int)(0.0588 * (double)(100 * letters / spaces) - 0.206 * (double)(100 * sentences / spaces) - 15.8);
    
    if (m <= 1)
    {
        printf("Before Grade 1\n");
    }
    
    if (m >= 16)
    {
        printf("Grade 16+\n");
    }
    
    if (m < 16)
    {
        printf("Grade %i\n", m);
    }
}

程序的典型运行：

Text: 
a quick brown fox jumped over a lazy dog
Grade 7

Answer 3

您可以简单地使用fgets() 来接受所有内容，直到换行符和strchr() 的一点帮助来删除换行符：

#define MAX_LENGTH 1024

char z[MAX_LENGTH];
fgets(z, MAX_LENGTH, stdin);

z[strlen(z) - 1] = '\0';

最好使用条件语句而不是循环：

int i = 0;

while (z[i++]) {
    if (isalpha(z[i]))
        a++;

    if (z[a] == ' ')
        d++;

    if (z[i] == '.' || z[i] == '!' || z[i] == '?' || z[i] == '/')
        e++;
}